Enable RSpec/NoExpectationExample

This commit is contained in:
Douglas Eichelberger 2023-01-24 20:30:29 -08:00
parent e4c18be302
commit 2b8b3a5cb6
3 changed files with 6 additions and 2 deletions

View File

@ -15,8 +15,6 @@ RSpec/MessageSpies:
Enabled: false Enabled: false
RSpec/StubbedMock: RSpec/StubbedMock:
Enabled: false Enabled: false
RSpec/NoExpectationExample:
Enabled: false
# TODO: try to reduce these # TODO: try to reduce these
RSpec/ExampleLength: RSpec/ExampleLength:

View File

@ -67,6 +67,8 @@ describe FormulaInstaller do
expect(f.libexec).to be_a_directory expect(f.libexec).to be_a_directory
end end
# This test wraps expect() calls in `test_basic_formula_setup`
# rubocop:disable RSpec/NoExpectationExample
specify "basic bottle install" do specify "basic bottle install" do
allow(DevelopmentTools).to receive(:installed?).and_return(false) allow(DevelopmentTools).to receive(:installed?).and_return(false)
Homebrew.install_args.parse(["testball_bottle"]) Homebrew.install_args.parse(["testball_bottle"])
@ -74,6 +76,7 @@ describe FormulaInstaller do
test_basic_formula_setup(f) test_basic_formula_setup(f)
end end
end end
# rubocop:enable RSpec/NoExpectationExample
specify "basic bottle install with cellar information on sha256 line" do specify "basic bottle install with cellar information on sha256 line" do
allow(DevelopmentTools).to receive(:installed?).and_return(false) allow(DevelopmentTools).to receive(:installed?).and_return(false)

View File

@ -268,6 +268,8 @@ describe Keg do
expect(lib.children.length).to eq(2) expect(lib.children.length).to eq(2)
end end
# This is a legacy violation that would benefit from a clear expectation.
# rubocop:disable RSpec/NoExpectationExample
it "removes broken symlinks that conflict with directories" do it "removes broken symlinks that conflict with directories" do
a = HOMEBREW_CELLAR/"a"/"1.0" a = HOMEBREW_CELLAR/"a"/"1.0"
(a/"lib"/"foo").mkpath (a/"lib"/"foo").mkpath
@ -280,6 +282,7 @@ describe Keg do
keg.link keg.link
end end
# rubocop:enable RSpec/NoExpectationExample
end end
describe "#optlink" do describe "#optlink" do