test: trivial cleanups

diff --git a/t/t7409-submodule-detached-worktree.sh b/t/t7409-submodule-detached-worktree.sh
old mode 100644 (file)
new mode 100755 (executable)
index db75642..c207171
--- a/t/t7409-submodule-detached-worktree.sh
+++ b/t/t7409-submodule-detached-worktree.sh
@@ -15,22 +15,43 @@ TEST_NO_CREATE_REPO=1
 test_expect_success 'submodule on detached working tree' '
        git init --bare remote &&
        test_create_repo bundle1 &&
-       (cd bundle1 && test_commit "shoot") &&
+       (
+               cd bundle1 &&
+               test_commit "shoot" &&
+               git rev-parse --verify HEAD >../expect
+       ) &&
        mkdir home &&
        (
                cd home &&
-               export GIT_WORK_TREE="$(pwd)" GIT_DIR="$(pwd)/.dotfiles" &&
+               GIT_WORK_TREE="$(pwd)" &&
+               GIT_DIR="$(pwd)/.dotfiles" &&
+               export GIT_WORK_TREE GIT_DIR &&
                git clone --bare ../remote .dotfiles &&
                git submodule add ../bundle1 .vim/bundle/sogood &&
                test_commit "sogood" &&
+               (
+                       unset GIT_WORK_TREE GIT_DIR &&
+                       cd .vim/bundle/sogood &&
+                       git rev-parse --verify HEAD >actual &&
+                       test_cmp ../../../../expect actual
+               ) &&
                git push origin master
        ) &&
        mkdir home2 &&
        (
                cd home2 &&
-               export GIT_WORK_TREE="$(pwd)" GIT_DIR="$(pwd)/.dotfiles" &&
                git clone --bare ../remote .dotfiles &&
-               git submodule update --init
+               GIT_WORK_TREE="$(pwd)" &&
+               GIT_DIR="$(pwd)/.dotfiles" &&
+               export GIT_WORK_TREE GIT_DIR &&
+               git checkout master &&
+               git submodule update --init &&
+               (
+                       unset GIT_WORK_TREE GIT_DIR &&
+                       cd .vim/bundle/sogood &&
+                       git rev-parse --verify HEAD >actual &&
+                       test_cmp ../../../../expect actual
+               )
        )
 '
 
@@ -38,23 +59,25 @@ test_expect_success 'submodule on detached working pointed by core.worktree' '
        mkdir home3 &&
        (
                cd home3 &&
-               export GIT_DIR="$(pwd)/.dotfiles" &&
+               GIT_DIR="$(pwd)/.dotfiles" &&
+               export GIT_DIR &&
                git clone --bare ../remote "$GIT_DIR" &&
                git config core.bare false &&
                git config core.worktree .. &&
+               git checkout master &&
                git submodule add ../bundle1 .vim/bundle/dupe &&
                test_commit "dupe" &&
                git push origin master
        ) &&
        (
                cd home &&
-               export GIT_DIR="$(pwd)/.dotfiles" &&
+               GIT_DIR="$(pwd)/.dotfiles" &&
+               export GIT_DIR &&
                git config core.bare false &&
                git config core.worktree .. &&
                git pull &&
-               git submodule update &&
-               git submodule status &&
-               test -d .vim/bundle/dupe
+               git submodule update --init &&
+               test -f .vim/bundle/dupe/shoot.t
        )
 '