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




src/tests/reservation_endpoints_tests.cpp (line 1035)
<https://reviews.apache.org/r/43779/#comment181408>

    s/ Reserve/Unreserve / `ReserveResources`



src/tests/reservation_endpoints_tests.cpp (line 1036)
<https://reviews.apache.org/r/43779/#comment181409>

    s/ reserve/unreserve / reserve


- Guangya Liu


On 二月 20, 2016, 1:10 a.m., Greg Mann wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/43779/
> -----------------------------------------------------------
> 
> (Updated 二月 20, 2016, 1:10 a.m.)
> 
> 
> Review request for mesos, Adam B, Jie Yu, Michael Park, and Neil Conway.
> 
> 
> Bugs: MESOS-4591
>     https://issues.apache.org/jira/browse/MESOS-4591
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Added '/reserve' tests with multiple roles.
> 
> Operators may reserve resources for multiple roles in the same operation; 
> this patch adds tests to confirm correct behavior of authorization in this 
> case. The tests `ReservationEndpointsTest.GoodReserveACLMultipleRoles` and 
> `ReservationEndpointsTest.BadReserveACLMultipleRoles` were added.
> 
> 
> Diffs
> -----
> 
>   src/tests/reservation_endpoints_tests.cpp 
> afe81b1d38a1b3a82583720f26482ddcde8f5e85 
> 
> Diff: https://reviews.apache.org/r/43779/diff/
> 
> 
> Testing
> -------
> 
> Ran `configure && make check` and `configure --enable-libevent --enable-ssl 
> && make check` on OSX; all tests passed.
> 
> 
> Thanks,
> 
> Greg Mann
> 
>

Reply via email to