chef 11.10.0.rc.0 → 11.10.0.rc.1

data/spec/unit/knife/cookbook_bulk_delete_spec.rb

@@ -27,18 +27,18 @@ describe Chef::Knife::CookbookBulkDelete do
     @knife.config = {:print_after => nil}
     @knife.name_args = ["."]
     @stdout = StringIO.new
-    @knife.ui.stub!(:stdout).and_return(@stdout)
-    @knife.ui.stub!(:confirm).and_return(true)
+    @knife.ui.stub(:stdout).and_return(@stdout)
+    @knife.ui.stub(:confirm).and_return(true)
     @cookbooks = Hash.new
     %w{cheezburger pizza lasagna}.each do |cookbook_name|
       cookbook = Chef::CookbookVersion.new(cookbook_name)
       @cookbooks[cookbook_name] = cookbook
     end
-    @rest = mock("Chef::REST")
-    @rest.stub!(:get_rest).and_return(@cookbooks)
-    @rest.stub!(:delete_rest).and_return(true)
-    @knife.stub!(:rest).and_return(@rest)
-    Chef::CookbookVersion.stub!(:list).and_return(@cookbooks)
+    @rest = double("Chef::REST")
+    @rest.stub(:get_rest).and_return(@cookbooks)
+    @rest.stub(:delete_rest).and_return(true)
+    @knife.stub(:rest).and_return(@rest)
+    Chef::CookbookVersion.stub(:list).and_return(@cookbooks)
 
   end
 
@@ -47,11 +47,11 @@ describe Chef::Knife::CookbookBulkDelete do
   describe "when there are several cookbooks on the server" do
     before do
       @cheezburger = {'cheezburger' => {"url" => "file:///dev/null", "versions" => [{"url" => "file:///dev/null-cheez", "version" => "1.0.0"}]}}
-      @rest.stub!(:get_rest).with('cookbooks/cheezburger').and_return(@cheezburger)
+      @rest.stub(:get_rest).with('cookbooks/cheezburger').and_return(@cheezburger)
       @pizza = {'pizza' => {"url" => "file:///dev/null", "versions" => [{"url" => "file:///dev/null-pizza", "version" => "2.0.0"}]}}
-      @rest.stub!(:get_rest).with('cookbooks/pizza').and_return(@pizza)
+      @rest.stub(:get_rest).with('cookbooks/pizza').and_return(@pizza)
       @lasagna = {'lasagna' => {"url" => "file:///dev/null", "versions" => [{"url" => "file:///dev/null-lasagna", "version" => "3.0.0"}]}}
-      @rest.stub!(:get_rest).with('cookbooks/lasagna').and_return(@lasagna)
+      @rest.stub(:get_rest).with('cookbooks/lasagna').and_return(@lasagna)
     end
 
     it "should print the cookbooks you are about to delete" do

data/spec/unit/knife/cookbook_create_spec.rb

@@ -26,7 +26,7 @@ describe Chef::Knife::CookbookCreate do
     @knife.config = {}
     @knife.name_args = ["foobar"]
     @stdout = StringIO.new
-    @knife.stub!(:stdout).and_return(@stdout)
+    @knife.stub(:stdout).and_return(@stdout)
   end
 
   describe "run" do
@@ -35,10 +35,10 @@ describe Chef::Knife::CookbookCreate do
     it "should expand the path of the cookbook directory" do
       File.should_receive(:expand_path).with("~/tmp/monkeypants")
       @knife.config = {:cookbook_path => "~/tmp/monkeypants"}
-      @knife.stub!(:create_cookbook)
-      @knife.stub!(:create_readme)
-      @knife.stub!(:create_changelog)
-      @knife.stub!(:create_metadata)
+      @knife.stub(:create_cookbook)
+      @knife.stub(:create_readme)
+      @knife.stub(:create_changelog)
+      @knife.stub(:create_metadata)
       @knife.run
     end
 

data/spec/unit/knife/cookbook_delete_spec.rb

@@ -24,9 +24,9 @@ describe Chef::Knife::CookbookDelete do
     @knife.name_args = ['foobar']
     @knife.cookbook_name = 'foobar'
     @stdout = StringIO.new
-    @knife.ui.stub!(:stdout).and_return(@stdout)
+    @knife.ui.stub(:stdout).and_return(@stdout)
     @stderr = StringIO.new
-    @knife.ui.stub!(:stderr).and_return(@stderr)
+    @knife.ui.stub(:stderr).and_return(@stderr)
   end
 
   describe 'run' do
@@ -125,7 +125,7 @@ describe Chef::Knife::CookbookDelete do
 
   describe 'available_versions' do
     before(:each) do
-      @rest_mock = mock('rest')
+      @rest_mock = double('rest')
       @knife.should_receive(:rest).and_return(@rest_mock)
       @cookbook_data = { 'foobar' => { 'versions' => [{'version' => '1.0.0'},
                                                       {'version' => '1.1.0'},
@@ -163,7 +163,7 @@ describe Chef::Knife::CookbookDelete do
 
   describe 'ask_which_version_to_delete' do
     before(:each) do
-      @knife.stub!(:available_versions).and_return(['1.0.0', '1.1.0', '2.0.0'])
+      @knife.stub(:available_versions).and_return(['1.0.0', '1.1.0', '2.0.0'])
     end
 
     it 'should prompt the user to select a version' do
@@ -202,7 +202,7 @@ describe Chef::Knife::CookbookDelete do
     end
 
     it 'should output that the cookbook was deleted' do
-      @knife.stub!(:delete_request)
+      @knife.stub(:delete_request)
       @knife.delete_version_without_confirmation('1.0.0')
       @stdout.string.should match /deleted cookbook\[foobar\]\[1.0.0\]/im
     end
@@ -211,7 +211,7 @@ describe Chef::Knife::CookbookDelete do
       it 'should display the cookbook data' do
         object = ''
         @knife.config[:print_after] = true
-        @knife.stub!(:delete_request).and_return(object)
+        @knife.stub(:delete_request).and_return(object)
         @knife.should_receive(:format_for_display).with(object)
         @knife.delete_version_without_confirmation('1.0.0')
       end

data/spec/unit/knife/cookbook_download_spec.rb

@@ -22,7 +22,7 @@ describe Chef::Knife::CookbookDownload do
   before(:each) do
     @knife = Chef::Knife::CookbookDownload.new
     @stdout = StringIO.new
-    @knife.ui.stub!(:stdout).and_return(@stdout)
+    @knife.ui.stub(:stdout).and_return(@stdout)
   end
 
   describe 'run' do
@@ -44,7 +44,7 @@ describe Chef::Knife::CookbookDownload do
       before(:each) do
         @knife.name_args = ['foobar']
         @knife.config[:download_directory] = '/var/tmp/chef'
-        @rest_mock = mock('rest')
+        @rest_mock = double('rest')
         @knife.stub(:rest).and_return(@rest_mock)
 
         @manifest_data = {
@@ -66,18 +66,18 @@ describe Chef::Knife::CookbookDownload do
           ]
         }
 
-        @cookbook_mock = mock('cookbook')
-        @cookbook_mock.stub!(:version).and_return('1.0.0')
-        @cookbook_mock.stub!(:manifest).and_return(@manifest_data)
+        @cookbook_mock = double('cookbook')
+        @cookbook_mock.stub(:version).and_return('1.0.0')
+        @cookbook_mock.stub(:manifest).and_return(@manifest_data)
         @rest_mock.should_receive(:get_rest).with('cookbooks/foobar/1.0.0').
                                              and_return(@cookbook_mock)
       end
 
       it 'should determine which version if one was not explicitly specified'do
-        @cookbook_mock.stub!(:manifest).and_return({})
+        @cookbook_mock.stub(:manifest).and_return({})
         @knife.should_receive(:determine_version).and_return('1.0.0')
         File.should_receive(:exists?).with('/var/tmp/chef/foobar-1.0.0').and_return(false)
-        Chef::CookbookVersion.stub!(:COOKBOOK_SEGEMENTS).and_return([])
+        Chef::CookbookVersion.stub(:COOKBOOK_SEGEMENTS).and_return([])
         @knife.run
       end
 
@@ -87,8 +87,8 @@ describe Chef::Knife::CookbookDownload do
           @files = @manifest_data.values.map { |v| v.map { |i| i['path'] } }.flatten.uniq
           @files_mocks = {}
           @files.map { |f| File.basename(f) }.flatten.uniq.each do |f|
-            @files_mocks[f] = mock("#{f}_mock")
-            @files_mocks[f].stub!(:path).and_return("/var/tmp/#{f}")
+            @files_mocks[f] = double("#{f}_mock")
+            @files_mocks[f].stub(:path).and_return("/var/tmp/#{f}")
           end
         end
 
@@ -207,7 +207,7 @@ describe Chef::Knife::CookbookDownload do
   describe 'ask_which_version' do
     before(:each) do
       @knife.cookbook_name = 'foobar'
-      @knife.stub!(:available_versions).and_return(['1.0.0', '1.1.0', '2.0.0'])
+      @knife.stub(:available_versions).and_return(['1.0.0', '1.1.0', '2.0.0'])
     end
 
     it 'should prompt the user to select a version' do

data/spec/unit/knife/cookbook_list_spec.rb

@@ -21,8 +21,8 @@ require 'spec_helper'
 describe Chef::Knife::CookbookList do
   before do
     @knife = Chef::Knife::CookbookList.new
-    @rest_mock = mock('rest')
-    @knife.stub!(:rest).and_return(@rest_mock)
+    @rest_mock = double('rest')
+    @knife.stub(:rest).and_return(@rest_mock)
     @cookbook_names = ['apache2', 'mysql']
     @base_url = 'https://server.example.com/cookbooks'
     @cookbook_data = {}
@@ -32,7 +32,7 @@ describe Chef::Knife::CookbookList do
                                               'url' => "#{@base_url}/#{item}/1.0.1"}]}
     end
     @stdout = StringIO.new
