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

Review request for Aurora and Maxim Khutornenko.


Repository: aurora


Description
-------

Realized i didn't cover tests in the last pass, this reduces another 1k LOC:
```
 79 files changed, 909 insertions(+), 1935 deletions(-)
```


Diffs
-----

  commons/src/test/java/org/apache/aurora/common/args/ArgFiltersTest.java 
b312bae52e1c7f66366858fb1b3f62554a41528f 
  commons/src/test/java/org/apache/aurora/common/args/ArgScannerTest.java 
de043041429890409004930f4aa81a9d7a7549ed 
  
commons/src/test/java/org/apache/aurora/common/collections/Iterables2Test.java 
5b832cd9d43b0da60a8de7ea59af431bbe35c166 
  
commons/src/test/java/org/apache/aurora/common/net/http/handlers/TimeSeriesDataSourceTest.java
 bdb237feabcf5bce2f9d349eebd0534e63156c01 
  
commons/src/test/java/org/apache/aurora/common/testing/easymock/EasyMockTestTest.java
 40c5b8c3f51d7db893fd1352812b0e82acd1821f 
  
commons/src/test/java/org/apache/aurora/common/util/templating/StringTemplateHelperTest.java
 0988abce58b414f360fc464c1d4ac04bd8854488 
  commons/src/test/java/org/apache/aurora/common/zookeeper/GroupTest.java 
edfc10907b9d5dab86540187c78c25865ead9e41 
  
commons/src/test/java/org/apache/aurora/common/zookeeper/ServerSetImplTest.java 
5eb18e3ed8ea5f725cc5600cfbe44e9fc03abf8b 
  
commons/src/test/java/org/apache/aurora/common/zookeeper/SingletonServiceTest.java
 167e24259474351e0d9fe2005c5b0526a7fd7c08 
  
commons/src/test/java/org/apache/aurora/common/zookeeper/ZooKeeperClientTest.java
 e074cf5d41e205ba7bf91964f564aeec1b0db6a8 
  src/jmh/java/org/apache/aurora/benchmark/SchedulingBenchmarks.java 
c13a3ba0d28f353d41325686b3031b97d59ffd3a 
  src/jmh/java/org/apache/aurora/benchmark/StatusUpdateBenchmark.java 
892d6ce6e07bb7ecb40a6b6ed9e843f9f7653f6b 
  src/jmh/java/org/apache/aurora/benchmark/ThriftApiBenchmarks.java 
06e22e139b749e8e79126c49127c44729b4e68b3 
  src/jmh/java/org/apache/aurora/benchmark/UpdateStoreBenchmarks.java 
9755ee5e336f2c9716851421ee9550ea67faf69e 
  src/jmh/java/org/apache/aurora/benchmark/fakes/FakeStatsProvider.java 
b1dfc019fb45825a81e52e82654186659e505149 
  src/main/java/org/apache/aurora/scheduler/TaskStatusHandlerImpl.java 
09be688060555a83332630019fad8ddd9823a76c 
  src/main/java/org/apache/aurora/scheduler/cron/quartz/AuroraCronJob.java 
9eda6e9f4582a1fc8a68a0956b12dc1494a5fb76 
  src/main/java/org/apache/aurora/scheduler/cron/quartz/CronJobManagerImpl.java 
675e73dd94ac2309e264b17c04864b900ea3ea2d 
  src/main/java/org/apache/aurora/scheduler/mesos/MesosSchedulerImpl.java 
c4af2fd9ae6ac9988cf676c8d60ad1c0d0d8c3ce 
  src/main/java/org/apache/aurora/scheduler/pruning/JobUpdateHistoryPruner.java 
f29b7cfea1a4f6f5b7e442cd7b8700637c9b385f 
  src/main/java/org/apache/aurora/scheduler/scheduling/TaskThrottler.java 
787309ab030ee5d28932a29e23ca300f611a5130 
  
src/main/java/org/apache/aurora/scheduler/storage/CallOrderEnforcingStorage.java
 6a5069f90a2202815ef31be90362f73b8ccbc47b 
  src/main/java/org/apache/aurora/scheduler/storage/backup/Recovery.java 
0f0218c44a8b5fb7909edad2f5d56b97effbafe5 
  
src/main/java/org/apache/aurora/scheduler/storage/backup/TemporaryStorage.java 
f683f798c39237932924fcbf45bae2012f5f3dbc 
  src/main/java/org/apache/aurora/scheduler/storage/db/RowGarbageCollector.java 
2684054aa601ff33c6cab34c914e966aa0d1bb0d 
  src/main/java/org/apache/aurora/scheduler/storage/log/LogStorage.java 
