diff options
Diffstat (limited to 'git/test/test_diff.py')
| -rw-r--r-- | git/test/test_diff.py | 24 | 
1 files changed, 11 insertions, 13 deletions
diff --git a/git/test/test_diff.py b/git/test/test_diff.py index 80652c8e..a59ee0bf 100644 --- a/git/test/test_diff.py +++ b/git/test/test_diff.py @@ -8,7 +8,7 @@ from git.test.lib import *  from git import *  class TestDiff(TestBase): -             +      def _assert_diff_format(self, diffs):          # verify that the format of the diff is sane          for diff in diffs: @@ -16,19 +16,19 @@ class TestDiff(TestBase):                  assert isinstance(diff.a_mode, int)              if diff.b_mode:                  assert isinstance(diff.b_mode, int) -                 +              if diff.a_blob:                  assert not diff.a_blob.path.endswith('\n')              if diff.b_blob:                  assert not diff.b_blob.path.endswith('\n')          # END for each diff          return diffs -     +      def test_list_from_string_new_mode(self):          output = StringProcessAdapter(fixture('diff_new_mode'))          diffs = Diff._index_from_patch_format(self.rorepo, output.stdout)          self._assert_diff_format(diffs) -         +          assert_equal(1, len(diffs))          assert_equal(10, len(diffs[0].diff.splitlines())) @@ -36,7 +36,7 @@ class TestDiff(TestBase):          output = StringProcessAdapter(fixture('diff_rename'))          diffs = Diff._index_from_patch_format(self.rorepo, output.stdout)          self._assert_diff_format(diffs) -         +          assert_equal(1, len(diffs))          diff = diffs[0] @@ -50,7 +50,7 @@ class TestDiff(TestBase):          fixtures = ("diff_2", "diff_2f", "diff_f", "diff_i", "diff_mode_only",                       "diff_new_mode", "diff_numstat", "diff_p", "diff_rename",                       "diff_tree_numstat_root" ) -         +          for fixture_name in fixtures:              diff_proc = StringProcessAdapter(fixture(fixture_name))              diffs = Diff._index_from_patch_format(self.rorepo, diff_proc.stdout) @@ -64,13 +64,13 @@ class TestDiff(TestBase):              if i%2 == 0:                  diff_item = commit.tree              # END use tree every second item -             +              for other in (None, commit.Index, commit.parents[0]):                  for paths in (None, "CHANGES", ("CHANGES", "lib")):                      for create_patch in range(2):                          diff_index = diff_item.diff(other, paths, create_patch)                          assert isinstance(diff_index, DiffIndex) -                         +                          if diff_index:                              self._assert_diff_format(diff_index)                              for ct in DiffIndex.change_type: @@ -78,7 +78,7 @@ class TestDiff(TestBase):                                  assertion_map.setdefault(key, 0)                                  assertion_map[key] = assertion_map[key]+len(list(diff_index.iter_change_type(ct)))                                # END for each changetype -                             +                              # check entries                              diff_set = set()                              diff_set.add(diff_index[0]) @@ -91,18 +91,16 @@ class TestDiff(TestBase):                  # END for each path option              # END for each other side          # END for each commit -         +          # assert we could always find at least one instance of the members we           # can iterate in the diff index - if not this indicates its not working correctly          # or our test does not span the whole range of possibilities          for key,value in assertion_map.items():              assert value, "Did not find diff for %s" % key          # END for each iteration type  -         +          # test path not existing in the index - should be ignored          c = self.rorepo.head.commit          cp = c.parents[0]          diff_index = c.diff(cp, ["does/not/exist"])          assert len(diff_index) == 0 -         -      | 
