test/deprecate_disable_spec: increase test coverage

Homebrew · Dec 20, 2024 · a153c9a · a153c9a
1 parent 21f3536
commit a153c9a
Showing 1 changed file with 22 additions and 0 deletions.
diff --git a/Library/Homebrew/test/deprecate_disable_spec.rb b/Library/Homebrew/test/deprecate_disable_spec.rb
@@ -3,14 +3,23 @@
 require "deprecate_disable"
 
 RSpec.describe DeprecateDisable do
+  let(:disable_date) { Date.parse("2020-01-01") }
   let(:deprecated_formula) do
     instance_double(Formula, deprecated?: true, disabled?: false, deprecation_reason: :does_not_build,
                     deprecation_replacement: nil, deprecation_date: nil, disable_date: nil)
   end
+  let(:deprecated_formula_with_disable_date) do
+    instance_double(Formula, deprecated?: true, disabled?: false, deprecation_reason: :does_not_build,
+                    deprecation_replacement: nil, deprecation_date: nil, disable_date: disable_date)
+  end
   let(:disabled_formula) do
     instance_double(Formula, deprecated?: false, disabled?: true, disable_reason: "is broken",
                     disable_replacement: nil, deprecation_date: nil, disable_date: nil)
   end
+  let(:disabled_formula_with_disable_date) do
+    instance_double(Formula, deprecated?: false, disabled?: true, disable_reason: :does_not_build,
+                    disable_replacement: nil, deprecation_date: nil, disable_date: disable_date)
+  end
   let(:deprecated_cask) do
     instance_double(Cask::Cask, deprecated?: true, disabled?: false, deprecation_reason: :discontinued,
                    deprecation_replacement: nil, deprecation_date: nil, disable_date: nil)
@@ -39,7 +48,9 @@
   before do
     formulae = [
       deprecated_formula,
+      deprecated_formula_with_disable_date,
       disabled_formula,
+      disabled_formula_with_disable_date,
       deprecated_formula_with_replacement,
       disabled_formula_with_replacement,
     ]
@@ -86,11 +97,22 @@
         .to eq "deprecated because it does not build!"
     end
 
+    it "returns a deprecation message with disable date" do
+      allow(Date).to receive(:today).and_return(disable_date << 1)
+      expect(described_class.message(disabled_formula_with_disable_date))
+        .to eq "disabled because it does not build! It will be disabled on #{disable_date}."
+    end
+
     it "returns a disable message with a custom reason" do
       expect(described_class.message(disabled_formula))
         .to eq "disabled because it is broken!"
     end
 
+    it "returns a disable message with disable date" do
+      expect(described_class.message(disabled_formula_with_disable_date))
+        .to eq "disabled because it does not build! It was disabled on #{disable_date}."
+    end
+
     it "returns a deprecation message with a preset cask reason" do
       expect(described_class.message(deprecated_cask))
         .to eq "deprecated because it is discontinued upstream!"