Skip to content
This repository was archived by the owner on Dec 8, 2021. It is now read-only.

Commit 9876b58

Browse files
authored
test: drop executable requirement for INTEGRATION_TEST_CONFIG (#1099)
we only `source` that script so it doesn't need to be executable, only readable.
1 parent 8d97487 commit 9876b58

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

ci/kokoro/docker/build-in-docker-bazel.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ readonly INTEGRATION_TESTS_CONFIG="/c/spanner-integration-tests-config.sh"
7676
# auto: only try to run integration tests if the config file is executable.
7777
if [[ "${RUN_INTEGRATION_TESTS}" == "yes" || \
7878
( "${RUN_INTEGRATION_TESTS}" == "auto" && \
79-
-x "${INTEGRATION_TESTS_CONFIG}" ) ]]; then
79+
-r "${INTEGRATION_TESTS_CONFIG}" ) ]]; then
8080
echo "================================================================"
8181
echo "Running the integration tests $(date)"
8282
echo "================================================================"

ci/kokoro/docker/build-in-docker-cmake.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ fi
9797
readonly INTEGRATION_TESTS_CONFIG="/c/spanner-integration-tests-config.sh"
9898
if [[ "${RUN_INTEGRATION_TESTS}" == "yes" || \
9999
( "${RUN_INTEGRATION_TESTS}" == "auto" && \
100-
-x "${INTEGRATION_TESTS_CONFIG}" ) ]]; then
100+
-r "${INTEGRATION_TESTS_CONFIG}" ) ]]; then
101101
echo "================================================================"
102102
echo "Running the integration tests $(date)"
103103
echo "================================================================"

0 commit comments

Comments
 (0)