dstandish commented on code in PR #38885:
URL: https://github.com/apache/airflow/pull/38885#discussion_r1558340800
##########
tests/core/test_settings.py:
##########
@@ -276,27 +286,22 @@ def test_encoding_absent_in_v2(is_v1, mock_conf):
("core", "internal_api_url"): "http://localhost:8888";,
}
)
-def test_get_traceback_session_if_aip_44_enabled():
+def test_get_traceback_session_if_aip_44_enabled(clear_internal_api):
# ensure we take the database_access_isolation config
InternalApiConfig._init_values()
assert InternalApiConfig.get_use_internal_api() is True
- # ensure that the Session object is TracebackSession
- configure_orm()
-
- from airflow.settings import Session
-
- assert Session == TracebackSession
Review Comment:
removed this part of the test so we can avoid reconfigure orm
it will me an that the behavior that settings.Session is traceback session
when using db isolation will be untested -- but, maybe that's acceptable
--
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]
