Skip to content

Commit 1c41180

Browse files
authored
Minor fixes to the release candidate command (#61764)
Not sure why the ls was failing but we can do without it
1 parent aab557b commit 1c41180

File tree

2 files changed

+2
-7
lines changed

2 files changed

+2
-7
lines changed

dev/breeze/src/airflow_breeze/commands/release_candidate_command.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -534,9 +534,7 @@ def move_artifacts_to_svn(
534534
check=True,
535535
shell=True,
536536
)
537-
console_print("[success]Moved artifacts to SVN:")
538-
run_command([f"ls {version}/"])
539-
run_command([f"ls task-sdk/{task_sdk_version}/"])
537+
console_print("[success]Moved artifacts to SVN")
540538

541539

542540
def push_artifacts_to_asf_repo(version, task_sdk_version, repo_root):

dev/breeze/tests/test_release_candidate_command.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -621,10 +621,7 @@ def fake_run_command(cmd: list[str] | str, **kwargs):
621621
and kwargs.get("shell") is True
622622
for cmd, kwargs in run_command_calls
623623
)
624-
assert "[success]Moved artifacts to SVN:" in console_messages
625-
# Verify ls commands
626-
assert any(cmd == [f"ls {version}/"] for cmd, kwargs in run_command_calls)
627-
assert any(cmd == [f"ls task-sdk/{task_sdk_version}/"] for cmd, kwargs in run_command_calls)
624+
assert "[success]Moved artifacts to SVN" in console_messages
628625

629626

630627
def test_push_artifacts_to_asf_repo_returns_early_when_user_declines(monkeypatch, rc_cmd):

0 commit comments

Comments
 (0)