knife-openstack 1.3.2 → 2.0.0

data/spec/spec_context.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Author:: Prabhu Das (<prabhu.das@clogeny.com>)
 # Copyright:: Copyright (c) 2014 Chef Software, Inc.
@@ -15,12 +16,12 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-shared_context '#validate!' do |instance|
+shared_context "#validate!" do |instance|
   before(:each) do
-    Chef::Config[:knife][:openstack_username] = 'testuser'
-    Chef::Config[:knife][:openstack_password] = 'testpassword'
-    Chef::Config[:knife][:openstack_auth_url] = 'tsturl'
-    Chef::Config[:knife][:openstack_region] = 'test-region'
+    Chef::Config[:knife][:openstack_username] = "testuser"
+    Chef::Config[:knife][:openstack_password] = "testpassword"
+    Chef::Config[:knife][:openstack_auth_url] = "tsturl"
+    Chef::Config[:knife][:openstack_region] = "test-region"
     allow(instance).to receive(:exit)
   end
 
@@ -31,23 +32,23 @@ shared_context '#validate!' do |instance|
     Chef::Config[:knife].delete(:openstack_region)
   end
 
-  it 'validate openstack mandatory options' do
+  it "validate openstack mandatory options" do
     expect { instance.validate! }.to_not raise_error
   end
 
-  it 'raise error on openstack_username missing' do
+  it "raise error on openstack_username missing" do
     Chef::Config[:knife].delete(:openstack_username)
     expect(instance.ui).to receive(:error).with("You did not provide a valid 'Openstack Username' value.")
     expect { instance.validate! }.to raise_error(Chef::Knife::Cloud::CloudExceptions::ValidationError)
   end
 
-  it 'raise error on openstack_password missing' do
+  it "raise error on openstack_password missing" do
     Chef::Config[:knife].delete(:openstack_password)
     expect(instance.ui).to receive(:error).with("You did not provide a valid 'Openstack Password' value.")
     expect { instance.validate! }.to raise_error(Chef::Knife::Cloud::CloudExceptions::ValidationError)
   end
 
-  it 'raise error on openstack_auth_url missing' do
+  it "raise error on openstack_auth_url missing" do
     Chef::Config[:knife].delete(:openstack_auth_url)
     expect(instance.ui).to receive(:error).with("You did not provide a valid 'Openstack Auth Url' value.")
     expect { instance.validate! }.to raise_error(Chef::Knife::Cloud::CloudExceptions::ValidationError)

data/spec/spec_helper.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Copyright:: Copyright (c) 2013-2014 Chef Software, Inc.
 # License: Apache License, Version 2.0
 #
@@ -15,31 +16,31 @@
 
 # Author:: Siddheshwar More (<siddheshwar.more@clogeny.com>)
 
-$LOAD_PATH.unshift File.expand_path('../../lib', __FILE__)
-require 'chef/knife/bootstrap'
-require 'chef/knife/openstack_helpers'
-require 'fog'
-require 'chef/knife/winrm_base'
-require 'chef/knife/bootstrap_windows_winrm'
-require 'chef/knife/openstack_server_create'
-require 'chef/knife/openstack_server_delete'
-require 'chef/knife/bootstrap_windows_ssh'
-require 'securerandom'
-require 'knife-openstack/version'
-require 'test/knife-utils/test_bed'
-require 'resource_spec_helper'
-require 'server_command_common_spec_helper'
-require 'tempfile'
-require 'spec_context'
+$LOAD_PATH.unshift File.expand_path("../../lib", __FILE__)
+require "chef/knife/bootstrap"
+require "chef/knife/openstack_helpers"
+require "fog"
+require "chef/knife/winrm_base"
+require "chef/knife/bootstrap_windows_winrm"
+require "chef/knife/openstack_server_create"
+require "chef/knife/openstack_server_delete"
+require "chef/knife/bootstrap_windows_ssh"
+require "securerandom"
+require "knife-openstack/version"
+require "test/knife-utils/test_bed"
+require "resource_spec_helper"
+require "server_command_common_spec_helper"
+require "tempfile"
+require "spec_context"
 
 def find_instance_id(instance_name, file)
   file.lines.each do |line|
-    return "#{line}".split(' ')[2].strip if line.include?("#{instance_name}")
+    return "#{line}".split(" ")[2].strip if line.include?("#{instance_name}")
   end
 end
 
 def is_config_present
