gopidesupavan commented on code in PR #47281:
URL: https://github.com/apache/airflow/pull/47281#discussion_r1980181329
##########
contributing-docs/testing/unit_tests.rst:
##########
@@ -1095,12 +1095,12 @@ are not part of the public API. We deal with it in one
of the following ways:
1) If the whole provider is supposed to only work for later airflow version,
we remove the whole provider
by excluding it from compatibility test configuration (see below)
-2) Some compatibility shims are defined in
``tests_common.test_utils/compat.py`` - and they can be used to make the
- tests compatible - for example importing ``ParseImportError`` after the
exception has been renamed from
- ``ImportError`` and it would fail in Airflow 2.9, but we have a fallback
import in ``compat.py`` that
- falls back to old import automatically, so all tests testing / expecting
``ParseImportError`` should import
- it from the ``tests.tests_utils.compat`` module. There are few other
compatibility shims defined there and
- you can add more if needed in a similar way.
+2) Some compatibility shims are defined in
``tests-ccommon/src/tests_common/test_utils/compat.py`` - and
Review Comment:
```suggestion
2) Some compatibility shims are defined in
``tests-common/src/tests_common/test_utils/compat.py`` - and
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
