karenbraganz commented on code in PR #65498:
URL: https://github.com/apache/airflow/pull/65498#discussion_r3119669768


##########
providers/cncf/kubernetes/tests/unit/cncf/kubernetes/utils/test_pod_manager.py:
##########
@@ -17,11 +17,11 @@
 from __future__ import annotations
 
 import logging
-from datetime import datetime
+from datetime import datetime, timedelta, timezone
 from json.decoder import JSONDecodeError
 from typing import TYPE_CHECKING, cast
 from unittest import mock
-from unittest.mock import MagicMock, PropertyMock
+from unittest.mock import MagicMock, PropertyMock, patch

Review Comment:
   Changed in the latest commit.



##########
providers/cncf/kubernetes/tests/unit/cncf/kubernetes/operators/test_pod.py:
##########
@@ -2933,7 +2933,7 @@ def test_write_logs_with_valid_since_time(self, 
mocked_client):
 
     @patch(KUB_OP_PATH.format("client"))
     def test_write_logs_with_invalid_since_time_falls_back_to_none(self, 
mocked_client):
-        """Test that a TypeError from an invalid since_time is caught, warns, 
and uses since_seconds=None."""
+        """Test that an invalid since_time is caught, warns, and since_seconds 
is not passed."""

Review Comment:
   Changed in the latest commit.



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