chef 12.6.0-universal-mingw32 → 12.7.2-universal-mingw32

data/spec/unit/knife/cookbook_delete_spec.rb

@@ -1,6 +1,6 @@
 #
 # Author:: Thomas Bishop (<bishop.thomas@gmail.com>)
-# Copyright:: Copyright (c) 2011 Thomas Bishop
+# Copyright:: Copyright 2011-2016, Thomas Bishop
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -16,54 +16,54 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Chef::Knife::CookbookDelete do
   before(:each) do
     @knife = Chef::Knife::CookbookDelete.new
-    @knife.name_args = ['foobar']
-    @knife.cookbook_name = 'foobar'
+    @knife.name_args = ["foobar"]
+    @knife.cookbook_name = "foobar"
     @stdout = StringIO.new
     allow(@knife.ui).to receive(:stdout).and_return(@stdout)
     @stderr = StringIO.new
     allow(@knife.ui).to receive(:stderr).and_return(@stderr)
   end
 
-  describe 'run' do
-    it 'should print usage and exit when a cookbook name is not provided' do
+  describe "run" do
+    it "should print usage and exit when a cookbook name is not provided" do
       @knife.name_args = []
       expect(@knife).to receive(:show_usage)
       expect(@knife.ui).to receive(:fatal)
       expect { @knife.run }.to raise_error(SystemExit)
     end
 
-    describe 'when specifying a cookbook name' do
-      it 'should delete the cookbook without a specific version' do
+    describe "when specifying a cookbook name" do
+      it "should delete the cookbook without a specific version" do
         expect(@knife).to receive(:delete_without_explicit_version)
         @knife.run
       end
 
-      describe 'and a version' do
-        it 'should delete the specific version of the cookbook' do
-          @knife.name_args << '1.0.0'
+      describe "and a version" do
+        it "should delete the specific version of the cookbook" do
+          @knife.name_args << "1.0.0"
           expect(@knife).to receive(:delete_explicit_version)
           @knife.run
         end
       end
 
-      describe 'with -a or --all' do
-        it 'should delete all versions of the cookbook' do
+      describe "with -a or --all" do
+        it "should delete all versions of the cookbook" do
           @knife.config[:all] = true
           expect(@knife).to receive(:delete_all_versions)
           @knife.run
         end
       end
 
-      describe 'with -p or --purge' do
-        it 'should prompt to purge the files' do
+      describe "with -p or --purge" do
+        it "should prompt to purge the files" do
           @knife.config[:purge] = true
           expect(@knife).to receive(:confirm).
-                 with(/.+Are you sure you want to purge files.+/)
+            with(/.+Are you sure you want to purge files.+/)
           expect(@knife).to receive(:delete_without_explicit_version)
           @knife.run
         end
@@ -71,30 +71,30 @@ describe Chef::Knife::CookbookDelete do
     end
   end
 
-  describe 'delete_explicit_version' do
-    it 'should delete the specific cookbook version' do
-      @knife.cookbook_name = 'foobar'
-      @knife.version = '1.0.0'
+  describe "delete_explicit_version" do
+    it "should delete the specific cookbook version" do
+      @knife.cookbook_name = "foobar"
+      @knife.version = "1.0.0"
       expect(@knife).to receive(:delete_object).with(Chef::CookbookVersion,
-                                                 'foobar version 1.0.0',
-                                                 'cookbook').and_yield()
-      expect(@knife).to receive(:delete_request).with('cookbooks/foobar/1.0.0')
+                                                 "foobar version 1.0.0",
+                                                 "cookbook").and_yield()
+      expect(@knife).to receive(:delete_request).with("cookbooks/foobar/1.0.0")
       @knife.delete_explicit_version
     end
   end
 
