Introduce standardrb; fix linter issues

Author: aramprice

bosh-stemcell/Gemfile

@@ -1,3 +1,3 @@
-source 'https://rubygems.org'
+source "https://rubygems.org"
 
-gemspec
+gemspec

bosh-stemcell/Rakefile

@@ -1,6 +1,7 @@
 require "rspec/core/rake_task"
+require "standard/rake"
 
-task default: [:spec]
+task default: [:standard, :spec]
 
 RSpec::Core::RakeTask.new(:spec)
 

bosh-stemcell/bosh-stemcell.gemspec

@@ -1,30 +1,28 @@
-# coding: utf-8
-require File.expand_path('../lib/bosh/stemcell/version', __FILE__)
+require File.expand_path("../lib/bosh/stemcell/version", __FILE__)
 
 version = Bosh::Stemcell::VERSION
 
 Gem::Specification.new do |spec|
-  spec.name          = 'bosh-stemcell'
-  spec.version       = version
-  spec.authors       = 'Pivotal'
-  spec.email         = 'support@cloudfoundry.com'
-  spec.description   = 'Bosh::Stemcell provides tools to manage stemcells'
-  spec.summary       = 'Bosh::Stemcell provides tools to manage stemcells'
-  spec.homepage      = 'https://github.com/cloudfoundry/bosh'
-  spec.license       = 'Apache 2.0'
+  spec.name = "bosh-stemcell"
+  spec.version = version
+  spec.authors = "Pivotal"
+  spec.email = "support@cloudfoundry.com"
+  spec.description = "Bosh::Stemcell provides tools to manage stemcells"
+  spec.summary = "Bosh::Stemcell provides tools to manage stemcells"
+  spec.homepage = "https://github.com/cloudfoundry/bosh"
+  spec.license = "Apache 2.0"
 
-  spec.required_ruby_version = Gem::Requirement.new('>= 1.9.3')
+  spec.required_ruby_version = Gem::Requirement.new(">= 1.9.3")
 
-  spec.files         = Dir['README.md', 'lib/**/*'].select{ |f| File.file? f }
-  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
-  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
+  spec.files = Dir["README.md", "lib/**/*"].select { |f| File.file? f }
+  spec.executables = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
   spec.require_paths = %w[lib]
 
-  spec.add_development_dependency 'fakefs'
-  spec.add_development_dependency 'rake'
-  spec.add_development_dependency 'rspec'
-  spec.add_development_dependency 'rspec-instafail'
-  spec.add_development_dependency 'rspec-its'
-  spec.add_development_dependency 'standard'
-  spec.add_development_dependency 'timecop'
+  spec.add_development_dependency "fakefs"
+  spec.add_development_dependency "rake"
+  spec.add_development_dependency "rspec"
+  spec.add_development_dependency "rspec-instafail"
+  spec.add_development_dependency "rspec-its"
+  spec.add_development_dependency "standard"
+  spec.add_development_dependency "timecop"
 end

bosh-stemcell/lib/bosh/core/shell.rb

@@ -21,12 +21,12 @@ def run_command(command, options)
       stdout.puts command if options[:output_command]
       lines = []
 
-      if options[:env]
+      popen_args = if options[:env]
         # Wrap in a shell because existing api to Shell#run takes a string
         # which makes it really hard to pass it to popen with custom environment.
-         popen_args = [options[:env], ENV['SHELL'] || 'bash', '-c', command]
+        [options[:env], ENV["SHELL"] || "bash", "-c", command]
       else
-        popen_args = command
+        command
       end
 
       io = IO.popen(popen_args)
@@ -50,7 +50,7 @@ def report(cmd, command_output, options)
 
       redacted_cmd = cmd
       if options[:redact]