-    @knife.ui.stub!(:stdout).and_return(@stdout)
+    @knife.ui.stub(:stdout).and_return(@stdout)
   end
 
   describe 'run' do

data/spec/unit/knife/cookbook_metadata_from_file_spec.rb

@@ -27,10 +27,10 @@ describe Chef::Knife::CookbookMetadataFromFile do
     @tgt = File.expand_path(File.join(CHEF_SPEC_DATA, "metadata", "quick_start", "metadata.json"))
     @knife = Chef::Knife::CookbookMetadataFromFile.new
     @knife.name_args = [ @src ]
-    @knife.stub!(:to_json_pretty).and_return(true)
+    @knife.stub(:to_json_pretty).and_return(true)
     @md = Chef::Cookbook::Metadata.new
     Chef::Cookbook::Metadata.stub(:new).and_return(@md)
-    $stdout.stub!(:write)
+    $stdout.stub(:write)
   end
 
   after do

data/spec/unit/knife/cookbook_metadata_spec.rb

@@ -26,8 +26,8 @@ describe Chef::Knife::CookbookMetadata do
     @json_data = '{ "version": "1.0.0" }'
     @stdout = StringIO.new
     @stderr = StringIO.new
-    @knife.ui.stub!(:stdout).and_return(@stdout)
-    @knife.ui.stub!(:stderr).and_return(@stderr)
+    @knife.ui.stub(:stdout).and_return(@stdout)
+    @knife.ui.stub(:stderr).and_return(@stderr)
   end
 
   describe 'run' do
@@ -84,7 +84,7 @@ describe Chef::Knife::CookbookMetadata do
   describe 'generate_metadata' do
     before(:each) do
       @knife.config[:cookbook_path] = @cookbook_dir
-      File.stub!(:expand_path).with("#{@cookbook_dir}/foobar/metadata.rb").
+      File.stub(:expand_path).with("#{@cookbook_dir}/foobar/metadata.rb").
                                     and_return("#{@cookbook_dir}/foobar/metadata.rb")
     end
 
@@ -105,12 +105,12 @@ describe Chef::Knife::CookbookMetadata do
 
   describe 'generate_metadata_from_file' do
     before(:each) do
-      @metadata_mock = mock('metadata')
-      @json_file_mock = mock('json_file')
+      @metadata_mock = double('metadata')
+      @json_file_mock = double('json_file')
     end
 
     it 'should generate the metatdata json from metatdata.rb' do
-      Chef::Cookbook::Metadata.stub!(:new).and_return(@metadata_mock)
+      Chef::Cookbook::Metadata.stub(:new).and_return(@metadata_mock)
       @metadata_mock.should_receive(:name).with('foobar')
       @metadata_mock.should_receive(:from_file).with("#{@cookbook_dir}/foobar/metadata.rb")
       File.should_receive(:open).with("#{@cookbook_dir}/foobar/metadata.json", 'w').
