inspec 4.3.2 → 4.6.3

data/lib/plugins/inspec-compliance/test/unit/target_test.rb

@@ -1,155 +1,155 @@
-require 'minitest/autorun'
-require 'mocha/setup'
-require_relative '../../lib/inspec-compliance/api.rb'
+require "minitest/autorun"
+require "mocha/setup"
+require_relative "../../lib/inspec-compliance/api.rb"
 
 describe InspecPlugins::Compliance::Fetcher do
-  let(:config) { { 'server' => 'myserver' } }
+  let(:config) { { "server" => "myserver" } }
 
-  describe 'the check_compliance_token method' do
-    let(:fetcher) { fetcher = InspecPlugins::Compliance::Fetcher.new('a/bad/url', config) }
+  describe "the check_compliance_token method" do
+    let(:fetcher) { InspecPlugins::Compliance::Fetcher.new("a/bad/url", config) }
 
-    it 'returns without error if token is set' do
-      config['token'] = 'my-token'
-      fetcher.class.check_compliance_token('http://test.com', config)
+    it "returns without error if token is set" do
+      config["token"] = "my-token"
+      fetcher.class.check_compliance_token("http://test.com", config)
     end
 
-    it 'returns an error when token is not set' do
-      ex = assert_raises(Inspec::FetcherFailure) { fetcher.class.check_compliance_token('http://test.com', config) }
+    it "returns an error when token is not set" do
+      ex = assert_raises(Inspec::FetcherFailure) { fetcher.class.check_compliance_token("http://test.com", config) }
       ex.message.must_include "Cannot fetch http://test.com because your compliance token has not been\nconfigured."
     end
   end
 
-  describe 'when the server is an automate2 server' do
+  describe "when the server is an automate2 server" do
     before { InspecPlugins::Compliance::API.expects(:is_automate2_server?).with(config).returns(true) }
 
-    it 'returns the correct owner and profile name' do
-      config['profile'] = ['admin', 'ssh-baseline', nil]
-      fetcher = InspecPlugins::Compliance::Fetcher.new('myserver/profile', config)
-      fetcher.send(:compliance_profile_name).must_equal 'admin/ssh-baseline'
+    it "returns the correct owner and profile name" do
+      config["profile"] = ["admin", "ssh-baseline", nil]
+      fetcher = InspecPlugins::Compliance::Fetcher.new("myserver/profile", config)
+      fetcher.send(:compliance_profile_name).must_equal "admin/ssh-baseline"
     end
   end
 
-  describe 'when the server is an automate server pre-0.8.0' do
+  describe "when the server is an automate server pre-0.8.0" do
     before { InspecPlugins::Compliance::API.expects(:is_automate_server_pre_080?).with(config).returns(true) }
 
-    it 'returns the correct profile name when the url is correct' do
-      fetcher = InspecPlugins::Compliance::Fetcher.new('myserver/myowner/myprofile/tar', config)
-      fetcher.send(:compliance_profile_name).must_equal 'myowner/myprofile'
+    it "returns the correct profile name when the url is correct" do
+      fetcher = InspecPlugins::Compliance::Fetcher.new("myserver/myowner/myprofile/tar", config)
+      fetcher.send(:compliance_profile_name).must_equal "myowner/myprofile"
     end
 
-    it 'raises an exception if the url is malformed' do
-      fetcher = InspecPlugins::Compliance::Fetcher.new('a/bad/url', config)
+    it "raises an exception if the url is malformed" do
+      fetcher = InspecPlugins::Compliance::Fetcher.new("a/bad/url", config)
       proc { fetcher.send(:compliance_profile_name) }.must_raise RuntimeError
     end
   end
 
-  describe 'when the server is an automate server 0.8.0-or-later' do
+  describe "when the server is an automate server 0.8.0-or-later" do
     before do
       InspecPlugins::Compliance::API.expects(:is_automate_server_pre_080?).with(config).returns(false)
       InspecPlugins::Compliance::API.expects(:is_automate_server_080_and_later?).with(config).returns(true)
     end
 
