chef 14.2.0 → 14.3.37

data/spec/unit/user_spec.rb

@@ -243,7 +243,7 @@ describe Chef::User do
     describe "create" do
       it "creates a new user via the API" do
         @user.password "password"
-        expect(@http_client).to receive(:post).with("users", { :name => "foobar", :admin => false, :password => "password" }).and_return({})
+        expect(@http_client).to receive(:post).with("users", { name: "foobar", admin: false, password: "password" }).and_return({})
         @user.create
       end
     end
@@ -260,7 +260,7 @@ describe Chef::User do
 
     describe "update" do
       it "updates an existing user on via the API" do
-        expect(@http_client).to receive(:put).with("users/foobar", { :name => "foobar", :admin => false }).and_return({})
+        expect(@http_client).to receive(:put).with("users/foobar", { name: "foobar", admin: false }).and_return({})
         @user.update
       end
     end

data/spec/unit/user_v1_spec.rb

@@ -311,7 +311,7 @@ describe Chef::UserV1 do
   end
 
   describe "Versioned API Interactions" do
-    let(:response_406) { OpenStruct.new(:code => "406") }
+    let(:response_406) { OpenStruct.new(code: "406") }
     let(:exception_406) { Net::HTTPServerException.new("406 Not Acceptable", response_406) }
 
     before (:each) do
@@ -334,13 +334,13 @@ describe Chef::UserV1 do
 
       let(:payload) do
         {
-          :username => "some_username",
-          :display_name => "some_display_name",
-          :first_name => "some_first_name",
-          :middle_name => "some_middle_name",
-          :last_name => "some_last_name",
-          :email => "some_email",
-          :password => "some_password",
+          username: "some_username",
+          display_name: "some_display_name",
+          first_name: "some_first_name",
+          middle_name: "some_middle_name",
+          last_name: "some_last_name",
+          email: "some_email",
+          password: "some_password",
         }
       end
 
@@ -356,14 +356,14 @@ describe Chef::UserV1 do
       context "when server API V1 is not valid on the Chef Server receiving the request" do
         let(:payload) do
           {
-            :username => "some_username",
-            :display_name => "some_display_name",
-            :first_name => "some_first_name",
-            :middle_name => "some_middle_name",
-            :last_name => "some_last_name",
-            :email => "some_email",
-            :password => "some_password",
-            :public_key => "some_public_key",
+            username: "some_username",
+            display_name: "some_display_name",
+            first_name: "some_first_name",
+            middle_name: "some_middle_name",
+            last_name: "some_last_name",
+            email: "some_email",
+            password: "some_password",
+            public_key: "some_public_key",
           }
         end
 
@@ -373,7 +373,7 @@ describe Chef::UserV1 do
         end
 
         context "when the server returns a 400" do
-          let(:response_400) { OpenStruct.new(:code => "400") }
+          let(:response_400) { OpenStruct.new(code: "400") }
           let(:exception_400) { Net::HTTPServerException.new("400 Bad Request", response_400) }
 
           context "when the 400 was due to public / private key fields no longer being supported" do
@@ -444,12 +444,12 @@ describe Chef::UserV1 do
     describe "create" do
       let(:payload) do
         {
-          :username => "some_username",
-          :display_name => "some_display_name",
-          :first_name => "some_first_name",
-          :last_name => "some_last_name",
-          :email => "some_email",
-          :password => "some_password",
+          username: "some_username",
+          display_name: "some_display_name",
+          first_name: "some_first_name",
+          last_name: "some_last_name",
+          email: "some_email",
+          password: "some_password",
         }
       end
       before do
@@ -473,7 +473,7 @@ describe Chef::UserV1 do
       context "when handling API V1" do
         it "creates a new user via the API with a middle_name when it exists" do
           @user.middle_name "some_middle_name"
-          expect(@user.chef_root_rest_v1).to receive(:post).with("users", payload.merge({ :middle_name => "some_middle_name" })).and_return({})
+          expect(@user.chef_root_rest_v1).to receive(:post).with("users", payload.merge({ middle_name: "some_middle_name" })).and_return({})
           @user.create
         end
       end # when server API V1 is valid on the Chef Server receiving the request
@@ -496,7 +496,7 @@ describe Chef::UserV1 do
 
         it "creates a new user via the API with a middle_name when it exists" do
           @user.middle_name "some_middle_name"
