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)

Reply via email to