[PATCH conductor] Fix pools_controller.spec failures due to formatting changes in rails

Richard Su rwsu at redhat.com
Mon Aug 20 18:42:05 UTC 2012


On 08/20/2012 11:34 AM, Richard Su wrote:
> The formatting changed with a different version of rails.
>
> The check on the text was removed. Check on the HTTP response code,
> 404, is sufficient.
> ---
>   src/spec/controllers/pools_controller_spec.rb |    6 ------
>   1 files changed, 0 insertions(+), 6 deletions(-)
>
> diff --git a/src/spec/controllers/pools_controller_spec.rb b/src/spec/controllers/pools_controller_spec.rb
> index 31cd31c..b890c19 100644
> --- a/src/spec/controllers/pools_controller_spec.rb
> +++ b/src/spec/controllers/pools_controller_spec.rb
> @@ -183,8 +183,6 @@ describe PoolsController do
>           response.status.should be_eql(404)
>           response.should have_content_type("application/xml")
>           response.body.should be_xml
> -        xml = Nokogiri::XML(response.body)
> -        xml.xpath("/error/message").text.should == "Couldn't find Pool with ID=-1"
>         end
>   
>       end
> @@ -237,8 +235,6 @@ describe PoolsController do
>           response.status.should be_eql(404)
>           response.should have_content_type("application/xml")
>           response.body.should be_xml
> -        xml = Nokogiri::XML(response.body)
> -        xml.xpath("/error/message").text.should == "Couldn't find Pool with ID=-1"
>         end
>   
>         it "update with blank name" do
> @@ -274,8 +270,6 @@ describe PoolsController do
>           response.status.should be_eql(404)
>           response.should have_content_type("application/xml")
>           response.body.should be_xml
> -        xml = Nokogiri::XML(response.body)
> -        xml.xpath("/error/message").text.should == "Couldn't find Pool with ID=-1"
>         end
>   
>         it "delete default pool should throw error" do

Pushed.



More information about the aeolus-devel mailing list