-          expect(@user.chef_root_rest_v0).to receive(:post).with("users", payload.merge({ :middle_name => "some_middle_name" })).and_return({})
+          expect(@user.chef_root_rest_v0).to receive(:post).with("users", payload.merge({ middle_name: "some_middle_name" })).and_return({})
           @user.create
         end
       end # when server API V1 is not valid on the Chef Server receiving the request

data/spec/unit/util/diff_spec.rb

@@ -546,7 +546,7 @@ shared_examples_for "a diff util" do
 
 end
 
-describe Chef::Util::Diff, :uses_diff => true do
+describe Chef::Util::Diff, uses_diff: true do
   let!(:old_file) { old_tempfile.path }
   let!(:new_file) { new_tempfile.path }
 

data/spec/unit/util/dsc/configuration_generator_spec.rb

@@ -76,7 +76,7 @@ describe Chef::Util::DSC::ConfigurationGenerator do
 
     context "when symbols are used as switches" do
       it "should merge the hash if there are no restricted switches" do
-        merged = conf_man.send(:get_merged_configuration_flags!, { :flag => "a" }, "hello")
+        merged = conf_man.send(:get_merged_configuration_flags!, { flag: "a" }, "hello")
         expect(merged).to include(:flag)
         expect(merged[:flag]).to eql("a")
         expect(merged).to include(:outputpath)
@@ -84,18 +84,18 @@ describe Chef::Util::DSC::ConfigurationGenerator do
 
       it "should raise an ArgumentError if you try to override outputpath" do
         expect do
-          conf_man.send(:get_merged_configuration_flags!, { :outputpath => "a" }, "hello")
+          conf_man.send(:get_merged_configuration_flags!, { outputpath: "a" }, "hello")
         end.to raise_error(ArgumentError)
       end
 
       it "should be case insensitive for switches that are not allowed" do
         expect do
-          conf_man.send(:get_merged_configuration_flags!, { :OutputPath => "a" }, "hello")
+          conf_man.send(:get_merged_configuration_flags!, { OutputPath: "a" }, "hello")
         end.to raise_error(ArgumentError)
       end
 
       it "should be case insensitive to switches that are allowed" do
-        merged = conf_man.send(:get_merged_configuration_flags!, { :FLAG => "a" }, "hello")
+        merged = conf_man.send(:get_merged_configuration_flags!, { FLAG: "a" }, "hello")
         expect(merged).to include(:flag)
       end
     end

data/spec/unit/util/dsc/lcm_output_parser_spec.rb

@@ -47,11 +47,11 @@ describe Chef::Util::DSC::LocalConfigurationManager::Parser do
 
   context "correctly formatted output from lcm for WhatIfParser" do
     it "returns a single resource when only 1 logged with the correct name" do
-      str = <<EOF
-logtype: [machinename]: LCM:  [ Start  Set      ]
-logtype: [machinename]: LCM:  [ Start  Resource ] [name]
-logtype: [machinename]: LCM:  [ End    Resource ] [name]
-logtype: [machinename]: LCM:  [ End    Set      ]
+      str = <<~EOF
+        logtype: [machinename]: LCM:  [ Start  Set      ]
+        logtype: [machinename]: LCM:  [ Start  Resource ] [name]
+        logtype: [machinename]: LCM:  [ End    Resource ] [name]
+        logtype: [machinename]: LCM:  [ End    Set      ]
 EOF
       resources = Chef::Util::DSC::LocalConfigurationManager::Parser.parse(str, false)
       expect(resources.length).to eq(1)
@@ -59,51 +59,51 @@ EOF
     end
 
     it "identifies when a resource changes the state of the system" do
-      str = <<EOF
-logtype: [machinename]: LCM:  [ Start  Set      ]
-logtype: [machinename]: LCM:  [ Start  Resource ] [name]
-logtype: [machinename]: LCM:  [ Start  Set      ] [name]
-logtype: [machinename]: LCM:  [ End    Set      ] [name]
-logtype: [machinename]: LCM:  [ End    Resource ] [name]
-logtype: [machinename]: LCM:  [ End    Set      ]
+      str = <<~EOF
+        logtype: [machinename]: LCM:  [ Start  Set      ]
+        logtype: [machinename]: LCM:  [ Start  Resource ] [name]
+        logtype: [machinename]: LCM:  [ Start  Set      ] [name]
+        logtype: [machinename]: LCM:  [ End    Set      ] [name]
+        logtype: [machinename]: LCM:  [ End    Resource ] [name]
+        logtype: [machinename]: LCM:  [ End    Set      ]
 EOF
       resources = Chef::Util::DSC::LocalConfigurationManager::Parser.parse(str, false)
       expect(resources[0].changes_state?).to be_truthy
     end
 
     it "preserves the log provided for how the system changed the state" do
