Stopped taking task_id in scheduler::SendAcknowledge action in tests. Taking task_id is not necessary because the status update itself (arg1) already has task_id in it.
Review: https://reviews.apache.org/r/53561 Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/f041787e Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/f041787e Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/f041787e Branch: refs/heads/master Commit: f041787e2356db53ef264e7872b2b99defca6cf7 Parents: 382b2ea Author: Jie Yu <yujie....@gmail.com> Authored: Mon Nov 7 16:40:13 2016 -0800 Committer: Jie Yu <yujie....@gmail.com> Committed: Mon Nov 7 17:05:37 2016 -0800 ---------------------------------------------------------------------- src/tests/mesos.hpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mesos/blob/f041787e/src/tests/mesos.hpp ---------------------------------------------------------------------- diff --git a/src/tests/mesos.hpp b/src/tests/mesos.hpp index f521d22..0d21175 100644 --- a/src/tests/mesos.hpp +++ b/src/tests/mesos.hpp @@ -1863,14 +1863,14 @@ ACTION_P(SendSubscribe, frameworkInfo) } -ACTION_P3(SendAcknowledge, frameworkId, taskId, agentId) +ACTION_P2(SendAcknowledge, frameworkId, agentId) { Call call; call.set_type(Call::ACKNOWLEDGE); call.mutable_framework_id()->CopyFrom(frameworkId); Call::Acknowledge* acknowledge = call.mutable_acknowledge(); - acknowledge->mutable_task_id()->CopyFrom(taskId); + acknowledge->mutable_task_id()->CopyFrom(arg1.status().task_id()); acknowledge->mutable_agent_id()->CopyFrom(agentId); acknowledge->set_uuid(arg1.status().uuid());
