amoghrajesh commented on code in PR #76718:
URL: https://github.com/apache/airflow/pull/76718#discussion_r4520415767
##########
providers/databricks/tests/unit/databricks/operators/test_databricks.py:
##########
@@ -2229,7 +2262,7 @@ def test_resubmits_when_stored_run_no_longer_exists(self,
db_mock_class):
db_mock = db_mock_class.return_value
db_mock.submit_run.return_value = RUN_ID
db_mock.get_run.side_effect = [
- DatabricksApiError("Response: RESOURCE_DOES_NOT_EXIST, Status
Code: 404", http_status_code=404),
+ make_api_error(400, "INVALID_PARAMETER_VALUE", "Run 999 does not
exist."),
Review Comment:
Added
`test_does_not_resubmit_when_the_stored_run_id_is_rejected_as_malformed` to
both durable classes: a malformed-input 400 propagates and submit_run / run_now
is never called. Checked it by making the helper return True for everything,
which turns both red.
--
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]
