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

        There are no Sub-Tasks for this issue.

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

              Created:
              Updated:
              Resolved: