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


Ship it!




- Joerg Schad


On Sept. 1, 2017, 9:27 p.m., Gastón Kleiman wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/62042/
> -----------------------------------------------------------
> 
> (Updated Sept. 1, 2017, 9:27 p.m.)
> 
> 
> Review request for mesos, Benjamin Bannier, Greg Mann, and Joerg Schad.
> 
> 
> Bugs: MESOS-7877
>     https://issues.apache.org/jira/browse/MESOS-7877
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Change `EXPECT` to `ASSERT` when relying on the assertion afterwards.
> 
> 
> Diffs
> -----
> 
>   src/tests/api_tests.cpp 0902a66772d178505e83b81878b7878a45c7bb25 
>   src/tests/check_tests.cpp cc7be317030d147c5617e784a50e42b0365b9df6 
>   src/tests/container_logger_tests.cpp 
> 97e79792d3ea8023890ad2a705db47f2aeb419cf 
>   src/tests/containerizer/cgroups_isolator_tests.cpp 
> 68b8181b9c6eb9ce2e8c3c8980451364d410516b 
>   src/tests/containerizer/cpu_isolator_tests.cpp 
> f7cab58b75994dc38eede6a84556ae73d412e1a1 
>   src/tests/containerizer/environment_secret_isolator_tests.cpp 
> 38fef2d5f677f768a0533d1ac085b1197b3b764d 
>   src/tests/containerizer/io_switchboard_tests.cpp 
> de42be9ae2019c7cdb24a0976a8565631a12fca3 
>   src/tests/containerizer/linux_capabilities_isolator_tests.cpp 
> a1ff9b5e67fcccb8aa9afd90ca704796c1b0a252 
>   src/tests/containerizer/linux_filesystem_isolator_tests.cpp 
> 1ad9464e99b7dd4dfd2c9bd1a4c6c9c5cce615bd 
>   src/tests/containerizer/memory_isolator_tests.cpp 
> a7028da62b27b109cb455048acbea0976da9a140 
>   src/tests/containerizer/memory_pressure_tests.cpp 
> eec2e05d399d097fcb997bb44e353b9c3e0563f7 
>   src/tests/containerizer/port_mapping_tests.cpp 
> 544eaef1215f309f67ae177249c209afc71c5d5e 
>   src/tests/containerizer/runtime_isolator_tests.cpp 
> 8bf6a11de85d4dacf081c009ace9e7b78e904849 
>   src/tests/containerizer/xfs_quota_tests.cpp 
> 11dc7f596671d373cd0c9b443a1d217053285a63 
>   src/tests/default_executor_tests.cpp 
> 186b8333c02ba3b9257e19437c6d689761085362 
>   src/tests/disk_quota_tests.cpp 3bf0508238a228d86737d6cc899fa68e2046f2e2 
>   src/tests/exception_tests.cpp 0e72cdfce2f12a6961efefb23fe14eadbadce01f 
>   src/tests/fault_tolerance_tests.cpp 
> 5ac38a897fefb6f40d69ca6e27a6f23176e42d36 
>   src/tests/gc_tests.cpp da9a31c30e759e30492963c199da2b16e9f91550 
>   src/tests/health_check_tests.cpp 2c43241a5cc245a1797ea16c68c94638cec6803d 
>   src/tests/hook_tests.cpp c5acbd0fde7cb0923779a925fe2f6a8214be4163 
>   src/tests/http_fault_tolerance_tests.cpp 
> f05aee8200e05b10595fd821413f44bc36839de2 
>   src/tests/master_authorization_tests.cpp 
> a776a5811b66cb93cfc3484c183e815248dea944 
>   src/tests/master_maintenance_tests.cpp 
> 7595e8b1d8360920f61af4685e210c9a9f4569af 
>   src/tests/master_slave_reconciliation_tests.cpp 
> 0b26953c73c3a41e7f5e4e62a41a0756fb2097e2 
>   src/tests/master_tests.cpp 59fbad468ae671a8fd39457632262015989f26c2 
>   src/tests/master_validation_tests.cpp 
> 710b25c1afdff4de7b2eb9b8a38f6856c373fb3c 
>   src/tests/oversubscription_tests.cpp 
> a4c4a6083a2cb55e2d69eba1719555a7a13ee8fb 
>   src/tests/persistent_volume_tests.cpp 
> 3e1d1fe468298186347b07b5882973c5a16231a3 
>   src/tests/reconciliation_tests.cpp 728961874239c7e5ce8accbaeb2a1c86a73a4f0f 
>   src/tests/registrar_zookeeper_tests.cpp 
> 96bae295bdbe839b979b879cc4b11fddcce6ad75 
>   src/tests/reservation_endpoints_tests.cpp 
> 3278732cb130c73be0f20c0204eddaee05123ff9 
>   src/tests/resource_offers_tests.cpp 
> dc9c230548ba49e92228c6625597b2294e65ca30 
>   src/tests/scheduler_driver_tests.cpp 
> d2aff699dc33d2a246b391d6322d271e7f0252b6 
>   src/tests/scheduler_tests.cpp 21f8825c214bb0c331f66bfb9b12a354d6673a43 
>   src/tests/slave_authorization_tests.cpp 
> 536a8efda0cc1ba08285c379b01af6ec64a82117 
>   src/tests/slave_recovery_tests.cpp 0e46748be266809c413fb10fc447516c51504fce 
>   src/tests/slave_tests.cpp 1bdadce4c50cbff958f2be2a4261e130b414acfd 
>   src/tests/status_update_manager_tests.cpp 
> 6922ee353123bc024c66a6bc4fadc96450287447 
>   src/tests/teardown_tests.cpp 8834333bbcdb6a04a95a6ed9632ad4ead0791f76 
> 
> 
> Diff: https://reviews.apache.org/r/62042/diff/1/
> 
> 
> Testing
> -------
> 
> `sudo bin/mesos-tests.sh` on GNU/Linux
> 
> 
> Thanks,
> 
> Gastón Kleiman
> 
>

Reply via email to