From be11d3ff790dd85e1b84ef9d8030b3200c74e7c5 Mon Sep 17 00:00:00 2001 From: codejedi365 Date: Sun, 5 Jul 2026 12:07:17 -0600 Subject: [PATCH] style: adjust formatting for clearer test implementations --- .../bump_version/git_flow/test_repo_1_channel.py | 5 ++--- .../bump_version/git_flow/test_repo_2_channels.py | 5 ++--- .../bump_version/git_flow/test_repo_3_channels.py | 5 ++--- .../bump_version/git_flow/test_repo_4_channels.py | 5 ++--- .../bump_version/github_flow/test_repo_1_channel.py | 5 ++--- .../test_repo_1_channel_branch_update_merge.py | 5 ++--- .../bump_version/github_flow/test_repo_2_channels.py | 5 ++--- .../github_flow_monorepo/test_monorepo_1_channel.py | 5 ++--- .../github_flow_monorepo/test_monorepo_2_channels.py | 5 ++--- .../bump_version/trunk_based_dev/test_repo_trunk.py | 5 ++--- .../test_repo_trunk_dual_version_support.py | 5 ++--- ...st_repo_trunk_dual_version_support_w_prereleases.py | 5 ++--- .../trunk_based_dev/test_repo_trunk_w_prereleases.py | 5 ++--- .../trunk_based_dev_monorepo/test_monorepo_trunk.py | 5 ++--- tests/e2e/cmd_version/test_version_build.py | 10 ++++------ 15 files changed, 32 insertions(+), 48 deletions(-) diff --git a/tests/e2e/cmd_version/bump_version/git_flow/test_repo_1_channel.py b/tests/e2e/cmd_version/bump_version/git_flow/test_repo_1_channel.py index 79d94b37b..ee17c5214 100644 --- a/tests/e2e/cmd_version/bump_version/git_flow/test_repo_1_channel.py +++ b/tests/e2e/cmd_version/bump_version/git_flow/test_repo_1_channel.py @@ -163,8 +163,7 @@ def test_gitflow_repo_rebuild_1_channel( assert expected_release_commit_text == actual_release_commit_text # Make sure tag is created assert curr_release_tag in [tag.name for tag in mirror_git_repo.tags] - assert ( - mocked_git_fetch.call_count == 1 - ) # fetch called to check for remote changes + # fetch called to check for remote changes + assert mocked_git_fetch.call_count == 1 assert mocked_git_push.call_count == 2 # 1 for commit, 1 for tag assert post_mocker.call_count == 1 # vcs release creation occurred diff --git a/tests/e2e/cmd_version/bump_version/git_flow/test_repo_2_channels.py b/tests/e2e/cmd_version/bump_version/git_flow/test_repo_2_channels.py index 2b8c046e1..27ff5f339 100644 --- a/tests/e2e/cmd_version/bump_version/git_flow/test_repo_2_channels.py +++ b/tests/e2e/cmd_version/bump_version/git_flow/test_repo_2_channels.py @@ -163,8 +163,7 @@ def test_gitflow_repo_rebuild_2_channels( assert expected_release_commit_text == actual_release_commit_text # Make sure tag is created assert curr_release_tag in [tag.name for tag in mirror_git_repo.tags] - assert ( - mocked_git_fetch.call_count == 1 - ) # fetch called to check for remote changes + # fetch called to check for remote changes + assert mocked_git_fetch.call_count == 1 assert mocked_git_push.call_count == 2 # 1 for commit, 1 for tag assert post_mocker.call_count == 1 # vcs release creation occurred diff --git a/tests/e2e/cmd_version/bump_version/git_flow/test_repo_3_channels.py b/tests/e2e/cmd_version/bump_version/git_flow/test_repo_3_channels.py index a26988646..5b32befd1 100644 --- a/tests/e2e/cmd_version/bump_version/git_flow/test_repo_3_channels.py +++ b/tests/e2e/cmd_version/bump_version/git_flow/test_repo_3_channels.py @@ -164,8 +164,7 @@ def test_gitflow_repo_rebuild_3_channels( assert expected_release_commit_text == actual_release_commit_text # Make sure tag is created assert curr_release_tag in [tag.name for tag in mirror_git_repo.tags] - assert ( - mocked_git_fetch.call_count == 1 - ) # fetch called to check for remote changes + # fetch called to check for remote changes + assert mocked_git_fetch.call_count == 1 assert mocked_git_push.call_count == 2 # 1 for commit, 1 for tag assert post_mocker.call_count == 1 # vcs release creation occurred diff --git a/tests/e2e/cmd_version/bump_version/git_flow/test_repo_4_channels.py b/tests/e2e/cmd_version/bump_version/git_flow/test_repo_4_channels.py index 0adde1fa3..2a135b116 100644 --- a/tests/e2e/cmd_version/bump_version/git_flow/test_repo_4_channels.py +++ b/tests/e2e/cmd_version/bump_version/git_flow/test_repo_4_channels.py @@ -163,8 +163,7 @@ def test_gitflow_repo_rebuild_4_channels( assert expected_release_commit_text == actual_release_commit_text # Make sure tag is created assert curr_release_tag in [tag.name for tag in mirror_git_repo.tags] - assert ( - mocked_git_fetch.call_count == 1 - ) # fetch called to check for remote changes + # fetch called to check for remote changes + assert mocked_git_fetch.call_count == 1 assert mocked_git_push.call_count == 2 # 1 for commit, 1 for tag assert post_mocker.call_count == 1 # vcs release creation occurred diff --git a/tests/e2e/cmd_version/bump_version/github_flow/test_repo_1_channel.py b/tests/e2e/cmd_version/bump_version/github_flow/test_repo_1_channel.py index edeb485ce..fedd861dd 100644 --- a/tests/e2e/cmd_version/bump_version/github_flow/test_repo_1_channel.py +++ b/tests/e2e/cmd_version/bump_version/github_flow/test_repo_1_channel.py @@ -163,8 +163,7 @@ def test_githubflow_repo_rebuild_1_channel( assert expected_release_commit_text == actual_release_commit_text # Make sure tag is created assert curr_release_tag in [tag.name for tag in mirror_git_repo.tags] - assert ( - mocked_git_fetch.call_count == 1 - ) # fetch called to check for remote changes + # fetch called to check for remote changes + assert mocked_git_fetch.call_count == 1 assert mocked_git_push.call_count == 2 # 1 for commit, 1 for tag assert post_mocker.call_count == 1 # vcs release creation occurred diff --git a/tests/e2e/cmd_version/bump_version/github_flow/test_repo_1_channel_branch_update_merge.py b/tests/e2e/cmd_version/bump_version/github_flow/test_repo_1_channel_branch_update_merge.py index 1d54aa459..a3018d346 100644 --- a/tests/e2e/cmd_version/bump_version/github_flow/test_repo_1_channel_branch_update_merge.py +++ b/tests/e2e/cmd_version/bump_version/github_flow/test_repo_1_channel_branch_update_merge.py @@ -176,8 +176,7 @@ def test_github_flow_repo_w_default_release_n_branch_update_merge( assert expected_release_commit_text == actual_release_commit_text # Make sure tag is created assert curr_release_tag in [tag.name for tag in mirror_git_repo.tags] - assert ( - mocked_git_fetch.call_count == 1 - ) # fetch called to check for remote changes + # fetch called to check for remote changes + assert mocked_git_fetch.call_count == 1 assert mocked_git_push.call_count == 2 # 1 for commit, 1 for tag assert post_mocker.call_count == 1 # vcs release creation occurred diff --git a/tests/e2e/cmd_version/bump_version/github_flow/test_repo_2_channels.py b/tests/e2e/cmd_version/bump_version/github_flow/test_repo_2_channels.py index ab6906eea..e86858b55 100644 --- a/tests/e2e/cmd_version/bump_version/github_flow/test_repo_2_channels.py +++ b/tests/e2e/cmd_version/bump_version/github_flow/test_repo_2_channels.py @@ -163,8 +163,7 @@ def test_githubflow_repo_rebuild_2_channels( assert expected_release_commit_text == actual_release_commit_text # Make sure tag is created assert curr_release_tag in [tag.name for tag in mirror_git_repo.tags] - assert ( - mocked_git_fetch.call_count == 1 - ) # fetch called to check for remote changes + # fetch called to check for remote changes + assert mocked_git_fetch.call_count == 1 assert mocked_git_push.call_count == 2 # 1 for commit, 1 for tag assert post_mocker.call_count == 1 # vcs release creation occurred diff --git a/tests/e2e/cmd_version/bump_version/github_flow_monorepo/test_monorepo_1_channel.py b/tests/e2e/cmd_version/bump_version/github_flow_monorepo/test_monorepo_1_channel.py index 1d82138f2..c0dafbf7c 100644 --- a/tests/e2e/cmd_version/bump_version/github_flow_monorepo/test_monorepo_1_channel.py +++ b/tests/e2e/cmd_version/bump_version/github_flow_monorepo/test_monorepo_1_channel.py @@ -249,8 +249,7 @@ def test_githubflow_monorepo_rebuild_1_channel( assert curr_release_str in [tag.name for tag in mirror_git_repo.tags] # Make sure publishing actions occurred - assert ( - mocked_git_fetch.call_count == 1 - ) # fetch called to check for remote changes + # fetch called to check for remote changes + assert mocked_git_fetch.call_count == 1 assert mocked_git_push.call_count == 2 # 1 for commit, 1 for tag assert post_mocker.call_count == 1 # vcs release creation occurred diff --git a/tests/e2e/cmd_version/bump_version/github_flow_monorepo/test_monorepo_2_channels.py b/tests/e2e/cmd_version/bump_version/github_flow_monorepo/test_monorepo_2_channels.py index 1352995b9..7c3e36390 100644 --- a/tests/e2e/cmd_version/bump_version/github_flow_monorepo/test_monorepo_2_channels.py +++ b/tests/e2e/cmd_version/bump_version/github_flow_monorepo/test_monorepo_2_channels.py @@ -248,8 +248,7 @@ def test_githubflow_monorepo_rebuild_2_channels( assert curr_release_str in [tag.name for tag in mirror_git_repo.tags] # Make sure publishing actions occurred - assert ( - mocked_git_fetch.call_count == 1 - ) # fetch called to check for remote changes + # fetch called to check for remote changes + assert mocked_git_fetch.call_count == 1 assert mocked_git_push.call_count == 2 # 1 for commit, 1 for tag assert post_mocker.call_count == 1 # vcs release creation occurred diff --git a/tests/e2e/cmd_version/bump_version/trunk_based_dev/test_repo_trunk.py b/tests/e2e/cmd_version/bump_version/trunk_based_dev/test_repo_trunk.py index c57c6d58c..b0f05c41b 100644 --- a/tests/e2e/cmd_version/bump_version/trunk_based_dev/test_repo_trunk.py +++ b/tests/e2e/cmd_version/bump_version/trunk_based_dev/test_repo_trunk.py @@ -165,8 +165,7 @@ def test_trunk_repo_rebuild_only_official_releases( assert expected_release_commit_text == actual_release_commit_text # Make sure tag is created assert curr_release_tag in [tag.name for tag in mirror_git_repo.tags] - assert ( - mocked_git_fetch.call_count == 1 - ) # fetch called to check for remote changes + # fetch called to check for remote changes + assert mocked_git_fetch.call_count == 1 assert mocked_git_push.call_count == 2 # 1 for commit, 1 for tag assert post_mocker.call_count == 1 # vcs release creation occurred diff --git a/tests/e2e/cmd_version/bump_version/trunk_based_dev/test_repo_trunk_dual_version_support.py b/tests/e2e/cmd_version/bump_version/trunk_based_dev/test_repo_trunk_dual_version_support.py index 6ea7e1f54..223e7a1f6 100644 --- a/tests/e2e/cmd_version/bump_version/trunk_based_dev/test_repo_trunk_dual_version_support.py +++ b/tests/e2e/cmd_version/bump_version/trunk_based_dev/test_repo_trunk_dual_version_support.py @@ -171,8 +171,7 @@ def test_trunk_repo_rebuild_dual_version_spt_official_releases_only( assert expected_release_commit_text == actual_release_commit_text # Make sure tag is created assert curr_release_tag in [tag.name for tag in mirror_git_repo.tags] - assert ( - mocked_git_fetch.call_count == 1 - ) # fetch called to check for remote changes + # fetch called to check for remote changes + assert mocked_git_fetch.call_count == 1 assert mocked_git_push.call_count == 2 # 1 for commit, 1 for tag assert post_mocker.call_count == 1 # vcs release creation occurred diff --git a/tests/e2e/cmd_version/bump_version/trunk_based_dev/test_repo_trunk_dual_version_support_w_prereleases.py b/tests/e2e/cmd_version/bump_version/trunk_based_dev/test_repo_trunk_dual_version_support_w_prereleases.py index 89819abe9..2ef42b490 100644 --- a/tests/e2e/cmd_version/bump_version/trunk_based_dev/test_repo_trunk_dual_version_support_w_prereleases.py +++ b/tests/e2e/cmd_version/bump_version/trunk_based_dev/test_repo_trunk_dual_version_support_w_prereleases.py @@ -172,8 +172,7 @@ def test_trunk_repo_rebuild_dual_version_spt_w_official_n_prereleases( assert expected_release_commit_text == actual_release_commit_text # Make sure tag is created assert curr_release_tag in [tag.name for tag in mirror_git_repo.tags] - assert ( - mocked_git_fetch.call_count == 1 - ) # fetch called to check for remote changes + # fetch called to check for remote changes + assert mocked_git_fetch.call_count == 1 assert mocked_git_push.call_count == 2 # 1 for commit, 1 for tag assert post_mocker.call_count == 1 # vcs release creation occurred diff --git a/tests/e2e/cmd_version/bump_version/trunk_based_dev/test_repo_trunk_w_prereleases.py b/tests/e2e/cmd_version/bump_version/trunk_based_dev/test_repo_trunk_w_prereleases.py index 78eca0bd6..5d3f1684f 100644 --- a/tests/e2e/cmd_version/bump_version/trunk_based_dev/test_repo_trunk_w_prereleases.py +++ b/tests/e2e/cmd_version/bump_version/trunk_based_dev/test_repo_trunk_w_prereleases.py @@ -163,8 +163,7 @@ def test_trunk_repo_rebuild_w_prereleases( assert expected_release_commit_text == actual_release_commit_text # Make sure tag is created assert curr_release_tag in [tag.name for tag in mirror_git_repo.tags] - assert ( - mocked_git_fetch.call_count == 1 - ) # fetch called to check for remote changes + # fetch called to check for remote changes + assert mocked_git_fetch.call_count == 1 assert mocked_git_push.call_count == 2 # 1 for commit, 1 for tag assert post_mocker.call_count == 1 # vcs release creation occurred diff --git a/tests/e2e/cmd_version/bump_version/trunk_based_dev_monorepo/test_monorepo_trunk.py b/tests/e2e/cmd_version/bump_version/trunk_based_dev_monorepo/test_monorepo_trunk.py index c68a35f5d..60aae38cb 100644 --- a/tests/e2e/cmd_version/bump_version/trunk_based_dev_monorepo/test_monorepo_trunk.py +++ b/tests/e2e/cmd_version/bump_version/trunk_based_dev_monorepo/test_monorepo_trunk.py @@ -249,8 +249,7 @@ def test_trunk_monorepo_rebuild_1_channel( assert curr_release_str in [tag.name for tag in mirror_git_repo.tags] # Make sure publishing actions occurred - assert ( - mocked_git_fetch.call_count == 1 - ) # fetch called to check for remote changes + # fetch called to check for remote changes + assert mocked_git_fetch.call_count == 1 assert mocked_git_push.call_count == 2 # 1 for commit, 1 for tag assert post_mocker.call_count == 1 # vcs release creation occurred diff --git a/tests/e2e/cmd_version/test_version_build.py b/tests/e2e/cmd_version/test_version_build.py index 40ac74388..068377543 100644 --- a/tests/e2e/cmd_version/test_version_build.py +++ b/tests/e2e/cmd_version/test_version_build.py @@ -128,9 +128,8 @@ def test_version_runs_build_command( ) assert built_wheel_file.exists() - assert ( - mocked_git_fetch.call_count == 1 - ) # fetch called to check for remote changes + # fetch called to check for remote changes + assert mocked_git_fetch.call_count == 1 assert mocked_git_push.call_count == 2 assert post_mocker.call_count == 1 @@ -230,9 +229,8 @@ def test_version_runs_build_command_windows( dist_file_exists = built_wheel_file.exists() assert dist_file_exists, f"\n Expected wheel file to be created at {built_wheel_file}, but it does not exist." - assert ( - mocked_git_fetch.call_count == 1 - ) # fetch called to check for remote changes + # fetch called to check for remote changes + assert mocked_git_fetch.call_count == 1 assert mocked_git_push.call_count == 2 assert post_mocker.call_count == 1