Adds tests for dump --describe

Homebrew · Feb 7, 2018 · ce55372 · ce55372
1 parent 132f9d7
commit ce55372
Showing 1 changed file with 93 additions and 0 deletions.
diff --git a/spec/brew_dumper_spec.rb b/spec/brew_dumper_spec.rb
@@ -477,6 +477,99 @@
     end
   end
 
+  context "when --describe is not set" do
+    before do
+      ARGV.delete("--describe") if ARGV.include?("--describe")
+      Bundle::BrewDumper.reset!
+      allow(Bundle::BrewDumper).to receive(:formulae_info).and_return [
+        {
+          name: "a",
+          full_name: "a",
+          desc: "z",
+          aliases: [],
+          args: [],
+          version: "1.0",
+          dependencies: ["b", "d", "c"],
+          recommended_dependencies: [],
+          optional_dependencies: [],
+          build_dependencies: [],
+          requirements: [],
+          conflicts_with: [],
+          pinned?: false,
+          outdated?: false,
+        },
+        {
+          name: "b",
+          full_name: "b",
+          aliases: [],
+          args: [],
+          version: "1.0",
+          dependencies: [],
+          recommended_dependencies: [],
+          optional_dependencies: [],
+          build_dependencies: [],
+          requirements: [],
+          conflicts_with: [],
+          pinned?: false,
+          outdated?: false,
+        },
+      ]
+    end
+    it "does not output a comment with dependency description" do
+      expect(Bundle::BrewDumper.dump).not_to include("#")
+    end
+  end
+
+  context "when --describe is set" do
+    before do
+      ARGV << "--describe" unless ARGV.include?("--describe")
+      Bundle::BrewDumper.reset!
+      allow(Bundle::BrewDumper).to receive(:formulae_info).and_return [
+        {
+          name: "a",
+          full_name: "a",
+          desc: "z",
+          aliases: [],
+          args: [],
+          version: "1.0",
+          dependencies: ["b", "d", "c"],
+          recommended_dependencies: [],
+          optional_dependencies: [],
+          build_dependencies: [],
+          requirements: [],
+          conflicts_with: [],
+          pinned?: false,
+          outdated?: false,
+        },
+        {
+          name: "b",
+          full_name: "b",
+          aliases: [],
+          args: [],
+          version: "1.0",
+          dependencies: [],
+          recommended_dependencies: [],
+          optional_dependencies: [],
+          build_dependencies: [],
+          requirements: [],
+          conflicts_with: [],
+          pinned?: false,
+          outdated?: false,
+        },
+      ]
+    end
+    after do
+      ARGV.delete("--describe") if ARGV.include?("--describe")
+    end
+    it "outputs a comment on the line before a dependency with a description" do
+      expect(Bundle::BrewDumper.dump).to include("# z")
+    end
+    it "does not output a comment if a formula lacks a description" do
+      lines_with_comments = Bundle::BrewDumper.dump.split.select { |line| line.include?("#") }
+      expect(lines_with_comments.size).to eq(1)
+    end
+  end
+
   context "when order of args for a formula is different in different environment" do
     it "dumps args in same order" do
       formula_info = [