vincbeck commented on code in PR #29732:
URL: https://github.com/apache/airflow/pull/29732#discussion_r1126564027
##########
tests/providers/amazon/aws/hooks/test_emr.py:
##########
@@ -191,6 +197,9 @@ def test_get_cluster_id_by_name(self):
assert no_match is None
+ with pytest.raises(AirflowException):
+ hook.get_cluster_id_by_name("test_cluster0", ["RUNNING",
"WAITING", "BOOTSTRAPPING"])
+
Review Comment:
I dont think that's what @o-nikolas is saying. I think he wants to have a
test to cover the use case **in case** the cluster you are looking for is in
the second page. The implementation should not change, just adding a test that
covers this use case would increase the testing coverage. And I agree with 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: commits-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
