2018-06-16 11:50:59 +00:00
|
|
|
require "../../../spec_helper"
|
2017-11-04 21:13:38 +00:00
|
|
|
|
|
|
|
module Ameba
|
2018-06-16 11:50:59 +00:00
|
|
|
subject = Rule::Style::VariableNames.new
|
2017-11-04 21:13:38 +00:00
|
|
|
|
2022-12-21 16:23:56 +00:00
|
|
|
private def it_reports_var_name(name, value, expected, *, file = __FILE__, line = __LINE__)
|
|
|
|
it "reports variable name #{expected}", file, line do
|
2021-10-27 23:13:29 +00:00
|
|
|
rule = Rule::Style::VariableNames.new
|
2022-12-21 16:23:56 +00:00
|
|
|
expect_issue rule, <<-CRYSTAL, name: name, file: file, line: line
|
2021-10-27 23:13:29 +00:00
|
|
|
%{name} = #{value}
|
|
|
|
# ^{name} error: Var name should be underscore-cased: #{expected}, not %{name}
|
|
|
|
CRYSTAL
|
2017-11-04 21:13:38 +00:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2018-06-16 11:50:59 +00:00
|
|
|
describe Rule::Style::VariableNames do
|
2017-11-04 21:13:38 +00:00
|
|
|
it "passes if var names are underscore-cased" do
|
2021-10-27 23:13:29 +00:00
|
|
|
expect_no_issues subject, <<-CRYSTAL
|
2017-11-04 21:13:38 +00:00
|
|
|
class Greeting
|
|
|
|
@@default_greeting = "Hello world"
|
|
|
|
|
|
|
|
def initialize(@custom_greeting = nil)
|
|
|
|
end
|
|
|
|
|
|
|
|
def print_greeting
|
|
|
|
greeting = @custom_greeting || @@default_greeting
|
|
|
|
puts greeting
|
|
|
|
end
|
|
|
|
end
|
2021-10-27 23:13:29 +00:00
|
|
|
CRYSTAL
|
2017-11-04 21:13:38 +00:00
|
|
|
end
|
|
|
|
|
2021-10-27 23:13:29 +00:00
|
|
|
it_reports_var_name "myBadNamedVar", "1", "my_bad_named_var"
|
|
|
|
it_reports_var_name "wrong_Name", "'y'", "wrong_name"
|
2017-11-04 21:13:38 +00:00
|
|
|
|
2021-10-27 23:13:29 +00:00
|
|
|
it "reports instance variable name" do
|
|
|
|
expect_issue subject, <<-CRYSTAL
|
|
|
|
class Greeting
|
|
|
|
def initialize(@badNamed = nil)
|
|
|
|
# ^ error: Var name should be underscore-cased: @bad_named, not @badNamed
|
|
|
|
end
|
2017-11-04 21:13:38 +00:00
|
|
|
end
|
2021-10-27 23:13:29 +00:00
|
|
|
CRYSTAL
|
|
|
|
end
|
|
|
|
|
2021-11-09 19:49:29 +00:00
|
|
|
it "reports method with multiple instance variables" do
|
|
|
|
expect_issue subject, <<-CRYSTAL
|
|
|
|
class Location
|
|
|
|
def at(@startLocation = nil, @endLocation = nil)
|
|
|
|
# ^ error: Var name should be underscore-cased: @start_location, not @startLocation
|
|
|
|
# ^ error: Var name should be underscore-cased: @end_location, not @endLocation
|
|
|
|
end
|
|
|
|
end
|
|
|
|
CRYSTAL
|
|
|
|
end
|
|
|
|
|
2021-10-27 23:13:29 +00:00
|
|
|
it "reports class variable name" do
|
|
|
|
expect_issue subject, <<-CRYSTAL
|
|
|
|
class Greeting
|
|
|
|
@@defaultGreeting = "Hello world"
|
|
|
|
# ^^^^^^^^^^^^^^^^^ error: Var name should be underscore-cased: @@default_greeting, not @@defaultGreeting
|
|
|
|
end
|
|
|
|
CRYSTAL
|
|
|
|
end
|
2017-11-04 21:13:38 +00:00
|
|
|
end
|
|
|
|
end
|