-    it 'returns the correct profile name when the url is correct' do
-      fetcher = InspecPlugins::Compliance::Fetcher.new('myserver/profiles/myowner/myprofile/tar', config)
-      fetcher.send(:compliance_profile_name).must_equal 'myowner/myprofile'
+    it "returns the correct profile name when the url is correct" do
+      fetcher = InspecPlugins::Compliance::Fetcher.new("myserver/profiles/myowner/myprofile/tar", config)
+      fetcher.send(:compliance_profile_name).must_equal "myowner/myprofile"
     end
 
-    it 'raises an exception if the url is malformed' do
-      fetcher = InspecPlugins::Compliance::Fetcher.new('a/bad/url', config)
+    it "raises an exception if the url is malformed" do
+      fetcher = InspecPlugins::Compliance::Fetcher.new("a/bad/url", config)
       proc { fetcher.send(:compliance_profile_name) }.must_raise RuntimeError
     end
   end
 
-  describe 'when the server is not an automate server (likely a compliance server)' do
+  describe "when the server is not an automate server (likely a compliance server)" do
     before do
       InspecPlugins::Compliance::API.expects(:is_automate_server_pre_080?).with(config).returns(false)
       InspecPlugins::Compliance::API.expects(:is_automate_server_080_and_later?).with(config).returns(false)
     end
 
-    it 'returns the correct profile name when the url is correct' do
-      fetcher = InspecPlugins::Compliance::Fetcher.new('myserver/owners/myowner/compliance/myprofile/tar', config)
-      fetcher.send(:compliance_profile_name).must_equal 'myowner/myprofile'
+    it "returns the correct profile name when the url is correct" do
+      fetcher = InspecPlugins::Compliance::Fetcher.new("myserver/owners/myowner/compliance/myprofile/tar", config)
+      fetcher.send(:compliance_profile_name).must_equal "myowner/myprofile"
     end
 
-    it 'raises an exception if the url is malformed' do
-      fetcher = InspecPlugins::Compliance::Fetcher.new('a/bad/url', config)
+    it "raises an exception if the url is malformed" do
+      fetcher = InspecPlugins::Compliance::Fetcher.new("a/bad/url", config)
       proc { fetcher.send(:compliance_profile_name) }.must_raise RuntimeError
     end
   end
 
-  describe 'when the server calls an automate profile' do
+  describe "when the server calls an automate profile" do
     let(:profiles_result) do
-      [{ 'name'=>'ssh-baseline',
-          'title'=>'InSpec Profile',
-          'maintainer'=>'The Authors',
-          'copyright'=>'The Authors',
-          'copyright_email'=>'you@example.com',
-          'license'=>'Apache-2.0',
-          'summary'=>'An InSpec Compliance Profile',
-          'version'=>'0.1.1',
-          'owner'=>'admin',
-          'supports'=>[],
-          'depends'=>[],
-          'sha256'=>'132j1kjdasfasdoaefaewo12312',
-          'groups'=>[],
-          'controls'=>[],
-          'attributes'=>[],
-          'latest_version'=>'' }]
+      [{ "name" => "ssh-baseline",
+         "title" => "InSpec Profile",
+         "maintainer" => "The Authors",
+         "copyright" => "The Authors",
+         "copyright_email" => "you@example.com",
+         "license" => "Apache-2.0",
+         "summary" => "An InSpec Compliance Profile",
+         "version" => "0.1.1",
+         "owner" => "admin",
+         "supports" => [],
+         "depends" => [],
+         "sha256" => "132j1kjdasfasdoaefaewo12312",
+         "groups" => [],
+         "controls" => [],
+         "attributes" => [],
+         "latest_version" => "" }]
     end
     before do
