Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 4e701bd

Browse filesBrowse files
committed
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.
1 parent b27a89f commit 4e701bd
Copy full SHA for 4e701bd

File tree

1 file changed

+2
-2
lines changed
Filter options

1 file changed

+2
-2
lines changed

‎test/test_submodule.py

Copy file name to clipboardExpand all lines: test/test_submodule.py
+2-2Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1031,8 +1031,8 @@ def test_branch_renames(self, rw_dir):
10311031
# This doesn't fail as our own submodule binsha didn't change, and the reset is only triggered if
10321032
# to latest revision is True.
10331033
parent_repo.submodule_update(to_latest_revision=False)
1034-
sm_mod.head.ref.name == sm_pfb.name, "should have been switched to past head"
1035-
sm_mod.commit() == sm_fb.commit, "Head wasn't reset"
1034+
assert sm_mod.head.ref.name == sm_pfb.name, "should have been switched to past head"
1035+
assert sm_mod.commit() == sm_fb.commit, "Head wasn't reset"
10361036

10371037
self.assertRaises(RepositoryDirtyError, parent_repo.submodule_update, to_latest_revision=True)
10381038
parent_repo.submodule_update(to_latest_revision=True, force_reset=True)

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.