testsuite: fix non-bourne shell test failure

chu11 · chu11 · commit 31014827b286 · 2022-09-02T16:15:52.000-07:00
Problem: Test in t2802-uri-cmd failed because of quoting issues with non-bourne shells. Test was recently broken due to change in: testsuite: make sure test ssh runs $SHELL c6a760c Solution: Set SHELL to /bin/sh in test.
diff --git a/t/t2802-uri-cmd.t b/t/t2802-uri-cmd.t
@@ -162,7 +162,7 @@ test_expect_success 'setup fake csm_allocation_query for mock lsf testing' '
 	export FLUX_SSH=${SHARNESS_TEST_SRCDIR}/scripts/tssh
 '
 test_expect_success 'flux-uri mock testing of lsf resolver works' '
-	result=$(flux uri --local lsf:12345) &&
+	result=$(SHELL=/bin/sh flux uri --local lsf:12345) &&
 	test "$result" = "$FLUX_URI"
 '
 test_expect_success 'cleanup jobs' '

Original file line number	Diff line number	Diff line change
`@@ -162,7 +162,7 @@ test_expect_success 'setup fake csm_allocation_query for mock lsf testing' '`
`162`	`162`	`export FLUX_SSH=${SHARNESS_TEST_SRCDIR}/scripts/tssh`
`163`	`163`	`'`
`164`	`164`	`test_expect_success 'flux-uri mock testing of lsf resolver works' '`
`165`		`- result=$(flux uri --local lsf:12345) &&`
	`165`	`+ result=$(SHELL=/bin/sh flux uri --local lsf:12345) &&`
`166`	`166`	`test "$result" = "$FLUX_URI"`
`167`	`167`	`'`
`168`	`168`	`test_expect_success 'cleanup jobs' '`