-      InspecPlugins::Compliance::Configuration.expects(:new).returns({ 'token' => '123abc', 'server' => 'https://a2.instance.com' })
+      InspecPlugins::Compliance::Configuration.expects(:new).returns({ "token" => "123abc", "server" => "https://a2.instance.com" })
     end
 
-    it 'returns the correct profile name when parsing url' do
-      InspecPlugins::Compliance::API.stubs(:profiles).returns(['success', profiles_result])
-      fetcher = InspecPlugins::Compliance::Fetcher.resolve('compliance://admin/ssh-baseline')
-      assert = ['admin', 'ssh-baseline', nil]
-      fetcher.instance_variable_get(:"@config")['profile'].must_equal assert
+    it "returns the correct profile name when parsing url" do
+      InspecPlugins::Compliance::API.stubs(:profiles).returns(["success", profiles_result])
+      fetcher = InspecPlugins::Compliance::Fetcher.resolve("compliance://admin/ssh-baseline")
+      assert = ["admin", "ssh-baseline", nil]
+      fetcher.instance_variable_get(:"@config")["profile"].must_equal assert
     end
 
-    it 'returns the correct profile name when parsing compliance hash' do
-      InspecPlugins::Compliance::API.stubs(:profiles).returns(['success', profiles_result])
+    it "returns the correct profile name when parsing compliance hash" do
+      InspecPlugins::Compliance::API.stubs(:profiles).returns(["success", profiles_result])
       hash = {
-        target: 'https://a2.instance.com/api/v0/compliance/tar',
-        compliance: 'admin/ssh-baseline',
-        sha256: '132j1kjdasfasdoaefaewo12312',
+        target: "https://a2.instance.com/api/v0/compliance/tar",
+        compliance: "admin/ssh-baseline",
+        sha256: "132j1kjdasfasdoaefaewo12312",
       }
       fetcher = InspecPlugins::Compliance::Fetcher.resolve(hash)
-      assert = ['admin', 'ssh-baseline', nil]
-      fetcher.instance_variable_get(:"@config")['profile'].must_equal assert
+      assert = ["admin", "ssh-baseline", nil]
+      fetcher.instance_variable_get(:"@config")["profile"].must_equal assert
     end
   end
 
-  describe 'when the server provides a sha256 in the profiles_result' do
+  describe "when the server provides a sha256 in the profiles_result" do
     let(:profiles_result) do
-      [{ 'name'=>'ssh-baseline',
-          'title'=>'InSpec Profile',
-          'maintainer'=>'The Authors',
-          'copyright'=>'The Authors',
-          'copyright_email'=>'you@example.com',
-          'license'=>'Apache-2.0',
-          'summary'=>'An InSpec Compliance Profile',
-          'version'=>'0.1.1',
-          'owner'=>'admin',
-          'supports'=>[],
-          'depends'=>[],
-          'sha256'=>'132j1kjdasfasdoaefaewo12312',
-          'groups'=>[],
-          'controls'=>[],
-          'attributes'=>[],
-          'latest_version'=>'' }]
+      [{ "name" => "ssh-baseline",
+         "title" => "InSpec Profile",
+         "maintainer" => "The Authors",
+         "copyright" => "The Authors",
+         "copyright_email" => "you@example.com",
+         "license" => "Apache-2.0",
+         "summary" => "An InSpec Compliance Profile",
+         "version" => "0.1.1",
+         "owner" => "admin",
+         "supports" => [],
+         "depends" => [],
+         "sha256" => "132j1kjdasfasdoaefaewo12312",
+         "groups" => [],
+         "controls" => [],
+         "attributes" => [],
+         "latest_version" => "" }]
     end
 
     before do
-      InspecPlugins::Compliance::Configuration.expects(:new).returns({ 'token' => '123abc', 'server' => 'https://a2.instance.com' })
+      InspecPlugins::Compliance::Configuration.expects(:new).returns({ "token" => "123abc", "server" => "https://a2.instance.com" })
     end
 