-  unless ENV['RUN_INTEGRATION_TESTS']
+  unless ENV["RUN_INTEGRATION_TESTS"]
     puts("\nPlease set RUN_INTEGRATION_TESTS environment variable to run integration tests")
     return false
   end
@@ -47,9 +48,9 @@ def is_config_present
   unset_env_var = []
   unset_config_options = []
   is_config = true
-  config_file_exist = File.exist?(File.expand_path('../integration/config/environment.yml', __FILE__))
-  openstack_config = YAML.load(File.read(File.expand_path('../integration/config/environment.yml', __FILE__))) if config_file_exist
-  %w(OPENSTACK_USERNAME OPENSTACK_PASSWORD OPENSTACK_AUTH_URL OPENSTACK_TENANT).each do |os_env_var|
+  config_file_exist = File.exist?(File.expand_path("../integration/config/environment.yml", __FILE__))
+  openstack_config = YAML.load(File.read(File.expand_path("../integration/config/environment.yml", __FILE__))) if config_file_exist
+  %w{OPENSTACK_USERNAME OPENSTACK_PASSWORD OPENSTACK_AUTH_URL OPENSTACK_TENANT}.each do |os_env_var|
     if ENV[os_env_var].nil?
       unset_env_var << os_env_var
       is_config = false
@@ -57,10 +58,10 @@ def is_config_present
   end
 
   err_msg = "\nPlease set #{unset_env_var.join(', ')} environment"
-  err_msg = err_msg + (unset_env_var.length > 1 ? ' variables ' : ' variable ') + 'for integration tests.'
+  err_msg = err_msg + (unset_env_var.length > 1 ? " variables " : " variable ") + "for integration tests."
   puts err_msg unless unset_env_var.empty?
 
-  %w(OS_SSH_USER OPENSTACK_PRI_KEY OPENSTACK_KEY_PAIR OS_WINDOWS_SSH_USER OS_WINDOWS_SSH_PASSWORD OS_WINRM_USER OS_WINRM_PASSWORD OS_LINUX_IMAGE OS_LINUX_FLAVOR OS_INVALID_FLAVOR OS_INVALID_FLOATING_IP OS_WINDOWS_FLAVOR OS_WINDOWS_IMAGE OS_WINDOWS_SSH_IMAGE OS_NETWORK_IDS OS_AVAILABILITY_ZONE).each do |os_config_opt|
+  %w{OS_SSH_USER OPENSTACK_PRI_KEY OPENSTACK_KEY_PAIR OS_WINDOWS_SSH_USER OS_WINDOWS_SSH_PASSWORD OS_WINRM_USER OS_WINRM_PASSWORD OS_LINUX_IMAGE OS_LINUX_FLAVOR OS_INVALID_FLAVOR OS_INVALID_FLOATING_IP OS_WINDOWS_FLAVOR OS_WINDOWS_IMAGE OS_WINDOWS_SSH_IMAGE OS_NETWORK_IDS OS_AVAILABILITY_ZONE}.each do |os_config_opt|
     option_value = ENV[os_config_opt] || (openstack_config[os_config_opt] if openstack_config)
     if option_value.nil?
       unset_config_options << os_config_opt
@@ -69,53 +70,53 @@ def is_config_present
   end
 
   config_err_msg = "\nPlease set #{unset_config_options.join(', ')} config"
-  config_err_msg = config_err_msg + (unset_config_options.length > 1 ? ' options in ../spec/integration/config/environment.yml or as environment variables' : ' option in ../spec/integration/config/environment.yml or as environment variable') + ' for integration tests.'
+  config_err_msg = config_err_msg + (unset_config_options.length > 1 ? " options in ../spec/integration/config/environment.yml or as environment variables" : " option in ../spec/integration/config/environment.yml or as environment variable") + " for integration tests."
   puts config_err_msg unless unset_config_options.empty?
 
   is_config
 end
 
 def get_gem_file_name
-  'knife-openstack-' + Knife::OpenStack::VERSION + '.gem'
+  "knife-openstack-" + Knife::OpenStack::VERSION + ".gem"
 end
 
 def delete_instance_cmd(stdout)
-  'knife openstack server delete ' + find_instance_id('Instance ID', stdout) +
-    append_openstack_creds(is_list_cmd = true) + ' --yes'
+  "knife openstack server delete " + find_instance_id("Instance ID", stdout) +
+    append_openstack_creds(is_list_cmd = true) + " --yes"
 end
 
 def create_node_name(name)
