On 02/15, David Lutterkort wrote:
> On Fri, 2013-02-15 at 11:36 +0100, mfoj...@redhat.com wrote:
> > From: Michal Fojtik <mfoj...@redhat.com>
> > 
> > This patch will make possible to run our unit-tests without
> > initializing the Deltacloud.configure classes, because we already
> > do that in initializers.
> > 
> > For CIMI tests, this will make sure the database directory exists.
> 
> ACK.
> 
> Comments inline:
> 
> > diff --git a/server/tests/cimi/collections/cloud_entry_point_test.rb 
> > b/server/tests/cimi/collections/cloud_entry_point_test.rb
> > index eed8721..a7fdd89 100644
> > --- a/server/tests/cimi/collections/cloud_entry_point_test.rb
> > +++ b/server/tests/cimi/collections/cloud_entry_point_test.rb
> > @@ -18,7 +18,7 @@ describe CIMI::Collections::CloudEntryPoint do
> >    it 'set the CIMI-Version header' do
> >      get root_url
> >      headers['CIMI-Specification-Version'].wont_be_nil
> > -    headers['CIMI-Specification-Version'].must_equal '1.0.0'
> > +    headers['CIMI-Specification-Version'].must_equal 
> > Deltacloud::CIMI_API_VERSION
> 

Yes, I removed al references to that header in code and tests. Thanks!

> You can actually completely remove this header - CIMI 1.0.1 doesn't
> mention it anymore, as we've revamped how versioning is done.
> 
> David
> 
> 

-- 
Michal Fojtik <mfoj...@redhat.com>
Deltacloud API, CloudForms

Reply via email to