You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by vi...@apache.org on 2014/11/08 06:53:00 UTC
mesos git commit: Fixed MesosContainerizerExecuteTest.IoRedirection
test.
Repository: mesos
Updated Branches:
refs/heads/master fae5a6020 -> 4a1bbf7d3
Fixed MesosContainerizerExecuteTest.IoRedirection test.
Review: https://reviews.apache.org/r/27765
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/4a1bbf7d
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/4a1bbf7d
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/4a1bbf7d
Branch: refs/heads/master
Commit: 4a1bbf7d3b3ecbb65c4dfd7ee6994e94df844931
Parents: fae5a60
Author: Vinod Kone <vi...@gmail.com>
Authored: Fri Nov 7 17:39:28 2014 -0800
Committer: Vinod Kone <vi...@gmail.com>
Committed: Fri Nov 7 21:52:45 2014 -0800
----------------------------------------------------------------------
src/tests/containerizer_tests.cpp | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/4a1bbf7d/src/tests/containerizer_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/containerizer_tests.cpp b/src/tests/containerizer_tests.cpp
index abe8074..a63897b 100644
--- a/src/tests/containerizer_tests.cpp
+++ b/src/tests/containerizer_tests.cpp
@@ -25,6 +25,8 @@
#include <process/future.hpp>
#include <process/owned.hpp>
+#include <stout/strings.hpp>
+
#include <mesos/mesos.hpp>
#include "slave/flags.hpp"
@@ -279,7 +281,12 @@ TEST_F(MesosContainerizerExecuteTest, IoRedirection)
EXPECT_EQ(0, wait.get().status());
// Check that std{err, out} was redirected.
- EXPECT_SOME_EQ(errMsg + "\n", os::read(path::join(directory, "stderr")));
+ // NOTE: Fetcher uses GLOG, which outputs extra information to
+ // stderr.
+ Try<string> stderr = os::read(path::join(directory, "stderr"));
+ ASSERT_SOME(stderr);
+ EXPECT_TRUE(strings::contains(stderr.get(), errMsg));
+
EXPECT_SOME_EQ(outMsg + "\n", os::read(path::join(directory, "stdout")));
delete containerizer.get();