potiuk commented on code in PR #40669:
URL: https://github.com/apache/airflow/pull/40669#discussion_r1684261259


##########
airflow/settings.py:
##########
@@ -256,6 +256,16 @@ def __init__(self):
     def remove(*args, **kwargs):
         pass
 
+    def get_bind(

Review Comment:
   The failing tests should be marked as @pytest.mark.db_test instead. Not sure 
which tests failed with it and why (somewhat the exception above would have to 
be ignored), but This session should have never been initialized in the 
"non-db" tests cases. See 
https://github.com/apache/airflow/blob/main/contributing-docs/testing/unit_tests.rst#db-and-non-db-tests
 for more explanation



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