-  describe 'delete_all_versions' do
-    it 'should prompt to delete all versions of the cookbook' do
-      @knife.cookbook_name = 'foobar'
-      expect(@knife).to receive(:confirm).with('Do you really want to delete all versions of foobar')
+  describe "delete_all_versions" do
+    it "should prompt to delete all versions of the cookbook" do
+      @knife.cookbook_name = "foobar"
+      expect(@knife).to receive(:confirm).with("Do you really want to delete all versions of foobar")
       expect(@knife).to receive(:delete_all_without_confirmation)
       @knife.delete_all_versions
     end
   end
 
-  describe 'delete_all_without_confirmation' do
-    it 'should delete all versions without confirmation' do
-      versions = ['1.0.0', '1.1.0']
+  describe "delete_all_without_confirmation" do
+    it "should delete all versions without confirmation" do
+      versions = ["1.0.0", "1.1.0"]
       expect(@knife).to receive(:available_versions).and_return(versions)
       versions.each do |v|
         expect(@knife).to receive(:delete_version_without_confirmation).with(v)
@@ -103,132 +103,132 @@ describe Chef::Knife::CookbookDelete do
     end
   end
 
-  describe 'delete_without_explicit_version' do
-    it 'should exit if there are no available versions' do
+  describe "delete_without_explicit_version" do
+    it "should exit if there are no available versions" do
       expect(@knife).to receive(:available_versions).and_return(nil)
       expect { @knife.delete_without_explicit_version }.to raise_error(SystemExit)
     end
 
-    it 'should delete the version if only one is found' do
-      expect(@knife).to receive(:available_versions).at_least(:once).and_return(['1.0.0'])
+    it "should delete the version if only one is found" do
+      expect(@knife).to receive(:available_versions).at_least(:once).and_return(["1.0.0"])
       expect(@knife).to receive(:delete_explicit_version)
       @knife.delete_without_explicit_version
     end
 
-    it 'should ask which version(s) to delete if multiple are found' do
-      expect(@knife).to receive(:available_versions).at_least(:once).and_return(['1.0.0', '1.1.0'])
-      expect(@knife).to receive(:ask_which_versions_to_delete).and_return(['1.0.0', '1.1.0'])
-      expect(@knife).to receive(:delete_versions_without_confirmation).with(['1.0.0', '1.1.0'])
+    it "should ask which version(s) to delete if multiple are found" do
+      expect(@knife).to receive(:available_versions).at_least(:once).and_return(["1.0.0", "1.1.0"])
+      expect(@knife).to receive(:ask_which_versions_to_delete).and_return(["1.0.0", "1.1.0"])
+      expect(@knife).to receive(:delete_versions_without_confirmation).with(["1.0.0", "1.1.0"])
       @knife.delete_without_explicit_version
     end
   end
 
-  describe 'available_versions' do
+  describe "available_versions" do
     before(:each) do
-      @rest_mock = double('rest')
+      @rest_mock = double("rest")
       expect(@knife).to receive(:rest).and_return(@rest_mock)