@@ -127,7 +127,7 @@ describe Chef::Knife::CookbookMetadata do
     }.each_pair do |klass, description|
       it "should print an error and exit when an #{description} syntax exception is encountered" do
         exception = klass.new("#{description} blah")
-        Chef::Cookbook::Metadata.stub!(:new).and_raise(exception)
+        Chef::Cookbook::Metadata.stub(:new).and_raise(exception)
         lambda {
           @knife.generate_metadata_from_file('foobar', "#{@cookbook_dir}/foobar/metadata.rb")
         }.should raise_error(SystemExit)
@@ -165,7 +165,7 @@ describe Chef::Knife::CookbookMetadata do
         IO.should_receive(:read).with("#{@cookbook_dir}/foobar/metadata.json").
                                  and_return(@json_data)
         exception = klass.new("#{description} blah")
-        Chef::Cookbook::Metadata.stub!(:validate_json).and_raise(exception)
+        Chef::Cookbook::Metadata.stub(:validate_json).and_raise(exception)
         lambda {
           @knife.validate_metadata_json(@cookbook_dir, 'foobar')
         }.should raise_error(SystemExit)

data/spec/unit/knife/cookbook_show_spec.rb

@@ -25,10 +25,10 @@ describe Chef::Knife::CookbookShow do
     @knife = Chef::Knife::CookbookShow.new
     @knife.config = { }
     @knife.name_args = [ "cookbook_name" ]
-    @rest = mock(Chef::REST)
-    @knife.stub!(:rest).and_return(@rest)
-    @knife.stub!(:pretty_print).and_return(true)
-    @knife.stub!(:output).and_return(true)
+    @rest = double(Chef::REST)
+    @knife.stub(:rest).and_return(@rest)
+    @knife.stub(:pretty_print).and_return(true)
+    @knife.stub(:output).and_return(true)
   end
 
   describe "run" do

data/spec/unit/knife/cookbook_site_download_spec.rb

@@ -24,7 +24,7 @@ describe Chef::Knife::CookbookSiteDownload do
     before do
       @knife            = Chef::Knife::CookbookSiteDownload.new
       @knife.name_args  = ['apache2']
-      @noauth_rest      = mock 'no auth rest'
+      @noauth_rest      = double('no auth rest')
       @stdout           = StringIO.new
       @cookbook_api_url = 'http://cookbooks.opscode.com/api/v1/cookbooks'
       @version          = '1.0.2'
@@ -58,7 +58,7 @@ describe Chef::Knife::CookbookSiteDownload do
       before do
         @cookbook_data = { 'version' => @version,
                            'file'    => "http://example.com/apache2_#{@version_us}.tgz" }
-        @temp_file     =  stub :path => "/tmp/apache2_#{@version_us}.tgz"
+        @temp_file     =  double( :path => "/tmp/apache2_#{@version_us}.tgz" )
         @file          = File.join(Dir.pwd, "apache2-#{@version}.tar.gz")
 
         @noauth_rest.should_receive(:sign_on_redirect=).with(false)
@@ -125,7 +125,7 @@ describe Chef::Knife::CookbookSiteDownload do
           @version_us      = @version.gsub '.', '_'
           @cookbook_data   = { 'version' => @version,
                                'file'    => "http://example.com/apache2_#{@version_us}.tgz" }
-          @temp_file       = stub :path => "/tmp/apache2_#{@version_us}.tgz"
+          @temp_file       = double(:path => "/tmp/apache2_#{@version_us}.tgz")
           @file            = File.join(Dir.pwd, "apache2-#{@version}.tar.gz")
           @knife.name_args << @version
         end

data/spec/unit/knife/cookbook_site_install_spec.rb

@@ -32,18 +32,18 @@ describe Chef::Knife::CookbookSiteInstall do
 
     @stdout = StringIO.new
     @stderr = StringIO.new
-    @knife.stub!(:stderr).and_return(@stdout)
-    @knife.stub!(:stdout).and_return(@stdout)
+    @knife.stub(:stderr).and_return(@stdout)
+    @knife.stub(:stdout).and_return(@stdout)
 
     #Assume all external commands would have succeed. :(