-    it 'contains the upstream_sha256' do
-      InspecPlugins::Compliance::API.stubs(:profiles).returns(['success', profiles_result])
+    it "contains the upstream_sha256" do
+      InspecPlugins::Compliance::API.stubs(:profiles).returns(["success", profiles_result])
       prof = profiles_result[0]
       target = "compliance://#{prof['owner']}/#{prof['name']}"
       fetcher = InspecPlugins::Compliance::Fetcher.resolve(target)
-      fetcher.upstream_sha256.must_equal prof['sha256']
+      fetcher.upstream_sha256.must_equal prof["sha256"]
     end
   end
 end

data/lib/plugins/inspec-habitat/Berksfile

@@ -1,5 +1,5 @@
-source 'https://supermarket.chef.io'
+source "https://supermarket.chef.io"
 
 group :integration do
-  cookbook 'inspec_habitat_fixture', path: 'test/cookbooks/inspec_habitat_fixture/'
+  cookbook "inspec_habitat_fixture", path: "test/cookbooks/inspec_habitat_fixture/"
 end

data/lib/plugins/inspec-habitat/lib/inspec-habitat.rb

@@ -3,7 +3,7 @@ module InspecPlugins
     class Plugin < Inspec.plugin(2)
       plugin_name :'inspec-habitat'
       cli_command :habitat do
-        require_relative 'inspec-habitat/cli'
+        require_relative "inspec-habitat/cli"
         InspecPlugins::Habitat::CLI
       end
     end

data/lib/plugins/inspec-habitat/lib/inspec-habitat/cli.rb

@@ -1,5 +1,5 @@
-# encoding: utf-8
-require_relative 'profile'
+require_relative "profile"
+require "inspec/dist"
 
 module InspecPlugins
   module Habitat
@@ -10,30 +10,32 @@ module InspecPlugins
         "#{basename} habitat profile #{command.usage}"
       end
 
-      desc 'create PATH', 'Create a Habitat artifact for the profile found at PATH'
+      desc "create PATH", "Create a Habitat artifact for the profile found at PATH"
       option :output_dir, type: :string, required: false,
-        desc: 'Output directory for the Habitat artifact. Default: current directory'
-      def create(path = '.')
+        desc: "Output directory for the Habitat artifact. Default: current directory"
+      def create(path = ".")
         InspecPlugins::Habitat::Profile.new(path, options).create
       end
 
-      desc 'setup PATH', 'Configure the profile at PATH for Habitat, including a plan and hooks'
-      def setup(path = '.')
+      desc "setup PATH", "Configure the profile at PATH for Habitat, including a plan and hooks"
+      def setup(path = ".")
         InspecPlugins::Habitat::Profile.new(path, options).setup
       end
 
-      desc 'upload PATH', 'Create then upload a Habitat artifact for the profile found at PATH to the Habitat Builder Depot'
-      def upload(path = '.')
+      desc "upload PATH", "Create then upload a Habitat artifact for the profile found at PATH to the Habitat Builder Depot"
+      def upload(path = ".")
         InspecPlugins::Habitat::Profile.new(path, options).upload
       end
     end
 
     class CLI < Inspec.plugin(2, :cli_command)
-      subcommand_desc 'habitat SUBCOMMAND', 'Manage Habitat with InSpec'
-      namespace 'habitat'
+      include Inspec::Dist
 
-      desc 'profile', 'Manage InSpec profiles as Habitat artifacts'
-      subcommand 'profile', ProfileCLI
+      subcommand_desc "habitat SUBCOMMAND", "Manage Habitat with #{PRODUCT_NAME}"
+      namespace "habitat"
+
+      desc "profile", "Manage #{PRODUCT_NAME} profiles as Habitat artifacts"
+      subcommand "profile", ProfileCLI
     end
   end
 end

