> On Feb. 6, 2014, 9:53 p.m., Kevin Sweeney wrote:
> > No ship. AbstractCronIT is used by QuartzCronIT in 
> > https://reviews.apache.org/r/17750/
> > 
> > Merging AbstractCronIT and QuartzCronIT is on the list of followup TODOs 
> > for that review, so this will be mitigated then.

AbstractCronIT depends on com.twitter.common.testing.easymock.EasyMockTest, 
will keep it and move to src/test rather than its current location.


- Jake


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


On Feb. 6, 2014, 8:57 p.m., Jake Farrell wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/17818/
> -----------------------------------------------------------
> 
> (Updated Feb. 6, 2014, 8:57 p.m.)
> 
> 
> Review request for Aurora, Kevin Sweeney and Bill Farner.
> 
> 
> Bugs: AURORA-145
>     https://issues.apache.org/jira/browse/AURORA-145
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> AURORA-145:Test dependencies leak into distribution
> 
> First part of Aurora-145 which makes easy mock and zookeeper testing
> part of the test compile only. Part two will be to look at 3rd party
> dependencies and see what we can do to make the dist smaller.
> 
> 
> Diffs
> -----
> 
>   build.gradle 4ba72c109b92d05581dbfe78194518653f20e12d 
>   src/main/java/org/apache/aurora/scheduler/cron/testing/AbstractCronIT.java 
> d8c40222809f525ce485a002873f1fe9dedc2748 
>   
> src/main/java/org/apache/aurora/scheduler/storage/log/testing/LogOpMatcher.java
>  041d1f9937672df64864be09bbc4fd2a38b9b1ab 
>   
> src/main/java/org/apache/aurora/scheduler/storage/testing/StorageTestUtil.java
>   
>   src/test/java/org/apache/aurora/scheduler/app/SchedulerIT.java 
> 6cfaf39ee41862d088ab9747b7820da3f5014a0c 
>   src/test/java/org/apache/aurora/scheduler/storage/log/LogStorageTest.java 
> f708ab3c2fa5d50b084e037f9cc0cb998935120a 
> 
> Diff: https://reviews.apache.org/r/17818/diff/
> 
> 
> Testing
> -------
> 
> ran all unit tests with no failures 
> 
> 
> Thanks,
> 
> Jake Farrell
> 
>

Reply via email to