[ https://issues.apache.org/jira/browse/MESOS-8094?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16227763#comment-16227763 ]
James Peach commented on MESOS-8094: ------------------------------------ {noformat} commit 8165188676ebc00707b61f6782a171170098b6be (HEAD -> master, origin/master, origin/HEAD) Author: James Peach <jpe...@apache.org> Date: Tue Oct 31 15:42:07 2017 -0700 Updated tests to use `createCallKill`. Updated all the tests that send v1 API KILL calls to use the `createCallKill` test helper. Review: https://reviews.apache.org/r/63419/ {noformat} > Leverage helper functions to reduce boilerplate code related to v1 API. > ----------------------------------------------------------------------- > > Key: MESOS-8094 > URL: https://issues.apache.org/jira/browse/MESOS-8094 > Project: Mesos > Issue Type: Improvement > Components: test > Reporter: Alexander Rukletsov > Labels: mesosphere, newbie > > https://reviews.apache.org/r/61982/ created an example how test code related > to scheduler v1 API can be simplified with appropriate usage of helper > function. For example, instead of crafting a subscribe call manually like in > {noformat} > { > v1::scheduler::Call call; > call.set_type(v1::scheduler::Call::SUBSCRIBE); > v1::scheduler::Call::Subscribe* subscribe = call.mutable_subscribe(); > subscribe->mutable_framework_info()->CopyFrom(v1::DEFAULT_FRAMEWORK_INFO); > mesos.send(call); > } > {noformat} > a helper function {{v1::scheduler::SendSubscribe()}} shall be invoked. > To find all occurrences that shall be fixed, one can grep the test codebase > for {{call.set_type}}. At the moment I see the following files: > {noformat} > api_tests.cpp > check_tests.cpp > http_fault_tolerant_tests.cpp > master_maintenance_tests.cpp > master_tests.cpp > scheduler_tests.cpp > slave_authorization_tests.cpp > slave_recovery_tests.cpp > slave_tests.cpp > {noformat} > The same applies for sending status update acks; > {{v1::scheduler::SendAcknowledge()}} action shall be used instead of manually > crafting acks. -- This message was sent by Atlassian JIRA (v6.4.14#64029)