mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Meng Zhu <m...@mesosphere.io>
Subject Review Request 66118: Updated an agent test to use mock authorizer.
Date Fri, 16 Mar 2018 18:17:40 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66118/
-----------------------------------------------------------

Review request for mesos and Greg Mann.


Repository: mesos


Description
-------

The test `RemoveExecutorUponFailedTaskAuthorization`
currently depends on catching and modifying the input
arguments of agent `_run()` function to mock the
authorizer behavior. Using mock authorizer gives better
readability and more functionality.


Diffs
-----

  src/tests/mesos.hpp 2c3d0c9350bbbbcd2223ff20c0797d1849d38c19 
  src/tests/mesos.cpp d82963195573dd9ed7d12a7708f64a236b28cdf1 
  src/tests/slave_tests.cpp f76500ebdb67f131a57a3b5aaae8c952d019e354 


Diff: https://reviews.apache.org/r/66118/diff/1/


Testing
-------

make check


Thanks,

Meng Zhu


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message