ashb commented on code in PR #47451:
URL: https://github.com/apache/airflow/pull/47451#discussion_r1983433830


##########
providers/openlineage/tests/unit/openlineage/extractors/test_manager.py:
##########
@@ -51,7 +51,7 @@
         # TODO: Remove once provider drops support for Airflow 2
         from airflow.utils.context import Context
 
-    from task_sdk.tests.conftest import MakeTIContextCallable
+    from conftest import MakeTIContextCallable

Review Comment:
   This isnt importing the right thing anymore. Turns out this needed the "I 
can improt task_sdk" at the top level.
   
   Given this is the only use of such an "outside of tasksdk test import" I 
think we should just re-implement that here instead.



-- 
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]

Reply via email to