-      str = <<EOF
-logtype: [machinename]: LCM:  [ Start  Set      ]
-logtype: [machinename]: LCM:  [ Start  Resource ] [name]
-logtype: [machinename]: LCM:  [ Start  Set      ] [name]
-logtype: [machinename]:                           [message]
-logtype: [machinename]: LCM:  [ End    Set      ] [name]
-logtype: [machinename]: LCM:  [ End    Resource ] [name]
-logtype: [machinename]: LCM:  [ End    Set      ]
+      str = <<~EOF
+        logtype: [machinename]: LCM:  [ Start  Set      ]
+        logtype: [machinename]: LCM:  [ Start  Resource ] [name]
+        logtype: [machinename]: LCM:  [ Start  Set      ] [name]
+        logtype: [machinename]:                           [message]
+        logtype: [machinename]: LCM:  [ End    Set      ] [name]
+        logtype: [machinename]: LCM:  [ End    Resource ] [name]
+        logtype: [machinename]: LCM:  [ End    Set      ]
 EOF
       resources = Chef::Util::DSC::LocalConfigurationManager::Parser.parse(str, false)
       expect(resources[0].change_log).to match_array(["[name]", "[message]", "[name]"])
     end
 
     it "returns false for changes_state?" do
-      str = <<EOF
-logtype: [machinename]: LCM:  [ Start  Set      ]
-logtype: [machinename]: LCM:  [ Start  Resource ] [name]
-logtype: [machinename]: LCM:  [ Skip   Set      ] [name]
-logtype: [machinename]: LCM:  [ End    Resource ] [name]
-logtype: [machinename]: LCM:  [ End    Set      ]
+      str = <<~EOF
+        logtype: [machinename]: LCM:  [ Start  Set      ]
+        logtype: [machinename]: LCM:  [ Start  Resource ] [name]
+        logtype: [machinename]: LCM:  [ Skip   Set      ] [name]
+        logtype: [machinename]: LCM:  [ End    Resource ] [name]
+        logtype: [machinename]: LCM:  [ End    Set      ]
 EOF
       resources = Chef::Util::DSC::LocalConfigurationManager::Parser.parse(str, false)
       expect(resources[0].changes_state?).to be_falsey
     end
 
     it "returns an empty array for change_log if changes_state? is false" do
-      str = <<EOF
-logtype: [machinename]: LCM:  [ Start  Set      ]
-logtype: [machinename]: LCM:  [ Start  Resource ] [name]
-logtype: [machinename]: LCM:  [ Skip   Set      ] [name]
-logtype: [machinename]: LCM:  [ End    Resource ] [name]
-logtype: [machinename]: LCM:  [ End    Set      ]
+      str = <<~EOF
+        logtype: [machinename]: LCM:  [ Start  Set      ]
+        logtype: [machinename]: LCM:  [ Start  Resource ] [name]
+        logtype: [machinename]: LCM:  [ Skip   Set      ] [name]
+        logtype: [machinename]: LCM:  [ End    Resource ] [name]
+        logtype: [machinename]: LCM:  [ End    Set      ]
 EOF
       resources = Chef::Util::DSC::LocalConfigurationManager::Parser.parse(str, false)
       expect(resources[0].change_log).to be_empty
@@ -112,12 +112,12 @@ EOF
 
   context "correctly formatted output from lcm for TestDSCParser" do
     it "returns a single resource when only 1 logged with the correct name" do
-      str = <<EOF
-InDesiredState            : False
-ResourcesInDesiredState   :
-ResourcesNotInDesiredState: [name]
-ReturnValue               : 0
-PSComputerName            : .
+      str = <<~EOF
+        InDesiredState            : False
+        ResourcesInDesiredState   :
+        ResourcesNotInDesiredState: [name]
+        ReturnValue               : 0
+        PSComputerName            : .
 EOF
       resources = Chef::Util::DSC::LocalConfigurationManager::Parser.parse(str, true)
       expect(resources.length).to eq(1)
@@ -125,36 +125,36 @@ EOF
     end
 
     it "identifies when a resource changes the state of the system" do
