bugraoz93 commented on code in PR #64586:
URL: https://github.com/apache/airflow/pull/64586#discussion_r3024072251
##########
airflow-ctl/tests/airflow_ctl/api/test_client.py:
##########
@@ -376,3 +379,16 @@ def test_retry_handling_ok(self):
response = client.get("http://error";)
assert response.status_code == 200
assert len(responses) == 1
+
+ def test_debug_mode_missing_debug_creds_reports_correct_error(self,
monkeypatch, tmp_path):
+ monkeypatch.setenv("AIRFLOW_HOME", str(tmp_path))
+ monkeypatch.setenv("AIRFLOW_CLI_DEBUG_MODE", "true")
+ monkeypatch.setenv("AIRFLOW_CLI_ENVIRONMENT", "TEST_DEBUG")
+
+ config_path = tmp_path / "TEST_DEBUG.json"
+ config_path.write_text(json.dumps({"api_url":
"http://localhost:8080"}), encoding="utf-8")
+ # 不建立 debug_creds_TEST_DEBUG.json，模擬缺檔
Review Comment:
I would love to understand what is written but I cannot :)
I would be in favor of either deleting since it is in test and test itself
is self explanatory or maybe translating into English to continue with default
What do you think?
--
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]
