shard-ameba/spec/ameba/rule/lint/literals_comparison_spec.cr

67 lines
1.9 KiB
Crystal
Raw Normal View History

2022-11-01 02:17:37 +00:00
require "../../../spec_helper"
module Ameba::Rule::Lint
subject = LiteralsComparison.new
2022-11-01 02:17:37 +00:00
describe LiteralsComparison do
2022-11-01 02:17:37 +00:00
it "passes for valid cases" do
expect_no_issues subject, <<-CRYSTAL
"foo" == foo
"foo" != foo
2022-11-01 02:17:37 +00:00
foo == "foo"
foo != "foo"
2022-11-01 02:17:37 +00:00
CRYSTAL
end
it "reports if there is a regex comparison possibly evaluating to the same" do
expect_issue subject, <<-CRYSTAL
/foo/ === "foo"
# ^^^^^^^^^^^^^ error: Comparison most likely evaluates to the same
CRYSTAL
end
it "reports if there is a static comparison evaluating to the same" do
expect_issue subject, <<-CRYSTAL
"foo" === "foo"
# ^^^^^^^^^^^^^ error: Comparison always evaluates to the same
CRYSTAL
end
it "reports if there is a static comparison evaluating to true (2)" do
2022-11-01 02:17:37 +00:00
expect_issue subject, <<-CRYSTAL
"foo" == "foo"
# ^^^^^^^^^^^^ error: Comparison always evaluates to true
CRYSTAL
end
it "reports if there is a static comparison evaluating to false" do
expect_issue subject, <<-CRYSTAL
"foo" != "foo"
# ^^^^^^^^^^^^ error: Comparison always evaluates to false
CRYSTAL
end
context "macro" do
it "reports in macro scope" do
expect_issue subject, <<-CRYSTAL
{{ "foo" == "foo" }}
# ^^^^^^^^^^^^^^ error: Comparison always evaluates to true
CRYSTAL
end
end
it "reports rule, pos and message" do
s = Source.new %(
"foo" == "foo"
), "source.cr"
subject.catch(s).should_not be_valid
issue = s.issues.first
issue.rule.should_not be_nil
issue.location.to_s.should eq "source.cr:1:1"
issue.end_location.to_s.should eq "source.cr:1:14"
issue.message.should eq "Comparison always evaluates to true"
end
end
end