Remove synchro converted_tests and associated CONTRIBUTING.md section

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Done
    • Priority: Unknown
    • None
    • Affects Version/s: None
    • Component/s: None
    • None
    • Python Drivers
    • 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

      Context

      Describe the background behind the problem.
      https://github.com/mongodb/mongo-python-driver/pull/2904 shows that the process for adding a new test suite and correctly having synchro convert changes is both non-obvious and poorly documented. This is partially due to the converted_tests list in tools/synchro.py that restricts which files in `test/asynchronous` are synchronized. This list serves no purpose now that all tests in `test/asynchronous` are synchronized, and so it should be removed.

      Definition of done

      What must be done to consider the task complete?
      The synchro hook must automatically convert all files in test/asynchronous not explicitly excluded as async-only to synchronous versions. The CONTRIBUTING.md section "Converting a test to async" and tools/convert_test_to_async.py should also be removed.

      Pitfalls

      What should the implementer watch out for? What are the risks?

            Assignee:
            Noah Stapp
            Reporter:
            Noah Stapp
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: