This is an automated email from the ASF dual-hosted git repository. qianzhang pushed a commit to branch 1.6.x in repository https://gitbox.apache.org/repos/asf/mesos.git
commit 3017a79d9a9918c7ee79b91447e9ac023091eb89 Author: Gilbert Song <[email protected]> AuthorDate: Wed Mar 27 12:03:00 2019 -0700 Fixed docker fetcher plugin unit test for v2s2 change. Review: https://reviews.apache.org/r/70290 --- src/tests/uri_fetcher_tests.cpp | 60 ++++++++++++++++++----------------------- 1 file changed, 26 insertions(+), 34 deletions(-) diff --git a/src/tests/uri_fetcher_tests.cpp b/src/tests/uri_fetcher_tests.cpp index 4b3027d..44f1cc9 100644 --- a/src/tests/uri_fetcher_tests.cpp +++ b/src/tests/uri_fetcher_tests.cpp @@ -302,29 +302,17 @@ TEST_F(DockerFetcherPluginTest, INTERNET_CURL_FetchManifest) AWAIT_READY_FOR(fetcher.get()->fetch(uri, dir), Seconds(60)); - // Version 2 schema 1 image manifest test - Try<string> _s1Manifest = os::read(path::join(dir, "manifest")); - ASSERT_SOME(_s1Manifest); - - Try<docker::spec::v2::ImageManifest> s1Manifest = - docker::spec::v2::parse(_s1Manifest.get()); - - ASSERT_SOME(s1Manifest); - EXPECT_EQ(1u, s1Manifest->schemaversion()); - EXPECT_EQ(TEST_REPOSITORY, s1Manifest->name()); - EXPECT_EQ("latest", s1Manifest->tag()); - -#ifdef __WINDOWS__ // Version 2 schema 2 image manifest test - Try<string> _s2Manifest = os::read(path::join(dir, "manifest_v2s2")); - ASSERT_SOME(_s2Manifest); + Try<string> _manifest = os::read(path::join(dir, "manifest")); + ASSERT_SOME(_manifest); - Try<docker::spec::v2_2::ImageManifest> s2Manifest = - docker::spec::v2_2::parse(_s2Manifest.get()); + Try<docker::spec::v2_2::ImageManifest> manifest = + docker::spec::v2_2::parse(_manifest.get()); - ASSERT_SOME(s2Manifest); - EXPECT_EQ(2u, s2Manifest->schemaversion()); -#endif // __WINDOWS__ + ASSERT_SOME(manifest); + EXPECT_EQ(2u, manifest->schemaversion()); + EXPECT_EQ("application/vnd.docker.distribution.manifest.v2+json", + manifest->mediatype()); } @@ -360,19 +348,21 @@ TEST_F(DockerFetcherPluginTest, INTERNET_CURL_FetchImage) Try<string> _manifest = os::read(path::join(dir, "manifest")); ASSERT_SOME(_manifest); - Try<docker::spec::v2::ImageManifest> manifest = - docker::spec::v2::parse(_manifest.get()); + Try<docker::spec::v2_2::ImageManifest> manifest = + docker::spec::v2_2::parse(_manifest.get()); ASSERT_SOME(manifest); - EXPECT_EQ(1u, manifest->schemaversion()); - EXPECT_EQ(TEST_REPOSITORY, manifest->name()); - EXPECT_EQ("latest", manifest->tag()); + EXPECT_EQ(2u, manifest->schemaversion()); + EXPECT_EQ("application/vnd.docker.distribution.manifest.v2+json", + manifest->mediatype()); - for (int i = 0; i < manifest->fslayers_size(); i++) { + EXPECT_TRUE(os::exists(path::join(dir, manifest->config().digest()))); + + for (int i = 0; i < manifest->layers_size(); i++) { EXPECT_TRUE(os::exists( DockerFetcherPlugin::getBlobPath( dir, - manifest->fslayers(i).blobsum()))); + manifest->layers(i).digest()))); } } @@ -395,19 +385,21 @@ TEST_F(DockerFetcherPluginTest, INTERNET_CURL_InvokeFetchByName) Try<string> _manifest = os::read(path::join(dir, "manifest")); ASSERT_SOME(_manifest); - Try<docker::spec::v2::ImageManifest> manifest = - docker::spec::v2::parse(_manifest.get()); + Try<docker::spec::v2_2::ImageManifest> manifest = + docker::spec::v2_2::parse(_manifest.get()); ASSERT_SOME(manifest); - EXPECT_EQ(1u, manifest->schemaversion()); - EXPECT_EQ(TEST_REPOSITORY, manifest->name()); - EXPECT_EQ("latest", manifest->tag()); + EXPECT_EQ(2u, manifest->schemaversion()); + EXPECT_EQ("application/vnd.docker.distribution.manifest.v2+json", + manifest->mediatype()); + + EXPECT_TRUE(os::exists(path::join(dir, manifest->config().digest()))); - for (int i = 0; i < manifest->fslayers_size(); i++) { + for (int i = 0; i < manifest->layers_size(); i++) { EXPECT_TRUE(os::exists( DockerFetcherPlugin::getBlobPath( dir, - manifest->fslayers(i).blobsum()))); + manifest->layers(i).digest()))); } }