-  @name_node = (name == 'linux') ? "os-integration-test-linux-#{SecureRandom.hex(4)}" : "os-integration-test-win-#{SecureRandom.hex(4)}"
+  @name_node = (name == "linux") ? "os-integration-test-linux-#{SecureRandom.hex(4)}" : "os-integration-test-win-#{SecureRandom.hex(4)}"
 end
 
 def init_openstack_test
   init_test
 
   begin
-    data_to_write = File.read(File.expand_path('../integration/config/incorrect_openstack.pem', __FILE__))
-    File.open("#{temp_dir}/incorrect_openstack.pem", 'w') { |f| f.write(data_to_write) }
+    data_to_write = File.read(File.expand_path("../integration/config/incorrect_openstack.pem", __FILE__))
+    File.open("#{temp_dir}/incorrect_openstack.pem", "w") { |f| f.write(data_to_write) }
   rescue
-    puts 'Error while creating file - incorrect_openstack.pem'
+    puts "Error while creating file - incorrect_openstack.pem"
   end
 
-  config_file_exist = File.exist?(File.expand_path('../integration/config/environment.yml', __FILE__))
-  openstack_config = YAML.load(File.read(File.expand_path('../integration/config/environment.yml', __FILE__))) if config_file_exist
+  config_file_exist = File.exist?(File.expand_path("../integration/config/environment.yml", __FILE__))
+  openstack_config = YAML.load(File.read(File.expand_path("../integration/config/environment.yml", __FILE__))) if config_file_exist
 
-  %w(OS_SSH_USER OPENSTACK_KEY_PAIR OPENSTACK_PRI_KEY OS_WINDOWS_SSH_USER OS_WINDOWS_SSH_PASSWORD OS_WINRM_USER OS_WINRM_PASSWORD OS_LINUX_IMAGE OS_LINUX_FLAVOR OS_INVALID_FLAVOR OS_INVALID_FLOATING_IP OS_WINDOWS_FLAVOR OS_WINDOWS_IMAGE OS_WINDOWS_SSH_IMAGE OS_NETWORK_IDS OS_AVAILABILITY_ZONE).each do |os_config_opt|
+  %w{OS_SSH_USER OPENSTACK_KEY_PAIR OPENSTACK_PRI_KEY OS_WINDOWS_SSH_USER OS_WINDOWS_SSH_PASSWORD OS_WINRM_USER OS_WINRM_PASSWORD OS_LINUX_IMAGE OS_LINUX_FLAVOR OS_INVALID_FLAVOR OS_INVALID_FLOATING_IP OS_WINDOWS_FLAVOR OS_WINDOWS_IMAGE OS_WINDOWS_SSH_IMAGE OS_NETWORK_IDS OS_AVAILABILITY_ZONE}.each do |os_config_opt|
     instance_variable_set("@#{os_config_opt.downcase}", (openstack_config[os_config_opt] if openstack_config) || ENV[os_config_opt])
   end
   begin
     key_file_path = @openstack_pri_key
     key_file_exist = File.exist?(File.expand_path(key_file_path, __FILE__))
     data_to_write = File.read(File.expand_path(key_file_path, __FILE__)) if key_file_exist
-    File.open("#{temp_dir}/openstack.pem", 'w') { |f| f.write(data_to_write) }
+    File.open("#{temp_dir}/openstack.pem", "w") { |f| f.write(data_to_write) }
   rescue
-    puts 'Error while creating file - openstack.pem'
+    puts "Error while creating file - openstack.pem"
   end
 end
 
 def create_sh_user_data_file
-  file = Tempfile.new(['test_user_data', '.sh'])
+  file = Tempfile.new(["test_user_data", ".sh"])
   file.write("echo 'sample user data file created' >> #{Dir.tmpdir}/testuserdata.txt")
   file.rewind
   file

data/spec/unit/openstack_flavor_list_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Author:: Mukta Aphale (<mukta.aphale@clogeny.com>)
 # Author:: Siddheshwar More (<siddheshwar.more@clogeny.com>)