data/lib/plugins/inspec-habitat/lib/inspec-habitat/profile.rb

@@ -1,19 +1,20 @@
-# encoding: utf-8
-
-require 'inspec/profile_vendor'
-require 'mixlib/shellout'
-require 'tomlrb'
-require 'ostruct'
+require "inspec/profile_vendor"
+require "mixlib/shellout"
+require "tomlrb"
+require "ostruct"
+require "inspec/dist"
 
 module InspecPlugins
   module Habitat
     class Profile
+      include Inspec::Dist
+
       attr_reader :logger
       def initialize(path, options = {})
         @path    = path
         @options = options
         @logger  = Inspec::Log
-        logger.level(options.fetch(:log_level, 'info').to_sym)
+        logger.level(options.fetch(:log_level, "info").to_sym)
       end
 
       def create
@@ -28,7 +29,7 @@ module InspecPlugins
         output_dir = @options[:output_dir] || Dir.pwd
         unless File.directory?(output_dir)
           exit_with_error("Output directory #{output_dir} is not a directory " \
-                          'or does not exist.')
+                          "or does not exist.")
         end
 
         duplicated_profile = duplicate_profile(@path, working_dir)
@@ -44,7 +45,7 @@ module InspecPlugins
         destination
       rescue => e
         logger.debug(e.backtrace.join("\n"))
-        exit_with_error('Unable to create Habitat artifact.')
+        exit_with_error("Unable to create Habitat artifact.")
       ensure
         if Dir.exist?(working_dir)
           logger.debug("Deleting working directory #{working_dir}")
@@ -56,34 +57,34 @@ module InspecPlugins
         path = profile.root_path
         logger.debug("Setting up #{path} for Habitat...")
 
-        plan_file = File.join(path, 'habitat', 'plan.sh')
+        plan_file = File.join(path, "habitat", "plan.sh")
         logger.info("Generating Habitat plan at #{plan_file}...")
         vars = {
           profile: profile,
-          habitat_origin: read_habitat_config['origin'],
+          habitat_origin: read_habitat_config["origin"],
         }
-        create_file_from_template(plan_file, 'plan.sh.erb', vars)
+        create_file_from_template(plan_file, "plan.sh.erb", vars)
 
-        run_hook_file = File.join(path, 'habitat', 'hooks', 'run')
+        run_hook_file = File.join(path, "habitat", "hooks", "run")
         logger.info("Generating a Habitat run hook at #{run_hook_file}...")
-        create_file_from_template(run_hook_file, 'hooks/run.erb')
+        create_file_from_template(run_hook_file, "hooks/run.erb")
 
-        default_toml = File.join(path, 'habitat', 'default.toml')
+        default_toml = File.join(path, "habitat", "default.toml")
         logger.info("Generating a Habitat default.toml at #{default_toml}...")
-        create_file_from_template(default_toml, 'default.toml.erb')
+        create_file_from_template(default_toml, "default.toml.erb")
 
-        config = File.join(path, 'habitat', 'config', 'inspec_exec_config.json')
-        logger.info("Generating #{config} for `inspec exec`...")
-        create_file_from_template(config, 'config/inspec_exec_config.json.erb')
+        config = File.join(path, "habitat", "config", "inspec_exec_config.json")
+        logger.info("Generating #{config} for `#{EXEC_NAME} exec`...")
+        create_file_from_template(config, "config/inspec_exec_config.json.erb")
       end
 
       def upload
         habitat_config = read_habitat_config
 
-        if habitat_config['auth_token'].nil?
+        if habitat_config["auth_token"].nil?
           exit_with_error(
-            'Unable to determine Habitat auth token for uploading.',
-            'Run `hab setup` or set the HAB_AUTH_TOKEN environment variable.',
+            "Unable to determine Habitat auth token for uploading.",
+            "Run `hab setup` or set the HAB_AUTH_TOKEN environment variable."
           )
         end
 
