diff options
| author | Russell Belfer <rb@github.com> | 2013-04-10 01:52:32 +0400 |
|---|---|---|
| committer | Russell Belfer <rb@github.com> | 2013-04-10 01:52:32 +0400 |
| commit | ad26434b3b8a5eafab8ec52b83aa99beaf48fb03 (patch) | |
| tree | ad5caff8bd3574a31040beaf1739f3ba3b6c8d9e /tests-clar/status | |
| parent | 9da187e83d1b8ab513a43fd54a9fe2be11b1703f (diff) | |
Tests and more fixes for submodule diffs
This adds tests for diffs with submodules in them and (perhaps
unsurprisingly) requires further fixes to be made. Specifically,
this fixes:
- when considering if a submodule is dirty in the workdir, it was
being treated as dirty even if only the index was dirty.
- git_diff_patch_to_str (and git_diff_patch_print) were "printing"
the headers for files (and submodules) that were unmodified or
had no meaningful content.
- added comment to previous fix and removed unneeded parens.
Diffstat (limited to 'tests-clar/status')
| -rw-r--r-- | tests-clar/status/submodules.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tests-clar/status/submodules.c b/tests-clar/status/submodules.c index 4656a87e3..8365a7f5a 100644 --- a/tests-clar/status/submodules.c +++ b/tests-clar/status/submodules.c @@ -165,7 +165,7 @@ void test_status_submodules__moved_head(void) cl_git_pass( git_status_foreach_ext(g_repo, &opts, cb_status__match, &counts)); cl_assert_equal_i(6, counts.entry_count); - + git_repository_free(smrepo); } @@ -219,3 +219,4 @@ void test_status_submodules__dirty_workdir_only(void) git_status_foreach_ext(g_repo, &opts, cb_status__match, &counts)); cl_assert_equal_i(6, counts.entry_count); } + |