@@ -17,13 +18,13 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require 'spec_helper'
-require 'chef/knife/openstack_flavor_list'
-require 'chef/knife/cloud/openstack_service'
-require 'support/shared_examples_for_command'
+require "spec_helper"
+require "chef/knife/openstack_flavor_list"
+require "chef/knife/cloud/openstack_service"
+require "support/shared_examples_for_command"
 
 describe Chef::Knife::Cloud::OpenstackFlavorList do
   it_behaves_like Chef::Knife::Cloud::Command, Chef::Knife::Cloud::OpenstackFlavorList.new
 
-  include_context '#validate!', Chef::Knife::Cloud::OpenstackFlavorList.new
+  include_context "#validate!", Chef::Knife::Cloud::OpenstackFlavorList.new
 end

data/spec/unit/openstack_floating_ip_allocate_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Author:: Vasundhara Jagdale (<vasundhara.jagdale@clogeny.com>)
 # Copyright:: Copyright (c) 2013-2015 Chef Software, Inc.
@@ -15,39 +16,39 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require 'spec_helper'
-require 'chef/knife/openstack_floating_ip_allocate'
-require 'chef/knife/cloud/openstack_service'
-require 'support/shared_examples_for_command'
+require "spec_helper"
+require "chef/knife/openstack_floating_ip_allocate"
+require "chef/knife/cloud/openstack_service"
+require "support/shared_examples_for_command"
 
 describe Chef::Knife::Cloud::OpenstackFloatingIpAllocate do
   it_behaves_like Chef::Knife::Cloud::Command, Chef::Knife::Cloud::OpenstackFloatingIpAllocate.new
-  include_context '#validate!', Chef::Knife::Cloud::OpenstackFloatingIpAllocate.new
+  include_context "#validate!", Chef::Knife::Cloud::OpenstackFloatingIpAllocate.new
 
   before(:each) do
     @instance = Chef::Knife::Cloud::OpenstackFloatingIpAllocate.new
     allow(@instance.ui).to receive(:error)
   end
 
-  describe 'create service instance' do
-    it 'return OpenstackService instance' do
+  describe "create service instance" do
+    it "return OpenstackService instance" do
       expect(@instance.create_service_instance).to be_an_instance_of(Chef::Knife::Cloud::OpenstackService)
     end
   end
 
-  describe 'allocate floating ip' do
-    it 'calls allocate address' do
+  describe "allocate floating ip" do
+    it "calls allocate address" do
       @instance.service = double
       expect(@instance.service).to receive(:allocate_address).and_return(true)
       @instance.execute_command
     end
   end
 
-  describe 'when user provides pool option ' do
-    it 'allocates floating ip in user specified pool' do
-      @instance = Chef::Knife::Cloud::OpenstackFloatingIpAllocate.new(['--pool', 'test-pool'])
+  describe "when user provides pool option " do
+    it "allocates floating ip in user specified pool" do
+      @instance = Chef::Knife::Cloud::OpenstackFloatingIpAllocate.new(["--pool", "test-pool"])
       @instance.service = Chef::Knife::Cloud::Service.new
-      response = { floating_ip: { 'id' => 'test-id', 'instance_id' => 'test-instance-id', 'floating_ip' => '127.0.0.1', 'fixed_ip' => 'nil', 'pool' => 'test-pool' } }
+      response = { floating_ip: { "id" => "test-id", "instance_id" => "test-instance-id", "floating_ip" => "127.0.0.1", "fixed_ip" => "nil", "pool" => "test-pool" } }
       expect(@instance.service).to receive(:allocate_address).and_return(response)
       @instance.execute_command
     end

data/spec/unit/openstack_floating_ip_associate_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Author:: Vasundhara Jagdale (<vasundhara.jagdale@clogeny.com>)
 # Copyright:: Copyright (c) 2013-2015 Chef Software, Inc.
@@ -14,21 +15,21 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
-require 'spec_helper'
-require 'chef/knife/openstack_floating_ip_associate'
-require 'chef/knife/cloud/openstack_service'
-require 'support/shared_examples_for_command'
-require 'ostruct'
+require "spec_helper"
+require "chef/knife/openstack_floating_ip_associate"
+require "chef/knife/cloud/openstack_service"
+require "support/shared_examples_for_command"
+require "ostruct"
 
 describe Chef::Knife::Cloud::OpenstackFloatingIpAssociate do
   before(:each) do
