ferruzzi commented on code in PR #31725:
URL: https://github.com/apache/airflow/pull/31725#discussion_r1224774589
##########
tests/core/test_otel_logger.py:
##########
@@ -173,13 +171,54 @@ def test_decr_with_rate_limit_works(self, mock_random,
name):
self.map[full_name(name)].add.assert_has_calls(expected_calls)
self.map[full_name(name)].add.call_count == 2
- @mock.patch("warnings.warn")
- def test_timer_warns_not_implemented(self, mock_warn):
- with self.stats.timer():
- mock_warn.assert_called_once_with("OpenTelemetry Timers are not
yet implemented.")
+ def test_gauge_new_metric(self, name):
Review Comment:
And I'll add a new test to verify that `delta` is working as intended :+1:
I think that should cover our bases on that?
--
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]