-        redacted_cmd = redacted_cmd.gsub(/#{options[:redact].join('|')}/, "[REDACTED]")
+        redacted_cmd = redacted_cmd.gsub(/#{options[:redact].join("|")}/, "[REDACTED]")
       end
 
       err_msg = "Failed: '#{redacted_cmd}' from #{pwd}, with exit status #{$?.to_i}\n\n #{command_output}"
@@ -64,7 +64,7 @@ def command_exited_successfully?
     def pwd
       Dir.pwd
     rescue Errno::ENOENT
-      'a deleted directory'
+      "a deleted directory"
     end
   end
 end

bosh-stemcell/lib/bosh/stemcell/arch.rb

@@ -1,11 +1,11 @@
 module Bosh::Stemcell
   class Arch
     def self.arch
-      RbConfig::CONFIG['host_cpu']
+      RbConfig::CONFIG["host_cpu"]
     end
 
     def self.x86_64?
-      arch == 'x86_64'
+      arch == "x86_64"
     end
   end
 end

bosh-stemcell/lib/bosh/stemcell/archive.rb

@@ -1,12 +1,12 @@
-require 'yaml'
-require 'rake/file_utils_ext'
-require 'bosh/stemcell/aws/region'
+require "yaml"
+require "rake/file_utils_ext"
+require "bosh/stemcell/aws/region"
 
 module Bosh::Stemcell
   class Archive
     attr_reader :path
 
-    def initialize(path = '')
+    def initialize(path = "")
       @path = path
       validate_stemcell
     end
@@ -16,25 +16,25 @@ def manifest
     end
 
     def name
-      manifest.fetch('name')
+      manifest.fetch("name")
     end
 
     def infrastructure
-      cloud_properties.fetch('infrastructure')
+      cloud_properties.fetch("infrastructure")
     end
 
     def version
-      cloud_properties.fetch('version')
+      cloud_properties.fetch("version")
     end
 
     def sha1
-      sha1 = manifest.fetch('sha1')
-      raise 'sha1 must not be nil' unless sha1
+      sha1 = manifest.fetch("sha1")
+      raise "sha1 must not be nil" unless sha1
       sha1.to_s
     end
 
     def light?
-      infrastructure == 'aws' && has_ami?
+      infrastructure == "aws" && has_ami?
     end
 
     def extract(tar_options = {}, &block)
@@ -51,11 +51,11 @@ def extract(tar_options = {}, &block)
     private
 
     def has_ami?
-      cloud_properties.has_key? 'ami'
+      cloud_properties.has_key? "ami"
     end
 
     def cloud_properties
-      manifest.fetch('cloud_properties')
+      manifest.fetch("cloud_properties")
     end
 
     def validate_stemcell

bosh-stemcell/lib/bosh/stemcell/archive_filename.rb

@@ -1,5 +1,5 @@
-require 'bosh/stemcell/arch'
-require 'forwardable'
+require "bosh/stemcell/arch"
+require "forwardable"
 
 module Bosh::Stemcell
   class ArchiveFilename
@@ -19,7 +19,7 @@ def to_s
         definition.stemcell_name(disk_format)
       ]
 
-      "#{stemcell_filename_parts.join('-')}.tgz"
+      "#{stemcell_filename_parts.join("-")}.tgz"
     end
 
     private
@@ -28,7 +28,7 @@ def to_s
       :base_name,
       :version,
       :definition,
-      :disk_format,
+      :disk_format
     )
   end
 end

bosh-stemcell/lib/bosh/stemcell/aws/region.rb

@@ -1,15 +1,15 @@
-require 'net/http'
+require "net/http"
 
 module Bosh
   module Stemcell
     module Aws
       class Region
-        DEFAULT = 'us-east-1'
-        REGIONS = %w{
+        DEFAULT = "us-east-1"
+        REGIONS = %w[
           us-east-1 us-west-1 us-west-2 eu-west-1 eu-central-1
           ap-southeast-1 ap-southeast-2 ap-northeast-1 ap-northeast-2
           sa-east-1
-        }
+        ]
       end
     end
   end