8aaff22c66d6c6a504464a57496387afa8319298 
  src/main/java/org/apache/aurora/scheduler/storage/log/SnapshotStoreImpl.java 
61058c7810f79e6369fe6208a5e2aab2e015b00b 
  
src/main/java/org/apache/aurora/scheduler/updater/JobUpdateControllerImpl.java 
94b612731b762ffdc2ff65f3a8dc4ef2a1bf32cc 
  src/test/java/org/apache/aurora/scheduler/ResourceSlotTest.java 
175151a111d18a7181558964a62f77e8da6b5b93 
  src/test/java/org/apache/aurora/scheduler/ResourcesTest.java 
430e27832f5d28770598ce7a71f95381e3a5dcdc 
  src/test/java/org/apache/aurora/scheduler/TaskStatusHandlerImplTest.java 
ea67c8d7d2f9b5d2b954f2fefff5688a2e31acd3 
  src/test/java/org/apache/aurora/scheduler/TaskVarsTest.java 
de4fbbcc929a80cdbd3b6593ad4fedce18a88057 
  src/test/java/org/apache/aurora/scheduler/app/SchedulerIT.java 
88253d00720ab99d7bf437c9d2758a5c9038478a 
  src/test/java/org/apache/aurora/scheduler/app/local/FakeMaster.java 
f8940edd2389806899ca75fe9c2851c26810fdc1 
  src/test/java/org/apache/aurora/scheduler/app/local/simulator/FakeSlaves.java 
46be83ccd378ee75ee4f1398e392939bd8c017cd 
  src/test/java/org/apache/aurora/scheduler/async/GatingDelayExecutorTest.java 
c62a1d539bcc2bbddcb4d4bc4f1d1799650426aa 
  src/test/java/org/apache/aurora/scheduler/base/AsyncUtilTest.java 
58b4dc914f09a615b2f218efc16102fb81105286 
  src/test/java/org/apache/aurora/scheduler/base/JobsTest.java 
986fd01c0885e713e41f0894b7d6c179ef50e118 
  src/test/java/org/apache/aurora/scheduler/cron/quartz/AuroraCronJobTest.java 
be921c3af95e7caa48d7e6e44fcaf640537fd4ea 
  src/test/java/org/apache/aurora/scheduler/cron/quartz/CronIT.java 
400f2955c148cd376b7acef2c9ceb3962cd13b09 
  
src/test/java/org/apache/aurora/scheduler/cron/quartz/CronJobManagerImplTest.java
 5b0405fed60711c9befa71a526450164e540a1df 
  
src/test/java/org/apache/aurora/scheduler/cron/quartz/CronPredictorImplTest.java
 b879d6f5a394ab016f3b1fc577ac0cf92ef43a3d 
  src/test/java/org/apache/aurora/scheduler/http/JettyServerModuleTest.java 
3794b967223baa9078b43c5dc112bf4ad2ed32d7 
  src/test/java/org/apache/aurora/scheduler/http/LeaderRedirectTest.java 
b831c9d94c5dd9a78fc5c202c158509071dd01c2 
  src/test/java/org/apache/aurora/scheduler/http/RequestLoggerTest.java 
4fd2abe14728bfb7dd4b399e32e2bc0d93229d04 
  src/test/java/org/apache/aurora/scheduler/log/mesos/MesosLogTest.java 
9189bde321573f5cc8a751fdd967e50daf5a90a7 
  src/test/java/org/apache/aurora/scheduler/mesos/Offers.java 
0e4e008e7057a4a7ce57c3d78b497f51e9e66f6d 
  
src/test/java/org/apache/aurora/scheduler/offers/RandomJitterReturnDelayTest.java
 d446cafc8efedb8c83b13bbf05482972eb2c43a5 
  
src/test/java/org/apache/aurora/scheduler/preemptor/PreemptionVictimFilterTest.java
 5cb850814996a9c37a74a231746393e301bd4cf3 
  src/test/java/org/apache/aurora/scheduler/pruning/TaskHistoryPrunerTest.java 
373e8b1e1766dc4bca09f727dd45274214593519 
  
src/test/java/org/apache/aurora/scheduler/reconciliation/TaskReconcilerTest.java
 e6a39acae29fce8cd9427f97612d9cf553035aca 
  
src/test/java/org/apache/aurora/scheduler/scheduling/RescheduleCalculatorImplTest.java
 71ce376777ce47c3a0f42fe7051041bb0462a152 
  src/test/java/org/apache/aurora/scheduler/scheduling/TaskGroupsTest.java 
