]> git.ipfire.org Git - thirdparty/git.git/commitdiff
Merge branch 'sg/t4051-fix'
authorJunio C Hamano <gitster@pobox.com>
Mon, 27 Aug 2018 21:33:45 +0000 (14:33 -0700)
committerJunio C Hamano <gitster@pobox.com>
Mon, 27 Aug 2018 21:33:45 +0000 (14:33 -0700)
Test fixes.

* sg/t4051-fix:
  t4051-diff-function-context: read the right file

1  2 
t/t4051-diff-function-context.sh

index 2d76a971c43f6850a1f443eab3e91bf3762fe952,3df24f2ef28a3af98ed703fc2cb0fdcb35ff8a91..4838a1df8b4369dc5024cdd7929d851b76482805
@@@ -64,31 -64,11 +64,31 @@@ test_expect_success 'setup' 
  
        grep -v "Begin of second part" <file.c >file.c.new &&
        mv file.c.new file.c &&
 -      commit_and_tag long_common_tail file.c
 +      commit_and_tag long_common_tail file.c &&
 +
 +      git checkout initial &&
 +      cat "$dir/hello.c" "$dir/dummy.c" >file.c &&
 +      commit_and_tag hello_dummy file.c &&
 +
 +      # overlap function context of 1st change and -u context of 2nd change
 +      grep -v "delete me from hello" <"$dir/hello.c" >file.c &&
 +      sed "2a\\
 +           extra line" <"$dir/dummy.c" >>file.c &&
 +      commit_and_tag changed_hello_dummy file.c &&
 +
 +      git checkout initial &&
 +      grep -v "delete me from hello" <file.c >file.c.new &&
 +      mv file.c.new file.c &&
 +      cat "$dir/appended1.c" >>file.c &&
 +      commit_and_tag changed_hello_appended file.c
  '
  
  check_diff changed_hello 'changed function'
  
 +test_expect_success ' context includes comment' '
 +      grep "^ .*Hello comment" changed_hello.diff
 +'
 +
  test_expect_success ' context includes begin' '
        grep "^ .*Begin of hello" changed_hello.diff
  '
@@@ -105,7 -85,7 +105,7 @@@ test_expect_success ' context does not 
        test "$(first_context_line <changed_hello.diff)" != " "
  '
  
 -test_expect_failure ' context does not include trailing empty lines' '
 +test_expect_success ' context does not include trailing empty lines' '
        test "$(last_context_line <changed_hello.diff)" != " "
  '
  
@@@ -123,7 -103,7 +123,7 @@@ test_expect_success ' context does not 
        test $(grep -c "^[ +-].*Begin" changed_includes.diff) -le 1
  '
  
 -test_expect_failure ' context does not include trailing empty lines' '
 +test_expect_success ' context does not include trailing empty lines' '
        test "$(last_context_line <changed_includes.diff)" != " "
  '
  
@@@ -137,7 -117,7 +137,7 @@@ test_expect_success ' context includes 
        grep "^[+].*End of first part" appended.diff
  '
  
 -test_expect_failure ' context does not include other functions' '
 +test_expect_success ' context does not include other functions' '
        test $(grep -c "^[ +-].*Begin" appended.diff) -le 1
  '
  
@@@ -151,7 -131,7 +151,7 @@@ test_expect_success ' context includes 
        grep "^[+].*End of second part" extended.diff
  '
  
 -test_expect_failure ' context does not include other functions' '
 +test_expect_success ' context does not include other functions' '
        test $(grep -c "^[ +-].*Begin" extended.diff) -le 2
  '
  
@@@ -165,7 -145,7 +165,7 @@@ test_expect_success ' context includes 
        grep "^ .*Begin of first part" long_common_tail.diff
  '
  
 -test_expect_failure ' context includes end' '
 +test_expect_success ' context includes end' '
        grep "^ .*End of second part" long_common_tail.diff
  '
  
@@@ -174,39 -154,7 +174,39 @@@ test_expect_success ' context does not 
  '
  
  test_expect_success ' context does not include preceding empty lines' '
-       test "$(first_context_line <long_common_tail.diff.diff)" != " "
+       test "$(first_context_line <long_common_tail.diff)" != " "
  '
  
 +check_diff changed_hello_appended 'changed function plus appended function'
 +
 +test_expect_success ' context includes begin' '
 +      grep "^ .*Begin of hello" changed_hello_appended.diff &&
 +      grep "^[+].*Begin of first part" changed_hello_appended.diff
 +'
 +
 +test_expect_success ' context includes end' '
 +      grep "^ .*End of hello" changed_hello_appended.diff &&
 +      grep "^[+].*End of first part" changed_hello_appended.diff
 +'
 +
 +test_expect_success ' context does not include other functions' '
 +      test $(grep -c "^[ +-].*Begin" changed_hello_appended.diff) -le 2
 +'
 +
 +check_diff changed_hello_dummy 'changed two consecutive functions'
 +
 +test_expect_success ' context includes begin' '
 +      grep "^ .*Begin of hello" changed_hello_dummy.diff &&
 +      grep "^ .*Begin of dummy" changed_hello_dummy.diff
 +'
 +
 +test_expect_success ' context includes end' '
 +      grep "^ .*End of hello" changed_hello_dummy.diff &&
 +      grep "^ .*End of dummy" changed_hello_dummy.diff
 +'
 +
 +test_expect_success ' overlapping hunks are merged' '
 +      test $(grep -c "^@@" changed_hello_dummy.diff) -eq 1
 +'
 +
  test_done