Re: [PATCH] submodule: check for NULL return of get_submodule_ref_store()

On Wed, Mar 28, 2018 at 11:57 AM, Eric Sunshine [off-list ref] wrote:

On Wed, Mar 28, 2018 at 2:38 PM, Stefan Beller [off-list ref] wrote:

quoted

diff --git a/t/t7400-submodule-basic.sh b/t/t7400-submodule-basic.sh

@@ -821,6 +821,18 @@ test_expect_success 'moving the superproject does not break submodules' '
+test_expect_success 'moving the submodule does not break the superproject' '
+       (
+               cd addtest2 &&
+
+               mv repo repo.bak &&
+               git submodule status >actual &&
+               grep -e "^-" -e repo actual &&
+
+               mv repo.bak repo

Should this "move back" be encapsulated in a test_when_finished?

I thought about that, but decided against it for some reason as I was debating
where to put the test_when_finished. I mostly saw those at the very beginning
of a test and wondered if it can be called from within a subshell.
(I'd not want to put it at the beginning but rather adjacent to the move.)