b9be4ade92f72a0b4bec0789dc65127068b8cd70 
  
src/test/java/org/apache/aurora/scheduler/scheduling/TaskSchedulerImplTest.java 
13c9f47941eb8ce6c79f3f1a7c530150e3e6fae2 
  src/test/java/org/apache/aurora/scheduler/sla/SlaModuleTest.java 
00a872cd56750a44e70b5a4bf988d73a8132ed44 
  src/test/java/org/apache/aurora/scheduler/state/LockManagerImplTest.java 
95b22ae5f821b5fb05661e998a68cab018a6f031 
  src/test/java/org/apache/aurora/scheduler/state/StateManagerImplTest.java 
01f5273d89d0f35faf3a00a1853326da1b0d62b9 
  src/test/java/org/apache/aurora/scheduler/state/TaskStateMachineTest.java 
e2c82b7f5a6ca096088b419a7b097ef71e09cc2e 
  src/test/java/org/apache/aurora/scheduler/stats/ResourceCounterTest.java 
24dcf6168b74380da2bd79bcfb7f09e8120dd6a2 
  
src/test/java/org/apache/aurora/scheduler/storage/AbstractCronJobStoreTest.java 
25af7da56e32ad5cc7920b76d91ff29186d2e530 
  src/test/java/org/apache/aurora/scheduler/storage/AbstractTaskStoreTest.java 
fa1d1cd3e718c79e0e6fc88726108bd3eef8086d 
  src/test/java/org/apache/aurora/scheduler/storage/StorageBackfillTest.java 
2e3ac5ae7d011cf211a4f05afb10b286eb296cfb 
  
src/test/java/org/apache/aurora/scheduler/storage/db/DbAttributeStoreTest.java 
20c9858d54378ac5ad9f2a8c0a90e246ec90bad9 
  
src/test/java/org/apache/aurora/scheduler/storage/db/DbJobUpdateStoreTest.java 
9d5468c74468e09e6665f499d32d7d4d39e70f1f 
  src/test/java/org/apache/aurora/scheduler/storage/db/DbLockStoreTest.java 
400e99fa3940cd6bab1d7cddd21a11969acf70d4 
  src/test/java/org/apache/aurora/scheduler/storage/db/DbQuotaStoreTest.java 
ff7afcf7828a5c61f5486801a7ed34df6578e0d2 
  
src/test/java/org/apache/aurora/scheduler/storage/db/DbSchedulerStoreTest.java 
36ff3a5bc60bed478324744f07d2a49745078fdd 
  src/test/java/org/apache/aurora/scheduler/storage/db/DbStorageTest.java 
a0bd34bbedb4ab004ec602b21fe23983ffd11594 
  src/test/java/org/apache/aurora/scheduler/storage/log/LogManagerTest.java 
36900a800b673c87e4cbcffb8e5cf9995e194457 
  src/test/java/org/apache/aurora/scheduler/storage/log/LogStorageTest.java 
cc777b2c976846cf46a34d3262decb53542641f3 
  src/test/java/org/apache/aurora/scheduler/storage/mem/InMemTaskStoreTest.java 
a39f6906a09143a4eee866081c24f2f4702bb64f 
  
src/test/java/org/apache/aurora/scheduler/storage/mem/StorageTransactionTest.java
 d40e75a66d20ba6372349a5f0787f8ae18934199 
  
src/test/java/org/apache/aurora/scheduler/storage/testing/StorageTestUtil.java 
dcc4ae92716607d6fc9185a473963bdc62382e6e 
  src/test/java/org/apache/aurora/scheduler/testing/FakeScheduledExecutor.java 
f1f9f771be1f5353774626b7242810f56ea8fc25 
  src/test/java/org/apache/aurora/scheduler/thrift/Fixtures.java 
76b047649cf97cd0f868e4091f136240678c096e 
  
src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java
 1fd106b6f2e3650ddc72f7836f6cd2940eff2527 
  
src/test/java/org/apache/aurora/scheduler/thrift/aop/AnnotatedAuroraAdminTest.java
 b7555043f6441cd1cfc047f8d8662b5f699d4c18 
  src/test/java/org/apache/aurora/scheduler/updater/AddTaskTest.java 
749e5b857055d2631acbf14b028e3c34def18e39 
  src/test/java/org/apache/aurora/scheduler/updater/JobUpdaterIT.java 
d98797406d9d5e9de10dbacc7640e513e291b2ff 

Diff: https://reviews.apache.org/r/41181/diff/


Testing
-------


Thanks,

Bill Farner

Reply via email to