andreahlert commented on code in PR #61784:
URL: https://github.com/apache/airflow/pull/61784#discussion_r2795196475
##########
shared/observability/tests/observability/metrics/test_stats.py:
##########
@@ -149,7 +148,8 @@ def test_load_invalid_custom_stats_client(self):
"Your custom StatsD client must extend the statsd."
"StatsClient in order to ensure backwards compatibility."
)
- with pytest.raises(AirflowConfigException, match=error_message):
+ # we assert for Exception here instead of AirflowConfigException
to not import from shared configuration
+ with pytest.raises(Exception, match=error_message):
Review Comment:
would it work to import AirflowConfigException from
airflow_shared.configuration here instead of catching bare Exception?
--
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]
