Fix non-determinism in skylark_repository_test.sh.

The test_skylark_repository_which_and_execute test would fail when run alone or when sharding puts it into a different group of tests, because it tried to run "bazel info" before it actually setup its test workspace. This only worked, because the files would have been created (and not cleaned up) by an earlier test.

PiperOrigin-RevId: 222393468

src/test/shell/bazel/skylark_repository_test.sh

@@ -367,8 +367,6 @@ EOF
 function test_skylark_repository_which_and_execute() {
   setup_skylark_repository
 
-  # Test we are using the client environment, not the server one
-  bazel info &> /dev/null  # Start up the server.
   echo "#!/bin/sh" > bin.sh
   echo "exit 0" >> bin.sh
   chmod +x bin.sh
@@ -393,6 +391,9 @@ def _impl(repository_ctx):
 repo = repository_rule(implementation=_impl, local=True)
 EOF
 
+  # Test we are using the client environment, not the server one
+  bazel info &> /dev/null  # Start up the server.
+
   FOO="BAZ" PATH="${PATH}:${PWD}" bazel build @foo//:bar >& $TEST_log \
       || fail "Failed to build"
   expect_log "version"