Refactor `CLI::Home`.

1 files changed, 8 insertions, 33 deletions

diff --git a/Library/Homebrew/test/cask/cli/home_spec.rb b/Library/Homebrew/test/cask/cli/home_spec.rb
index 7be26dd4c..e985fb6cd 100644
--- a/Library/Homebrew/test/cask/cli/home_spec.rb
+++ b/Library/Homebrew/test/cask/cli/home_spec.rb
@@ -1,46 +1,21 @@
-# monkeypatch for testing
-module Hbc
-  class CLI
-    class Home
-      def self.system(*command)
-        system_commands << command
-      end
-
-      def self.reset!
-        @system_commands = []
-      end
-
-      def self.system_commands
-        @system_commands ||= []
-      end
-    end
-  end
-end
-
 describe Hbc::CLI::Home, :cask do
   before do
-    Hbc::CLI::Home.reset!
+    allow(described_class).to receive(:open_url)
   end
 
   it "opens the homepage for the specified Cask" do
-    Hbc::CLI::Home.run("local-caffeine")
-    expect(Hbc::CLI::Home.system_commands).to eq [
-      ["/usr/bin/open", "--", "http://example.com/local-caffeine"],
-    ]
+    expect(described_class).to receive(:open_url).with("http://example.com/local-caffeine")
+    described_class.run("local-caffeine")
   end
 
   it "works for multiple Casks" do
-    Hbc::CLI::Home.run("local-caffeine", "local-transmission")
-    expect(Hbc::CLI::Home.system_commands).to eq [
-      ["/usr/bin/open", "--", "http://example.com/local-caffeine"],
-      ["/usr/bin/open", "--", "http://example.com/local-transmission"],
-    ]
+    expect(described_class).to receive(:open_url).with("http://example.com/local-caffeine")
+    expect(described_class).to receive(:open_url).with("http://example.com/local-transmission")
+    described_class.run("local-caffeine", "local-transmission")
   end
 
   it "opens the project page when no Cask is specified" do
-    Hbc::CLI::Home.run
-    expect(Hbc::CLI::Home.system_commands).to eq [
-      ["/usr/bin/open", "--", "https://caskroom.github.io/"],
-    ]
+    expect(described_class).to receive(:open_url).with("https://caskroom.github.io/")
+    described_class.run
   end
 end