----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41001/ -----------------------------------------------------------
Review request for mesos. Repository: mesos Description ------- Improved 'ReservationTest.ACLMultipleOperations'. Diffs ----- src/tests/reservation_tests.cpp 3fdf5e121840fe99057e917cca48f1425eff6624 Diff: https://reviews.apache.org/r/41001/diff/ Testing ------- `GTEST_FILTER="ReservationTest.ACLMultipleOperations" bin/mesos-tests.sh --gtest_repeat=1000 --gtest_break_on_failure` This test was originally written to test the interaction of multiple RESERVE/UNRESERVE/LAUNCH operations in a single `acceptOffers()` call when authorization is enabled. In order to probe the effect of authorization on this interaction, some operations should fail due to failed authorization. However, this test included operations that failed simply because they were malformed. I altered the test so that now operations will fail due to failed authorization, which tests the desired functionality in a more thorough way. Thanks, Greg Mann