-      @cookbook_data = { 'foobar' => { 'versions' => [{'version' => '1.0.0'},
-                                                      {'version' => '1.1.0'},
-                                                      {'version' => '2.0.0'} ]}
+      @cookbook_data = { "foobar" => { "versions" => [{ "version" => "1.0.0" },
+                                                      { "version" => "1.1.0" },
+                                                      { "version" => "2.0.0" } ] }
       }
     end
 
-    it 'should return the list of versions of the cookbook' do
-      expect(@rest_mock).to receive(:get_rest).with('cookbooks/foobar').and_return(@cookbook_data)
-      expect(@knife.available_versions).to eq(['1.0.0', '1.1.0', '2.0.0'])
+    it "should return the list of versions of the cookbook" do
+      expect(@rest_mock).to receive(:get).with("cookbooks/foobar").and_return(@cookbook_data)
+      expect(@knife.available_versions).to eq(["1.0.0", "1.1.0", "2.0.0"])
     end
 
-    it 'should raise if an error other than HTTP 404 is returned' do
-      exception = Net::HTTPServerException.new('500 Internal Server Error', '500')
-      expect(@rest_mock).to receive(:get_rest).and_raise(exception)
+    it "should raise if an error other than HTTP 404 is returned" do
+      exception = Net::HTTPServerException.new("500 Internal Server Error", "500")
+      expect(@rest_mock).to receive(:get).and_raise(exception)
       expect { @knife.available_versions }.to raise_error Net::HTTPServerException
     end
 
     describe "if the cookbook can't be found" do
       before(:each) do
-        expect(@rest_mock).to receive(:get_rest).
-          and_raise(Net::HTTPServerException.new('404 Not Found', '404'))
+        expect(@rest_mock).to receive(:get).
+          and_raise(Net::HTTPServerException.new("404 Not Found", "404"))
       end
 
-      it 'should print an error' do
+      it "should print an error" do
         @knife.available_versions
         expect(@stderr.string).to match /error.+cannot find a cookbook named foobar/i
       end
 
-      it 'should return nil' do
+      it "should return nil" do
         expect(@knife.available_versions).to eq(nil)
       end
     end
   end
 
-  describe 'ask_which_version_to_delete' do
+  describe "ask_which_version_to_delete" do
     before(:each) do
-      allow(@knife).to receive(:available_versions).and_return(['1.0.0', '1.1.0', '2.0.0'])
+      allow(@knife).to receive(:available_versions).and_return(["1.0.0", "1.1.0", "2.0.0"])
     end
 
-    it 'should prompt the user to select a version' do
+    it "should prompt the user to select a version" do
       prompt = /Which version\(s\) do you want to delete\?.+1\. foobar 1\.0\.0.+2\. foobar 1\.1\.0.+3\. foobar 2\.0\.0.+4\. All versions.+/m
-      expect(@knife).to receive(:ask_question).with(prompt).and_return('1')
+      expect(@knife).to receive(:ask_question).with(prompt).and_return("1")
       @knife.ask_which_versions_to_delete
     end
 
     it "should print an error and exit if a version wasn't specified" do
-      expect(@knife).to receive(:ask_question).and_return('')
+      expect(@knife).to receive(:ask_question).and_return("")
       expect(@knife.ui).to receive(:error).with(/no versions specified/i)
       expect { @knife.ask_which_versions_to_delete }.to raise_error(SystemExit)
     end
 
-    it 'should print an error if an invalid choice was selected' do
-      expect(@knife).to receive(:ask_question).and_return('100')
+    it "should print an error if an invalid choice was selected" do
+      expect(@knife).to receive(:ask_question).and_return("100")
       expect(@knife.ui).to receive(:error).with(/100 is not a valid choice/i)
       @knife.ask_which_versions_to_delete
     end
 
-    it 'should return the selected versions' do
-      expect(@knife).to receive(:ask_question).and_return('1, 3')
-      expect(@knife.ask_which_versions_to_delete).to eq(['1.0.0', '2.0.0'])
+    it "should return the selected versions" do
+      expect(@knife).to receive(:ask_question).and_return("1, 3")
+      expect(@knife.ask_which_versions_to_delete).to eq(["1.0.0", "2.0.0"])
     end
 
     it "should return all of the versions if 'all' was selected" do
-      expect(@knife).to receive(:ask_question).and_return('4')
+      expect(@knife).to receive(:ask_question).and_return("4")
       expect(@knife.ask_which_versions_to_delete).to eq([:all])
     end
   end
 
-  describe 'delete_version_without_confirmation' do
-    it 'should delete the cookbook version' do
-      expect(@knife).to receive(:delete_request).with('cookbooks/foobar/1.0.0')
-      @knife.delete_version_without_confirmation('1.0.0')
+  describe "delete_version_without_confirmation" do
+    it "should delete the cookbook version" do
+      expect(@knife).to receive(:delete_request).with("cookbooks/foobar/1.0.0")
+      @knife.delete_version_without_confirmation("1.0.0")
     end
 
-    it 'should output that the cookbook was deleted' do
+    it "should output that the cookbook was deleted" do
       allow(@knife).to receive(:delete_request)
-      @knife.delete_version_without_confirmation('1.0.0')
+      @knife.delete_version_without_confirmation("1.0.0")
       expect(@stderr.string).to match /deleted cookbook\[foobar\]\[1.0.0\]/im
     end
 
-    describe 'with --print-after' do
-      it 'should display the cookbook data' do
-        object = ''
+    describe "with --print-after" do
+      it "should display the cookbook data" do
+        object = ""
         @knife.config[:print_after] = true
         allow(@knife).to receive(:delete_request).and_return(object)
         expect(@knife).to receive(:format_for_display).with(object)
-        @knife.delete_version_without_confirmation('1.0.0')
+        @knife.delete_version_without_confirmation("1.0.0")
       end
     end
   end
 
-  describe 'delete_versions_without_confirmation' do
-    it 'should delete each version without confirmation' do
-      versions = ['1.0.0', '1.1.0']
+  describe "delete_versions_without_confirmation" do
+    it "should delete each version without confirmation" do
+      versions = ["1.0.0", "1.1.0"]
       versions.each do |v|
         expect(@knife).to receive(:delete_version_without_confirmation).with(v)
       end
       @knife.delete_versions_without_confirmation(versions)
     end
 
-    describe 'with -a or --all' do
-      it 'should delete all versions without confirmation' do
+    describe "with -a or --all" do
+      it "should delete all versions without confirmation" do
         versions = [:all]
         expect(@knife).to receive(:delete_all_without_confirmation)
         @knife.delete_versions_without_confirmation(versions)

data/spec/unit/knife/cookbook_download_spec.rb

@@ -1,6 +1,6 @@
 #
 # Author:: Thomas Bishop (<bishop.thomas@gmail.com>)
-# Copyright:: Copyright (c) 2011 Thomas Bishop
+# Copyright:: Copyright 2011-2016, Thomas Bishop
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -16,7 +16,7 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Chef::Knife::CookbookDownload do
   before(:each) do
@@ -25,66 +25,66 @@ describe Chef::Knife::CookbookDownload do
     allow(@knife.ui).to receive(:stderr).and_return(@stderr)
   end
 
-  describe 'run' do
-    it 'should print usage and exit when a cookbook name is not provided' do
+  describe "run" do
+    it "should print usage and exit when a cookbook name is not provided" do
       @knife.name_args = []
       expect(@knife).to receive(:show_usage)
       expect(@knife.ui).to receive(:fatal).with(/must specify a cookbook name/)
       expect { @knife.run }.to raise_error(SystemExit)
     end
 
-    it 'should exit with a fatal error when there is no cookbook on the server' do
-      @knife.name_args = ['foobar', nil]
+    it "should exit with a fatal error when there is no cookbook on the server" do
+      @knife.name_args = ["foobar", nil]
       expect(@knife).to receive(:determine_version).and_return(nil)
-      expect(@knife.ui).to receive(:fatal).with('No such cookbook found')
+      expect(@knife.ui).to receive(:fatal).with("No such cookbook found")
       expect { @knife.run }.to raise_error(SystemExit)
     end
 
-    describe 'with a cookbook name' do
+    describe "with a cookbook name" do
       before(:each) do
-        @knife.name_args = ['foobar']
-        @knife.config[:download_directory] = '/var/tmp/chef'
-        @rest_mock = double('rest')
+        @knife.name_args = ["foobar"]
+        @knife.config[:download_directory] = "/var/tmp/chef"
+        @rest_mock = double("rest")
         allow(@knife).to receive(:rest).and_return(@rest_mock)
 
         @manifest_data = {
           :recipes => [
-            {'path' => 'recipes/foo.rb',
-             'url' => 'http://example.org/files/foo.rb'},
-            {'path' => 'recipes/bar.rb',
-             'url' => 'http://example.org/files/bar.rb'}
+            { "path" => "recipes/foo.rb",
+              "url" => "http://example.org/files/foo.rb" },
+            { "path" => "recipes/bar.rb",
+              "url" => "http://example.org/files/bar.rb" },
           ],
           :templates => [
-            {'path' => 'templates/default/foo.erb',
-             'url' => 'http://example.org/files/foo.erb'},
-            {'path' => 'templates/default/bar.erb',
-             'url' => 'http://example.org/files/bar.erb'}
+            { "path" => "templates/default/foo.erb",
+              "url" => "http://example.org/files/foo.erb" },
+            { "path" => "templates/default/bar.erb",
+              "url" => "http://example.org/files/bar.erb" },
           ],
           :attributes => [
-            {'path' => 'attributes/default.rb',
-             'url' => 'http://example.org/files/default.rb'}
-          ]
+            { "path" => "attributes/default.rb",
+              "url" => "http://example.org/files/default.rb" },
+          ],
         }
 
