ruby-lint 0.0.1a → 0.0.1a1
Sign up to get free protection for your applications and to get access to all the features.
- data/.yardopts +1 -1
- data/MANIFEST +65 -62
- data/README.md +114 -13
- data/bin/ruby-lint +6 -0
- data/lib/ruby-lint.rb +36 -0
- data/lib/{rlint → ruby-lint}/analyze/coding_style.rb +32 -32
- data/lib/{rlint → ruby-lint}/analyze/definitions.rb +13 -13
- data/lib/{rlint → ruby-lint}/analyze/method_validation.rb +5 -5
- data/lib/{rlint → ruby-lint}/analyze/shadowing_variables.rb +5 -5
- data/lib/{rlint → ruby-lint}/analyze/undefined_variables.rb +7 -7
- data/lib/{rlint → ruby-lint}/analyze/unused_variables.rb +6 -6
- data/lib/{rlint → ruby-lint}/callback.rb +11 -11
- data/lib/{rlint → ruby-lint}/cli.rb +17 -17
- data/lib/{rlint → ruby-lint}/constant_importer.rb +18 -8
- data/lib/{rlint → ruby-lint}/definition.rb +10 -10
- data/lib/{rlint → ruby-lint}/formatter/text.rb +6 -6
- data/lib/{rlint → ruby-lint}/helper/definition_resolver.rb +11 -11
- data/lib/{rlint → ruby-lint}/helper/scoping.rb +14 -14
- data/lib/{rlint → ruby-lint}/iterator.rb +22 -22
- data/lib/{rlint → ruby-lint}/options.rb +9 -9
- data/lib/{rlint → ruby-lint}/parser.rb +111 -111
- data/lib/{rlint → ruby-lint}/parser_error.rb +3 -3
- data/lib/{rlint → ruby-lint}/report.rb +8 -8
- data/lib/{rlint → ruby-lint}/token/assignment_token.rb +4 -4
- data/lib/{rlint → ruby-lint}/token/begin_rescue_token.rb +7 -7
- data/lib/{rlint → ruby-lint}/token/block_token.rb +12 -3
- data/lib/{rlint → ruby-lint}/token/case_token.rb +5 -5
- data/lib/{rlint → ruby-lint}/token/class_token.rb +3 -3
- data/lib/{rlint → ruby-lint}/token/method_definition_token.rb +8 -8
- data/lib/{rlint → ruby-lint}/token/method_token.rb +9 -7
- data/lib/{rlint → ruby-lint}/token/parameters_token.rb +6 -6
- data/lib/{rlint → ruby-lint}/token/regexp_token.rb +2 -2
- data/lib/{rlint → ruby-lint}/token/statement_token.rb +6 -6
- data/lib/{rlint → ruby-lint}/token/token.rb +8 -6
- data/lib/{rlint → ruby-lint}/token/variable_token.rb +3 -3
- data/lib/ruby-lint/version.rb +3 -0
- data/ruby-lint.gemspec +5 -5
- data/spec/benchmarks/memory.rb +7 -7
- data/spec/benchmarks/parse_parser.rb +5 -5
- data/spec/fixtures/stdlib/un.rb +348 -0
- data/spec/helper.rb +3 -1
- data/spec/{rlint → ruby-lint}/analyze/coding_style.rb +30 -30
- data/spec/ruby-lint/analyze/complex/un.rb +29 -0
- data/spec/{rlint → ruby-lint}/analyze/definitions/classes.rb +25 -25
- data/spec/{rlint → ruby-lint}/analyze/definitions/methods.rb +22 -22
- data/spec/{rlint → ruby-lint}/analyze/definitions/modules.rb +42 -42
- data/spec/{rlint → ruby-lint}/analyze/definitions/variables.rb +27 -27
- data/spec/{rlint → ruby-lint}/analyze/method_validation.rb +31 -31
- data/spec/{rlint → ruby-lint}/analyze/shadowing_variables.rb +6 -6
- data/spec/{rlint → ruby-lint}/analyze/undefined_variables.rb +37 -37
- data/spec/{rlint → ruby-lint}/analyze/unused_variables.rb +21 -21
- data/spec/{rlint → ruby-lint}/callback.rb +7 -7
- data/spec/{rlint → ruby-lint}/constant_importer.rb +6 -6
- data/spec/{rlint → ruby-lint}/definition.rb +25 -25
- data/spec/{rlint → ruby-lint}/formatter/text.rb +4 -4
- data/spec/{rlint → ruby-lint}/iterator.rb +38 -38
- data/spec/{rlint → ruby-lint}/parser/arrays.rb +28 -28
- data/spec/{rlint → ruby-lint}/parser/classes.rb +23 -23
- data/spec/{rlint → ruby-lint}/parser/errors.rb +4 -4
- data/spec/{rlint → ruby-lint}/parser/hashes.rb +24 -24
- data/spec/{rlint → ruby-lint}/parser/methods.rb +50 -50
- data/spec/{rlint → ruby-lint}/parser/modules.rb +8 -8
- data/spec/{rlint → ruby-lint}/parser/objects.rb +8 -8
- data/spec/{rlint → ruby-lint}/parser/operators.rb +14 -14
- data/spec/{rlint → ruby-lint}/parser/procs.rb +26 -26
- data/spec/{rlint → ruby-lint}/parser/ranges.rb +9 -9
- data/spec/{rlint → ruby-lint}/parser/regexp.rb +5 -5
- data/spec/{rlint → ruby-lint}/parser/scalars.rb +17 -17
- data/spec/{rlint → ruby-lint}/parser/statements.rb +94 -94
- data/spec/{rlint → ruby-lint}/parser/variables.rb +37 -37
- data/spec/{rlint → ruby-lint}/report.rb +4 -4
- data/task/manifest.rake +8 -0
- data/task/test.rake +1 -1
- metadata +69 -66
- data/bin/rlint +0 -6
- data/lib/rlint.rb +0 -36
- data/lib/rlint/version.rb +0 -3
@@ -1,6 +1,6 @@
|
|
1
1
|
require File.expand_path('../../../../helper', __FILE__)
|
2
2
|
|
3
|
-
describe '
|
3
|
+
describe 'RubyLint::Analyze::Definitions: variables' do
|
4
4
|
it 'Build a list of variables defined in the global scope' do
|
5
5
|
code = <<-CODE
|
6
6
|
number = 10
|
@@ -10,15 +10,15 @@ $number = 10
|
|
10
10
|
NUMBER = 10
|
11
11
|
CODE
|
12
12
|
|
13
|
-
tokens =
|
14
|
-
iterator =
|
13
|
+
tokens = RubyLint::Parser.new(code).parse
|
14
|
+
iterator = RubyLint::Iterator.new
|
15
15
|
|
16
|
-
iterator.bind(
|
16
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
17
17
|
iterator.run(tokens)
|
18
18
|
|
19
19
|
scope = iterator.storage[:scope]
|
20
20
|
|
21
|
-
scope.class.should ==
|
21
|
+
scope.class.should == RubyLint::Definition
|
22
22
|
|
23
23
|
lvar = scope.lookup(:local_variable, 'number')
|
24
24
|
ivar = scope.lookup(:instance_variable, '@number')
|
@@ -26,43 +26,43 @@ NUMBER = 10
|
|
26
26
|
gvar = scope.lookup(:global_variable, '$number')
|
27
27
|
const = scope.lookup(:constant, 'NUMBER')
|
28
28
|
|
29
|
-
lvar.class.should ==
|
30
|
-
lvar.token.class.should ==
|
29
|
+
lvar.class.should == RubyLint::Definition
|
30
|
+
lvar.token.class.should == RubyLint::Token::AssignmentToken
|
31
31
|
lvar.token.name.should == 'number'
|
32
32
|
lvar.token.type.should == :local_variable
|
33
|
-
lvar.token.value.class.should ==
|
33
|
+
lvar.token.value.class.should == RubyLint::Token::Token
|
34
34
|
lvar.token.value.type.should == :integer
|
35
35
|
lvar.token.value.value.should == '10'
|
36
36
|
|
37
|
-
ivar.class.should ==
|
38
|
-
ivar.token.class.should ==
|
37
|
+
ivar.class.should == RubyLint::Definition
|
38
|
+
ivar.token.class.should == RubyLint::Token::AssignmentToken
|
39
39
|
ivar.token.name.should == '@number'
|
40
40
|
ivar.token.type.should == :instance_variable
|
41
|
-
ivar.token.value.class.should ==
|
41
|
+
ivar.token.value.class.should == RubyLint::Token::Token
|
42
42
|
ivar.token.value.type.should == :integer
|
43
43
|
ivar.token.value.value.should == '10'
|
44
44
|
|
45
|
-
cvar.class.should ==
|
46
|
-
cvar.token.class.should ==
|
45
|
+
cvar.class.should == RubyLint::Definition
|
46
|
+
cvar.token.class.should == RubyLint::Token::AssignmentToken
|
47
47
|
cvar.token.name.should == '@@number'
|
48
48
|
cvar.token.type.should == :class_variable
|
49
|
-
cvar.token.value.class.should ==
|
49
|
+
cvar.token.value.class.should == RubyLint::Token::Token
|
50
50
|
cvar.token.value.type.should == :integer
|
51
51
|
cvar.token.value.value.should == '10'
|
52
52
|
|
53
|
-
gvar.class.should ==
|
54
|
-
gvar.token.class.should ==
|
53
|
+
gvar.class.should == RubyLint::Definition
|
54
|
+
gvar.token.class.should == RubyLint::Token::AssignmentToken
|
55
55
|
gvar.token.name.should == '$number'
|
56
56
|
gvar.token.type.should == :global_variable
|
57
|
-
gvar.token.value.class.should ==
|
57
|
+
gvar.token.value.class.should == RubyLint::Token::Token
|
58
58
|
gvar.token.value.type.should == :integer
|
59
59
|
gvar.token.value.value.should == '10'
|
60
60
|
|
61
|
-
const.class.should ==
|
62
|
-
const.token.class.should ==
|
61
|
+
const.class.should == RubyLint::Definition
|
62
|
+
const.token.class.should == RubyLint::Token::AssignmentToken
|
63
63
|
const.token.name.should == 'NUMBER'
|
64
64
|
const.token.type.should == :constant
|
65
|
-
const.token.value.class.should ==
|
65
|
+
const.token.value.class.should == RubyLint::Token::Token
|
66
66
|
const.token.value.type.should == :integer
|
67
67
|
const.token.value.value.should == '10'
|
68
68
|
end
|
@@ -74,10 +74,10 @@ def example_method
|
|
74
74
|
end
|
75
75
|
CODE
|
76
76
|
|
77
|
-
tokens =
|
78
|
-
iterator =
|
77
|
+
tokens = RubyLint::Parser.new(code).parse
|
78
|
+
iterator = RubyLint::Iterator.new
|
79
79
|
|
80
|
-
iterator.bind(
|
80
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
81
81
|
iterator.run(tokens)
|
82
82
|
|
83
83
|
scope = iterator.storage[:scope]
|
@@ -86,17 +86,17 @@ end
|
|
86
86
|
|
87
87
|
method = scope.lookup(:instance_method, 'example_method')
|
88
88
|
|
89
|
-
method.class.should ==
|
89
|
+
method.class.should == RubyLint::Definition
|
90
90
|
|
91
|
-
method.token.class.should ==
|
91
|
+
method.token.class.should == RubyLint::Token::MethodDefinitionToken
|
92
92
|
method.token.name.should == 'example_method'
|
93
93
|
method.token.value.nil?.should == true
|
94
94
|
|
95
95
|
variable = method.lookup(:local_variable, 'number')
|
96
96
|
|
97
|
-
variable.class.should ==
|
97
|
+
variable.class.should == RubyLint::Definition
|
98
98
|
variable.token.name.should == 'number'
|
99
|
-
variable.token.value.class.should ==
|
99
|
+
variable.token.value.class.should == RubyLint::Token::Token
|
100
100
|
variable.token.value.type.should == :integer
|
101
101
|
variable.token.value.value.should == '10'
|
102
102
|
end
|
@@ -1,6 +1,6 @@
|
|
1
1
|
require File.expand_path('../../../helper', __FILE__)
|
2
2
|
|
3
|
-
describe '
|
3
|
+
describe 'RubyLint::Analyze::MethodValidation' do
|
4
4
|
it 'Calling undefined methods' do
|
5
5
|
code = <<-CODE
|
6
6
|
def defined_method; end
|
@@ -9,13 +9,13 @@ undefined_method
|
|
9
9
|
defined_method
|
10
10
|
CODE
|
11
11
|
|
12
|
-
tokens =
|
13
|
-
report =
|
14
|
-
iterator =
|
12
|
+
tokens = RubyLint::Parser.new(code).parse
|
13
|
+
report = RubyLint::Report.new
|
14
|
+
iterator = RubyLint::Iterator.new(report)
|
15
15
|
|
16
|
-
iterator.bind(
|
17
|
-
iterator.bind(
|
18
|
-
iterator.bind(
|
16
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
17
|
+
iterator.bind(RubyLint::Analyze::MethodValidation)
|
18
|
+
iterator.bind(RubyLint::Analyze::UndefinedVariables)
|
19
19
|
iterator.run(tokens)
|
20
20
|
|
21
21
|
report.messages[:error].class.should == Array
|
@@ -37,13 +37,13 @@ String.newx
|
|
37
37
|
Foobar.new
|
38
38
|
CODE
|
39
39
|
|
40
|
-
tokens =
|
41
|
-
report =
|
42
|
-
iterator =
|
40
|
+
tokens = RubyLint::Parser.new(code).parse
|
41
|
+
report = RubyLint::Report.new
|
42
|
+
iterator = RubyLint::Iterator.new(report)
|
43
43
|
|
44
|
-
iterator.bind(
|
45
|
-
iterator.bind(
|
46
|
-
iterator.bind(
|
44
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
45
|
+
iterator.bind(RubyLint::Analyze::MethodValidation)
|
46
|
+
iterator.bind(RubyLint::Analyze::UndefinedVariables)
|
47
47
|
iterator.run(tokens)
|
48
48
|
|
49
49
|
report.messages[:error].class.should == Array
|
@@ -72,13 +72,13 @@ foo.upcase
|
|
72
72
|
''.downcasex
|
73
73
|
CODE
|
74
74
|
|
75
|
-
tokens =
|
76
|
-
report =
|
77
|
-
iterator =
|
75
|
+
tokens = RubyLint::Parser.new(code).parse
|
76
|
+
report = RubyLint::Report.new
|
77
|
+
iterator = RubyLint::Iterator.new(report)
|
78
78
|
|
79
|
-
iterator.bind(
|
80
|
-
iterator.bind(
|
81
|
-
iterator.bind(
|
79
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
80
|
+
iterator.bind(RubyLint::Analyze::MethodValidation)
|
81
|
+
iterator.bind(RubyLint::Analyze::UndefinedVariables)
|
82
82
|
iterator.run(tokens)
|
83
83
|
|
84
84
|
report.messages[:error].class.should == Array
|
@@ -109,13 +109,13 @@ def uppercase(name = 'Ruby', *args, &block)
|
|
109
109
|
end
|
110
110
|
CODE
|
111
111
|
|
112
|
-
tokens =
|
113
|
-
report =
|
114
|
-
iterator =
|
112
|
+
tokens = RubyLint::Parser.new(code).parse
|
113
|
+
report = RubyLint::Report.new
|
114
|
+
iterator = RubyLint::Iterator.new(report)
|
115
115
|
|
116
|
-
iterator.bind(
|
117
|
-
iterator.bind(
|
118
|
-
iterator.bind(
|
116
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
117
|
+
iterator.bind(RubyLint::Analyze::MethodValidation)
|
118
|
+
iterator.bind(RubyLint::Analyze::UndefinedVariables)
|
119
119
|
iterator.run(tokens)
|
120
120
|
|
121
121
|
report.messages[:error].class.should == Array
|
@@ -148,13 +148,13 @@ A::B.invalid_method
|
|
148
148
|
A::B.valid_method
|
149
149
|
CODE
|
150
150
|
|
151
|
-
tokens =
|
152
|
-
report =
|
153
|
-
iterator =
|
151
|
+
tokens = RubyLint::Parser.new(code).parse
|
152
|
+
report = RubyLint::Report.new
|
153
|
+
iterator = RubyLint::Iterator.new(report)
|
154
154
|
|
155
|
-
iterator.bind(
|
156
|
-
iterator.bind(
|
157
|
-
iterator.bind(
|
155
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
156
|
+
iterator.bind(RubyLint::Analyze::UndefinedVariables)
|
157
|
+
iterator.bind(RubyLint::Analyze::MethodValidation)
|
158
158
|
iterator.run(tokens)
|
159
159
|
|
160
160
|
report.messages[:error].class.should == Array
|
@@ -1,6 +1,6 @@
|
|
1
1
|
require File.expand_path('../../../helper', __FILE__)
|
2
2
|
|
3
|
-
describe '
|
3
|
+
describe 'RubyLint::Analyze::ShadowingVariables' do
|
4
4
|
it 'Warn for shadowing outer variables' do
|
5
5
|
code = <<-CODE
|
6
6
|
number = 10
|
@@ -10,12 +10,12 @@ number = 10
|
|
10
10
|
end
|
11
11
|
CODE
|
12
12
|
|
13
|
-
tokens =
|
14
|
-
report =
|
15
|
-
iterator =
|
13
|
+
tokens = RubyLint::Parser.new(code).parse
|
14
|
+
report = RubyLint::Report.new
|
15
|
+
iterator = RubyLint::Iterator.new(report)
|
16
16
|
|
17
|
-
iterator.bind(
|
18
|
-
iterator.bind(
|
17
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
18
|
+
iterator.bind(RubyLint::Analyze::ShadowingVariables)
|
19
19
|
iterator.run(tokens)
|
20
20
|
|
21
21
|
report.messages[:warning].class.should == Array
|
@@ -1,6 +1,6 @@
|
|
1
1
|
require File.expand_path('../../../helper', __FILE__)
|
2
2
|
|
3
|
-
describe '
|
3
|
+
describe 'RubyLint::Analyze::UndefinedVariables' do
|
4
4
|
it 'Use of undefined variables' do
|
5
5
|
code = <<-CODE
|
6
6
|
number = 10
|
@@ -25,12 +25,12 @@ puts $number
|
|
25
25
|
puts NUMBER
|
26
26
|
CODE
|
27
27
|
|
28
|
-
tokens =
|
29
|
-
report =
|
30
|
-
iterator =
|
28
|
+
tokens = RubyLint::Parser.new(code).parse
|
29
|
+
report = RubyLint::Report.new
|
30
|
+
iterator = RubyLint::Iterator.new(report)
|
31
31
|
|
32
|
-
iterator.bind(
|
33
|
-
iterator.bind(
|
32
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
33
|
+
iterator.bind(RubyLint::Analyze::UndefinedVariables)
|
34
34
|
iterator.run(tokens)
|
35
35
|
|
36
36
|
report.messages[:error].class.should == Array
|
@@ -61,12 +61,12 @@ puts NUMBER
|
|
61
61
|
|
62
62
|
it 'Default global variables should not trigger errors' do
|
63
63
|
code = Kernel.global_variables.join("\n")
|
64
|
-
tokens =
|
65
|
-
report =
|
66
|
-
iterator =
|
64
|
+
tokens = RubyLint::Parser.new(code).parse
|
65
|
+
report = RubyLint::Report.new
|
66
|
+
iterator = RubyLint::Iterator.new(report)
|
67
67
|
|
68
|
-
iterator.bind(
|
69
|
-
iterator.bind(
|
68
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
69
|
+
iterator.bind(RubyLint::Analyze::UndefinedVariables)
|
70
70
|
iterator.run(tokens)
|
71
71
|
|
72
72
|
report.messages[:error].nil?.should == true
|
@@ -89,12 +89,12 @@ puts a
|
|
89
89
|
puts b # b was defined inside the method and isn't available outside it
|
90
90
|
CODE
|
91
91
|
|
92
|
-
tokens =
|
93
|
-
report =
|
94
|
-
iterator =
|
92
|
+
tokens = RubyLint::Parser.new(code).parse
|
93
|
+
report = RubyLint::Report.new
|
94
|
+
iterator = RubyLint::Iterator.new(report)
|
95
95
|
|
96
|
-
iterator.bind(
|
97
|
-
iterator.bind(
|
96
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
97
|
+
iterator.bind(RubyLint::Analyze::UndefinedVariables)
|
98
98
|
iterator.run(tokens)
|
99
99
|
|
100
100
|
report.messages[:error].class.should == Array
|
@@ -124,12 +124,12 @@ end
|
|
124
124
|
puts @number
|
125
125
|
CODE
|
126
126
|
|
127
|
-
tokens =
|
128
|
-
report =
|
129
|
-
iterator =
|
127
|
+
tokens = RubyLint::Parser.new(code).parse
|
128
|
+
report = RubyLint::Report.new
|
129
|
+
iterator = RubyLint::Iterator.new(report)
|
130
130
|
|
131
|
-
iterator.bind(
|
132
|
-
iterator.bind(
|
131
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
132
|
+
iterator.bind(RubyLint::Analyze::UndefinedVariables)
|
133
133
|
iterator.run(tokens)
|
134
134
|
|
135
135
|
report.messages[:error].nil?.should == true
|
@@ -149,12 +149,12 @@ class Person
|
|
149
149
|
end
|
150
150
|
CODE
|
151
151
|
|
152
|
-
tokens =
|
153
|
-
report =
|
154
|
-
iterator =
|
152
|
+
tokens = RubyLint::Parser.new(code).parse
|
153
|
+
report = RubyLint::Report.new
|
154
|
+
iterator = RubyLint::Iterator.new(report)
|
155
155
|
|
156
|
-
iterator.bind(
|
157
|
-
iterator.bind(
|
156
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
157
|
+
iterator.bind(RubyLint::Analyze::UndefinedVariables)
|
158
158
|
iterator.run(tokens)
|
159
159
|
|
160
160
|
report.messages[:error].class.should == Array
|
@@ -178,12 +178,12 @@ end
|
|
178
178
|
puts A::B
|
179
179
|
CODE
|
180
180
|
|
181
|
-
tokens =
|
182
|
-
report =
|
183
|
-
iterator =
|
181
|
+
tokens = RubyLint::Parser.new(code).parse
|
182
|
+
report = RubyLint::Report.new
|
183
|
+
iterator = RubyLint::Iterator.new(report)
|
184
184
|
|
185
|
-
iterator.bind(
|
186
|
-
iterator.bind(
|
185
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
186
|
+
iterator.bind(RubyLint::Analyze::UndefinedVariables)
|
187
187
|
iterator.run(tokens)
|
188
188
|
|
189
189
|
report.messages[:error].class.should == Array
|
@@ -202,7 +202,7 @@ puts A::B
|
|
202
202
|
|
203
203
|
it 'Look up a constant using an implicit constant path' do
|
204
204
|
code = <<-CODE
|
205
|
-
module
|
205
|
+
module RubyLint
|
206
206
|
module Derp
|
207
207
|
Foobar.name
|
208
208
|
ConstantImporter.name
|
@@ -210,12 +210,12 @@ module Rlint
|
|
210
210
|
end
|
211
211
|
CODE
|
212
212
|
|
213
|
-
tokens =
|
214
|
-
report =
|
215
|
-
iterator =
|
213
|
+
tokens = RubyLint::Parser.new(code).parse
|
214
|
+
report = RubyLint::Report.new
|
215
|
+
iterator = RubyLint::Iterator.new(report)
|
216
216
|
|
217
|
-
iterator.bind(
|
218
|
-
iterator.bind(
|
217
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
218
|
+
iterator.bind(RubyLint::Analyze::UndefinedVariables)
|
219
219
|
iterator.run(tokens)
|
220
220
|
|
221
221
|
report.messages[:error].class.should == Array
|
@@ -1,6 +1,6 @@
|
|
1
1
|
require File.expand_path('../../../helper', __FILE__)
|
2
2
|
|
3
|
-
describe '
|
3
|
+
describe 'RubyLint::Analyze::UnusedVariables' do
|
4
4
|
it 'Add warnings for unused variables in the global scope' do
|
5
5
|
code = <<-CODE
|
6
6
|
number = 10
|
@@ -19,12 +19,12 @@ puts @@number_2
|
|
19
19
|
puts $number_2
|
20
20
|
CODE
|
21
21
|
|
22
|
-
tokens =
|
23
|
-
report =
|
24
|
-
iterator =
|
22
|
+
tokens = RubyLint::Parser.new(code).parse
|
23
|
+
report = RubyLint::Report.new
|
24
|
+
iterator = RubyLint::Iterator.new(report)
|
25
25
|
|
26
|
-
iterator.bind(
|
27
|
-
iterator.bind(
|
26
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
27
|
+
iterator.bind(RubyLint::Analyze::UnusedVariables)
|
28
28
|
iterator.run(tokens)
|
29
29
|
|
30
30
|
report.messages[:warning].class.should == Array
|
@@ -75,12 +75,12 @@ def example
|
|
75
75
|
end
|
76
76
|
CODE
|
77
77
|
|
78
|
-
tokens =
|
79
|
-
report =
|
80
|
-
iterator =
|
78
|
+
tokens = RubyLint::Parser.new(code).parse
|
79
|
+
report = RubyLint::Report.new
|
80
|
+
iterator = RubyLint::Iterator.new(report)
|
81
81
|
|
82
|
-
iterator.bind(
|
83
|
-
iterator.bind(
|
82
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
83
|
+
iterator.bind(RubyLint::Analyze::UnusedVariables)
|
84
84
|
iterator.run(tokens)
|
85
85
|
|
86
86
|
report.messages[:warning].class.should == Array
|
@@ -131,12 +131,12 @@ class Example
|
|
131
131
|
end
|
132
132
|
CODE
|
133
133
|
|
134
|
-
tokens =
|
135
|
-
report =
|
136
|
-
iterator =
|
134
|
+
tokens = RubyLint::Parser.new(code).parse
|
135
|
+
report = RubyLint::Report.new
|
136
|
+
iterator = RubyLint::Iterator.new(report)
|
137
137
|
|
138
|
-
iterator.bind(
|
139
|
-
iterator.bind(
|
138
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
139
|
+
iterator.bind(RubyLint::Analyze::UnusedVariables)
|
140
140
|
iterator.run(tokens)
|
141
141
|
|
142
142
|
report.messages[:warning].class.should == Array
|
@@ -187,12 +187,12 @@ module Example
|
|
187
187
|
end
|
188
188
|
CODE
|
189
189
|
|
190
|
-
tokens =
|
191
|
-
report =
|
192
|
-
iterator =
|
190
|
+
tokens = RubyLint::Parser.new(code).parse
|
191
|
+
report = RubyLint::Report.new
|
192
|
+
iterator = RubyLint::Iterator.new(report)
|
193
193
|
|
194
|
-
iterator.bind(
|
195
|
-
iterator.bind(
|
194
|
+
iterator.bind(RubyLint::Analyze::Definitions)
|
195
|
+
iterator.bind(RubyLint::Analyze::UnusedVariables)
|
196
196
|
iterator.run(tokens)
|
197
197
|
|
198
198
|
report.messages[:warning].class.should == Array
|