phanikumv commented on code in PR #69450:
URL: https://github.com/apache/airflow/pull/69450#discussion_r3527950349


##########
providers/snowflake/tests/unit/snowflake/operators/test_snowflake.py:
##########
@@ -359,6 +359,49 @@ def 
test_poll_on_queries_raises_runtime_error_on_status_check_failure(
         with pytest.raises(RuntimeError, match="Failed to get status for query 
uuid1"):
             operator.poll_on_queries()
 
+    def test_poll_on_queries_no_sleep_when_all_resolved(
+        self, mock_execute_query, mock_get_sql_api_query_status
+    ):

Review Comment:
   Since `poll_on_queries()` is called directly and we set `operator.query_ids` 
in the test , so `mock_execute_query` never fires — it can be dropped from the 
signature



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