Re: Review Request 63015: Cleaned up test ordering in "master_tests.cpp".

2017-10-16 Thread Michael Park

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


Ship it!




Ship It!

- Michael Park


On Oct. 15, 2017, 3:41 p.m., Alexander Rukletsov wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/63015/
> ---
> 
> (Updated Oct. 15, 2017, 3:41 p.m.)
> 
> 
> Review request for mesos and Michael Park.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> Move all tests from the `MasterTest` fixture before tests from the
> `MasterTestPrePostReservationRefinement` fixture.
> 
> 
> Diffs
> -
> 
>   src/tests/master_tests.cpp 4c776012e84c34dda7410841e063faab2a84f2df 
> 
> 
> Diff: https://reviews.apache.org/r/63015/diff/1/
> 
> 
> Testing
> ---
> 
> make check on Mac OS 10.11.6
> make check + root tests on various Linux distros
> tests from all modified test fixtures in the chain run at least 1000 on Mac 
> OS 10.11.6
> 
> 
> Thanks,
> 
> Alexander Rukletsov
> 
>



Re: Review Request 63015: Cleaned up test ordering in "master_tests.cpp".

2017-10-15 Thread Mesos Reviewbot Windows

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



PASS: Mesos patch 63015 was successfully built and tested.

Reviews applied: `['63002', '63003', '63004', '63005', '63006', '63007', 
'63008', '63009', '63010', '63011', '63012', '63013', '63014', '63015']`

All the build artifacts available at: 
http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/63015

- Mesos Reviewbot Windows


On Oct. 15, 2017, 10:41 p.m., Alexander Rukletsov wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/63015/
> ---
> 
> (Updated Oct. 15, 2017, 10:41 p.m.)
> 
> 
> Review request for mesos and Michael Park.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> Move all tests from the `MasterTest` fixture before tests from the
> `MasterTestPrePostReservationRefinement` fixture.
> 
> 
> Diffs
> -
> 
>   src/tests/master_tests.cpp 4c776012e84c34dda7410841e063faab2a84f2df 
> 
> 
> Diff: https://reviews.apache.org/r/63015/diff/1/
> 
> 
> Testing
> ---
> 
> make check on Mac OS 10.11.6
> make check + root tests on various Linux distros
> tests from all modified test fixtures in the chain run at least 1000 on Mac 
> OS 10.11.6
> 
> 
> Thanks,
> 
> Alexander Rukletsov
> 
>