Good point, updated :)



On Wed, May 7, 2014 at 2:53 PM, Dan Carley <notifications@github.com> wrote:

> In spec/vcloud/core/vapp_spec.rb:
>
> > @@ -136,7 +136,10 @@ module Core
> >
> >            it "should power on a vapp that is not powered on" do
> >              vapp = Vapp.new(@vapp_id)
> > -            @mock_fog_interface.should_receive(:get_vapp).twice().and_return({:status => 8}, {:status => 4})
> > +            @mock_fog_interface.should_receive(:get_vapp).twice().and_return(
> > +              {:status => Vapp::STATUS::POWERED_OFF},
>
> Can you use Vcloud::Core::Vapp::STATUS so that we have less things to
> rename when we come to change the describe block at the top?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/alphagov/vcloud-core/pull/43/files#r12377266>
> .
>