@@ -95,7 +96,7 @@ module InspecPlugins
         logger.info("Habitat artifact #{hart} uploaded.")
       rescue => e
         logger.debug(e.backtrace.join("\n"))
-        exit_with_error('Unable to upload Habitat artifact.')
+        exit_with_error("Unable to upload Habitat artifact.")
       end
 
       private
@@ -121,14 +122,14 @@ module InspecPlugins
       def profile_from_path(path)
         Inspec::Profile.for_target(
           path,
-          backend: Inspec::Backend.create(Inspec::Config.mock),
+          backend: Inspec::Backend.create(Inspec::Config.mock)
         )
       end
 
       def copy_profile_to_working_dir(profile, working_dir)
-        logger.debug('Copying profile contents to the working directory...')
+        logger.debug("Copying profile contents to the working directory...")
         profile.files.each do |profile_file|
-          next if File.extname(profile_file) == '.hart'
+          next if File.extname(profile_file) == ".hart"
 
           src = File.join(profile.root_path, profile_file)
           dst = File.join(working_dir, profile_file)
@@ -143,86 +144,86 @@ module InspecPlugins
       end
 
       def verify_profile(profile)
-        logger.debug('Checking to see if the profile is valid...')
+        logger.debug("Checking to see if the profile is valid...")
 
         unless profile.check[:summary][:valid]
-          exit_with_error('Profile check failed. Please fix the profile ' \
-                          'before creating a Habitat artifact.')
+          exit_with_error("Profile check failed. Please fix the profile " \
+                          "before creating a Habitat artifact.")
         end
 
-        logger.debug('Profile is valid.')
+        logger.debug("Profile is valid.")
       end
 
       def vendor_profile_dependencies!(profile)
         profile_vendor = Inspec::ProfileVendor.new(profile.root_path)
         if profile_vendor.lockfile.exist? && profile_vendor.cache_path.exist?
           logger.debug("Profile's dependencies are already vendored, skipping " \
-                    'vendor process.')
+                    "vendor process.")
         else
           logger.debug("Vendoring the profile's dependencies...")
           profile_vendor.vendor!
 
-          logger.debug('Ensuring all vendored content has read permissions...')
+          logger.debug("Ensuring all vendored content has read permissions...")
           profile_vendor.make_readable
         end
 
         # Return new profile since it has changed
         Inspec::Profile.for_target(
           profile.root_path,
-          backend: Inspec::Backend.create(Inspec::Config.mock),
+          backend: Inspec::Backend.create(Inspec::Config.mock)
         )
       end
 
       def verify_habitat_setup(habitat_config)
-        logger.debug('Checking to see if Habitat is installed...')
-        cmd = Mixlib::ShellOut.new('hab --version')
+        logger.debug("Checking to see if Habitat is installed...")
+        cmd = Mixlib::ShellOut.new("hab --version")
         cmd.run_command
         if cmd.error?
-          exit_with_error('Unable to run Habitat commands.', cmd.stderr)
+          exit_with_error("Unable to run Habitat commands.", cmd.stderr)
         end
 
-        if habitat_config['origin'].nil?
+        if habitat_config["origin"].nil?
           exit_with_error(
-            'Unable to determine Habitat origin name.',
-            'Run `hab setup` or set the HAB_ORIGIN environment variable.',
+            "Unable to determine Habitat origin name.",
+            "Run `hab setup` or set the HAB_ORIGIN environment variable."
           )
         end
       end
 
       def create_file_from_template(file, template, vars = {})
         FileUtils.mkdir_p(File.dirname(file))
-        template_path = File.join(__dir__, '../../templates/habitat', template)
+        template_path = File.join(__dir__, "../../templates/habitat", template)
         contents = ERB.new(File.read(template_path))
                       .result(OpenStruct.new(vars).instance_eval { binding })
         File.write(file, contents)
       end
 
       def build_hart(working_dir, habitat_config)
