2018-01-31 17:46:44 +00:00
|
|
|
require "../../spec_helper"
|
|
|
|
|
|
|
|
module Ameba::Rule
|
|
|
|
describe UnneededDisableDirective do
|
|
|
|
subject = UnneededDisableDirective.new
|
|
|
|
|
|
|
|
it "passes if there are no comments" do
|
|
|
|
s = Source.new %(
|
|
|
|
a = 1
|
|
|
|
)
|
|
|
|
subject.catch(s).should be_valid
|
|
|
|
end
|
|
|
|
|
|
|
|
it "passes if there is disable directive" do
|
|
|
|
s = Source.new %(
|
|
|
|
a = 1 # my super var
|
|
|
|
)
|
|
|
|
subject.catch(s).should be_valid
|
|
|
|
end
|
|
|
|
|
2018-05-08 21:14:46 +00:00
|
|
|
it "doesn't report if there is disable directive and it is needed" do
|
|
|
|
s = Source.new %Q(
|
|
|
|
# ameba:disable #{NamedRule.name}
|
|
|
|
a = 1
|
|
|
|
)
|
|
|
|
s.error NamedRule.new, 3, 9, "Useless assignment", :disabled
|
|
|
|
subject.catch(s).should be_valid
|
|
|
|
end
|
|
|
|
|
|
|
|
it "passes if there is inline disable directive and it is needed" do
|
2018-01-31 17:46:44 +00:00
|
|
|
s = Source.new %Q(
|
|
|
|
a = 1 # ameba:disable #{NamedRule.name}
|
|
|
|
)
|
|
|
|
s.error NamedRule.new, 2, 1, "Alarm!", :disabled
|
|
|
|
subject.catch(s).should be_valid
|
|
|
|
end
|
|
|
|
|
|
|
|
it "ignores commented out disable directive" do
|
|
|
|
s = Source.new %Q(
|
|
|
|
# # ameba:disable #{NamedRule.name}
|
|
|
|
a = 1
|
|
|
|
)
|
|
|
|
s.error NamedRule.new, 3, 1, "Alarm!", :disabled
|
|
|
|
subject.catch(s).should be_valid
|
|
|
|
end
|
|
|
|
|
2018-05-03 15:57:47 +00:00
|
|
|
it "fails if there is unneeded directive" do
|
2018-01-31 17:46:44 +00:00
|
|
|
s = Source.new %Q(
|
|
|
|
# ameba:disable #{NamedRule.name}
|
|
|
|
a = 1
|
|
|
|
)
|
|
|
|
subject.catch(s).should_not be_valid
|
|
|
|
s.errors.first.message.should eq(
|
|
|
|
"Unnecessary disabling of #{NamedRule.name}"
|
|
|
|
)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "fails if there is inline unneeded directive" do
|
|
|
|
s = Source.new %Q(a = 1 # ameba:disable #{NamedRule.name})
|
|
|
|
subject.catch(s).should_not be_valid
|
|
|
|
s.errors.first.message.should eq(
|
|
|
|
"Unnecessary disabling of #{NamedRule.name}"
|
|
|
|
)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "detects mixed inline directives" do
|
|
|
|
s = Source.new %Q(
|
|
|
|
# ameba:disable Rule1, Rule2
|
|
|
|
a = 1 # ameba:disable Rule3
|
|
|
|
), "source.cr"
|
|
|
|
subject.catch(s).should_not be_valid
|
|
|
|
s.errors.size.should eq 2
|
|
|
|
s.errors.first.message.should contain "Rule1, Rule2"
|
|
|
|
s.errors.last.message.should contain "Rule3"
|
|
|
|
end
|
|
|
|
|
2018-02-02 08:52:21 +00:00
|
|
|
it "fails if there is disabled UnneededDisableDirective" do
|
|
|
|
s = Source.new %Q(
|
2018-02-02 20:11:18 +00:00
|
|
|
# ameba:disable #{UnneededDisableDirective.rule_name}
|
2018-02-02 08:52:21 +00:00
|
|
|
a = 1
|
|
|
|
), "source.cr"
|
|
|
|
s.error UnneededDisableDirective.new, 3, 1, "Alarm!", :disabled
|
|
|
|
subject.catch(s).should_not be_valid
|
|
|
|
end
|
|
|
|
|
2018-01-31 17:46:44 +00:00
|
|
|
it "reports error, location and message" do
|
|
|
|
s = Source.new %Q(
|
|
|
|
# ameba:disable Rule1, Rule2
|
|
|
|
a = 1
|
|
|
|
), "source.cr"
|
|
|
|
subject.catch(s).should_not be_valid
|
|
|
|
error = s.errors.first
|
|
|
|
error.rule.should_not be_nil
|
|
|
|
error.location.to_s.should eq "source.cr:2:9"
|
|
|
|
error.message.should eq "Unnecessary disabling of Rule1, Rule2"
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|