reek 1.3.1 → 1.3.2
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +7 -0
- data/CHANGELOG +6 -0
- data/README.md +15 -9
- data/bin/reek +1 -1
- data/config/defaults.reek +71 -86
- data/features/command_line_interface/options.feature +0 -15
- data/features/reports/reports.feature +23 -0
- data/features/samples.feature +3 -12
- data/lib/reek.rb +3 -3
- data/lib/reek/cli/application.rb +1 -1
- data/lib/reek/cli/command_line.rb +10 -8
- data/lib/reek/cli/reek_command.rb +6 -7
- data/lib/reek/cli/report.rb +34 -38
- data/lib/reek/cli/version_command.rb +1 -1
- data/lib/reek/cli/yaml_command.rb +1 -1
- data/lib/reek/core/code_parser.rb +4 -4
- data/lib/reek/core/hash_extensions.rb +2 -2
- data/lib/reek/core/method_context.rb +2 -2
- data/lib/reek/core/module_context.rb +4 -4
- data/lib/reek/core/singleton_method_context.rb +1 -1
- data/lib/reek/core/smell_repository.rb +7 -6
- data/lib/reek/core/sniffer.rb +4 -4
- data/lib/reek/examiner.rb +10 -3
- data/lib/reek/smell_warning.rb +0 -2
- data/lib/reek/smells.rb +22 -21
- data/lib/reek/smells/attribute.rb +4 -8
- data/lib/reek/smells/boolean_parameter.rb +2 -2
- data/lib/reek/smells/class_variable.rb +3 -2
- data/lib/reek/smells/{control_couple.rb → control_parameter.rb} +5 -5
- data/lib/reek/smells/data_clump.rb +13 -29
- data/lib/reek/smells/{duplication.rb → duplicate_method_call.rb} +9 -11
- data/lib/reek/smells/feature_envy.rb +2 -2
- data/lib/reek/smells/irresponsible_module.rb +3 -2
- data/lib/reek/smells/long_parameter_list.rb +6 -10
- data/lib/reek/smells/long_yield_list.rb +4 -8
- data/lib/reek/smells/nested_iterators.rb +31 -25
- data/lib/reek/smells/nil_check.rb +11 -12
- data/lib/reek/smells/{simulated_polymorphism.rb → repeated_conditional.rb} +6 -10
- data/lib/reek/smells/smell_detector.rb +3 -6
- data/lib/reek/smells/too_many_instance_variables.rb +60 -0
- data/lib/reek/smells/too_many_methods.rb +62 -0
- data/lib/reek/smells/{long_method.rb → too_many_statements.rb} +7 -12
- data/lib/reek/smells/uncommunicative_method_name.rb +3 -7
- data/lib/reek/smells/uncommunicative_module_name.rb +3 -7
- data/lib/reek/smells/uncommunicative_parameter_name.rb +4 -8
- data/lib/reek/smells/uncommunicative_variable_name.rb +5 -9
- data/lib/reek/smells/unused_parameters.rb +62 -13
- data/lib/reek/smells/utility_function.rb +3 -7
- data/lib/reek/source.rb +8 -8
- data/lib/reek/source/core_extras.rb +1 -1
- data/lib/reek/source/source_code.rb +2 -2
- data/lib/reek/source/source_file.rb +2 -2
- data/lib/reek/source/source_locator.rb +1 -1
- data/lib/reek/source/source_repository.rb +4 -2
- data/lib/reek/spec.rb +9 -3
- data/lib/reek/spec/should_reek.rb +2 -2
- data/lib/reek/spec/should_reek_of.rb +1 -1
- data/lib/reek/spec/should_reek_only_of.rb +2 -2
- data/lib/reek/version.rb +1 -1
- data/reek.gemspec +3 -1
- data/spec/gem/updates_spec.rb +1 -1
- data/spec/gem/yard_spec.rb +1 -1
- data/spec/matchers/smell_of_matcher.rb +53 -19
- data/spec/reek/cli/help_command_spec.rb +2 -2
- data/spec/reek/cli/reek_command_spec.rb +6 -6
- data/spec/reek/cli/report_spec.rb +6 -6
- data/spec/reek/cli/version_command_spec.rb +2 -2
- data/spec/reek/cli/yaml_command_spec.rb +2 -2
- data/spec/reek/core/code_context_spec.rb +4 -4
- data/spec/reek/core/code_parser_spec.rb +2 -2
- data/spec/reek/core/config_spec.rb +4 -4
- data/spec/reek/core/method_context_spec.rb +3 -3
- data/spec/reek/core/module_context_spec.rb +3 -3
- data/spec/reek/core/object_refs_spec.rb +3 -3
- data/spec/reek/core/singleton_method_context_spec.rb +4 -4
- data/spec/reek/core/smell_configuration_spec.rb +2 -2
- data/spec/reek/core/stop_context_spec.rb +2 -2
- data/spec/reek/core/warning_collector_spec.rb +3 -3
- data/spec/reek/examiner_spec.rb +13 -4
- data/spec/reek/smell_warning_spec.rb +2 -2
- data/spec/reek/smells/attribute_spec.rb +4 -4
- data/spec/reek/smells/boolean_parameter_spec.rb +3 -3
- data/spec/reek/smells/class_variable_spec.rb +4 -4
- data/spec/reek/smells/{control_couple_spec.rb → control_parameter_spec.rb} +10 -10
- data/spec/reek/smells/data_clump_spec.rb +3 -3
- data/spec/reek/smells/{duplication_spec.rb → duplicate_method_call_spec.rb} +42 -26
- data/spec/reek/smells/feature_envy_spec.rb +3 -3
- data/spec/reek/smells/irresponsible_module_spec.rb +3 -3
- data/spec/reek/smells/long_parameter_list_spec.rb +3 -3
- data/spec/reek/smells/long_yield_list_spec.rb +3 -3
- data/spec/reek/smells/nested_iterators_spec.rb +42 -4
- data/spec/reek/smells/nil_check_spec.rb +23 -11
- data/spec/reek/smells/{simulated_polymorphism_spec.rb → repeated_conditional_spec.rb} +6 -6
- data/spec/reek/smells/smell_detector_shared.rb +2 -2
- data/spec/reek/smells/too_many_instance_variables_spec.rb +62 -0
- data/spec/reek/smells/{large_class_spec.rb → too_many_methods_spec.rb} +11 -56
- data/spec/reek/smells/{long_method_spec.rb → too_many_statements_spec.rb} +17 -17
- data/spec/reek/smells/uncommunicative_method_name_spec.rb +5 -5
- data/spec/reek/smells/uncommunicative_module_name_spec.rb +5 -5
- data/spec/reek/smells/uncommunicative_parameter_name_spec.rb +4 -4
- data/spec/reek/smells/uncommunicative_variable_name_spec.rb +5 -5
- data/spec/reek/smells/unused_parameters_spec.rb +19 -4
- data/spec/reek/smells/utility_function_spec.rb +3 -3
- data/spec/reek/source/code_comment_spec.rb +2 -2
- data/spec/reek/source/object_source_spec.rb +1 -1
- data/spec/reek/source/reference_collector_spec.rb +2 -2
- data/spec/reek/source/sexp_formatter_spec.rb +2 -2
- data/spec/reek/source/source_code_spec.rb +2 -2
- data/spec/reek/source/tree_dresser_spec.rb +2 -2
- data/spec/reek/spec/should_reek_of_spec.rb +2 -2
- data/spec/reek/spec/should_reek_only_of_spec.rb +2 -2
- data/spec/reek/spec/should_reek_spec.rb +2 -2
- data/spec/samples/all_but_one_masked/masked.reek +1 -1
- data/spec/samples/clean_due_to_masking/masked.reek +1 -1
- data/spec/samples/config/allow_duplication.reek +2 -2
- data/spec/samples/inline_config/dirty.rb +2 -2
- data/spec/samples/mask_some/some.reek +1 -1
- data/spec/samples/masked_by_dotfile/dirty.rb +8 -0
- data/spec/samples/not_quite_masked/smelly.rb +3 -0
- data/spec/samples/overrides/masked/lower.reek +1 -1
- data/spec/samples/overrides/upper.reek +1 -1
- data/spec/spec_helper.rb +4 -9
- data/tasks/test.rake +0 -2
- metadata +253 -263
- data/lib/reek/smells/large_class.rb +0 -87
- data/lib/xp.reek +0 -66
- data/spec/gem/manifest_spec.rb +0 -22
- data/spec/spec.opts +0 -1
@@ -1,8 +1,8 @@
|
|
1
|
-
require
|
2
|
-
require
|
3
|
-
require
|
4
|
-
require
|
5
|
-
require
|
1
|
+
require 'spec_helper'
|
2
|
+
require 'reek/smells/uncommunicative_method_name'
|
3
|
+
require 'reek/smells/smell_detector_shared'
|
4
|
+
require 'reek/core/code_parser'
|
5
|
+
require 'reek/core/sniffer'
|
6
6
|
|
7
7
|
include Reek
|
8
8
|
include Reek::Smells
|
@@ -1,8 +1,8 @@
|
|
1
|
-
require
|
2
|
-
require
|
3
|
-
require
|
4
|
-
require
|
5
|
-
require
|
1
|
+
require 'spec_helper'
|
2
|
+
require 'reek/smells/uncommunicative_module_name'
|
3
|
+
require 'reek/smells/smell_detector_shared'
|
4
|
+
require 'reek/core/code_parser'
|
5
|
+
require 'reek/core/sniffer'
|
6
6
|
|
7
7
|
include Reek
|
8
8
|
include Reek::Smells
|
@@ -1,7 +1,7 @@
|
|
1
|
-
require
|
2
|
-
require
|
3
|
-
require
|
4
|
-
require
|
1
|
+
require 'spec_helper'
|
2
|
+
require 'reek/smells/uncommunicative_parameter_name'
|
3
|
+
require 'reek/smells/smell_detector_shared'
|
4
|
+
require 'reek/core/method_context'
|
5
5
|
|
6
6
|
include Reek::Core
|
7
7
|
include Reek::Smells
|
@@ -1,8 +1,8 @@
|
|
1
|
-
require
|
2
|
-
require
|
3
|
-
require
|
4
|
-
require
|
5
|
-
require
|
1
|
+
require 'spec_helper'
|
2
|
+
require 'reek/smells/uncommunicative_variable_name'
|
3
|
+
require 'reek/smells/smell_detector_shared'
|
4
|
+
require 'reek/core/code_parser'
|
5
|
+
require 'reek/core/sniffer'
|
6
6
|
|
7
7
|
include Reek
|
8
8
|
include Reek::Smells
|
@@ -1,7 +1,6 @@
|
|
1
|
-
|
2
|
-
require
|
3
|
-
require
|
4
|
-
require File.join(File.dirname(File.expand_path(__FILE__)), 'smell_detector_shared')
|
1
|
+
require 'spec_helper'
|
2
|
+
require 'reek/smells/unused_parameters'
|
3
|
+
require 'reek/smells/smell_detector_shared'
|
5
4
|
|
6
5
|
include Reek
|
7
6
|
include Reek::Smells
|
@@ -39,9 +38,25 @@ describe UnusedParameters do
|
|
39
38
|
'def simple(_); end'.should_not smell_of(UnusedParameters)
|
40
39
|
end
|
41
40
|
|
41
|
+
it 'should report nothing for named parameters prefixed with _' do
|
42
|
+
'def simple(_name); end'.should_not smell_of(UnusedParameters)
|
43
|
+
end
|
44
|
+
|
42
45
|
it 'should report nothing for unused anonymous splatted parameter' do
|
43
46
|
'def simple(*); end'.should_not smell_of(UnusedParameters)
|
44
47
|
end
|
45
48
|
|
49
|
+
it 'should report nothing when using super with implicit arguments' do
|
50
|
+
'def simple(*args); super; end'.should_not smell_of(UnusedParameters)
|
51
|
+
end
|
52
|
+
|
53
|
+
it 'should report something when using super explicitely passing no arguments' do
|
54
|
+
'def simple(*args); super(); end'.should smell_of(UnusedParameters)
|
55
|
+
end
|
56
|
+
|
57
|
+
it 'should report nothing when using super explicitely passing all arguments' do
|
58
|
+
'def simple(*args); super(*args); end'.should_not smell_of(UnusedParameters)
|
59
|
+
end
|
60
|
+
|
46
61
|
end
|
47
62
|
end
|
@@ -1,6 +1,6 @@
|
|
1
|
-
require
|
2
|
-
require
|
3
|
-
require
|
1
|
+
require 'spec_helper'
|
2
|
+
require 'reek/smells/utility_function'
|
3
|
+
require 'reek/smells/smell_detector_shared'
|
4
4
|
|
5
5
|
include Reek
|
6
6
|
include Reek::Smells
|
@@ -1,5 +1,5 @@
|
|
1
|
-
require
|
2
|
-
require
|
1
|
+
require 'spec_helper'
|
2
|
+
require 'reek/source/code_comment'
|
3
3
|
|
4
4
|
include Reek::Source
|
5
5
|
|
@@ -1,5 +1,5 @@
|
|
1
|
-
require
|
2
|
-
require
|
1
|
+
require 'spec_helper'
|
2
|
+
require 'reek/source/reference_collector'
|
3
3
|
|
4
4
|
include Reek::Source
|
5
5
|
|
@@ -1,5 +1,5 @@
|
|
1
|
-
require
|
2
|
-
require
|
1
|
+
require 'spec_helper'
|
2
|
+
require 'reek/source/sexp_formatter'
|
3
3
|
|
4
4
|
include Reek::Source
|
5
5
|
|
@@ -1,6 +1,6 @@
|
|
1
|
-
require
|
1
|
+
require 'spec_helper'
|
2
2
|
require 'stringio'
|
3
|
-
require
|
3
|
+
require 'reek/source/source_code'
|
4
4
|
|
5
5
|
include Reek::Source
|
6
6
|
|
@@ -1,5 +1,5 @@
|
|
1
|
-
require
|
2
|
-
require
|
1
|
+
require 'spec_helper'
|
2
|
+
require 'reek/source/tree_dresser'
|
3
3
|
|
4
4
|
include Reek::Source
|
5
5
|
|
@@ -1,5 +1,5 @@
|
|
1
|
-
require
|
2
|
-
require
|
1
|
+
require 'spec_helper'
|
2
|
+
require 'reek/spec'
|
3
3
|
|
4
4
|
include Reek
|
5
5
|
include Reek::Spec
|
@@ -1,5 +1,5 @@
|
|
1
|
-
require
|
2
|
-
require
|
1
|
+
require 'spec_helper'
|
2
|
+
require 'reek/spec'
|
3
3
|
|
4
4
|
include Reek
|
5
5
|
include Reek::Spec
|
@@ -1,5 +1,5 @@
|
|
1
|
-
require
|
2
|
-
require
|
1
|
+
require 'spec_helper'
|
2
|
+
require 'reek/spec'
|
3
3
|
|
4
4
|
include Reek
|
5
5
|
include Reek::Spec
|
@@ -1,3 +1,3 @@
|
|
1
1
|
---
|
2
|
-
|
3
|
-
enabled: false
|
2
|
+
DuplicateMethodCall:
|
3
|
+
enabled: false
|
@@ -1,4 +1,4 @@
|
|
1
|
-
# :reek:
|
1
|
+
# :reek:DuplicateMethodCall: { allow_calls: [ puts ] }
|
2
2
|
# smells of :reek:NestedIterators but ignores them
|
3
3
|
class Dirty
|
4
4
|
def a
|
@@ -7,7 +7,7 @@ class Dirty
|
|
7
7
|
puts @s.title
|
8
8
|
end
|
9
9
|
|
10
|
-
# :reek:
|
10
|
+
# :reek:DuplicateMethodCall: { max_calls: 2 }
|
11
11
|
def b
|
12
12
|
puts @s.title
|
13
13
|
@s = fred.map {|x| x.each {|key| key += 3}}
|
data/spec/spec_helper.rb
CHANGED
@@ -1,14 +1,9 @@
|
|
1
|
-
require '
|
2
|
-
require '
|
3
|
-
Bundler.require :development, :default # Explicitly necessary here.
|
1
|
+
require 'reek/spec'
|
2
|
+
require 'reek/source/tree_dresser'
|
4
3
|
|
5
|
-
|
6
|
-
require File.join((File.dirname(File.dirname(File.expand_path(__FILE__)))), 'lib', 'reek', 'spec')
|
7
|
-
require File.join((File.dirname(File.dirname(File.expand_path(__FILE__)))), 'lib', 'reek', 'source', 'tree_dresser')
|
4
|
+
require 'matchers/smell_of_matcher'
|
8
5
|
|
9
|
-
|
10
|
-
|
11
|
-
SAMPLES_DIR = 'spec/samples' unless Object.const_defined?('SAMPLES_DIR')
|
6
|
+
SAMPLES_DIR = 'spec/samples'
|
12
7
|
|
13
8
|
def ast(*args)
|
14
9
|
result = Reek::Source::TreeDresser.new.dress(s(*args))
|
data/tasks/test.rake
CHANGED
@@ -9,7 +9,6 @@ namespace 'test' do
|
|
9
9
|
|
10
10
|
RSpec::Core::RakeTask.new('spec') do |t|
|
11
11
|
t.pattern = UNIT_TESTS
|
12
|
-
t.rspec_opts = ['--color']
|
13
12
|
t.ruby_opts = ['-Ilib']
|
14
13
|
t.rcov = false
|
15
14
|
end
|
@@ -23,7 +22,6 @@ namespace 'test' do
|
|
23
22
|
desc 'Tests code quality'
|
24
23
|
RSpec::Core::RakeTask.new('quality') do |t|
|
25
24
|
t.pattern = FileList['quality/**/*_spec.rb']
|
26
|
-
t.rspec_opts = ['--color']
|
27
25
|
t.ruby_opts = ['-Ilib']
|
28
26
|
t.rcov = false
|
29
27
|
end
|
metadata
CHANGED
@@ -1,156 +1,155 @@
|
|
1
|
-
--- !ruby/object:Gem::Specification
|
1
|
+
--- !ruby/object:Gem::Specification
|
2
2
|
name: reek
|
3
|
-
version: !ruby/object:Gem::Version
|
4
|
-
|
5
|
-
prerelease:
|
6
|
-
segments:
|
7
|
-
- 1
|
8
|
-
- 3
|
9
|
-
- 1
|
10
|
-
version: 1.3.1
|
3
|
+
version: !ruby/object:Gem::Version
|
4
|
+
version: 1.3.2
|
11
5
|
platform: ruby
|
12
|
-
authors:
|
6
|
+
authors:
|
13
7
|
- Kevin Rutherford
|
14
8
|
- Timo Roessner
|
15
9
|
- Matijs van Zuijlen
|
16
10
|
autorequire:
|
17
11
|
bindir: bin
|
18
12
|
cert_chain: []
|
19
|
-
|
20
|
-
|
21
|
-
|
22
|
-
- !ruby/object:Gem::Dependency
|
23
|
-
prerelease: false
|
13
|
+
date: 2013-08-09 00:00:00.000000000 Z
|
14
|
+
dependencies:
|
15
|
+
- !ruby/object:Gem::Dependency
|
24
16
|
name: ruby_parser
|
25
|
-
|
26
|
-
|
27
|
-
requirements:
|
17
|
+
requirement: !ruby/object:Gem::Requirement
|
18
|
+
requirements:
|
28
19
|
- - ~>
|
29
|
-
- !ruby/object:Gem::Version
|
30
|
-
hash: 1
|
31
|
-
segments:
|
32
|
-
- 3
|
33
|
-
- 1
|
34
|
-
- 1
|
20
|
+
- !ruby/object:Gem::Version
|
35
21
|
version: 3.1.1
|
36
|
-
requirement: *id001
|
37
22
|
type: :runtime
|
38
|
-
- !ruby/object:Gem::Dependency
|
39
23
|
prerelease: false
|
24
|
+
version_requirements: !ruby/object:Gem::Requirement
|
25
|
+
requirements:
|
26
|
+
- - ~>
|
27
|
+
- !ruby/object:Gem::Version
|
28
|
+
version: 3.1.1
|
29
|
+
- !ruby/object:Gem::Dependency
|
40
30
|
name: sexp_processor
|
41
|
-
|
42
|
-
|
43
|
-
|
44
|
-
|
45
|
-
|
46
|
-
hash: 3
|
47
|
-
segments:
|
48
|
-
- 0
|
49
|
-
version: "0"
|
50
|
-
requirement: *id002
|
31
|
+
requirement: !ruby/object:Gem::Requirement
|
32
|
+
requirements:
|
33
|
+
- - '>='
|
34
|
+
- !ruby/object:Gem::Version
|
35
|
+
version: '0'
|
51
36
|
type: :runtime
|
52
|
-
- !ruby/object:Gem::Dependency
|
53
37
|
prerelease: false
|
38
|
+
version_requirements: !ruby/object:Gem::Requirement
|
39
|
+
requirements:
|
40
|
+
- - '>='
|
41
|
+
- !ruby/object:Gem::Version
|
42
|
+
version: '0'
|
43
|
+
- !ruby/object:Gem::Dependency
|
54
44
|
name: ruby2ruby
|
55
|
-
|
56
|
-
|
57
|
-
requirements:
|
45
|
+
requirement: !ruby/object:Gem::Requirement
|
46
|
+
requirements:
|
58
47
|
- - ~>
|
59
|
-
- !ruby/object:Gem::Version
|
60
|
-
hash: 11
|
61
|
-
segments:
|
62
|
-
- 2
|
63
|
-
- 0
|
64
|
-
- 2
|
48
|
+
- !ruby/object:Gem::Version
|
65
49
|
version: 2.0.2
|
66
|
-
requirement: *id003
|
67
50
|
type: :runtime
|
68
|
-
- !ruby/object:Gem::Dependency
|
69
51
|
prerelease: false
|
52
|
+
version_requirements: !ruby/object:Gem::Requirement
|
53
|
+
requirements:
|
54
|
+
- - ~>
|
55
|
+
- !ruby/object:Gem::Version
|
56
|
+
version: 2.0.2
|
57
|
+
- !ruby/object:Gem::Dependency
|
70
58
|
name: bundler
|
71
|
-
|
72
|
-
|
73
|
-
requirements:
|
59
|
+
requirement: !ruby/object:Gem::Requirement
|
60
|
+
requirements:
|
74
61
|
- - ~>
|
75
|
-
- !ruby/object:Gem::Version
|
76
|
-
|
77
|
-
segments:
|
78
|
-
- 1
|
79
|
-
- 1
|
80
|
-
version: "1.1"
|
81
|
-
requirement: *id004
|
62
|
+
- !ruby/object:Gem::Version
|
63
|
+
version: '1.1'
|
82
64
|
type: :development
|
83
|
-
- !ruby/object:Gem::Dependency
|
84
65
|
prerelease: false
|
66
|
+
version_requirements: !ruby/object:Gem::Requirement
|
67
|
+
requirements:
|
68
|
+
- - ~>
|
69
|
+
- !ruby/object:Gem::Version
|
70
|
+
version: '1.1'
|
71
|
+
- !ruby/object:Gem::Dependency
|
85
72
|
name: rake
|
86
|
-
|
87
|
-
|
88
|
-
|
89
|
-
|
90
|
-
|
91
|
-
hash: 3
|
92
|
-
segments:
|
93
|
-
- 0
|
94
|
-
version: "0"
|
95
|
-
requirement: *id005
|
73
|
+
requirement: !ruby/object:Gem::Requirement
|
74
|
+
requirements:
|
75
|
+
- - '>='
|
76
|
+
- !ruby/object:Gem::Version
|
77
|
+
version: '0'
|
96
78
|
type: :development
|
97
|
-
- !ruby/object:Gem::Dependency
|
98
79
|
prerelease: false
|
80
|
+
version_requirements: !ruby/object:Gem::Requirement
|
81
|
+
requirements:
|
82
|
+
- - '>='
|
83
|
+
- !ruby/object:Gem::Version
|
84
|
+
version: '0'
|
85
|
+
- !ruby/object:Gem::Dependency
|
99
86
|
name: cucumber
|
100
|
-
|
101
|
-
|
102
|
-
|
103
|
-
|
104
|
-
|
105
|
-
hash: 3
|
106
|
-
segments:
|
107
|
-
- 0
|
108
|
-
version: "0"
|
109
|
-
requirement: *id006
|
87
|
+
requirement: !ruby/object:Gem::Requirement
|
88
|
+
requirements:
|
89
|
+
- - '>='
|
90
|
+
- !ruby/object:Gem::Version
|
91
|
+
version: '0'
|
110
92
|
type: :development
|
111
|
-
- !ruby/object:Gem::Dependency
|
112
93
|
prerelease: false
|
94
|
+
version_requirements: !ruby/object:Gem::Requirement
|
95
|
+
requirements:
|
96
|
+
- - '>='
|
97
|
+
- !ruby/object:Gem::Version
|
98
|
+
version: '0'
|
99
|
+
- !ruby/object:Gem::Dependency
|
113
100
|
name: rspec
|
114
|
-
|
115
|
-
|
116
|
-
|
101
|
+
requirement: !ruby/object:Gem::Requirement
|
102
|
+
requirements:
|
103
|
+
- - ~>
|
104
|
+
- !ruby/object:Gem::Version
|
105
|
+
version: '2.12'
|
106
|
+
type: :development
|
107
|
+
prerelease: false
|
108
|
+
version_requirements: !ruby/object:Gem::Requirement
|
109
|
+
requirements:
|
117
110
|
- - ~>
|
118
|
-
- !ruby/object:Gem::Version
|
119
|
-
|
120
|
-
|
121
|
-
|
122
|
-
|
123
|
-
|
124
|
-
|
111
|
+
- !ruby/object:Gem::Version
|
112
|
+
version: '2.12'
|
113
|
+
- !ruby/object:Gem::Dependency
|
114
|
+
name: flay
|
115
|
+
requirement: !ruby/object:Gem::Requirement
|
116
|
+
requirements:
|
117
|
+
- - '>='
|
118
|
+
- !ruby/object:Gem::Version
|
119
|
+
version: '0'
|
125
120
|
type: :development
|
126
|
-
- !ruby/object:Gem::Dependency
|
127
121
|
prerelease: false
|
122
|
+
version_requirements: !ruby/object:Gem::Requirement
|
123
|
+
requirements:
|
124
|
+
- - '>='
|
125
|
+
- !ruby/object:Gem::Version
|
126
|
+
version: '0'
|
127
|
+
- !ruby/object:Gem::Dependency
|
128
128
|
name: yard
|
129
|
-
|
130
|
-
|
131
|
-
|
132
|
-
|
133
|
-
|
134
|
-
hash: 3
|
135
|
-
segments:
|
136
|
-
- 0
|
137
|
-
version: "0"
|
138
|
-
requirement: *id008
|
129
|
+
requirement: !ruby/object:Gem::Requirement
|
130
|
+
requirements:
|
131
|
+
- - '>='
|
132
|
+
- !ruby/object:Gem::Version
|
133
|
+
version: '0'
|
139
134
|
type: :development
|
135
|
+
prerelease: false
|
136
|
+
version_requirements: !ruby/object:Gem::Requirement
|
137
|
+
requirements:
|
138
|
+
- - '>='
|
139
|
+
- !ruby/object:Gem::Version
|
140
|
+
version: '0'
|
140
141
|
description: |
|
141
142
|
Reek is a tool that examines Ruby classes, modules and methods
|
142
143
|
and reports any code smells it finds.
|
143
|
-
|
144
|
-
email:
|
144
|
+
email:
|
145
145
|
- timo.roessner@googlemail.com
|
146
|
-
executables:
|
146
|
+
executables:
|
147
147
|
- reek
|
148
148
|
extensions: []
|
149
|
-
|
150
|
-
extra_rdoc_files:
|
149
|
+
extra_rdoc_files:
|
151
150
|
- CHANGELOG
|
152
151
|
- License.txt
|
153
|
-
files:
|
152
|
+
files:
|
154
153
|
- .yardopts
|
155
154
|
- CHANGELOG
|
156
155
|
- License.txt
|
@@ -158,213 +157,204 @@ files:
|
|
158
157
|
- Rakefile
|
159
158
|
- bin/reek
|
160
159
|
- config/defaults.reek
|
161
|
-
- features/ruby_api/api.feature
|
162
|
-
- features/command_line_interface/stdin.feature
|
163
|
-
- features/command_line_interface/options.feature
|
164
|
-
- features/step_definitions/reek_steps.rb
|
165
160
|
- features/support/env.rb
|
161
|
+
- features/configuration_files/masking_smells.feature
|
166
162
|
- features/samples.feature
|
167
|
-
- features/reports/reports.feature
|
168
163
|
- features/reports/yaml.feature
|
169
|
-
- features/
|
164
|
+
- features/reports/reports.feature
|
165
|
+
- features/command_line_interface/options.feature
|
166
|
+
- features/command_line_interface/stdin.feature
|
167
|
+
- features/step_definitions/reek_steps.rb
|
170
168
|
- features/rake_task/rake_task.feature
|
171
|
-
-
|
172
|
-
- lib/reek.rb
|
173
|
-
- lib/reek/spec.rb
|
169
|
+
- features/ruby_api/api.feature
|
174
170
|
- lib/reek/spec/should_reek_of.rb
|
175
171
|
- lib/reek/spec/should_reek_only_of.rb
|
176
172
|
- lib/reek/spec/should_reek.rb
|
173
|
+
- lib/reek/source/source_code.rb
|
174
|
+
- lib/reek/source/code_comment.rb
|
175
|
+
- lib/reek/source/source_file.rb
|
176
|
+
- lib/reek/source/config_file.rb
|
177
|
+
- lib/reek/source/tree_dresser.rb
|
178
|
+
- lib/reek/source/source_locator.rb
|
179
|
+
- lib/reek/source/reference_collector.rb
|
180
|
+
- lib/reek/source/sexp_formatter.rb
|
181
|
+
- lib/reek/source/core_extras.rb
|
182
|
+
- lib/reek/source/source_repository.rb
|
183
|
+
- lib/reek/core/object_refs.rb
|
184
|
+
- lib/reek/core/module_context.rb
|
185
|
+
- lib/reek/core/smell_repository.rb
|
186
|
+
- lib/reek/core/warning_collector.rb
|
187
|
+
- lib/reek/core/method_context.rb
|
188
|
+
- lib/reek/core/code_parser.rb
|
189
|
+
- lib/reek/core/hash_extensions.rb
|
190
|
+
- lib/reek/core/stop_context.rb
|
191
|
+
- lib/reek/core/singleton_method_context.rb
|
192
|
+
- lib/reek/core/code_context.rb
|
193
|
+
- lib/reek/core/sniffer.rb
|
194
|
+
- lib/reek/core/smell_configuration.rb
|
195
|
+
- lib/reek/spec.rb
|
196
|
+
- lib/reek/source.rb
|
197
|
+
- lib/reek/examiner.rb
|
198
|
+
- lib/reek/rake/task.rb
|
199
|
+
- lib/reek/smells.rb
|
200
|
+
- lib/reek/version.rb
|
177
201
|
- lib/reek/cli/version_command.rb
|
178
202
|
- lib/reek/cli/application.rb
|
179
|
-
- lib/reek/cli/command_line.rb
|
180
|
-
- lib/reek/cli/yaml_command.rb
|
181
|
-
- lib/reek/cli/help_command.rb
|
182
203
|
- lib/reek/cli/report.rb
|
183
204
|
- lib/reek/cli/reek_command.rb
|
184
|
-
- lib/reek/
|
185
|
-
- lib/reek/
|
186
|
-
- lib/reek/
|
187
|
-
- lib/reek/
|
188
|
-
- lib/reek/smells/duplication.rb
|
189
|
-
- lib/reek/smells/boolean_parameter.rb
|
190
|
-
- lib/reek/smells/uncommunicative_parameter_name.rb
|
191
|
-
- lib/reek/smells/unused_parameters.rb
|
192
|
-
- lib/reek/smells/long_method.rb
|
193
|
-
- lib/reek/smells/control_couple.rb
|
205
|
+
- lib/reek/cli/yaml_command.rb
|
206
|
+
- lib/reek/cli/command_line.rb
|
207
|
+
- lib/reek/cli/help_command.rb
|
208
|
+
- lib/reek/smells/irresponsible_module.rb
|
194
209
|
- lib/reek/smells/long_yield_list.rb
|
195
|
-
- lib/reek/smells/
|
196
|
-
- lib/reek/smells/simulated_polymorphism.rb
|
210
|
+
- lib/reek/smells/nested_iterators.rb
|
197
211
|
- lib/reek/smells/uncommunicative_method_name.rb
|
198
|
-
- lib/reek/smells/
|
212
|
+
- lib/reek/smells/boolean_parameter.rb
|
199
213
|
- lib/reek/smells/class_variable.rb
|
200
|
-
- lib/reek/smells/
|
201
|
-
- lib/reek/smells/feature_envy.rb
|
202
|
-
- lib/reek/smells/irresponsible_module.rb
|
214
|
+
- lib/reek/smells/control_parameter.rb
|
203
215
|
- lib/reek/smells/utility_function.rb
|
204
|
-
- lib/reek/smells/
|
205
|
-
- lib/reek/smells/
|
216
|
+
- lib/reek/smells/too_many_instance_variables.rb
|
217
|
+
- lib/reek/smells/too_many_methods.rb
|
218
|
+
- lib/reek/smells/long_parameter_list.rb
|
219
|
+
- lib/reek/smells/nil_check.rb
|
220
|
+
- lib/reek/smells/attribute.rb
|
206
221
|
- lib/reek/smells/uncommunicative_module_name.rb
|
207
|
-
- lib/reek/smells/
|
222
|
+
- lib/reek/smells/repeated_conditional.rb
|
223
|
+
- lib/reek/smells/uncommunicative_parameter_name.rb
|
208
224
|
- lib/reek/smells/smell_detector.rb
|
209
|
-
- lib/reek/smells/
|
210
|
-
- lib/reek/smells.rb
|
211
|
-
- lib/reek/
|
212
|
-
- lib/reek/
|
213
|
-
- lib/reek/
|
214
|
-
- lib/reek/
|
215
|
-
- lib/reek/
|
216
|
-
- lib/reek
|
217
|
-
-
|
218
|
-
-
|
219
|
-
- lib/reek/source/source_locator.rb
|
220
|
-
- lib/reek/source/tree_dresser.rb
|
221
|
-
- lib/reek/source.rb
|
222
|
-
- lib/reek/core/smell_configuration.rb
|
223
|
-
- lib/reek/core/method_context.rb
|
224
|
-
- lib/reek/core/code_parser.rb
|
225
|
-
- lib/reek/core/module_context.rb
|
226
|
-
- lib/reek/core/stop_context.rb
|
227
|
-
- lib/reek/core/code_context.rb
|
228
|
-
- lib/reek/core/sniffer.rb
|
229
|
-
- lib/reek/core/hash_extensions.rb
|
230
|
-
- lib/reek/core/smell_repository.rb
|
231
|
-
- lib/reek/core/object_refs.rb
|
232
|
-
- lib/reek/core/singleton_method_context.rb
|
233
|
-
- lib/reek/core/warning_collector.rb
|
234
|
-
- spec/matchers/smell_of_matcher.rb
|
235
|
-
- spec/spec_helper.rb
|
236
|
-
- spec/spec.opts
|
225
|
+
- lib/reek/smells/feature_envy.rb
|
226
|
+
- lib/reek/smells/too_many_statements.rb
|
227
|
+
- lib/reek/smells/uncommunicative_variable_name.rb
|
228
|
+
- lib/reek/smells/unused_parameters.rb
|
229
|
+
- lib/reek/smells/duplicate_method_call.rb
|
230
|
+
- lib/reek/smells/data_clump.rb
|
231
|
+
- lib/reek/smell_warning.rb
|
232
|
+
- lib/reek.rb
|
233
|
+
- spec/gem/yard_spec.rb
|
234
|
+
- spec/gem/updates_spec.rb
|
237
235
|
- spec/reek/spec/should_reek_of_spec.rb
|
238
|
-
- spec/reek/spec/should_reek_spec.rb
|
239
236
|
- spec/reek/spec/should_reek_only_of_spec.rb
|
237
|
+
- spec/reek/spec/should_reek_spec.rb
|
238
|
+
- spec/reek/source/object_source_spec.rb
|
239
|
+
- spec/reek/source/reference_collector_spec.rb
|
240
|
+
- spec/reek/source/code_comment_spec.rb
|
241
|
+
- spec/reek/source/source_code_spec.rb
|
242
|
+
- spec/reek/source/sexp_formatter_spec.rb
|
243
|
+
- spec/reek/source/tree_dresser_spec.rb
|
244
|
+
- spec/reek/core/module_context_spec.rb
|
245
|
+
- spec/reek/core/singleton_method_context_spec.rb
|
246
|
+
- spec/reek/core/code_parser_spec.rb
|
247
|
+
- spec/reek/core/code_context_spec.rb
|
248
|
+
- spec/reek/core/warning_collector_spec.rb
|
249
|
+
- spec/reek/core/object_refs_spec.rb
|
250
|
+
- spec/reek/core/smell_configuration_spec.rb
|
251
|
+
- spec/reek/core/stop_context_spec.rb
|
252
|
+
- spec/reek/core/config_spec.rb
|
253
|
+
- spec/reek/core/method_context_spec.rb
|
254
|
+
- spec/reek/smell_warning_spec.rb
|
240
255
|
- spec/reek/examiner_spec.rb
|
241
|
-
- spec/reek/cli/help_command_spec.rb
|
242
|
-
- spec/reek/cli/reek_command_spec.rb
|
243
256
|
- spec/reek/cli/yaml_command_spec.rb
|
257
|
+
- spec/reek/cli/reek_command_spec.rb
|
258
|
+
- spec/reek/cli/help_command_spec.rb
|
244
259
|
- spec/reek/cli/report_spec.rb
|
245
260
|
- spec/reek/cli/version_command_spec.rb
|
246
|
-
- spec/reek/smells/boolean_parameter_spec.rb
|
247
|
-
- spec/reek/smells/long_method_spec.rb
|
248
|
-
- spec/reek/smells/uncommunicative_method_name_spec.rb
|
249
|
-
- spec/reek/smells/duplication_spec.rb
|
250
|
-
- spec/reek/smells/uncommunicative_module_name_spec.rb
|
251
|
-
- spec/reek/smells/class_variable_spec.rb
|
252
|
-
- spec/reek/smells/long_yield_list_spec.rb
|
253
|
-
- spec/reek/smells/control_couple_spec.rb
|
254
261
|
- spec/reek/smells/unused_parameters_spec.rb
|
255
|
-
- spec/reek/smells/
|
262
|
+
- spec/reek/smells/irresponsible_module_spec.rb
|
263
|
+
- spec/reek/smells/too_many_methods_spec.rb
|
264
|
+
- spec/reek/smells/too_many_statements_spec.rb
|
265
|
+
- spec/reek/smells/uncommunicative_module_name_spec.rb
|
266
|
+
- spec/reek/smells/utility_function_spec.rb
|
256
267
|
- spec/reek/smells/long_parameter_list_spec.rb
|
268
|
+
- spec/reek/smells/control_parameter_spec.rb
|
269
|
+
- spec/reek/smells/behaves_like_variable_detector.rb
|
270
|
+
- spec/reek/smells/class_variable_spec.rb
|
257
271
|
- spec/reek/smells/attribute_spec.rb
|
258
|
-
- spec/reek/smells/
|
272
|
+
- spec/reek/smells/uncommunicative_variable_name_spec.rb
|
259
273
|
- spec/reek/smells/data_clump_spec.rb
|
274
|
+
- spec/reek/smells/uncommunicative_parameter_name_spec.rb
|
260
275
|
- spec/reek/smells/feature_envy_spec.rb
|
261
276
|
- spec/reek/smells/smell_detector_shared.rb
|
262
|
-
- spec/reek/smells/utility_function_spec.rb
|
263
|
-
- spec/reek/smells/nested_iterators_spec.rb
|
264
|
-
- spec/reek/smells/behaves_like_variable_detector.rb
|
265
|
-
- spec/reek/smells/large_class_spec.rb
|
266
277
|
- spec/reek/smells/nil_check_spec.rb
|
267
|
-
- spec/reek/smells/
|
268
|
-
- spec/reek/smells/
|
269
|
-
- spec/reek/
|
270
|
-
- spec/reek/
|
271
|
-
- spec/reek/
|
272
|
-
- spec/reek/
|
273
|
-
- spec/reek/
|
274
|
-
- spec/
|
275
|
-
- spec/
|
276
|
-
- spec/
|
277
|
-
- spec/reek/core/warning_collector_spec.rb
|
278
|
-
- spec/reek/core/stop_context_spec.rb
|
279
|
-
- spec/reek/core/config_spec.rb
|
280
|
-
- spec/reek/core/code_parser_spec.rb
|
281
|
-
- spec/reek/core/smell_configuration_spec.rb
|
282
|
-
- spec/reek/core/module_context_spec.rb
|
283
|
-
- spec/reek/core/method_context_spec.rb
|
284
|
-
- spec/reek/core/code_context_spec.rb
|
285
|
-
- spec/reek/core/singleton_method_context_spec.rb
|
286
|
-
- spec/gem/updates_spec.rb
|
287
|
-
- spec/gem/yard_spec.rb
|
288
|
-
- spec/gem/manifest_spec.rb
|
289
|
-
- spec/samples/three_clean_files/clean_one.rb
|
278
|
+
- spec/reek/smells/repeated_conditional_spec.rb
|
279
|
+
- spec/reek/smells/uncommunicative_method_name_spec.rb
|
280
|
+
- spec/reek/smells/nested_iterators_spec.rb
|
281
|
+
- spec/reek/smells/long_yield_list_spec.rb
|
282
|
+
- spec/reek/smells/duplicate_method_call_spec.rb
|
283
|
+
- spec/reek/smells/boolean_parameter_spec.rb
|
284
|
+
- spec/reek/smells/too_many_instance_variables_spec.rb
|
285
|
+
- spec/spec_helper.rb
|
286
|
+
- spec/samples/inline_config/dirty.rb
|
287
|
+
- spec/samples/inline_config/masked.reek
|
290
288
|
- spec/samples/three_clean_files/clean_two.rb
|
291
289
|
- spec/samples/three_clean_files/clean_three.rb
|
292
|
-
- spec/samples/
|
293
|
-
- spec/samples/
|
294
|
-
- spec/samples/
|
295
|
-
- spec/samples/
|
296
|
-
- spec/samples/
|
290
|
+
- spec/samples/three_clean_files/clean_one.rb
|
291
|
+
- spec/samples/clean_due_to_masking/dirty_one.rb
|
292
|
+
- spec/samples/clean_due_to_masking/masked.reek
|
293
|
+
- spec/samples/clean_due_to_masking/clean_two.rb
|
294
|
+
- spec/samples/clean_due_to_masking/dirty_two.rb
|
295
|
+
- spec/samples/clean_due_to_masking/clean_three.rb
|
296
|
+
- spec/samples/clean_due_to_masking/clean_one.rb
|
297
|
+
- spec/samples/not_quite_masked/dirty.rb
|
298
|
+
- spec/samples/not_quite_masked/masked.reek
|
299
|
+
- spec/samples/not_quite_masked/smelly.rb
|
300
|
+
- spec/samples/optparse.rb
|
301
|
+
- spec/samples/all_but_one_masked/dirty.rb
|
302
|
+
- spec/samples/all_but_one_masked/masked.reek
|
303
|
+
- spec/samples/all_but_one_masked/clean_one.rb
|
297
304
|
- spec/samples/demo/demo.rb
|
305
|
+
- spec/samples/exceptions.reek
|
306
|
+
- spec/samples/masked_by_dotfile/dirty.rb
|
307
|
+
- spec/samples/inline.rb
|
298
308
|
- spec/samples/redcloth.rb
|
299
|
-
- spec/samples/
|
300
|
-
- spec/samples/
|
301
|
-
- spec/samples/
|
309
|
+
- spec/samples/overrides/masked/dirty.rb
|
310
|
+
- spec/samples/overrides/masked/lower.reek
|
311
|
+
- spec/samples/overrides/upper.reek
|
312
|
+
- spec/samples/two_smelly_files/dirty_one.rb
|
313
|
+
- spec/samples/two_smelly_files/dirty_two.rb
|
314
|
+
- spec/samples/masked/dirty.rb
|
315
|
+
- spec/samples/masked/masked.reek
|
316
|
+
- spec/samples/config/deeper_nested_iterators.reek
|
317
|
+
- spec/samples/config/allow_duplication.reek
|
302
318
|
- spec/samples/corrupt_config_file/dirty.rb
|
319
|
+
- spec/samples/corrupt_config_file/corrupt.reek
|
303
320
|
- spec/samples/empty_config_file/dirty.rb
|
304
321
|
- spec/samples/empty_config_file/empty.reek
|
305
|
-
- spec/samples/masked/masked.reek
|
306
|
-
- spec/samples/masked/dirty.rb
|
307
|
-
- spec/samples/all_but_one_masked/masked.reek
|
308
|
-
- spec/samples/all_but_one_masked/clean_one.rb
|
309
|
-
- spec/samples/all_but_one_masked/dirty.rb
|
310
|
-
- spec/samples/inline.rb
|
311
|
-
- spec/samples/mask_some/dirty.rb
|
312
|
-
- spec/samples/mask_some/some.reek
|
313
|
-
- spec/samples/not_quite_masked/masked.reek
|
314
|
-
- spec/samples/not_quite_masked/dirty.rb
|
315
|
-
- spec/samples/exceptions.reek
|
316
|
-
- spec/samples/mixed_results/dirty_two.rb
|
317
322
|
- spec/samples/mixed_results/dirty_one.rb
|
318
|
-
- spec/samples/mixed_results/clean_one.rb
|
319
323
|
- spec/samples/mixed_results/clean_two.rb
|
324
|
+
- spec/samples/mixed_results/dirty_two.rb
|
320
325
|
- spec/samples/mixed_results/clean_three.rb
|
321
|
-
- spec/samples/
|
322
|
-
- spec/samples/
|
323
|
-
- spec/samples/
|
324
|
-
- spec/
|
325
|
-
- spec/samples/clean_due_to_masking/clean_two.rb
|
326
|
-
- spec/samples/clean_due_to_masking/clean_three.rb
|
327
|
-
- spec/samples/optparse.rb
|
328
|
-
- spec/samples/two_smelly_files/dirty_two.rb
|
329
|
-
- spec/samples/two_smelly_files/dirty_one.rb
|
326
|
+
- spec/samples/mixed_results/clean_one.rb
|
327
|
+
- spec/samples/mask_some/dirty.rb
|
328
|
+
- spec/samples/mask_some/some.reek
|
329
|
+
- spec/matchers/smell_of_matcher.rb
|
330
330
|
- tasks/test.rake
|
331
|
-
- tasks/reek.rake
|
332
331
|
- tasks/develop.rake
|
332
|
+
- tasks/reek.rake
|
333
333
|
- reek.gemspec
|
334
334
|
homepage: http://wiki.github.com/troessner/reek
|
335
335
|
licenses: []
|
336
|
-
|
336
|
+
metadata: {}
|
337
337
|
post_install_message: Thank you for downloading Reek. For info see the reek wiki http://wiki.github.com/troessner/reek
|
338
|
-
rdoc_options:
|
338
|
+
rdoc_options:
|
339
339
|
- --main
|
340
340
|
- README.md
|
341
|
-
require_paths:
|
341
|
+
require_paths:
|
342
342
|
- lib
|
343
|
-
required_ruby_version: !ruby/object:Gem::Requirement
|
344
|
-
|
345
|
-
|
346
|
-
|
347
|
-
|
348
|
-
|
349
|
-
|
350
|
-
|
351
|
-
|
352
|
-
|
353
|
-
none: false
|
354
|
-
requirements:
|
355
|
-
- - ">="
|
356
|
-
- !ruby/object:Gem::Version
|
357
|
-
hash: 3
|
358
|
-
segments:
|
359
|
-
- 0
|
360
|
-
version: "0"
|
343
|
+
required_ruby_version: !ruby/object:Gem::Requirement
|
344
|
+
requirements:
|
345
|
+
- - '>='
|
346
|
+
- !ruby/object:Gem::Version
|
347
|
+
version: '0'
|
348
|
+
required_rubygems_version: !ruby/object:Gem::Requirement
|
349
|
+
requirements:
|
350
|
+
- - '>='
|
351
|
+
- !ruby/object:Gem::Version
|
352
|
+
version: '0'
|
361
353
|
requirements: []
|
362
|
-
|
363
354
|
rubyforge_project: reek
|
364
|
-
rubygems_version:
|
355
|
+
rubygems_version: 2.0.6
|
365
356
|
signing_key:
|
366
|
-
specification_version:
|
357
|
+
specification_version: 4
|
367
358
|
summary: Code smell detector for Ruby
|
368
359
|
test_files: []
|
369
|
-
|
370
360
|
has_rdoc:
|