auto-fixing some easy copschefstyle/fixes

   3 Performance/CaseWhenSplat: Place when conditions with a splat at the end of the when branches.
   1 Style/ElseAlignment: Align else with if.
   1 Style/EvenOdd: Replace with Fixnum#even?.
   1 Style/MethodDefParentheses: Use def with parentheses when there are parameters.
   3 Style/PercentQLiterals: Do not use %Q unless interpolation is needed.  Use %q.
   3 Style/SpaceAfterColon: Space missing after colon.
   2 Style/SpaceAfterMethodName: Do not put a space between a method name and the opening parenthesis.
   2 Style/Tab: Tab detected.
   1 Style/TrailingUnderscoreVariable: Do not use trailing _s in parallel assignment.
   1 Style/UnneededCapitalW: Do not use %W unless interpolation is needed.  If not, use %w.

4 files changed, 4 insertions, 4 deletions

diff --git a/spec/unit/application/knife_spec.rb b/spec/unit/application/knife_spec.rb
index 416dba4233..ba02e44481 100644
--- a/spec/unit/application/knife_spec.rb
+++ b/spec/unit/application/knife_spec.rb
@@ -191,7 +191,7 @@ describe Chef::Application::Knife do
     end
 
     it "should load the environment from the CLI options" do
-      with_argv(*%W{noop knife command -E development}) do
+      with_argv(*%w{noop knife command -E development}) do
         expect(@knife).to receive(:exit).with(0)
         @knife.run
       end
diff --git a/spec/unit/formatters/error_inspectors/cookbook_resolve_error_inspector_spec.rb b/spec/unit/formatters/error_inspectors/cookbook_resolve_error_inspector_spec.rb
index c6170d4e01..700d7fa7ed 100644
--- a/spec/unit/formatters/error_inspectors/cookbook_resolve_error_inspector_spec.rb
+++ b/spec/unit/formatters/error_inspectors/cookbook_resolve_error_inspector_spec.rb
@@ -32,7 +32,7 @@ describe Chef::Formatters::ErrorInspectors::CookbookResolveErrorInspector do
   describe "when explaining a 403 error" do
     before do
 
-      @response_body = %Q({"error": [{"message": "gtfo"}])
+      @response_body = %q({"error": [{"message": "gtfo"}])
       @response = Net::HTTPForbidden.new("1.1", "403", "(response) forbidden")
       allow(@response).to receive(:body).and_return(@response_body)
       @exception = Net::HTTPServerException.new("(exception) forbidden", @response)
diff --git a/spec/unit/key_spec.rb b/spec/unit/key_spec.rb
index fa200762cb..5912016577 100644
--- a/spec/unit/key_spec.rb
+++ b/spec/unit/key_spec.rb
@@ -231,7 +231,7 @@ EOS
 
       it "should include the expiration_date field when present" do
         new_key.expiration_date "2020-12-24T21:00:00Z"
-        expect(new_key.to_json).to include(%Q{"expiration_date":"2020-12-24T21:00:00Z"})
+        expect(new_key.to_json).to include(%q{"expiration_date":"2020-12-24T21:00:00Z"})
       end
 
       it "should not include the expiration_date if not present" do
diff --git a/spec/unit/provider/service/debian_service_spec.rb b/spec/unit/provider/service/debian_service_spec.rb
index 669d715cfc..2400676b84 100644
--- a/spec/unit/provider/service/debian_service_spec.rb
+++ b/spec/unit/provider/service/debian_service_spec.rb
@@ -276,7 +276,7 @@ insserv: dryrun, not creating .depend.boot, .depend.start, and .depend.stop
     context "when the service is enabled" do
       before do
         @current_resource.enabled(true)
-	@current_resource.priority(80)
+  @current_resource.priority(80)
       end
 
       context "and the service sets no priority" do