Migrate driver callback API style tests to promises

XMLWordPrintableJSON

    • 13
    • Not Needed
    • 🔵 Done
    • 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
    • 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 Closed Pavel Safronov 7.0.0
        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 Closed Sergey Zelenov 7.0.0
        17.
        Migrate `test/integration/crud/find` tests NODE-7195 Sub-task Closed Sergey Zelenov 7.0.0
        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 Closed Sergey Zelenov 7.0.0
        20.
        Migrate `test/integration/crud/find_cursor_methods` tests NODE-7197 Sub-task Closed Sergey Zelenov 7.0.0
        21.
        Migrate `test/integration/crud/insert` tests NODE-7198 Sub-task Closed Sergey Zelenov 7.0.0
        22.
        Migrate `test/integration/crud/misc_cursor` tests NODE-7199 Sub-task Closed Pavel Safronov  
        23.
        Migrate `test/integration/crud/remove` tests NODE-7200 Sub-task Closed Sergey Zelenov 7.0.0
        24.
        Migrate `test/integration/crud/server_errors` tests NODE-7202 Sub-task Closed Sergey Zelenov 7.0.0
        25.
        Migrate `test/integration/node-specific/bson-options` tests NODE-7203 Sub-task Closed Sergey Zelenov 7.0.0
        26.
        Migrate `test/integration/node-specific/cursor_stream` tests NODE-7204 Sub-task Closed Sergey Zelenov 7.0.0
        27.
        Migrate `test/integration/node-specific/db` tests NODE-7205 Sub-task Closed Sergey Zelenov 7.0.0
        28.
        Migrate `test/integration/node-specific/mongo_client` tests NODE-7206 Sub-task Closed Unassigned  
        29.
        Migrate `test/integration/node-specific/operation-examples` tests NODE-7254 Sub-task Closed Sergey Zelenov 7.0.0
        30.
        Change imports (import from 'src') NODE-7179 Sub-task Closed Sergey Zelenov 7.0.0
        31.
        Remove `mongodb-legacy` dependency and its' usage NODE-7189 Sub-task Closed Pavel Safronov 7.0.0
        32.
        Remove outdated tests NODE-7219 Sub-task Closed Durran Jordan  
        33.
        Remove mongodb.ts from the root test folder and its' usage NODE-7280 Sub-task Closed Sergey Zelenov 7.0.0

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

              Created:
              Updated:
              Resolved: