[GitHub] [airflow] potiuk commented on a diff in pull request #33702: Inspect container state rather than last_state when deciding whether to skip

["potiuk (via GitHub)" <gi...@apache.org>]

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


##########
kubernetes_tests/test_kubernetes_pod_operator.py:
##########
@@ -225,6 +225,22 @@ def test_delete_operator_pod(self, mock_get_connection):
         assert self.expected_pod["spec"] == actual_pod["spec"]
         assert self.expected_pod["metadata"]["labels"] == actual_pod["metadata"]["labels"]
 
+    def test_skip_on_specified_exit_code(self, mock_get_connection):

Review Comment:
   :heart:  the integration test ! 



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org