You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by ji...@apache.org on 2019/01/12 00:35:30 UTC

[mesos] 06/09: Fixed a bug in docker_containerizer_tests.cpp.

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

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

commit 787d0298276d4484d4dcfe2024f29d810c8a0e56
Author: Jie Yu <yu...@gmail.com>
AuthorDate: Thu Jan 10 21:41:12 2019 -0800

    Fixed a bug in docker_containerizer_tests.cpp.
    
    Forgot to call MesosTest::SetUp() and MesosTest::TearDown() in the
    override methods.
    
    Review: https://reviews.apache.org/r/69714
    (cherry picked from commit 2bfd152e8ff1d42d5cff90b65cc9fdff52297800)
---
 src/tests/containerizer/docker_containerizer_tests.cpp | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/tests/containerizer/docker_containerizer_tests.cpp b/src/tests/containerizer/docker_containerizer_tests.cpp
index 2feead9..a621758 100644
--- a/src/tests/containerizer/docker_containerizer_tests.cpp
+++ b/src/tests/containerizer/docker_containerizer_tests.cpp
@@ -192,6 +192,8 @@ public:
 
   void SetUp() override
   {
+    MesosTest::SetUp();
+
     Future<std::tuple<Nothing, Nothing>> pulls = process::collect(
         pullDockerImage(DOCKER_TEST_IMAGE),
         pullDockerImage(DOCKER_INKY_IMAGE));
@@ -227,6 +229,8 @@ public:
     foreach (const Docker::Container& container, containers.get()) {
       AWAIT_READY_FOR(docker.get()->rm(container.id, true), Seconds(30));
     }
+
+    MesosTest::TearDown();
   }
 };