From ddf5583208e473ef00447834a3a712411e61e190 Mon Sep 17 00:00:00 2001
From: Markus Reiter <me@reitermark.us>
Date: Tue, 4 Oct 2016 15:24:58 +0200
Subject: [PATCH] =?UTF-8?q?Refactor=20Cask=E2=80=99s=20extensions.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 Library/Homebrew/cask/lib/hbc.rb                           | 1 -
 Library/Homebrew/cask/lib/hbc/artifact/artifact.rb         | 4 +++-
 Library/Homebrew/cask/lib/hbc/artifact/pkg.rb              | 4 +++-
 Library/Homebrew/cask/lib/hbc/artifact/relocated.rb        | 4 +++-
 Library/Homebrew/cask/lib/hbc/cli.rb                       | 2 ++
 Library/Homebrew/cask/lib/hbc/extend.rb                    | 6 ------
 Library/Homebrew/cask/lib/hbc/extend/string.rb             | 5 -----
 Library/Homebrew/cask/lib/hbc/system_command.rb            | 7 ++++++-
 .../lib/hbc/{extend/hash.rb => utils/hash_validator.rb}    | 2 +-
 Library/Homebrew/diagnostic.rb                             | 2 +-
 Library/Homebrew/{cask/lib/hbc => }/extend/io.rb           | 0
 Library/Homebrew/{cask/lib/hbc => }/extend/optparse.rb     | 0
 Library/Homebrew/test/test_diagnostic.rb                   | 2 +-
 13 files changed, 20 insertions(+), 19 deletions(-)
 delete mode 100644 Library/Homebrew/cask/lib/hbc/extend.rb
 delete mode 100644 Library/Homebrew/cask/lib/hbc/extend/string.rb
 rename Library/Homebrew/cask/lib/hbc/{extend/hash.rb => utils/hash_validator.rb} (91%)
 rename Library/Homebrew/{cask/lib/hbc => }/extend/io.rb (100%)
 rename Library/Homebrew/{cask/lib/hbc => }/extend/optparse.rb (100%)

diff --git a/Library/Homebrew/cask/lib/hbc.rb b/Library/Homebrew/cask/lib/hbc.rb
index 62b391638a..69b6e8f21c 100644
--- a/Library/Homebrew/cask/lib/hbc.rb
+++ b/Library/Homebrew/cask/lib/hbc.rb
@@ -1,7 +1,6 @@
 module Hbc; end
 
 require "hardware"
-require "hbc/extend"
 require "hbc/artifact"
 require "hbc/audit"
 require "hbc/auditor"
diff --git a/Library/Homebrew/cask/lib/hbc/artifact/artifact.rb b/Library/Homebrew/cask/lib/hbc/artifact/artifact.rb
index cb35821ccf..b42db877d0 100644
--- a/Library/Homebrew/cask/lib/hbc/artifact/artifact.rb
+++ b/Library/Homebrew/cask/lib/hbc/artifact/artifact.rb
@@ -1,5 +1,7 @@
 require "hbc/artifact/moved"
 
+require "hbc/utils/hash_validator"
+
 module Hbc
   module Artifact
     class Artifact < Moved
@@ -16,7 +18,7 @@ module Hbc
         raise CaskInvalidError.new(@cask.token, "no source given for artifact") if source_string.nil?
         @source = @cask.staged_path.join(source_string)
         raise CaskInvalidError.new(@cask.token, "target required for generic artifact #{source_string}") unless target_hash.is_a?(Hash)
-        target_hash.assert_valid_keys(:target)
+        target_hash.extend(HashValidator).assert_valid_keys(:target)
         @target = Pathname.new(target_hash[:target])
       end
     end
diff --git a/Library/Homebrew/cask/lib/hbc/artifact/pkg.rb b/Library/Homebrew/cask/lib/hbc/artifact/pkg.rb
index e590a9082d..372740631a 100644
--- a/Library/Homebrew/cask/lib/hbc/artifact/pkg.rb
+++ b/Library/Homebrew/cask/lib/hbc/artifact/pkg.rb
@@ -1,5 +1,7 @@
 require "hbc/artifact/base"
 
+require "hbc/utils/hash_validator"
+
 module Hbc
   module Artifact
     class Pkg < Base
@@ -14,7 +16,7 @@ module Hbc
         @pkg_install_opts = pkg_description.shift
         begin
           if @pkg_install_opts.respond_to?(:keys)
-            @pkg_install_opts.assert_valid_keys(:allow_untrusted)
+            @pkg_install_opts.extend(HashValidator).assert_valid_keys(:allow_untrusted)
           elsif @pkg_install_opts
             raise
           end
diff --git a/Library/Homebrew/cask/lib/hbc/artifact/relocated.rb b/Library/Homebrew/cask/lib/hbc/artifact/relocated.rb
index 953045b32b..1b5a4ff474 100644
--- a/Library/Homebrew/cask/lib/hbc/artifact/relocated.rb
+++ b/Library/Homebrew/cask/lib/hbc/artifact/relocated.rb
@@ -1,5 +1,7 @@
 require "hbc/artifact/base"
 
+require "hbc/utils/hash_validator"
+
 module Hbc
   module Artifact
     class Relocated < Base
@@ -46,7 +48,7 @@ module Hbc
         @source = @cask.staged_path.join(source_string)
         if target_hash
           raise CaskInvalidError unless target_hash.respond_to?(:keys)
-          target_hash.assert_valid_keys(:target)
+          target_hash.extend(HashValidator).assert_valid_keys(:target)
           @target = Hbc.send(self.class.artifact_dirmethod).join(target_hash[:target])
         else
           @target = Hbc.send(self.class.artifact_dirmethod).join(source.basename)
