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 2016/07/13 19:11:00 UTC

[5/8] mesos git commit: Added test case for 'GetFrameworks' call in v1 agent API.

Added test case for 'GetFrameworks' call in v1 agent API.

Review: https://reviews.apache.org/r/49797/


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/4990e7c4
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/4990e7c4
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/4990e7c4

Branch: refs/heads/master
Commit: 4990e7c496583d46d8355c2bcd3934c968c31015
Parents: a298179
Author: haosdent huang <ha...@gmail.com>
Authored: Wed Jul 13 12:10:28 2016 -0700
Committer: Vinod Kone <vi...@gmail.com>
Committed: Wed Jul 13 12:10:28 2016 -0700

----------------------------------------------------------------------
 src/tests/api_tests.cpp | 103 +++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 103 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/4990e7c4/src/tests/api_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/api_tests.cpp b/src/tests/api_tests.cpp
index 55e825e..40f5077 100644
--- a/src/tests/api_tests.cpp
+++ b/src/tests/api_tests.cpp
@@ -2659,6 +2659,109 @@ TEST_P(AgentAPITest, ReadFileInvalidPath)
   AWAIT_EXPECT_RESPONSE_STATUS_EQ(NotFound().status, response);
 }
 
+
+TEST_P(AgentAPITest, GetFrameworks)
+{
+  Try<Owned<cluster::Master>> master = StartMaster();
+  ASSERT_SOME(master);
+
+  Owned<MasterDetector> detector = master.get()->createDetector();
+  Try<Owned<cluster::Slave>> slave = StartSlave(detector.get());
+  ASSERT_SOME(slave);
+
+  MockScheduler sched;
+  MesosSchedulerDriver driver(
+      &sched, DEFAULT_FRAMEWORK_INFO, master.get()->pid, DEFAULT_CREDENTIAL);
+
+  EXPECT_CALL(sched, registered(_, _, _));
+  Future<vector<Offer>> offers;
+  EXPECT_CALL(sched, resourceOffers(&driver, _))
+    .WillOnce(FutureArg<1>(&offers))
+    .WillRepeatedly(Return()); // Ignore subsequent offers.
+
+  driver.start();
+
+  AWAIT_READY(offers);
+  EXPECT_NE(0u, offers.get().size());
+  const Offer& offer = offers.get()[0];
+
+  TaskInfo task;
+  task.set_name("");
+  task.mutable_task_id()->set_value("1");
+  task.mutable_slave_id()->MergeFrom(offer.slave_id());
+  task.mutable_resources()->MergeFrom(offer.resources());
+
+  CommandInfo command;
+  command.set_value("sleep 1000");
+  task.mutable_command()->MergeFrom(command);
+
+  Future<TaskStatus> statusRunning;
+  EXPECT_CALL(sched, statusUpdate(&driver, _))
+    .WillOnce(FutureArg<1>(&statusRunning));
+
+  ContentType contentType = GetParam();
+
+  // No tasks launched, we should expect zero frameworks in Response.
+  {
+    v1::agent::Call v1Call;
+    v1Call.set_type(v1::agent::Call::GET_FRAMEWORKS);
+
+    Future<v1::agent::Response> v1Response =
+      post(slave.get()->pid, v1Call, contentType);
+
+    AWAIT_READY(v1Response);
+    ASSERT_TRUE(v1Response.get().IsInitialized());
+    ASSERT_EQ(v1::agent::Response::GET_FRAMEWORKS, v1Response.get().type());
+    ASSERT_EQ(0, v1Response.get().get_frameworks().frameworks_size());
+    ASSERT_EQ(0, v1Response.get().get_frameworks().completed_frameworks_size());
+  }
+
+  driver.launchTasks(offer.id(), {task});
+
+  AWAIT_READY(statusRunning);
+  EXPECT_EQ(TASK_RUNNING, statusRunning->state());
+
+  // A task launched, we expect one framework in Response.
+  {
+    v1::agent::Call v1Call;
+    v1Call.set_type(v1::agent::Call::GET_FRAMEWORKS);
+
+    Future<v1::agent::Response> v1Response =
+      post(slave.get()->pid, v1Call, contentType);
+
+    AWAIT_READY(v1Response);
+    ASSERT_TRUE(v1Response.get().IsInitialized());
+    ASSERT_EQ(v1::agent::Response::GET_FRAMEWORKS, v1Response.get().type());
+    ASSERT_EQ(1, v1Response.get().get_frameworks().frameworks_size());
+    ASSERT_EQ(0, v1Response.get().get_frameworks().completed_frameworks_size());
+  }
+
+  // Make sure the executor terminated.
+  Future<Nothing> executorTerminated =
+    FUTURE_DISPATCH(_, &Slave::executorTerminated);
+
+  driver.stop();
+  driver.join();
+
+  AWAIT_READY(executorTerminated);
+
+  // After the executor terminated, we should expect one completed framework in
+  // Response.
+  {
+    v1::agent::Call v1Call;
+    v1Call.set_type(v1::agent::Call::GET_FRAMEWORKS);
+
+    Future<v1::agent::Response> v1Response =
+      post(slave.get()->pid, v1Call, contentType);
+
+    AWAIT_READY(v1Response);
+    ASSERT_TRUE(v1Response.get().IsInitialized());
+    ASSERT_EQ(v1::agent::Response::GET_FRAMEWORKS, v1Response.get().type());
+    ASSERT_EQ(0, v1Response.get().get_frameworks().frameworks_size());
+    ASSERT_EQ(1, v1Response.get().get_frameworks().completed_frameworks_size());
+  }
+}
+
 } // namespace tests {
 } // namespace internal {
 } // namespace mesos {