Good call
On Jan 8, 2013 4:56 PM, "Dan Croak" <notifications@github.com> wrote:

> In spec/spec_helper.rb:
>
> > @@ -64,7 +64,10 @@
> >   URL_HELPERS = Rails.application.routes.url_helpers
> >
> >    def garbage_collect_once_per_second_for_faster_tests
> > -    if time_to_run_gc?
> > +    if ENV['TDDIUM'].present?
> > +      GC.enable
> > +      GC.start
> > +    elsif time_to_run_gc?
> >        GC.enable
> >        GC.start
>
> Thoughts on moving the ENV['TDDIUM'].present? logic into the
> time_to_run_gc? logic? Seems like this duplicates the GC.enable and
> GC.start lines.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/yammer/sched.do/pull/272/files#r2585157>.
>
>