Somehow github ate this comment in the web context but -

I agree with you here re: the style guide. I'm making the change.

And Caleb I think your approach is synonymous but I think I prefer seeing
the setup explicitly created before the assertion in this case.


On Wed, Feb 27, 2013 at 12:07 PM, Gabe Berke-Williams <
notifications@github.com> wrote:

> In spec/griddler/configuration_spec.rb:
>
> > @@ -35,5 +35,19 @@
> >
> >        Griddler.configuration.processor_class.should eq dummy_processor
> >      end
> > +
> > +    it 'defaults the email service to SendGrid' do
> > +      Griddler.configuration.email_service.should eq(Griddler::Adapters::SendgridAdapter)
> > +    end
> > +
> > +    it 'raises an error when setting a non-existent email service adapter' do
> > +      config = -> {
>
> Yes - using { and } for a block that spans more than one line doesn't
> match our style guide.
>
> I've usually seen lambda do in RSpec.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/thoughtbot/griddler/pull/25/files#r3174138>
> .
>