-        @cookbook_mock = double('cookbook')
-        allow(@cookbook_mock).to receive(:version).and_return('1.0.0')
+        @cookbook_mock = double("cookbook")
+        allow(@cookbook_mock).to receive(:version).and_return("1.0.0")
         allow(@cookbook_mock).to receive(:manifest).and_return(@manifest_data)
-        expect(@rest_mock).to receive(:get_rest).with('cookbooks/foobar/1.0.0').
-                                             and_return(@cookbook_mock)
+        expect(Chef::CookbookVersion).to receive(:load).with("foobar", "1.0.0").
+          and_return(@cookbook_mock)
       end
 
-      it 'should determine which version if one was not explicitly specified'do
+      it "should determine which version if one was not explicitly specified"do
         allow(@cookbook_mock).to receive(:manifest).and_return({})
-        expect(@knife).to receive(:determine_version).and_return('1.0.0')
-        expect(File).to receive(:exists?).with('/var/tmp/chef/foobar-1.0.0').and_return(false)
+        expect(@knife).to receive(:determine_version).and_return("1.0.0")
+        expect(File).to receive(:exists?).with("/var/tmp/chef/foobar-1.0.0").and_return(false)
         allow(Chef::CookbookVersion).to receive(:COOKBOOK_SEGEMENTS).and_return([])
         @knife.run
       end
 
