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

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

              Created:
              Updated: