commit 1e45eb3e41e7c742c68ef4c717135e44b681b7dd
author Yun Peng <pcloudy@google.com> Fri Jul 21 09:50:21 2017 +0200
committer Yun Peng <pcloudy@google.com> Fri Jul 21 10:30:09 2017 +0000
tree a0488434517a323d9e25e9025eaa454e57647947
parent a2d53a6f813315301c5f9c4ec067e6daf95cc865

Add test env for TEMP environment variable

A workaround for ASSERT_DEATH on Windows

Change-Id: I379bd873e6219167da03a0240663bb457225ec65

jenkins/jobs/BUILD

diff --git a/jenkins/jobs/BUILD b/jenkins/jobs/BUILD
index e4e06c5..6073f48 100644
--- a/jenkins/jobs/BUILD
+++ b/jenkins/jobs/BUILD
@@ -179,6 +179,11 @@
         # TODO(pcloudy): This is only for Windows.
         # Remove it after wrapper-less CROSSTOOL becomes default
         "--action_env=NO_MSVC_WRAPPER=1",
+        # TODO(pcloudy): Remove this after TEMP is properly set.
+        # A workaround for ASSERT_DEATH in gtest on Windows.
+        # When running tests, Bazel doesn't set TEMP,
+        # ASSERT_DEATH will try to write to C:/Windows, then fails.
+        "--test_env=TEMP",
     ],
     config = ":configs/bazel-tests.json",
     configure = [

jenkins/jobs/configs/bazel-tests.json

diff --git a/jenkins/jobs/configs/bazel-tests.json b/jenkins/jobs/configs/bazel-tests.json
index 780368f..6f859f9 100644
--- a/jenkins/jobs/configs/bazel-tests.json
+++ b/jenkins/jobs/configs/bazel-tests.json
@@ -86,6 +86,13 @@
                 // Remove it after wrapper-less CROSSTOOL becomes default
                 "--action_env=NO_MSVC_WRAPPER=1"
             ],
+            "test_opts": [
+                // TODO(pcloudy): Remove this after TEMP is properly set.
+                // A workaround for ASSERT_DEATH in gtest on Windows.
+                // When running tests, Bazel doesn't set TEMP,
+                // ASSERT_DEATH will try to write to C:/Windows, then fails.
+                "--test_env=TEMP",
+            ],
             "test_tag_filters": ["-no_windows"],
             "tests": [
                 "//src/test/py/...",