Content-Length: 1419 | pFad | http://github.com/gitpython-developers/GitPython/pull/1678.patch

thub.com From 4e701bdae829bb02abd7f9acf5e6508e242b6977 Mon Sep 17 00:00:00 2001 From: Eliah Kagan Date: Mon, 25 Sep 2023 05:54:47 -0400 Subject: [PATCH] Add missing assert keywords This turns two tuple expression statements--each of an equality comparison and message--that were not being checked or otherwise used, and that were intended to be assertions, into assertions. --- test/test_submodule.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/test_submodule.py b/test/test_submodule.py index 4a9c9c582..0aa80e5ce 100644 --- a/test/test_submodule.py +++ b/test/test_submodule.py @@ -1031,8 +1031,8 @@ def test_branch_renames(self, rw_dir): # This doesn't fail as our own submodule binsha didn't change, and the reset is only triggered if # to latest revision is True. parent_repo.submodule_update(to_latest_revision=False) - sm_mod.head.ref.name == sm_pfb.name, "should have been switched to past head" - sm_mod.commit() == sm_fb.commit, "Head wasn't reset" + assert sm_mod.head.ref.name == sm_pfb.name, "should have been switched to past head" + assert sm_mod.commit() == sm_fb.commit, "Head wasn't reset" self.assertRaises(RepositoryDirtyError, parent_repo.submodule_update, to_latest_revision=True) parent_repo.submodule_update(to_latest_revision=True, force_reset=True)

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier! Saves Data!