Merge branch 'js/t6026-clean-up'
authorJunio C Hamano <gitster@pobox.com>
Mon, 12 Sep 2016 22:34:37 +0000 (15:34 -0700)
committerJunio C Hamano <gitster@pobox.com>
Mon, 12 Sep 2016 22:34:37 +0000 (15:34 -0700)
A test spawned a short-lived background process, which sometimes
prevented the test directory from getting removed at the end of the
script on some platforms.

* js/t6026-clean-up:
t6026-merge-attr: clean up background process at end of test case

t/t6026-merge-attr.sh
index dd8f88d18782f15e35aa3105668d9a713ba9fe3b..7a6e33e6732eaf69e298b56729ba4ead9ffaee0e 100755 (executable)
@@ -185,7 +185,9 @@ test_expect_success 'custom merge does not lock index' '
        git reset --hard anchor &&
        write_script sleep-one-second.sh <<-\EOF &&
                sleep 1 &
+               echo $! >sleep.pid
        EOF
+       test_when_finished "kill \$(cat sleep.pid)" &&
 
        test_write_lines >.gitattributes \
                "* merge=ours" "text merge=sleep-one-second" &&