diff --git a/Library/Homebrew/cask/lib/hbc/cli.rb b/Library/Homebrew/cask/lib/hbc/cli.rb
index f637ae7af5..2173a52d00 100644
--- a/Library/Homebrew/cask/lib/hbc/cli.rb
+++ b/Library/Homebrew/cask/lib/hbc/cli.rb
@@ -1,6 +1,8 @@
 require "optparse"
 require "shellwords"
 
+require "extend/optparse"
+
 require "hbc/cli/base"
 require "hbc/cli/audit"
 require "hbc/cli/cat"
diff --git a/Library/Homebrew/cask/lib/hbc/extend.rb b/Library/Homebrew/cask/lib/hbc/extend.rb
deleted file mode 100644
index e836b6bc8e..0000000000
--- a/Library/Homebrew/cask/lib/hbc/extend.rb
+++ /dev/null
@@ -1,6 +0,0 @@
-# monkeypatching
-require "hbc/extend/hash"
-require "hbc/extend/io"
-require "hbc/extend/optparse"
-require "extend/pathname"
-require "hbc/extend/string"
diff --git a/Library/Homebrew/cask/lib/hbc/extend/string.rb b/Library/Homebrew/cask/lib/hbc/extend/string.rb
deleted file mode 100644
index 38c284194b..0000000000
--- a/Library/Homebrew/cask/lib/hbc/extend/string.rb
+++ /dev/null
@@ -1,5 +0,0 @@
-class String
-  def undent
-    gsub(%r{^.{#{(slice(%r{^ +}) || '').length}}}, "")
-  end
-end
diff --git a/Library/Homebrew/cask/lib/hbc/system_command.rb b/Library/Homebrew/cask/lib/hbc/system_command.rb
index 160aadff9e..aa4b9b2957 100644
--- a/Library/Homebrew/cask/lib/hbc/system_command.rb
+++ b/Library/Homebrew/cask/lib/hbc/system_command.rb
@@ -1,6 +1,10 @@
 require "open3"
 require "shellwords"
 
+require "extend/io"
+
+require "hbc/utils/hash_validator"
+
 module Hbc
   class SystemCommand
     attr_reader :command
@@ -43,7 +47,8 @@ module Hbc
     attr_reader :executable, :options, :processed_output, :processed_status
 
     def process_options!
-      options.assert_valid_keys :input, :print_stdout, :print_stderr, :args, :must_succeed, :sudo, :bsexec
+      options.extend(HashValidator)
+             .assert_valid_keys :input, :print_stdout, :print_stderr, :args, :must_succeed, :sudo, :bsexec
       sudo_prefix = %w[/usr/bin/sudo -E --]
       bsexec_prefix = ["/bin/launchctl", "bsexec", options[:bsexec] == :startup ? "/" : options[:bsexec]]
       @command = [executable]
diff --git a/Library/Homebrew/cask/lib/hbc/extend/hash.rb b/Library/Homebrew/cask/lib/hbc/utils/hash_validator.rb
similarity index 91%
rename from Library/Homebrew/cask/lib/hbc/extend/hash.rb
rename to Library/Homebrew/cask/lib/hbc/utils/hash_validator.rb
index 3a06ba3af9..b9e11720d1 100644
--- a/Library/Homebrew/cask/lib/hbc/extend/hash.rb
+++ b/Library/Homebrew/cask/lib/hbc/utils/hash_validator.rb
@@ -1,4 +1,4 @@
-class Hash
+module HashValidator
   def assert_valid_keys(*valid_keys)
     unknown_keys = keys - valid_keys
     return if unknown_keys.empty?
diff --git a/Library/Homebrew/diagnostic.rb b/Library/Homebrew/diagnostic.rb
index 0db92592c4..394b766ae2 100644
--- a/Library/Homebrew/diagnostic.rb
+++ b/Library/Homebrew/diagnostic.rb
@@ -79,7 +79,7 @@ module Homebrew
       end
 
       def inject_file_list(list, string)
-        list.inject(string) { |a, e| a << "    #{e}\n" }
+        list.inject(string) { |a, e| a << "  #{e}\n" }
       end
       ############# END HELPERS
 
diff --git a/Library/Homebrew/cask/lib/hbc/extend/io.rb b/Library/Homebrew/extend/io.rb
similarity index 100%
rename from Library/Homebrew/cask/lib/hbc/extend/io.rb
rename to Library/Homebrew/extend/io.rb
diff --git a/Library/Homebrew/cask/lib/hbc/extend/optparse.rb b/Library/Homebrew/extend/optparse.rb
similarity index 100%
rename from Library/Homebrew/cask/lib/hbc/extend/optparse.rb
rename to Library/Homebrew/extend/optparse.rb
diff --git a/Library/Homebrew/test/test_diagnostic.rb b/Library/Homebrew/test/test_diagnostic.rb
index b9b995f0ff..33cf27b746 100644
--- a/Library/Homebrew/test/test_diagnostic.rb
+++ b/Library/Homebrew/test/test_diagnostic.rb
@@ -16,7 +16,7 @@ class DiagnosticChecksTest < Homebrew::TestCase
   def test_inject_file_list
     assert_equal "foo:\n",
       @checks.inject_file_list([], "foo:\n")
-    assert_equal "foo:\n    /a\n    /b\n",
+    assert_equal "foo:\n  /a\n  /b\n",
       @checks.inject_file_list(%w[/a /b], "foo:\n")
   end