Fixed all.


On Wed, Aug 21, 2013 at 1:16 PM, cheddar <notifications@github.com> wrote:

> In
> realtime/src/test/java/com/metamx/druid/realtime/plumber/ServerTimeRejectionPolicyFactoryTest.java:
>
> > +import org.joda.time.Period;
> > +import org.junit.Test;
> > +
> > +/**
> > + */
> > +public class ServerTimeRejectionPolicyFactoryTest
> > +{
> > +  @Test
> > +  public void testAccept() throws Exception
> > +  {
> > +    RejectionPolicy rejectionPolicy = new ServerTimeRejectionPolicyFactory().create(
> > +        new Period("PT10M")
> > +    );
> > +
> > +    Assert.assertTrue(rejectionPolicy.accept(new DateTime().getMillis()));
> > +    Assert.assertFalse(rejectionPolicy.accept(new DateTime("2000").getMillis()));
>
> It should be sufficient (and more better) to pull this out into a
>
> DateTime now = new DateTime();
> DateTime past = now.minus(Minutes.something(10)).minus(Milliseconds.ONE);
>
> And then test.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/metamx/druid/pull/222/files#r5907329>
> .
>