Merge pull request #3289 from reitermarkus/cask-loader1.3.5

Fix regexes in `CaskLoader`.

2 files changed, 78 insertions, 0 deletions

diff --git a/Library/Homebrew/test/cask/cask_loader/from_content_loader_spec.rb b/Library/Homebrew/test/cask/cask_loader/from_content_loader_spec.rb
new file mode 100644
index 000000000..e735d5eca
--- /dev/null
+++ b/Library/Homebrew/test/cask/cask_loader/from_content_loader_spec.rb
@@ -0,0 +1,57 @@
+describe Hbc::CaskLoader::FromContentLoader do
+  alias_matcher :be_able_to_load, :be_can_load
+
+  describe "::can_load?" do
+    it "returns true for Casks specified with `cask \"token\" do … end`" do
+      expect(described_class).to be_able_to_load <<~EOS
+        cask "token" do
+        end
+      EOS
+    end
+
+    it "returns true for Casks specified with `cask \"token\" do; end`" do
+      expect(described_class).to be_able_to_load <<~EOS
+        cask "token" do; end
+      EOS
+    end
+
+    it "returns true for Casks specified with `cask 'token' do … end`" do
+      expect(described_class).to be_able_to_load <<~EOS
+        cask 'token' do
+        end
+      EOS
+    end
+
+    it "returns true for Casks specified with `cask 'token' do; end`" do
+      expect(described_class).to be_able_to_load <<~EOS
+        cask 'token' do; end
+      EOS
+    end
+
+    it "returns true for Casks specified with `cask(\"token\") { … }`" do
+      expect(described_class).to be_able_to_load <<~EOS
+        cask("token") {
+        }
+      EOS
+    end
+
+    it "returns true for Casks specified with `cask(\"token\") {}`" do
+      expect(described_class).to be_able_to_load <<~EOS
+        cask("token") {}
+      EOS
+    end
+
+    it "returns true for Casks specified with `cask('token') { … }`" do
+      expect(described_class).to be_able_to_load <<~EOS
+        cask('token') {
+        }
+      EOS
+    end
+
+    it "returns true for Casks specified with `cask('token') {}`" do
+      expect(described_class).to be_able_to_load <<~EOS
+        cask('token') {}
+      EOS
+    end
+  end
+end
diff --git a/Library/Homebrew/test/cask/cask_loader/from_uri_loader_spec.rb b/Library/Homebrew/test/cask/cask_loader/from_uri_loader_spec.rb
new file mode 100644
index 000000000..df2de1f82
--- /dev/null
+++ b/Library/Homebrew/test/cask/cask_loader/from_uri_loader_spec.rb
@@ -0,0 +1,21 @@
+describe Hbc::CaskLoader::FromURILoader do
+  alias_matcher :be_able_to_load, :be_can_load
+
+  describe "::can_load?" do
+    it "returns true when given an URI" do
+      expect(described_class).to be_able_to_load(URI("http://example.com/"))
+    end
+
+    it "returns true when given a String which can be parsed to a URI" do
+      expect(described_class).to be_able_to_load("http://example.com/")
+    end
+
+    it "returns false when given a String with Cask contents containing a URL" do
+      expect(described_class).not_to be_able_to_load <<~EOS
+        cask 'token' do
+          url 'http://example.com/'
+        end
+      EOS
+    end
+  end
+end