commit 0d107498c64303099e3a86acd08313beb587d31f
author Laszlo Csomor <laszlocsomor@google.com> Wed Mar 13 09:04:27 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Wed Mar 13 09:07:23 2019 -0700
tree 14eebcd6ab9282874b96e217c38a7539db30c9ce
parent 20f0345824ead79a2aebed72d0bae82f2578328b

Bazel client: add and use blaze::GetPathEnv()

Use GetPathEnv() instead of GetEnv() for envvars
with paths.

On Linux/macOS/POSIX, GetPathEnv and GetEnv do the
same.

On Windows, GetPathEnv removes the UNC prefix from
the result, calls AsWindowsPath, then converts
backslashes to forward slashes. (As callers expect
the result.)

Fixes https://github.com/bazelbuild/bazel/issues/7705

Closes #7707.

PiperOrigin-RevId: 238236143

src/test/cpp/workspace_layout_test.cc

diff --git a/src/test/cpp/workspace_layout_test.cc b/src/test/cpp/workspace_layout_test.cc
index 4e17d64..865005a 100644
--- a/src/test/cpp/workspace_layout_test.cc
+++ b/src/test/cpp/workspace_layout_test.cc
@@ -27,10 +27,10 @@
 
 class WorkspaceLayoutTest : public ::testing::Test {
  protected:
-  WorkspaceLayoutTest() :
-      build_root_(blaze_util::JoinPath(
-          blaze::GetEnv("TEST_TMPDIR"), "build_root")),
-      workspace_layout_(new WorkspaceLayout()) {}
+  WorkspaceLayoutTest()
+      : build_root_(blaze_util::JoinPath(blaze::GetPathEnv("TEST_TMPDIR"),
+                                         "build_root")),
+        workspace_layout_(new WorkspaceLayout()) {}
 
   void SetUp() override {
     ASSERT_TRUE(blaze_util::MakeDirectories(build_root_, 0755));