-    @instance = Chef::Knife::Cloud::OpenstackFloatingIpAssociate.new(['--instance-id', '23849038438240934n3294839248'])
-    @instance.name_args = ['127.0.0.1']
+    @instance = Chef::Knife::Cloud::OpenstackFloatingIpAssociate.new(["--instance-id", "23849038438240934n3294839248"])
+    @instance.name_args = ["127.0.0.1"]
   end
 
-  describe 'associate floating ip' do
-    it 'calls associate address' do
-      success_message = 'Floating IP 127.0.0.1 associated with Instance 23849038438240934n3294839248'
+  describe "associate floating ip" do
+    it "calls associate address" do
+      success_message = "Floating IP 127.0.0.1 associated with Instance 23849038438240934n3294839248"
       @instance.service = Chef::Knife::Cloud::Service.new
       response = OpenStruct.new(status: 202)
       expect(@instance.service).to receive(:associate_address).and_return(response)

data/spec/unit/openstack_floating_ip_disassociate_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Author:: Vasundhara Jagdale (<vasundhara.jagdale@clogeny.com>)
 # Copyright:: Copyright (c) 2013-2015 Chef Software, Inc.
@@ -14,24 +15,24 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
-require 'spec_helper'
-require 'chef/knife/openstack_floating_ip_disassociate'
-require 'chef/knife/cloud/openstack_service'
-require 'support/shared_examples_for_command'
-require 'ostruct'
+require "spec_helper"
+require "chef/knife/openstack_floating_ip_disassociate"
+require "chef/knife/cloud/openstack_service"
+require "support/shared_examples_for_command"
+require "ostruct"
 
 describe Chef::Knife::Cloud::OpenstackFloatingIpDisassociate do
   before(:each) do
-    @instance = Chef::Knife::Cloud::OpenstackFloatingIpDisassociate.new(['--instance-id', '23849038438240934n3294839248'])
-    @instance.name_args = ['127.0.0.1']
+    @instance = Chef::Knife::Cloud::OpenstackFloatingIpDisassociate.new(["--instance-id", "23849038438240934n3294839248"])
+    @instance.name_args = ["127.0.0.1"]
   end
 
-  describe 'associate floating ip' do
-    it 'calls associate address' do
+  describe "associate floating ip" do
+    it "calls associate address" do
       @instance.service = Chef::Knife::Cloud::Service.new
       response = OpenStruct.new(status: 202)
-      expect(@instance.service).to receive(:disassociate_address).with('23849038438240934n3294839248', '127.0.0.1').and_return(response)
-      expect(@instance.ui).to receive(:info).and_return('Floating IP 127.0.0.1 disassociated with Instance 23849038438240934n3294839248')
+      expect(@instance.service).to receive(:disassociate_address).with("23849038438240934n3294839248", "127.0.0.1").and_return(response)
+      expect(@instance.ui).to receive(:info).and_return("Floating IP 127.0.0.1 disassociated with Instance 23849038438240934n3294839248")
       @instance.execute_command
     end
   end

data/spec/unit/openstack_floating_ip_list_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Author:: Vasundhara Jagdale (<vasundhara.jagdale@clogeny.com>)
 # Copyright:: Copyright (c) 2013-2015 Chef Software, Inc.
@@ -15,12 +16,12 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require 'spec_helper'
-require 'chef/knife/openstack_floating_ip_list'
-require 'chef/knife/cloud/openstack_service'
-require 'support/shared_examples_for_command'
+require "spec_helper"
+require "chef/knife/openstack_floating_ip_list"
+require "chef/knife/cloud/openstack_service"
+require "support/shared_examples_for_command"
 
 describe Chef::Knife::Cloud::OpenstackFloatingIpList do
   it_behaves_like Chef::Knife::Cloud::Command, Chef::Knife::Cloud::OpenstackFloatingIpList.new
-  include_context '#validate!', Chef::Knife::Cloud::OpenstackFloatingIpList.new
+  include_context "#validate!", Chef::Knife::Cloud::OpenstackFloatingIpList.new
 end

data/spec/unit/openstack_floating_ip_release_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Author:: Vasundhara Jagdale (<vasundhara.jagdale@clogeny.com>)
 # Copyright:: Copyright (c) 2013-2015 Chef Software, Inc.
