Skip to content

Commit

Permalink
Merge pull request #1678 from EliahKagan/missing-asserts
Browse files Browse the repository at this point in the history
Add missing assert keywords
  • Loading branch information
Byron authored Sep 25, 2023
2 parents b27a89f + 4e701bd commit abd445f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/test_submodule.py
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down

0 comments on commit abd445f

Please sign in to comment.