Uploaded image for project: 'PHP Driver: Library'
  1. PHP Driver: Library
  2. PHPLIB-887

Allow extra fields in results of legacy test runner find operations

    • Type: Icon: Bug Bug
    • Resolution: Fixed
    • Priority: Icon: Unknown Unknown
    • 1.13.0-beta1, 1.13.0
    • Affects Version/s: None
    • Component/s: Tests
    • None

      A failure was observed in the fle2-BypassQueryAnalysis.yml test due to an additional field appearing in the actual result from a find operation.

      The CSFLE legacy test format references the Transactions spec, which in turn references the CRUD spec, where Extra Fields in Actual Documents should apply. It looks like this was never the case, as most all legacy test runner operations use ASSERT_SAME_DOCUMENTS instead of ASSERT_DOCUMENTS_MATCH.

      Since existing spec tests are passing and this pertains to the legacy test runner, this ticket need only revise the assertion logic for find operations at this time (in getResultAssertionTypeForCollection).

            Assignee:
            jmikola@mongodb.com Jeremy Mikola
            Reporter:
            jmikola@mongodb.com Jeremy Mikola
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: