deltacloud-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Michal Fojtik <mfoj...@redhat.com>
Subject Re: Modular Deltacloud (rev 2)
Date Thu, 17 May 2012 09:15:11 GMT
On 05/17/12, David Lutterkort wrote:

Thanks! I applied the patches and both tasks now run fine under 1.8.
I'll continue with swithing the rest of unit tests we have.

  -- Michal

> On Wed, 2012-05-16 at 15:18 +0200, Michal Fojtik wrote:
> > On 05/15/12, Michal Fojtik wrote:
> 
> > === Work with Sinatra::Base and should be all GREEN ===
> > 
> > $ rake test:mock # => Minitest for Mock drivers and the overall API
> > $ rake cucumber:ec2:test # => EC2 Cucumber scenarios
> > $ rake cucumber:mock:test # => Mock Cucumber scenarios
> > $ rake cimi # => CIMI cucumber scenarios
> 
> This fails for me because it can't find
> lib/deltacloud/helpers/assets_helper - did you forget to check that in ?
> 
> > $ rake spec # => CIMI models rspec tests
> 
> I had to add a few patches of my own to make that work (attached) ;)
> Just fold them into yours.
> 
> I now get another test failure:
>         Failure/Error: should_properly_serialize_model CIMI::Model::MachineImage, @xml,
@json
>                XML documents do not match
>                entries differ[/MachineImage[0]/property[0]/name]: "status" != "locked"
>                entries differ[/MachineImage[0]/property[0]/content]: "BUILD" != "true"
>                entries differ[/MachineImage[0]/property[1]/name]: "locked" != "status"
>                entries differ[/MachineImage[0]/property[1]/content]: "true" != "BUILD"
> 
> This is because in 1.8, Hash has no order; we could import
> ActiveSupport::OrderedHash to achieve this (or only use data for the
> CIMI tests that has at most one property)
> 
> David
> 





-- 
Michal Fojtik
Sr. Software Engineer, Deltacloud API (http://deltacloud.org)

Mime
View raw message