-    File.stub!(:unlink)
-    File.stub!(:rmtree)
-    @knife.stub!(:shell_out!).and_return(true)
+    File.stub(:unlink)
+    File.stub(:rmtree)
+    @knife.stub(:shell_out!).and_return(true)
 
     #CookbookSiteDownload Stup
     @downloader = {}
-    @knife.stub!(:download_cookbook_to).and_return(@downloader)
-    @downloader.stub!(:version).and_return do
+    @knife.stub(:download_cookbook_to).and_return(@downloader)
+    @downloader.stub(:version).and_return do
       if @knife.name_args.size == 2
         @knife.name_args[1]
       else
@@ -55,7 +55,7 @@ describe Chef::Knife::CookbookSiteInstall do
     @repo = stub(:sanity_check => true, :reset_to_default_state => true,
                  :prepare_to_import => true, :finalize_updates_to => true,
                  :merge_updates_from => true)
-    Chef::Knife::CookbookSCMRepo.stub!(:new).and_return(@repo)
+    Chef::Knife::CookbookSCMRepo.stub(:new).and_return(@repo)
   end
 
 

data/spec/unit/knife/cookbook_site_share_spec.rb

@@ -29,25 +29,25 @@ describe Chef::Knife::CookbookSiteShare do
 
     @cookbook = Chef::CookbookVersion.new('cookbook_name')
 
-    @cookbook_loader = mock('Chef::CookbookLoader')
-    @cookbook_loader.stub!(:cookbook_exists?).and_return(true)
-    @cookbook_loader.stub!(:[]).and_return(@cookbook)
-    Chef::CookbookLoader.stub!(:new).and_return(@cookbook_loader)
+    @cookbook_loader = double('Chef::CookbookLoader')
+    @cookbook_loader.stub(:cookbook_exists?).and_return(true)
+    @cookbook_loader.stub(:[]).and_return(@cookbook)
+    Chef::CookbookLoader.stub(:new).and_return(@cookbook_loader)
 
     @cookbook_uploader = Chef::CookbookUploader.new('herpderp', File.join(CHEF_SPEC_DATA, 'cookbooks'), :rest => "norest")
-    Chef::CookbookUploader.stub!(:new).and_return(@cookbook_uploader)
-    @cookbook_uploader.stub!(:validate_cookbooks).and_return(true)
-    Chef::CookbookSiteStreamingUploader.stub!(:create_build_dir).and_return(Dir.mktmpdir)
+    Chef::CookbookUploader.stub(:new).and_return(@cookbook_uploader)
+    @cookbook_uploader.stub(:validate_cookbooks).and_return(true)
+    Chef::CookbookSiteStreamingUploader.stub(:create_build_dir).and_return(Dir.mktmpdir)
 
     Chef::Mixin::Command.stub(:run_command).and_return(true)
     @stdout = StringIO.new
-    @knife.ui.stub!(:stdout).and_return(@stdout)
+    @knife.ui.stub(:stdout).and_return(@stdout)
   end
 
   describe 'run' do
 
     before(:each) do
-      @knife.stub!(:do_upload).and_return(true)
+      @knife.stub(:do_upload).and_return(true)
     end
 
     it 'should should print usage and exit when given no arguments' do
@@ -83,7 +83,7 @@ describe Chef::Knife::CookbookSiteShare do
     end
 
     it 'should exit and log to error when the tarball creation fails' do
-      Chef::Mixin::Command.stub!(:run_command).and_raise(Chef::Exceptions::Exec)
+      Chef::Mixin::Command.stub(:run_command).and_raise(Chef::Exceptions::Exec)
       @knife.ui.should_receive(:error)
       lambda { @knife.run }.should raise_error(SystemExit)
     end
@@ -98,44 +98,44 @@ describe Chef::Knife::CookbookSiteShare do
   describe 'do_upload' do
 
     before(:each) do
-      @upload_response = mock('Net::HTTPResponse')
-      Chef::CookbookSiteStreamingUploader.stub!(:post).and_return(@upload_response)
+      @upload_response = double('Net::HTTPResponse')
+      Chef::CookbookSiteStreamingUploader.stub(:post).and_return(@upload_response)
 
       @stdout = StringIO.new
       @stderr = StringIO.new
