From 6ab4e16b89aca166fdee85628e264eb6ca5288af Mon Sep 17 00:00:00 2001 From: Laszlo Csomor Date: Mon, 13 Aug 2018 08:31:31 -0700 Subject: Windows,tests: port cpp_test //src/test/shell/integration:cpp_test now runs on Windows. See https://github.com/bazelbuild/bazel/issues/4292 Change-Id: I7dd4ce2b5702e5853ee357ab4f2a728af7d3018d Closes #5874. Change-Id: I7dd4ce2b5702e5853ee357ab4f2a728af7d3018d PiperOrigin-RevId: 208484274 --- src/test/shell/integration/BUILD | 6 +- src/test/shell/integration/cpp_test.sh | 104 ++++++++++++++++++++++----------- 2 files changed, 73 insertions(+), 37 deletions(-) diff --git a/src/test/shell/integration/BUILD b/src/test/shell/integration/BUILD index 4717180d16..66e8eb3688 100644 --- a/src/test/shell/integration/BUILD +++ b/src/test/shell/integration/BUILD @@ -270,8 +270,10 @@ sh_test( name = "cpp_test", size = "large", srcs = ["cpp_test.sh"], - data = [":test-deps"], - tags = ["no_windows"], + data = [ + ":test-deps", + "@bazel_tools//tools/bash/runfiles", + ], ) sh_test( diff --git a/src/test/shell/integration/cpp_test.sh b/src/test/shell/integration/cpp_test.sh index bb7a8f693a..7c777e7eac 100755 --- a/src/test/shell/integration/cpp_test.sh +++ b/src/test/shell/integration/cpp_test.sh @@ -13,91 +13,125 @@ # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. # See the License for the specific language governing permissions and # limitations under the License. -# -# Load the test setup defined in the parent directory -CURRENT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)" -source "${CURRENT_DIR}/../integration_test_setup.sh" \ + +# --- begin runfiles.bash initialization --- +# Copy-pasted from Bazel's Bash runfiles library (tools/bash/runfiles/runfiles.bash). +set -euo pipefail +if [[ ! -d "${RUNFILES_DIR:-/dev/null}" && ! -f "${RUNFILES_MANIFEST_FILE:-/dev/null}" ]]; then + if [[ -f "$0.runfiles_manifest" ]]; then + export RUNFILES_MANIFEST_FILE="$0.runfiles_manifest" + elif [[ -f "$0.runfiles/MANIFEST" ]]; then + export RUNFILES_MANIFEST_FILE="$0.runfiles/MANIFEST" + elif [[ -f "$0.runfiles/bazel_tools/tools/bash/runfiles/runfiles.bash" ]]; then + export RUNFILES_DIR="$0.runfiles" + fi +fi +if [[ -f "${RUNFILES_DIR:-/dev/null}/bazel_tools/tools/bash/runfiles/runfiles.bash" ]]; then + source "${RUNFILES_DIR}/bazel_tools/tools/bash/runfiles/runfiles.bash" +elif [[ -f "${RUNFILES_MANIFEST_FILE:-/dev/null}" ]]; then + source "$(grep -m1 "^bazel_tools/tools/bash/runfiles/runfiles.bash " \ + "$RUNFILES_MANIFEST_FILE" | cut -d ' ' -f 2-)" +else + echo >&2 "ERROR: cannot find @bazel_tools//tools/bash/runfiles:runfiles.bash" + exit 1 +fi +# --- end runfiles.bash initialization --- + +source "$(rlocation "io_bazel/src/test/shell/integration_test_setup.sh")" \ || { echo "integration_test_setup.sh not found!" >&2; exit 1; } -#### SETUP ############################################################# +case "$(uname -s | tr [:upper:] [:lower:])" in +msys*|mingw*|cygwin*) + declare -r is_windows=true + ;; +*) + declare -r is_windows=false + ;; +esac -set -e +if "$is_windows"; then + export MSYS_NO_PATHCONV=1 + export MSYS2_ARG_CONV_EXCL="*" +fi #### TESTS ############################################################# function test_no_rebuild_on_irrelevant_header_change() { - mkdir -p a - cat > a/BUILD < $pkg/BUILD < a/a.cc < $pkg/a.cc < a/b1.h < $pkg/b1.h < a/b2.h < $pkg/b2.h < a/b2.h - bazel build //a >& $TEST_log || fail "build failed" - expect_not_log "Compiling a/a.cc" + bazel build //$pkg:a || fail "build failed" + echo "CHERRY" > $pkg/b2.h + bazel build //$pkg:a >& $TEST_log || fail "build failed" + expect_not_log "Compiling $pkg/a.cc" } function test_new_header_is_required() { - mkdir -p a - cat > a/BUILD < $pkg/BUILD < a/a.cc << EOF -#include "a/b1.h" + cat > $pkg/a.cc << EOF +#include "$pkg/b1.h" int main(void) { return B1; } EOF - cat > a/b1.h < $pkg/b1.h < a/b2.h < $pkg/b2.h < a/a.cc << EOF -#include "a/b1.h" -#include "a/b2.h" + bazel build //$pkg:a || fail "build failed" + cat > $pkg/a.cc << EOF +#include "$pkg/b1.h" +#include "$pkg/b2.h" int main(void) { return B1 + B2; } EOF - bazel build //a || fail "build failled" + bazel build //$pkg:a || fail "build failled" } function test_no_recompile_on_shutdown() { - mkdir -p a - cat > a/BUILD < $pkg/BUILD < a/a.cc < $pkg/a.cc < a/b.h < $pkg/b.h <& $TEST_log || fail "build failed" - expect_log "Compiling a/a.cc" + bazel build -s //$pkg:a >& $TEST_log || fail "build failed" + expect_log "Compiling $pkg/a.cc" bazel shutdown >& /dev/null || fail "query failed" - bazel build -s //a >& $TEST_log || fail "build failed" - expect_not_log "Compiling a/a.cc" + bazel build -s //$pkg:a >& $TEST_log || fail "build failed" + expect_not_log "Compiling $pkg/a.cc" } run_suite "Tests for Bazel's C++ rules" -- cgit v1.2.3