-      str = <<EOF
-InDesiredState            : False
-ResourcesInDesiredState   :
-ResourcesNotInDesiredState: [name]
-ReturnValue               : 0
-PSComputerName            : .
+      str = <<~EOF
+        InDesiredState            : False
+        ResourcesInDesiredState   :
+        ResourcesNotInDesiredState: [name]
+        ReturnValue               : 0
+        PSComputerName            : .
 EOF
       resources = Chef::Util::DSC::LocalConfigurationManager::Parser.parse(str, true)
       expect(resources[0].changes_state?).to be_truthy
     end
 
     it "returns false for changes_state?" do
-      str = <<EOF
-InDesiredState            : True
-ResourcesInDesiredState   : [name]
-ResourcesNotInDesiredState:
-ReturnValue               : 0
-PSComputerName            : .
+      str = <<~EOF
+        InDesiredState            : True
+        ResourcesInDesiredState   : [name]
+        ResourcesNotInDesiredState:
+        ReturnValue               : 0
+        PSComputerName            : .
 EOF
       resources = Chef::Util::DSC::LocalConfigurationManager::Parser.parse(str, true)
       expect(resources[0].changes_state?).to be_falsey
     end
 
     it "returns an empty array for change_log if changes_state? is false" do
-      str = <<EOF
-InDesiredState            : True
-ResourcesInDesiredState   : [name]
-ResourcesNotInDesiredState:
-ReturnValue               : 0
-PSComputerName            : .
+      str = <<~EOF
+        InDesiredState            : True
+        ResourcesInDesiredState   : [name]
+        ResourcesNotInDesiredState:
+        ReturnValue               : 0
+        PSComputerName            : .
 EOF
       resources = Chef::Util::DSC::LocalConfigurationManager::Parser.parse(str, true)
       expect(resources[0].change_log).to be_empty
@@ -163,19 +163,19 @@ EOF
 
   context "Incorrectly formatted output from LCM for WhatIfParser" do
     it "allows missing [End Resource] when its the last one and still find all the resource" do
-      str = <<-EOF
-logtype: [machinename]: LCM:  [ Start  Set      ]
-logtype: [machinename]: LCM:  [ Start  Resource ]  [name]
-logtype: [machinename]: LCM:  [ Start  Test     ]
-logtype: [machinename]: LCM:  [ End    Test     ]
-logtype: [machinename]: LCM:  [ Skip   Set      ]
-logtype: [machinename]: LCM:  [ End    Resource ]
-logtype: [machinename]: LCM:  [ Start  Resource ]  [name2]
-logtype: [machinename]: LCM:  [ Start  Test     ]
-logtype: [machinename]: LCM:  [ End    Test     ]
-logtype: [machinename]: LCM:  [ Start  Set      ]
-logtype: [machinename]: LCM:  [ End    Set      ]
-logtype: [machinename]: LCM:  [ End    Set      ]
+      str = <<~EOF
+        logtype: [machinename]: LCM:  [ Start  Set      ]
+        logtype: [machinename]: LCM:  [ Start  Resource ]  [name]
+        logtype: [machinename]: LCM:  [ Start  Test     ]
+        logtype: [machinename]: LCM:  [ End    Test     ]
+        logtype: [machinename]: LCM:  [ Skip   Set      ]
+        logtype: [machinename]: LCM:  [ End    Resource ]
+        logtype: [machinename]: LCM:  [ Start  Resource ]  [name2]
+        logtype: [machinename]: LCM:  [ Start  Test     ]
+        logtype: [machinename]: LCM:  [ End    Test     ]
+        logtype: [machinename]: LCM:  [ Start  Set      ]
+        logtype: [machinename]: LCM:  [ End    Set      ]
+        logtype: [machinename]: LCM:  [ End    Set      ]
 EOF
 
       resources = Chef::Util::DSC::LocalConfigurationManager::Parser.parse(str, false)
@@ -184,19 +184,19 @@ EOF
     end
 
     it "allow missing [End Resource] when its the first one and still find all the resource" do