-      @knife.ui.stub!(:stdout).and_return(@stdout)
-      @knife.ui.stub!(:stderr).and_return(@stderr)
+      @knife.ui.stub(:stdout).and_return(@stdout)
+      @knife.ui.stub(:stderr).and_return(@stderr)
       File.stub(:open).and_return(true)
     end
 
     it 'should post the cookbook to "http://cookbooks.opscode.com"' do
       response_text = {:uri => 'http://cookbooks.opscode.com/cookbooks/cookbook_name'}.to_json
-      @upload_response.stub!(:body).and_return(response_text)
-      @upload_response.stub!(:code).and_return(201)
+      @upload_response.stub(:body).and_return(response_text)
+      @upload_response.stub(:code).and_return(201)
       Chef::CookbookSiteStreamingUploader.should_receive(:post).with(/cookbooks\.opscode\.com/, anything(), anything(), anything())
       @knife.run
     end
 
     it 'should alert the user when a version already exists' do
       response_text = {:error_messages => ['Version already exists']}.to_json
-      @upload_response.stub!(:body).and_return(response_text)
-      @upload_response.stub!(:code).and_return(409)
+      @upload_response.stub(:body).and_return(response_text)
+      @upload_response.stub(:code).and_return(409)
       lambda { @knife.run }.should raise_error(SystemExit)
       @stderr.string.should match(/ERROR(.+)cookbook already exists/)
     end
 
     it 'should pass any errors on to the user' do
       response_text = {:error_messages => ["You're holding it wrong"]}.to_json
-      @upload_response.stub!(:body).and_return(response_text)
-      @upload_response.stub!(:code).and_return(403)
+      @upload_response.stub(:body).and_return(response_text)
+      @upload_response.stub(:code).and_return(403)
       lambda { @knife.run }.should raise_error(SystemExit)
       @stderr.string.should match("ERROR(.*)You're holding it wrong")
     end
 
     it 'should print the body if no errors are exposed on failure' do
       response_text = {:system_error => "Your call was dropped", :reason => "There's a map for that"}.to_json
-      @upload_response.stub!(:body).and_return(response_text)
-      @upload_response.stub!(:code).and_return(500)
+      @upload_response.stub(:body).and_return(response_text)
+      @upload_response.stub(:code).and_return(500)
       @knife.ui.should_receive(:error).with(/#{Regexp.escape(response_text)}/)#.ordered
       @knife.ui.should_receive(:error).with(/Unknown error/)#.ordered
       lambda { @knife.run }.should raise_error(SystemExit)

data/spec/unit/knife/cookbook_site_unshare_spec.rb

@@ -24,13 +24,13 @@ describe Chef::Knife::CookbookSiteUnshare do
   before(:each) do
     @knife = Chef::Knife::CookbookSiteUnshare.new
     @knife.name_args = ['cookbook_name']
-    @knife.stub!(:confirm).and_return(true)
+    @knife.stub(:confirm).and_return(true)
 
-    @rest = mock('Chef::REST')
-    @rest.stub!(:delete_rest).and_return(true)
-    @knife.stub!(:rest).and_return(@rest)
+    @rest = double('Chef::REST')
+    @rest.stub(:delete_rest).and_return(true)
+    @knife.stub(:rest).and_return(@rest)
     @stdout = StringIO.new
-    @knife.ui.stub!(:stdout).and_return(@stdout)
+    @knife.ui.stub(:stdout).and_return(@stdout)
   end
 
   describe 'run' do
@@ -55,14 +55,14 @@ describe Chef::Knife::CookbookSiteUnshare do
     end
 
     it 'should log an error and exit when forbidden' do
-      exception = mock('403 "Forbidden"', :code => '403')
-      @rest.stub!(:delete_rest).and_raise(Net::HTTPServerException.new('403 "Forbidden"', exception))
+      exception = double('403 "Forbidden"', :code => '403')
+      @rest.stub(:delete_rest).and_raise(Net::HTTPServerException.new('403 "Forbidden"', exception))
       @knife.ui.should_receive(:error)
       lambda { @knife.run }.should raise_error(SystemExit)
     end
 
     it 'should re-raise any non-forbidden errors on delete_rest' do
-      exception = mock('500 "Application Error"', :code => '500')
+      exception = double('500 "Application Error"', :code => '500')
       @rest.stub(:delete_rest).and_raise(Net::HTTPServerException.new('500 "Application Error"', exception))
       lambda { @knife.run }.should raise_error(Net::HTTPServerException)
     end