-        logger.debug('Building our Habitat artifact...')
+        logger.debug("Building our Habitat artifact...")
 
         env = {
-          'TERM'               => 'vt100',
-          'HAB_ORIGIN'         => habitat_config['origin'],
-          'HAB_NONINTERACTIVE' => 'true',
+          "TERM" => "vt100",
+          "HAB_ORIGIN" => habitat_config["origin"],
+          "HAB_NONINTERACTIVE" => "true",
         }
 
-        env['RUST_LOG'] = 'debug' if logger.level == :debug
+        env["RUST_LOG"] = "debug" if logger.level == :debug
 
         # TODO: Would love to use Mixlib::ShellOut here, but it doesn't
         # seem to preserve the STDIN tty, and docker gets angry.
         Dir.chdir(working_dir) do
-          unless system(env, 'hab pkg build .')
-            exit_with_error('Unable to build the Habitat artifact.')
+          unless system(env, "hab pkg build .")
+            exit_with_error("Unable to build the Habitat artifact.")
           end
         end
 
-        hart_files = Dir.glob(File.join(working_dir, 'results', '*.hart'))
+        hart_files = Dir.glob(File.join(working_dir, "results", "*.hart"))
 
         if hart_files.length > 1
-          exit_with_error('More than one Habitat artifact was created which ' \
-                          'was not expected.')
+          exit_with_error("More than one Habitat artifact was created which " \
+                          "was not expected.")
         elsif hart_files.empty?
-          exit_with_error('No Habitat artifact was created.')
+          exit_with_error("No Habitat artifact was created.")
         end
 
         hart_files.first
@@ -234,33 +235,33 @@ module InspecPlugins
         config = habitat_config
 
         env = {
-          'HAB_AUTH_TOKEN'     => config['auth_token'],
-          'HAB_NONINTERACTIVE' => 'true',
-          'HAB_ORIGIN'         => config['origin'],
-          'TERM'               => 'vt100',
+          "HAB_AUTH_TOKEN" => config["auth_token"],
+          "HAB_NONINTERACTIVE" => "true",
+          "HAB_ORIGIN" => config["origin"],
+          "TERM" => "vt100",
         }
 
-        env['HAB_DEPOT_URL'] = ENV['HAB_DEPOT_URL'] if ENV['HAB_DEPOT_URL']
+        env["HAB_DEPOT_URL"] = ENV["HAB_DEPOT_URL"] if ENV["HAB_DEPOT_URL"]
 
         cmd = Mixlib::ShellOut.new("hab pkg upload #{hart_file}", env: env)
         cmd.run_command
         if cmd.error?
           exit_with_error(
-            'Unable to upload Habitat artifact to the Depot.',
+            "Unable to upload Habitat artifact to the Depot.",
             cmd.stdout,
-            cmd.stderr,
+            cmd.stderr
           )
         end
 
-        logger.debug('Upload complete!')
+        logger.debug("Upload complete!")
       end
 
       def read_habitat_config
-        cli_toml = File.join(ENV['HOME'], '.hab', 'etc', 'cli.toml')
-        cli_toml = '/hab/etc/cli.toml' unless File.exist?(cli_toml)
+        cli_toml = File.join(ENV["HOME"], ".hab", "etc", "cli.toml")
+        cli_toml = "/hab/etc/cli.toml" unless File.exist?(cli_toml)
         cli_config = File.exist?(cli_toml) ? Tomlrb.load_file(cli_toml) : {}
-        cli_config['origin'] ||= ENV['HAB_ORIGIN']
-        cli_config['auth_token'] ||= ENV['HAB_AUTH_TOKEN']
+        cli_config["origin"] ||= ENV["HAB_ORIGIN"]
+        cli_config["auth_token"] ||= ENV["HAB_AUTH_TOKEN"]
         cli_config
       end