-      str = <<-EOF
-logtype: [machinename]: LCM:  [ Start  Set      ]
-logtype: [machinename]: LCM:  [ Start  Resource ]  [name]
-logtype: [machinename]: LCM:  [ Start  Test     ]
-logtype: [machinename]: LCM:  [ End    Test     ]
-logtype: [machinename]: LCM:  [ Skip   Set      ]
-logtype: [machinename]: LCM:  [ Start  Resource ]  [name2]
-logtype: [machinename]: LCM:  [ Start  Test     ]
-logtype: [machinename]: LCM:  [ End    Test     ]
-logtype: [machinename]: LCM:  [ Start  Set      ]
-logtype: [machinename]: LCM:  [ End    Set      ]
-logtype: [machinename]: LCM:  [ End    Resource ]
-logtype: [machinename]: LCM:  [ End    Set      ]
+      str = <<~EOF
+        logtype: [machinename]: LCM:  [ Start  Set      ]
+        logtype: [machinename]: LCM:  [ Start  Resource ]  [name]
+        logtype: [machinename]: LCM:  [ Start  Test     ]
+        logtype: [machinename]: LCM:  [ End    Test     ]
+        logtype: [machinename]: LCM:  [ Skip   Set      ]
+        logtype: [machinename]: LCM:  [ Start  Resource ]  [name2]
+        logtype: [machinename]: LCM:  [ Start  Test     ]
+        logtype: [machinename]: LCM:  [ End    Test     ]
+        logtype: [machinename]: LCM:  [ Start  Set      ]
+        logtype: [machinename]: LCM:  [ End    Set      ]
+        logtype: [machinename]: LCM:  [ End    Resource ]
+        logtype: [machinename]: LCM:  [ End    Set      ]
 EOF
 
       resources = Chef::Util::DSC::LocalConfigurationManager::Parser.parse(str, false)
@@ -205,18 +205,18 @@ EOF
     end
 
     it "allows missing set and end resource and assume an unconverged resource in this case" do
-      str = <<-EOF
-logtype: [machinename]: LCM:  [ Start  Set      ]
-logtype: [machinename]: LCM:  [ Start  Resource ]  [name]
-logtype: [machinename]: LCM:  [ Start  Test     ]
-logtype: [machinename]: LCM:  [ End    Test     ]
-logtype: [machinename]: LCM:  [ Start  Resource ]  [name2]
-logtype: [machinename]: LCM:  [ Start  Test     ]
-logtype: [machinename]: LCM:  [ End    Test     ]
-logtype: [machinename]: LCM:  [ Start  Set      ]
-logtype: [machinename]: LCM:  [ End    Set      ]
-logtype: [machinename]: LCM:  [ End    Resource ]
-logtype: [machinename]: LCM:  [ End    Set      ]
+      str = <<~EOF
+        logtype: [machinename]: LCM:  [ Start  Set      ]
+        logtype: [machinename]: LCM:  [ Start  Resource ]  [name]
+        logtype: [machinename]: LCM:  [ Start  Test     ]
+        logtype: [machinename]: LCM:  [ End    Test     ]
+        logtype: [machinename]: LCM:  [ Start  Resource ]  [name2]
+        logtype: [machinename]: LCM:  [ Start  Test     ]
+        logtype: [machinename]: LCM:  [ End    Test     ]
+        logtype: [machinename]: LCM:  [ Start  Set      ]
+        logtype: [machinename]: LCM:  [ End    Set      ]
+        logtype: [machinename]: LCM:  [ End    Resource ]
+        logtype: [machinename]: LCM:  [ End    Set      ]
 EOF
       resources = Chef::Util::DSC::LocalConfigurationManager::Parser.parse(str, false)
       expect(resources[0].changes_state?).to be_truthy
@@ -228,17 +228,17 @@ EOF
 
   context "Incorrectly formatted output from LCM for TestDSCParser" do
     it "allows missing [End Resource] when its the last one and still find all the resource" do
-      str = <<EOF
-InDesiredState            : True
-ResourcesInDesiredState   :
-ResourcesNotInDesiredState: [name]
-ReturnValue               : 0
-PSComputerName            : .
-InDesiredState            : True
-ResourcesInDesiredState   :
-ResourcesNotInDesiredState: [name2]
-ReturnValue               : 0
-PSComputerName            : .
+      str = <<~EOF
+        InDesiredState            : True
+        ResourcesInDesiredState   :
+        ResourcesNotInDesiredState: [name]
+        ReturnValue               : 0
+        PSComputerName            : .
+        InDesiredState            : True
+        ResourcesInDesiredState   :
+        ResourcesNotInDesiredState: [name2]
+        ReturnValue               : 0
+        PSComputerName            : .
 EOF
 
       resources = Chef::Util::DSC::LocalConfigurationManager::Parser.parse(str, true)

data/spec/unit/util/dsc/local_configuration_manager_spec.rb

@@ -24,33 +24,33 @@ describe Chef::Util::DSC::LocalConfigurationManager do
   let(:lcm) { Chef::Util::DSC::LocalConfigurationManager.new(nil, "tmp") }
 
   let(:normal_lcm_output) do
-    <<-EOH
-logtype: [machinename]: LCM:  [ Start  Set      ]
-logtype: [machinename]: LCM:  [ Start  Resource ] [name]
-logtype: [machinename]: LCM:  [ End    Resource ] [name]
-logtype: [machinename]: LCM:  [ End    Set      ]
+    <<~EOH
+      logtype: [machinename]: LCM:  [ Start  Set      ]
+      logtype: [machinename]: LCM:  [ Start  Resource ] [name]
+      logtype: [machinename]: LCM:  [ End    Resource ] [name]
+      logtype: [machinename]: LCM:  [ End    Set      ]
 EOH
   end
 
   let(:no_whatif_lcm_output) do
-    <<-EOH
-Start-DscConfiguration : A parameter cannot be found\r\n that matches parameter name 'whatif'.
-At line:1 char:123
-+ run-somecommand -whatif
-+                        ~~~~~~~~
-    + CategoryInfo          : InvalidArgument: (:) [Start-DscConfiguration], ParameterBindingException
-    + FullyQualifiedErrorId : NamedParameterNotFound,SomeCompany.SomeAssembly.Commands.RunSomeCommand
+    <<~EOH
+      Start-DscConfiguration : A parameter cannot be found\r\n that matches parameter name 'whatif'.
+      At line:1 char:123
+      + run-somecommand -whatif
+      +                        ~~~~~~~~
+          + CategoryInfo          : InvalidArgument: (:) [Start-DscConfiguration], ParameterBindingException
+          + FullyQualifiedErrorId : NamedParameterNotFound,SomeCompany.SomeAssembly.Commands.RunSomeCommand
 EOH
   end
 
   let(:dsc_resource_import_failure_output) do
-    <<-EOH
-PowerShell provider MSFT_xWebsite failed to execute Test-TargetResource functionality with error message: Please ensure that WebAdministration module is installed. + CategoryInfo : InvalidOperation: (:) [], CimException + FullyQualifiedErrorId : ProviderOperationExecutionFailure + PSComputerName : . PowerShell provider MSFT_xWebsite failed to execute Test-TargetResource functionality with error message: Please ensure that WebAdministration module is installed. + CategoryInfo : InvalidOperation: (:) [], CimException + FullyQualifiedErrorId : ProviderOperationExecutionFailure + PSComputerName : . The SendConfigurationApply function did not succeed. + CategoryInfo : NotSpecified: (root/Microsoft/...gurationManager:String) [], CimException + FullyQualifiedErrorId : MI RESULT 1 + PSComputerName : .
+    <<~EOH
+      PowerShell provider MSFT_xWebsite failed to execute Test-TargetResource functionality with error message: Please ensure that WebAdministration module is installed. + CategoryInfo : InvalidOperation: (:) [], CimException + FullyQualifiedErrorId : ProviderOperationExecutionFailure + PSComputerName : . PowerShell provider MSFT_xWebsite failed to execute Test-TargetResource functionality with error message: Please ensure that WebAdministration module is installed. + CategoryInfo : InvalidOperation: (:) [], CimException + FullyQualifiedErrorId : ProviderOperationExecutionFailure + PSComputerName : . The SendConfigurationApply function did not succeed. + CategoryInfo : NotSpecified: (root/Microsoft/...gurationManager:String) [], CimException + FullyQualifiedErrorId : MI RESULT 1 + PSComputerName : .
 EOH
   end
 
   let(:lcm_status) do
-    double("LCM cmdlet status", :stderr => lcm_standard_error, :return_value => lcm_standard_output, :succeeded? => lcm_cmdlet_success)
+    double("LCM cmdlet status", stderr: lcm_standard_error, return_value: lcm_standard_output, succeeded?: lcm_cmdlet_success)
   end
 
   describe "test_configuration method invocation" do
@@ -192,7 +192,7 @@ EOH
     context "when invalid dsc script is given" do
       it "raises exception" do
         configuration_document = "invalid-config"
-        shellout_flags = { :cwd => nil, :environment => nil, :timeout => nil }
+        shellout_flags = { cwd: nil, environment: nil, timeout: nil }
         expect { lcm.send(:run_configuration_cmdlet, configuration_document, true, shellout_flags) }.to raise_error(Chef::Exceptions::PowershellCmdletException)
       end
     end