-      describe 'and a version' do
+      describe "and a version" do
         before(:each) do
-          @knife.name_args << '1.0.0'
-          @files = @manifest_data.values.map { |v| v.map { |i| i['path'] } }.flatten.uniq
+          @knife.name_args << "1.0.0"
+          @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] = double("#{f}_mock")
@@ -92,46 +92,45 @@ describe Chef::Knife::CookbookDownload do
           end
         end
 
-        it 'should print an error and exit if the cookbook download directory already exists' do
-          expect(File).to receive(:exists?).with('/var/tmp/chef/foobar-1.0.0').and_return(true)
+        it "should print an error and exit if the cookbook download directory already exists" do
+          expect(File).to receive(:exists?).with("/var/tmp/chef/foobar-1.0.0").and_return(true)
           expect(@knife.ui).to receive(:fatal).with(/\/var\/tmp\/chef\/foobar-1\.0\.0 exists/i)
           expect { @knife.run }.to raise_error(SystemExit)
         end
 
-        describe 'when downloading the cookbook' do
+        describe "when downloading the cookbook" do
           before(:each) do
             @files.map { |f| File.dirname(f) }.flatten.uniq.each do |dir|
               expect(FileUtils).to receive(:mkdir_p).with("/var/tmp/chef/foobar-1.0.0/#{dir}").
-              at_least(:once)
+                at_least(:once)
             end
 
             @files_mocks.each_pair do |file, mock|
-              expect(@rest_mock).to receive(:get_rest).with("http://example.org/files/#{file}", true).
-              and_return(mock)
+              expect(@rest_mock).to receive(:streaming_request).with("http://example.org/files/#{file}").
+                and_return(mock)
             end
 
-            expect(@rest_mock).to receive(:sign_on_redirect=).with(false).at_least(:once)
             @files.each do |f|
               expect(FileUtils).to receive(:mv).
-                        with("/var/tmp/#{File.basename(f)}", "/var/tmp/chef/foobar-1.0.0/#{f}")
+                with("/var/tmp/#{File.basename(f)}", "/var/tmp/chef/foobar-1.0.0/#{f}")
             end
           end
 
           it "should download the cookbook when the cookbook download directory doesn't exist" do
-            expect(File).to receive(:exists?).with('/var/tmp/chef/foobar-1.0.0').and_return(false)
+            expect(File).to receive(:exists?).with("/var/tmp/chef/foobar-1.0.0").and_return(false)
             @knife.run
-            ['attributes', 'recipes', 'templates'].each do |segment|
+            ["attributes", "recipes", "templates"].each do |segment|
               expect(@stderr.string).to match /downloading #{segment}/im
             end
             expect(@stderr.string).to match /downloading foobar cookbook version 1\.0\.0/im
             expect(@stderr.string).to match /cookbook downloaded to \/var\/tmp\/chef\/foobar-1\.0\.0/im
           end
 
-          describe 'with -f or --force' do
-            it 'should remove the existing the cookbook download directory if it exists' do
+          describe "with -f or --force" do
+            it "should remove the existing the cookbook download directory if it exists" do
               @knife.config[:force] = true
-              expect(File).to receive(:exists?).with('/var/tmp/chef/foobar-1.0.0').and_return(true)
-              expect(FileUtils).to receive(:rm_rf).with('/var/tmp/chef/foobar-1.0.0')
+              expect(File).to receive(:exists?).with("/var/tmp/chef/foobar-1.0.0").and_return(true)
+              expect(FileUtils).to receive(:rm_rf).with("/var/tmp/chef/foobar-1.0.0")
               @knife.run
             end
           end
@@ -142,94 +141,94 @@ describe Chef::Knife::CookbookDownload do
 
   end
 
-  describe 'determine_version' do
+  describe "determine_version" do
 
-    it 'should return nil if there are no versions' do
+    it "should return nil if there are no versions" do
       expect(@knife).to receive(:available_versions).and_return(nil)
       expect(@knife.determine_version).to eq(nil)
       expect(@knife.version).to eq(nil)
     end
 
-    it 'should return and set the version if there is only one version' do
-      expect(@knife).to receive(:available_versions).at_least(:once).and_return(['1.0.0'])
-      expect(@knife.determine_version).to eq('1.0.0')
-      expect(@knife.version).to eq('1.0.0')
+    it "should return and set the version if there is only one version" do
+      expect(@knife).to receive(:available_versions).at_least(:once).and_return(["1.0.0"])
+      expect(@knife.determine_version).to eq("1.0.0")
+      expect(@knife.version).to eq("1.0.0")
     end
 
-    it 'should ask which version to download and return it if there is more than one' do
-      expect(@knife).to receive(:available_versions).at_least(:once).and_return(['1.0.0', '2.0.0'])
-      expect(@knife).to receive(:ask_which_version).and_return('1.0.0')
-      expect(@knife.determine_version).to eq('1.0.0')
+    it "should ask which version to download and return it if there is more than one" do
+      expect(@knife).to receive(:available_versions).at_least(:once).and_return(["1.0.0", "2.0.0"])
+      expect(@knife).to receive(:ask_which_version).and_return("1.0.0")
+      expect(@knife.determine_version).to eq("1.0.0")
     end
 
-    describe 'with -N or --latest' do
-      it 'should return and set the version to the latest version' do
+    describe "with -N or --latest" do
+      it "should return and set the version to the latest version" do
         @knife.config[:latest] = true
         expect(@knife).to receive(:available_versions).at_least(:once).
-                                                   and_return(['1.0.0', '1.1.0', '2.0.0'])
+          and_return(["1.0.0", "1.1.0", "2.0.0"])
         @knife.determine_version