@@ -14,34 +15,34 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
-require 'spec_helper'
-require 'chef/knife/openstack_floating_ip_release'
-require 'chef/knife/cloud/openstack_service'
-require 'support/shared_examples_for_command'
+require "spec_helper"
+require "chef/knife/openstack_floating_ip_release"
+require "chef/knife/cloud/openstack_service"
+require "support/shared_examples_for_command"
 
 describe Chef::Knife::Cloud::OpenstackFloatingIpRelease do
   it_behaves_like Chef::Knife::Cloud::Command, Chef::Knife::Cloud::OpenstackFloatingIpRelease.new
-  include_context '#validate!', Chef::Knife::Cloud::OpenstackFloatingIpRelease.new
+  include_context "#validate!", Chef::Knife::Cloud::OpenstackFloatingIpRelease.new
 
   before(:each) do
     @instance = Chef::Knife::Cloud::OpenstackFloatingIpRelease.new
     allow(@instance.ui).to receive(:error)
-    @instance.name_args = ['23849038438240934n3294839248']
+    @instance.name_args = ["23849038438240934n3294839248"]
   end
 
-  describe 'create service instance' do
-    it 'return OpenstackService instance' do
+  describe "create service instance" do
+    it "return OpenstackService instance" do
       expect(@instance.create_service_instance).to be_an_instance_of(Chef::Knife::Cloud::OpenstackService)
     end
   end
 
-  describe 'release floating ip' do
-    it 'calls release address' do
-      address_id = '23849038438240934n3294839248'
+  describe "release floating ip" do
+    it "calls release address" do
+      address_id = "23849038438240934n3294839248"
       @instance.service = double
       response = OpenStruct.new(status: 202)
       expect(@instance.service).to receive(:release_address).and_return(response)
-      expect(@instance.ui).to receive(:info).and_return('Floating IP released successfully.')
+      expect(@instance.ui).to receive(:info).and_return("Floating IP released successfully.")
       @instance.execute_command
     end
   end

data/spec/unit/openstack_group_list_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Author:: Mukta Aphale (<mukta.aphale@clogeny.com>)
 # Author:: Siddheshwar More (<siddheshwar.more@clogeny.com>)
@@ -17,26 +18,26 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require 'spec_helper'
-require 'chef/knife/openstack_group_list'
-require 'chef/knife/cloud/openstack_service'
-require 'support/shared_examples_for_command'
+require "spec_helper"
+require "chef/knife/openstack_group_list"
+require "chef/knife/cloud/openstack_service"
+require "support/shared_examples_for_command"
 
 describe Chef::Knife::Cloud::OpenstackGroupList do
   it_behaves_like Chef::Knife::Cloud::Command, Chef::Knife::Cloud::OpenstackGroupList.new
 
-  include_context '#validate!', Chef::Knife::Cloud::OpenstackGroupList.new
+  include_context "#validate!", Chef::Knife::Cloud::OpenstackGroupList.new
 
   let (:instance) { Chef::Knife::Cloud::OpenstackGroupList.new }
 
-  context '#list' do
+  context "#list" do
     before(:each) do
-      @security_groups = [TestResource.new('name' => 'Unrestricted', 'description' => 'testdescription', 'security_group_rules' => [TestResource.new('from_port' => 636, 'group' => {}, 'ip_protocol' => 'tcp', 'to_port' => 636, 'parent_group_id' => 14, 'ip_range' => { 'cidr' => '0.0.0.0/0' }, 'id' => 183)])]
-      instance.config[:format] = 'summary'
+      @security_groups = [TestResource.new("name" => "Unrestricted", "description" => "testdescription", "security_group_rules" => [TestResource.new("from_port" => 636, "group" => {}, "ip_protocol" => "tcp", "to_port" => 636, "parent_group_id" => 14, "ip_range" => { "cidr" => "0.0.0.0/0" }, "id" => 183)])]
+      instance.config[:format] = "summary"
     end
 
-    it 'returns group list' do
-      expect(instance.ui).to receive(:list).with(['Name', 'Protocol', 'From', 'To', 'CIDR', 'Description', 'Unrestricted', 'tcp', '636', '636', '0.0.0.0/0', 'testdescription'], :uneven_columns_across, 6)
+    it "returns group list" do
+      expect(instance.ui).to receive(:list).with(["Name", "Protocol", "From", "To", "CIDR", "Description", "Unrestricted", "tcp", "636", "636", "0.0.0.0/0", "testdescription"], :uneven_columns_across, 6)
       instance.list(@security_groups)
     end
   end