2017-11-05 20:08:01 +00:00
|
|
|
require "../../spec_helper"
|
|
|
|
|
|
|
|
module Ameba
|
2017-11-07 21:50:25 +00:00
|
|
|
subject = Rule::ConstantNames.new
|
2017-11-05 20:08:01 +00:00
|
|
|
|
2017-11-07 20:02:51 +00:00
|
|
|
private def it_reports_constant(code, expected)
|
2017-11-05 20:08:01 +00:00
|
|
|
it "reports constant name #{expected}" do
|
2017-11-07 20:02:51 +00:00
|
|
|
s = Source.new code
|
2017-11-07 21:50:25 +00:00
|
|
|
Rule::ConstantNames.new.catch(s).should_not be_valid
|
2018-06-10 21:15:12 +00:00
|
|
|
s.issues.first.message.should contain expected
|
2017-11-05 20:08:01 +00:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2017-11-07 21:50:25 +00:00
|
|
|
describe Rule::ConstantNames do
|
2017-11-05 20:08:01 +00:00
|
|
|
it "passes if type names are screaming-cased" do
|
|
|
|
s = Source.new %(
|
|
|
|
LUCKY_NUMBERS = [3, 7, 11]
|
|
|
|
DOCUMENTATION_URL = "http://crystal-lang.org/docs"
|
|
|
|
|
|
|
|
Int32
|
|
|
|
|
|
|
|
s : String = "str"
|
|
|
|
|
|
|
|
def works(n : Int32)
|
|
|
|
end
|
|
|
|
|
|
|
|
a = 1
|
|
|
|
myVar = 2
|
|
|
|
m_var = 3
|
|
|
|
)
|
|
|
|
subject.catch(s).should be_valid
|
|
|
|
end
|
|
|
|
|
|
|
|
it_reports_constant "MyBadConstant=1", "MYBADCONSTANT"
|
|
|
|
it_reports_constant "Wrong_NAME=2", "WRONG_NAME"
|
|
|
|
it_reports_constant "Wrong_Name=3", "WRONG_NAME"
|
|
|
|
|
|
|
|
it "reports rule, pos and message" do
|
|
|
|
s = Source.new %(
|
|
|
|
Const = 1
|
2017-11-07 20:02:51 +00:00
|
|
|
), "source.cr"
|
2017-11-05 20:08:01 +00:00
|
|
|
subject.catch(s).should_not be_valid
|
2018-06-10 21:15:12 +00:00
|
|
|
issue = s.issues.first
|
|
|
|
issue.rule.should_not be_nil
|
|
|
|
issue.location.to_s.should eq "source.cr:2:9"
|
|
|
|
issue.message.should eq(
|
2017-11-05 20:08:01 +00:00
|
|
|
"Constant name should be screaming-cased: CONST, not Const"
|
|
|
|
)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|