Migrate driver callback API style tests to promises

XMLWordPrintableJSON

    • 13
    • Not Needed
    • None
    • Hide

      Create a copy of the Kickoff Template with the issue key (NODE-XXX) in the filename and share a link to the new doc via this field.

      Show
      Create a copy of the Kickoff Template with the issue key (NODE-XXX) in the filename and share a link to the new doc via this field.
    • Not Needed
    • Hide

      1. What would you like to communicate to the user about this feature?
      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      3. Which versions of the driver/connector does this apply to?

      Show
      1. What would you like to communicate to the user about this feature? 2. Would you like the user to see examples of the syntax and/or executable code and its output? 3. Which versions of the driver/connector does this apply to?
    • None
    • None
    • None
    • None
    • None
    • None

      Use Case

      We want to modify all tests to no longer use callbacks as the driver is a promise only API now. This is a requirement for deprecating the legacy driver.

      User Impact

      • N/A

      Dependencies

      • upstream and/or downstream requirements and timelines to bear in mind

      Unknowns

      • N/A

      Acceptance Criteria

      Implementation Requirements

      • convert any JS files to TS if file is small / requires minimal effort for the conversion, otherwise leave as JS; keep the TS conversion in a separate commit from any other changes to the file
      • convert callback to async but don't change relative test order or make any structural modifications to describe blocks (for ease of comparative review)
        • deleting redundant tests is ok
        • do not change test case titles, but if you have an idea for a better test name, leave it as a TODO comment below the current test name - we can update all test names separately later
      • If you see potential improvements as you go along that would require structural changes to the file, file a new ticket and leave TODOs as desired
      • Update the client import in each updated file to use src instead of legacy

      Testing Requirements

      • N/A

      Documentation Requirements

      • N/A

      Follow Up Requirements

      • File follow up tickets for TODOs as needed

        1.
        Migrate `node-mongodb-native/test/unit/assorted` tests NODE-7160 Sub-task Closed Pavel Safronov 7.0.0
        2.
        Migrate `test/integration/bson-decimal128` tests NODE-7165 Sub-task Closed Sergey Zelenov 7.0.0
        3.
        Migrate `node-mongodb-native/test/unit` tests NODE-7169 Sub-task Closed Pavel Safronov 7.0.0
        4.
        Migrate `test/integration/collection-management` tests NODE-7177 Sub-task Closed Sergey Zelenov 7.0.0
        5.
        Migrate `test/integration/change-streams` tests NODE-7180 Sub-task In Code Review Pavel Safronov  
        6.
        Migrate `test/integration/connection-monitoring-and-pooling` tests NODE-7182 Sub-task Closed Sergey Zelenov 7.0.0
        7.
        Migrate `test/integration/gridfs` tests NODE-7183 Sub-task Closed Sergey Zelenov 7.0.0
        8.
        Migrate `test/integration/read-write-concern` tests NODE-7184 Sub-task Closed Sergey Zelenov 7.0.0
        9.
        Migrate `test/integration/server-selection` tests NODE-7185 Sub-task Closed Sergey Zelenov 7.0.0
        10.
        Migrate `test/integration/sessions` tests NODE-7186 Sub-task Closed Pavel Safronov 7.0.0
        11.
        Migrate `test/integration/transactions` tests NODE-7187 Sub-task Closed Pavel Safronov 7.0.0
        12.
        Migrate `test/integration/uri-options` tests NODE-7188 Sub-task Closed Pavel Safronov 7.0.0
        13.
        Migrate `test/integration/crud/aggregation` tests NODE-7192 Sub-task Closed Pavel Safronov 7.0.0
        14.
        Migrate `test/integration/crud/bulk` tests NODE-7193 Sub-task Closed Sergey Zelenov 7.0.0
        15.
        Migrate `test/integration/crud/crud_api` tests NODE-7194 Sub-task Closed Sergey Zelenov 7.0.0
        16.
        Migrate `test/integration/crud/document_validation` tests NODE-7191 Sub-task Needs Verification Sergey Zelenov  
        17.
        Migrate `test/integration/crud/find` tests NODE-7195 Sub-task In Code Review Sergey Zelenov  
        18.
        Migrate `test/integration/crud/promise_stats` tests NODE-7190 Sub-task Closed Pavel Safronov 7.0.0
        19.
        Migrate `test/integration/crud/unicode` tests NODE-7196 Sub-task Needs Verification Sergey Zelenov  
        20.
        Migrate `test/integration/crud/find_cursor_methods` tests NODE-7197 Sub-task In Code Review Sergey Zelenov  
        21.
        Migrate `test/integration/crud/insert` tests NODE-7198 Sub-task In Progress Sergey Zelenov  
        22.
        Migrate `test/integration/crud/misc_cursor` tests NODE-7199 Sub-task Ready for Work Unassigned  
        23.
        Migrate `test/integration/crud/remove` tests NODE-7200 Sub-task Ready for Work Unassigned  
        24.
        Migrate `test/integration/crud/server_errors` tests NODE-7202 Sub-task Ready for Work Unassigned  
        25.
        Migrate `test/integration/node-specific/bson-options` tests NODE-7203 Sub-task Ready for Work Unassigned  
        26.
        Migrate `test/integration/node-specific/cursor_stream` and `client_encryption` tests NODE-7204 Sub-task Ready for Work Unassigned  
        27.
        Migrate `test/integration/node-specific/db` tests NODE-7205 Sub-task Ready for Work Unassigned  
        28.
        Migrate `test/integration/node-specific/mongo_client` tests NODE-7206 Sub-task Ready for Work Unassigned  
        29.
        Change imports (import from 'src') NODE-7179 Sub-task Ready for Work Unassigned  
        30.
        Remove `mongodb-legacy` dependency and its' usage NODE-7189 Sub-task Ready for Work Unassigned  
        31.
        Remove outdated tests NODE-7219 Sub-task Backlog Unassigned  

            Assignee:
            Sergey Zelenov
            Reporter:
            Neal Beeken
            None
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated: