You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by qi...@apache.org on 2019/04/15 10:35:15 UTC

[mesos] 16/17: Fixed the URI fetcher image fetch test failure on windows.

This is an automated email from the ASF dual-hosted git repository.

qianzhang pushed a commit to branch 1.7.x
in repository https://gitbox.apache.org/repos/asf/mesos.git

commit f8b64a407237ff21f9cce6c87356ca0a1623692b
Author: Gilbert Song <so...@gmail.com>
AuthorDate: Fri Apr 12 17:13:58 2019 +0800

    Fixed the URI fetcher image fetch test failure on windows.
    
    Review: https://reviews.apache.org/r/70398/
---
 src/tests/uri_fetcher_tests.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/tests/uri_fetcher_tests.cpp b/src/tests/uri_fetcher_tests.cpp
index bb224d8..96175f4 100644
--- a/src/tests/uri_fetcher_tests.cpp
+++ b/src/tests/uri_fetcher_tests.cpp
@@ -356,7 +356,8 @@ TEST_F(DockerFetcherPluginTest, INTERNET_CURL_FetchImage)
   EXPECT_EQ("application/vnd.docker.distribution.manifest.v2+json",
             manifest->mediatype());
 
-  EXPECT_TRUE(os::exists(path::join(dir, manifest->config().digest())));
+  EXPECT_TRUE(os::exists(DockerFetcherPlugin::getBlobPath(
+      dir, manifest->config().digest())));
 
   for (int i = 0; i < manifest->layers_size(); i++) {
     EXPECT_TRUE(os::exists(
@@ -393,7 +394,8 @@ TEST_F(DockerFetcherPluginTest, INTERNET_CURL_InvokeFetchByName)
   EXPECT_EQ("application/vnd.docker.distribution.manifest.v2+json",
             manifest->mediatype());
 
-  EXPECT_TRUE(os::exists(path::join(dir, manifest->config().digest())));
+  EXPECT_TRUE(os::exists(DockerFetcherPlugin::getBlobPath(
+      dir, manifest->config().digest())));
 
   for (int i = 0; i < manifest->layers_size(); i++) {
     EXPECT_TRUE(os::exists(