-        expect(@knife.version.to_s).to eq('2.0.0')
+        expect(@knife.version.to_s).to eq("2.0.0")
       end
     end
   end
 
-  describe 'available_versions' do
+  describe "available_versions" do
     before(:each) do
-      @knife.cookbook_name = 'foobar'
+      @knife.cookbook_name = "foobar"
     end
 
-    it 'should return nil if there are no versions' do
+    it "should return nil if there are no versions" do
       expect(Chef::CookbookVersion).to receive(:available_versions).
-                            with('foobar').
-                            and_return(nil)
+        with("foobar").
+        and_return(nil)
       expect(@knife.available_versions).to eq(nil)
     end
 
-    it 'should return the available versions' do
+    it "should return the available versions" do
       expect(Chef::CookbookVersion).to receive(:available_versions).
-                            with('foobar').
-                            and_return(['1.1.0', '2.0.0', '1.0.0'])
-      expect(@knife.available_versions).to eq([Chef::Version.new('1.0.0'),
-                                           Chef::Version.new('1.1.0'),
-                                           Chef::Version.new('2.0.0')])
+        with("foobar").
+        and_return(["1.1.0", "2.0.0", "1.0.0"])
+      expect(@knife.available_versions).to eq([Chef::Version.new("1.0.0"),
+                                           Chef::Version.new("1.1.0"),
+                                           Chef::Version.new("2.0.0")])
     end
 
-    it 'should avoid multiple API calls to the server' do
+    it "should avoid multiple API calls to the server" do
       expect(Chef::CookbookVersion).to receive(:available_versions).
-                            once.
-                            with('foobar').
-                            and_return(['1.1.0', '2.0.0', '1.0.0'])
+        once.
+        with("foobar").
+        and_return(["1.1.0", "2.0.0", "1.0.0"])
       @knife.available_versions
       @knife.available_versions
     end
   end
 
-  describe 'ask_which_version' do
+  describe "ask_which_version" do
     before(:each) do
-      @knife.cookbook_name = 'foobar'
-      allow(@knife).to receive(:available_versions).and_return(['1.0.0', '1.1.0', '2.0.0'])
+      @knife.cookbook_name = "foobar"
+      allow(@knife).to receive(:available_versions).and_return(["1.0.0", "1.1.0", "2.0.0"])
     end
 
-    it 'should prompt the user to select a version' do
+    it "should prompt the user to select a version" do
       prompt = /Which version do you want to download\?.+1\. foobar 1\.0\.0.+2\. foobar 1\.1\.0.+3\. foobar 2\.0\.0.+/m
-      expect(@knife).to receive(:ask_question).with(prompt).and_return('1')
+      expect(@knife).to receive(:ask_question).with(prompt).and_return("1")
       @knife.ask_which_version
     end
 
     it "should set the version to the user's selection" do
-      expect(@knife).to receive(:ask_question).and_return('1')
+      expect(@knife).to receive(:ask_question).and_return("1")
       @knife.ask_which_version
-      expect(@knife.version).to eq('1.0.0')
+      expect(@knife.version).to eq("1.0.0")
     end
 
     it "should print an error and exit if a version wasn't specified" do
-      expect(@knife).to receive(:ask_question).and_return('')
+      expect(@knife).to receive(:ask_question).and_return("")
       expect(@knife.ui).to receive(:error).with(/is not a valid value/i)
       expect { @knife.ask_which_version }.to raise_error(SystemExit)
     end
 
-    it 'should print an error if an invalid choice was selected' do
-      expect(@knife).to receive(:ask_question).and_return('100')
+    it "should print an error if an invalid choice was selected" do
+      expect(@knife).to receive(:ask_question).and_return("100")
       expect(@knife.ui).to receive(:error).with(/'100' is not a valid value/i)
       expect { @knife.ask_which_version }.to raise_error(SystemExit)
     end