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 abd445f

Browse filesBrowse files
authored
Merge pull request #1678 from EliahKagan/missing-asserts
Add missing assert keywords
2 parents b27a89f + 4e701bd commit abd445f
Copy full SHA for abd445f

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.