Uploaded image for project: 'Drivers'
  1. Drivers
  2. DRIVERS-2360

Ensure "does not fail" in CSE Prose Test 13 accounts for (no) matching documents

    • Type: Icon: Improvement Improvement
    • Resolution: Done
    • Priority: Icon: Unknown Unknown
    • None
    • Component/s: None
    • None
    • $i18n.getText("admin.common.words.hide")
      Key Status/Resolution FixVersion
      CDRIVER-4407 Fixed 1.22.0
      CXX-2528 Done 3.8.0
      CSHARP-4215 Fixed 2.17.0
      GODRIVER-2462 Done
      JAVA-4651 Done
      NODE-4314 Fixed 4.9.0
      MOTOR-982 Duplicate
      PYTHON-3306 Duplicate
      PHPLIB-897 Fixed 1.15.0
      RUBY-3029 Fixed 2.18.0
      RUST-1367 Duplicate
      SWIFT-1585 Duplicate
      $i18n.getText("admin.common.words.show")
      #scriptField, #scriptField *{ border: 1px solid black; } #scriptField{ border-collapse: collapse; } #scriptField td { text-align: center; /* Center-align text in table cells */ } #scriptField td.key { text-align: left; /* Left-align text in the Key column */ } #scriptField a { text-decoration: none; /* Remove underlines from links */ border: none; /* Remove border from links */ } /* Add green background color to cells with FixVersion */ #scriptField td.hasFixVersion { background-color: #00FF00; /* Green color code */ } /* Center-align the first row headers */ #scriptField th { text-align: center; } Key Status/Resolution FixVersion CDRIVER-4407 Fixed 1.22.0 CXX-2528 Done 3.8.0 CSHARP-4215 Fixed 2.17.0 GODRIVER-2462 Done JAVA-4651 Done NODE-4314 Fixed 4.9.0 MOTOR-982 Duplicate PYTHON-3306 Duplicate PHPLIB-897 Fixed 1.15.0 RUBY-3029 Fixed 2.18.0 RUST-1367 Duplicate SWIFT-1585 Duplicate

      Summary

      The current wording in Client Side Encryption Prose Test 13 requires certain addKeyAltName() operations "does not fail". However, this description does not take into account potential failure to find a matching document to add a keyAltName to, which also satisfies the "does not fail" condition (returning a null value instead of the previous state of the matched key document).

      The prose test description should be improved to ensure the addKeyAltName() operations are succeeding because it successfully found and added a keyAltName  to a matching document, not because it did not find a matching document.

      Motivation

      Who is the affected end user?

      Drivers that have implemented, or will implement, CSE Prose Test 13.

      How does this affect the end user?

      Drivers' implementation of CSE Prose Test 13 will need to assert the result of addKeyAltName() operations include the keyAltName added in a previous operation if they are not already doing so.

      How likely is it that this problem or use case will occur?

      During implementation, Drivers might construct an incorrect filter argument to addKeyAltName() and fail to correctly test the success conditions intended by the prose test. This is more likely on Drivers that must convert to/from Native UUID Types.

      If the problem does occur, what are the consequences and how severe are they?

      Minor.

      Is this issue urgent?

      No.

      Is this ticket required by a downstream team?

      No.

      Is this ticket only for tests?

      Yes.

            Assignee:
            ezra.chung@mongodb.com Ezra Chung
            Reporter:
            ezra.chung@mongodb.com Ezra Chung
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: