Minor linting with the latest Rubocop

Some things the old Rubocop in Chefstyle misses.

Signed-off-by: Tim Smith <[email protected]>

Gemfile

@@ -9,8 +9,8 @@ group :test do
 end
 
 group :debug do
-  gem "pry"
   gem "byebug"
+  gem "pry"
 end
 
 group :chefstyle do

kitchen-google.gemspec

@@ -1,4 +1,4 @@
-$:.push File.expand_path("lib", __dir__)
+$LOAD_PATH.push File.expand_path("lib", __dir__)
 require "kitchen/driver/gce_version"
 
 Gem::Specification.new do |s|

lib/kitchen/driver/gce.rb

@@ -554,9 +554,7 @@ def instance_guest_accelerators
 
           count = 1
 
-          if guest_accelerator.key?(:count)
-            count = guest_accelerator[:count]
-          end
+          count = guest_accelerator[:count] if guest_accelerator.key?(:count)
 
           guest_accelerator_obj.accelerator_count = count
 

lib/kitchen/driver/gce_version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 #
 # Author:: Chef Partner Engineering (<[email protected]>)
 # Copyright:: Copyright (c) Chef Software, Inc.
@@ -18,6 +20,6 @@
 
 module Kitchen
   module Driver
-    GCE_VERSION = "2.3.0".freeze
+    GCE_VERSION = "2.3.0"
   end
 end

spec/kitchen/driver/gce_spec.rb

@@ -735,7 +735,6 @@
 
   describe "#create_disks" do
     it "sets up a boot disk if boot is present and returns it" do
-
       config = {
         disks: {
           disk1: {
@@ -879,13 +878,12 @@
   end
 
   describe "#instance_metadata" do
-    let(:item1   ) { double("item1") }
-    let(:item2   ) { double("item2") }
-    let(:item3   ) { double("item3") }
+    let(:item1) { double("item1") }
+    let(:item2) { double("item2") }
+    let(:item3) { double("item3") }
     let(:metadata) { double("metadata") }
 
     it "returns a properly-configured metadata object" do
-
       expect(instance).to receive(:name).and_return("instance_name")
       expect(driver).to receive(:env_user).and_return("env_user")
       expect(Google::Apis::ComputeV1::Metadata).to receive(:new).and_return(metadata)
@@ -1129,15 +1127,18 @@
     it "returns an array containing a properly-formatted service account" do
       service_account = double("service_account")
 
-      allow(driver).to receive(:config).and_return(service_account_name: "account_name", service_account_scopes: %w{scope1 scope2})
+      allow(driver).to receive(:config).and_return(service_account_name: "account_name",
+                                                   service_account_scopes: %w{
+                                                     scope1 scope2
+                                                   })
       expect(Google::Apis::ComputeV1::ServiceAccount).to receive(:new).and_return(service_account)
       expect(service_account).to receive(:email=).with("account_name")
       expect(driver).to receive(:service_account_scope_url).with("scope1").and_return("https://www.googleapis.com/auth/scope1")
       expect(driver).to receive(:service_account_scope_url).with("scope2").and_return("https://www.googleapis.com/auth/scope2")
       expect(service_account).to receive(:scopes=).with([
-        "https://www.googleapis.com/auth/scope1",
-        "https://www.googleapis.com/auth/scope2",
-      ])
+                                                          "https://www.googleapis.com/auth/scope1",
+                                                          "https://www.googleapis.com/auth/scope2",
+                                                        ])
 
       expect(driver.instance_service_accounts).to eq([service_account])
     end

spec/spec_helper.rb

@@ -31,4 +31,4 @@
   # test failures related to randomization by passing the same `--seed` value
   # as the one that triggered the failure.
   Kernel.srand config.seed
-end
+end