aboutsummaryrefslogtreecommitdiffstats
path: root/Library/Homebrew/test
diff options
context:
space:
mode:
authorGautham Goli2017-08-14 02:43:54 +0530
committerGautham Goli2017-08-14 02:43:54 +0530
commit5a7cbb762f1a9e715ae03e82eb6cb8ef4ac4a0bb (patch)
treef0fc617a0988b25ab646d639a00a2f212577301c /Library/Homebrew/test
parent02a1406a2e4c2d0506861248eb767d99f5ce4515 (diff)
downloadbrew-5a7cbb762f1a9e715ae03e82eb6cb8ef4ac4a0bb.tar.bz2
add test for build.with? "--with-foo"
Diffstat (limited to 'Library/Homebrew/test')
-rw-r--r--Library/Homebrew/test/rubocops/lines_cop_spec.rb26
1 files changed, 25 insertions, 1 deletions
diff --git a/Library/Homebrew/test/rubocops/lines_cop_spec.rb b/Library/Homebrew/test/rubocops/lines_cop_spec.rb
index 0c225a5b6..273014cf1 100644
--- a/Library/Homebrew/test/rubocops/lines_cop_spec.rb
+++ b/Library/Homebrew/test/rubocops/lines_cop_spec.rb
@@ -716,7 +716,7 @@ describe RuboCop::Cop::FormulaAudit::Miscellaneous do
end
end
- it "with negated build.with?" do
+ it "with duplicated build.without?" do
source = <<-EOS.undent
class Foo < Formula
desc "foo"
@@ -739,6 +739,30 @@ describe RuboCop::Cop::FormulaAudit::Miscellaneous do
expect_offense(expected, actual)
end
end
+
+ it "with duplicated build.with?" do
+ source = <<-EOS.undent
+ class Foo < Formula
+ desc "foo"
+ url 'http://example.com/foo-1.0.tgz'
+ def post_install
+ return if build.with? "--with-bar"
+ end
+ end
+ EOS
+
+ expected_offenses = [{ message: "Don't duplicate 'with': Use `build.with? \"bar\"` to check for \"--with-bar\"",
+ severity: :convention,
+ line: 5,
+ column: 27,
+ source: source }]
+
+ inspect_source(cop, source)
+
+ expected_offenses.zip(cop.offenses).each do |expected, actual|
+ expect_offense(expected, actual)
+ end
+ end
end
def expect_offense(expected, actual)
expect(actual.message).to eq(expected[:message])