----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31176/ -----------------------------------------------------------
(Updated Feb. 23, 2015, 2:39 p.m.) Review request for Aurora and Bill Farner. Changes ------- static import for Fixtures Repository: aurora Description ------- Split out ReadOnlySchedulerImplTest. * Factored out a common Fixtures class for Thrift test data (not married to the name). * Added a unit test for LoggingInterceptor (preferable to relying on SchedulerThriftInterfaceTest to test it indirectly). Diffs (updated) ----- src/main/java/org/apache/aurora/scheduler/thrift/aop/LoggingInterceptor.java 6484642b6416466807f76267d66941c9cf7b3346 src/test/java/org/apache/aurora/scheduler/thrift/Fixtures.java PRE-CREATION src/test/java/org/apache/aurora/scheduler/thrift/ReadOnlySchedulerImplTest.java PRE-CREATION src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java 15147757dfca889f5b5b5ec6967dda98ddfe3075 src/test/java/org/apache/aurora/scheduler/thrift/aop/LoggingInterceptorTest.java PRE-CREATION Diff: https://reviews.apache.org/r/31176/diff/ Testing ------- ./gradlew -Pq build Thanks, Kevin Sweeney