reek 4.7.3 → 5.0.0

data/features/configuration_files/reject_setting.feature

@@ -3,20 +3,25 @@ Feature: `reject` configuration setting
   As a user
   I want to be able to reject specific patterns and names to include them into reporting
 
-  Scenario: reject names as list and as single item
+  Scenario: reject names
     Given a file named "config.reek" with:
       """
       ---
-      UncommunicativeMethodName:
-        reject:
-          - awesome_helper
-          - little_helper
-      UncommunicativeParameterName:
-        reject:
-          - solid_argument
-          - nifty_argument
-      UncommunicativeModuleName:
-        reject: Dummy
+      detectors:
+        UncommunicativeMethodName:
+          reject:
+            - awesome_helper
+            - little_helper
+        UncommunicativeParameterName:
+          reject:
+            - solid_argument
+            - nifty_argument
+        UncommunicativeModuleName:
+          reject:
+            - Dummy
+        UncommunicativeVariableName:
+          reject:
+            - fine_name
       """
     And a file named "smelly.rb" with:
       """
@@ -28,51 +33,57 @@ Feature: `reject` configuration setting
         def little_helper(nifty_argument); nifty_argument; end
         # Should not report UncommunicativeMethodName and UncommunicativeParameterName
         def meth(argument); argument; end
+        fine_name = 2 # Should report UncommunicativeMethodName
+        other_name = 2 # Should not report UncommunicativeMethodName
       end
       """
-    When I run `reek -c config.reek smelly.rb`
+    When I run reek -c config.reek smelly.rb
     Then it reports:
     """
-    smelly.rb -- 5 warnings:
-      [4]:UncommunicativeMethodName: Dummy#awesome_helper has the name 'awesome_helper' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Method-Name.md]
-      [6]:UncommunicativeMethodName: Dummy#little_helper has the name 'little_helper' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Method-Name.md]
-      [2]:UncommunicativeModuleName: Dummy has the name 'Dummy' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Module-Name.md]
-      [4]:UncommunicativeParameterName: Dummy#awesome_helper has the parameter name 'solid_argument' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Parameter-Name.md]
-      [6]:UncommunicativeParameterName: Dummy#little_helper has the parameter name 'nifty_argument' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Parameter-Name.md]
+    smelly.rb -- 6 warnings:
+      [4]:UncommunicativeMethodName: Dummy#awesome_helper has the name 'awesome_helper'
+      [6]:UncommunicativeMethodName: Dummy#little_helper has the name 'little_helper'
+      [2]:UncommunicativeModuleName: Dummy has the name 'Dummy'
+      [4]:UncommunicativeParameterName: Dummy#awesome_helper has the parameter name 'solid_argument'
+      [6]:UncommunicativeParameterName: Dummy#little_helper has the parameter name 'nifty_argument'
+      [9]:UncommunicativeVariableName: Dummy has the variable name 'fine_name'
     """
 
-  Scenario: reject regexes as list and as single item
+  Scenario: Reject regexes
     Given a file named "config.reek" with:
       """
       ---
-      UncommunicativeMethodName:
-        reject: !ruby/regexp /helper/
-      UncommunicativeParameterName:
-        reject:
-          - !ruby/regexp /solid/
-          - !ruby/regexp /nifty/
-      UncommunicativeModuleName:
-        reject: !ruby/regexp /ummy/
+      detectors:
+        UncommunicativeMethodName:
+          accept:
+            - /oobar/
+        UncommunicativeParameterName:
+          accept:
+            - /ola/
+        UncommunicativeModuleName:
+          accept:
+            - /lassy/
+        UncommunicativeVariableName:
+          reject:
+            - /^fine_name/
       """
     And a file named "smelly.rb" with:
       """
-      # Should report UncommunicativeModuleName
-      class Dummy
-        # Should report UncommunicativeMethodName and UncommunicativeParameterName
-        def awesome_helper(solid_argument); solid_argument; end
-        # Should report UncommunicativeMethodName and UncommunicativeParameterName
-        def little_helper(nifty_argument); nifty_argument; end
+      # Should not report UncommunicativeModuleName
+      class Classy1
         # Should not report UncommunicativeMethodName and UncommunicativeParameterName
-        def meth(argument); argument; end
+        def foobar1(hola1); hola1; end
+        # Should report UncommunicativeMethodName and UncommunicativeParameterName
+        def m2(a2); a2; end
+        fine_name = 2 # Should report UncommunicativeMethodName
+        my_fine_name = 2 # Should not report UncommunicativeMethodName
       end
       """
-    When I run `reek -c config.reek smelly.rb`
+    When I run reek -c config.reek smelly.rb
     Then it reports:
     """
-    smelly.rb -- 5 warnings:
-      [4]:UncommunicativeMethodName: Dummy#awesome_helper has the name 'awesome_helper' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Method-Name.md]
-      [6]:UncommunicativeMethodName: Dummy#little_helper has the name 'little_helper' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Method-Name.md]
-      [2]:UncommunicativeModuleName: Dummy has the name 'Dummy' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Module-Name.md]
-      [4]:UncommunicativeParameterName: Dummy#awesome_helper has the parameter name 'solid_argument' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Parameter-Name.md]
-      [6]:UncommunicativeParameterName: Dummy#little_helper has the parameter name 'nifty_argument' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Parameter-Name.md]
+    smelly.rb -- 3 warnings:
+      [6]:UncommunicativeMethodName: Classy1#m2 has the name 'm2'
+      [6]:UncommunicativeParameterName: Classy1#m2 has the parameter name 'a2'
+      [7]:UncommunicativeVariableName: Classy1 has the variable name 'fine_name'
     """

data/features/configuration_files/schema_validation.feature

@@ -0,0 +1,59 @@
+Feature: Validate schema
+  In order to ensure that I am using the right configuration
+  As a user
+  I want to be notified when I am using a configuration that violates our schema
+
+  Scenario: Our generated default configuration
+    Given our default configuration file
+    And the clean file 'clean.rb'
+    When I run reek -c defaults.reek clean.rb
+    Then it succeeds
+    And it reports nothing
+
+  Scenario: Detectors, directories and exclude paths all mixed
+    Given a file named "config.reek" with:
+    """
+    ---
+    detectors:
+      IrresponsibleModule:
+        enabled: false
+      NestedIterators:
+        exclude:
+          - "MyWorker#self.class_method"
+          - "AnotherWorker#instance_method"
+      DataClump:
+        max_copies: 3
+        min_clump_size: 3
+
+    directories:
+      "web_app/app/controllers":
+        NestedIterators:
+          enabled: false
+      "web_app/app/helpers":
+        UtilityFunction:
+          enabled: false
+
+    exclude_paths:
+      - lib/legacy
+      - lib/rake/legacy_tasks
+    """
+    And the clean file 'clean.rb'
+    When I run reek -c config.reek clean.rb
+    Then it succeeds
+    And it reports nothing
+
+  Scenario: Invalid detector name
+    Given a file named "config.reek" with:
+    """
+    ---
+    detectors:
+      DoesNotExist:
+        enabled: true
+    """
+    And the clean file 'clean.rb'
+    When I run reek -c config.reek clean.rb
+    Then the exit status indicates an error
+    And stderr reports:
+    """
+    Error: We found some problems with your configuration file: [/detectors/DoesNotExist] key 'DoesNotExist:' is undefined.
+    """

data/features/configuration_files/unused_private_method.feature

@@ -7,12 +7,13 @@ Feature: Unused Private Method detector
     Given a file named "config.reek" with:
       """
       ---
-      IrresponsibleModule:
-        enabled: false
-      UnusedPrivateMethod:
-        enabled: true
-        exclude:
-          - "Outer::Awesome#foobar"
+      detectors:
+        IrresponsibleModule:
+          enabled: false
+        UnusedPrivateMethod:
+          enabled: true
+          exclude:
+            - Outer::Awesome#foobar
       """
     And a file named "sample.rb" with:
       """
@@ -27,23 +28,24 @@ Feature: Unused Private Method detector
         end
       end
       """
-    When I run `reek -c config.reek sample.rb`
+    When I run reek -c config.reek sample.rb
     Then it reports:
     """
     sample.rb -- 1 warning:
-      [4]:UnusedPrivateMethod: Outer::Smelly has the unused private instance method 'foobar' [https://github.com/troessner/reek/blob/master/docs/Unused-Private-Method.md]
+      [4]:UnusedPrivateMethod: Outer::Smelly has the unused private instance method 'foobar'
     """
 
   Scenario: Use regexes for excluding contexts
     Given a file named "config.reek" with:
       """
       ---
-      IrresponsibleModule:
-        enabled: false
-      UnusedPrivateMethod:
-        enabled: true
-        exclude:
-          - !ruby/regexp /(.*)wesome#foo(.*)/
+      detectors:
+        IrresponsibleModule:
+          enabled: false
+        UnusedPrivateMethod:
+          enabled: true
+          exclude:
+            - wesome#foo
       """
     And a file named "sample.rb" with:
       """
@@ -58,9 +60,9 @@ Feature: Unused Private Method detector
         end
       end
       """
-    When I run `reek -c config.reek sample.rb`
+    When I run reek -c config.reek sample.rb
     Then it reports:
     """
     sample.rb -- 1 warning:
-      [4]:UnusedPrivateMethod: Outer::Smelly has the unused private instance method 'foobar' [https://github.com/troessner/reek/blob/master/docs/Unused-Private-Method.md]
+      [4]:UnusedPrivateMethod: Outer::Smelly has the unused private instance method 'foobar'
     """

data/features/configuration_loading.feature

@@ -3,7 +3,7 @@ Feature: Offer different ways how to load configuration
   Reek can be configured in two ways:
   - Using the cli "-c" switch to pass a configuration file on the command line.
   - Having a Reek configuration file that is automatically found. Reek will
-    look for a file ending in .reek in the following places, in order:
+    look for a file ending in .reek.yml in the following places, in order:
     - The current working directory
     - The working directory's ancestor directories, traversing all the way up
       to the root.
@@ -17,8 +17,8 @@ Feature: Offer different ways how to load configuration
     And it reports:
       """
       smelly.rb -- 2 warnings:
-        [4]:UncommunicativeMethodName: Smelly#x has the name 'x' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Method-Name.md]
-        [5]:UncommunicativeVariableName: Smelly#x has the variable name 'y' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+        [4]:UncommunicativeMethodName: Smelly#x has the name 'x'
+        [5]:UncommunicativeVariableName: Smelly#x has the variable name 'y'
       """
 
   Scenario: Configuration via CLI
@@ -30,19 +30,62 @@ Feature: Offer different ways how to load configuration
 
   Scenario: Configuration file in working directory
     Given the smelly file 'smelly.rb'
-    And a configuration file 'full_mask.reek'
+    And a file named ".reek.yml" with:
+      """
+      ---
+      detectors:
+        UncommunicativeMethodName:
+          enabled: false
+        UncommunicativeVariableName:
+          enabled: false
+      """
     When I run reek smelly.rb
     Then it reports no errors
     And it succeeds
 
-  Scenario: Two opposing configuration files and we stop after the first one
-    Given the smelly file 'smelly.rb' in a subdirectory
-    And a configuration file 'partial_mask.reek' in a subdirectory
-    And a configuration file 'full_mask.reek'
-    When I run "reek smelly.rb" in a subdirectory
+  Scenario: Do not use the default config file when we explicitly specify one configuration file
+    Given the smelly file 'smelly.rb'
+    And a file named "config.reek" with:
+      """
+      ---
+      detectors:
+        UncommunicativeMethodName:
+          enabled: false
+      """
+    And a file named ".reek.yml" with:
+      """
+      ---
+      UncommunicativeVariableName:
+        enabled: false
+      """
+    When I run reek -c config.reek smelly.rb
+    Then the exit status indicates smells
+    And it reports:
+      """
+      smelly.rb -- 1 warning:
+        [5]:UncommunicativeVariableName: Smelly#x has the variable name 'y'
+      """
+
+  Scenario: Prefer the default config file over other configuration files
+    Given the smelly file 'smelly.rb'
+    And a file named "config.reek" with:
+      """
+      ---
+      detectors:
+        UncommunicativeMethodName:
+          enabled: false
+      """
+    And a file named ".reek.yml" with:
+      """
+      ---
+      detectors:
+        UncommunicativeVariableName:
+          enabled: false
+      """
+    When I run reek smelly.rb
     Then the exit status indicates smells
     And it reports:
       """
       smelly.rb -- 1 warning:
-        [5]:UncommunicativeVariableName: Smelly#x has the variable name 'y' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+        [4]:UncommunicativeMethodName: Smelly#x has the name 'x'
       """

data/features/configuration_via_source_comments/erroneous_source_comments.feature

@@ -33,8 +33,8 @@ Feature: Erroneous source comments are handled properly
     And it reports:
       """
       smelly.rb -- 2 warnings:
-        [4]:UncommunicativeMethodName: Smelly#x has the name 'x' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Method-Name.md]
-        [5]:UncommunicativeVariableName: Smelly#x has the variable name 'y' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+        [4]:UncommunicativeMethodName: Smelly#x has the name 'x'
+        [5]:UncommunicativeVariableName: Smelly#x has the variable name 'y'
       2 total warnings
       """
 

data/features/configuration_via_source_comments/well_formed_source_comments.feature

@@ -48,7 +48,7 @@ Feature: Well formed source comments are handled properly
     And it reports:
       """
       enable_detector_explicitly.rb -- 1 warning:
-        [3]:TooManyInstanceVariables: Alfa has at least 5 instance variables [https://github.com/troessner/reek/blob/master/docs/Too-Many-Instance-Variables.md]
+        [3]:TooManyInstanceVariables: Alfa has at least 5 instance variables
       """
 
   Scenario: Configure smell detector with the basic exclude option as string
@@ -112,5 +112,5 @@ Feature: Well formed source comments are handled properly
     And it reports:
       """
       configure_detector_specific_option.rb -- 1 warning:
-        [3]:TooManyInstanceVariables: Alfa has at least 3 instance variables [https://github.com/troessner/reek/blob/master/docs/Too-Many-Instance-Variables.md]
+        [3]:TooManyInstanceVariables: Alfa has at least 3 instance variables
       """

data/features/locales.feature

@@ -9,7 +9,7 @@ Feature: Handling different locales
     """
     puts 'こんにちは世界'
     """
-    When I run `reek -V konnichiwa.rb`
+    When I run reek -V konnichiwa.rb
     Then it succeeds
     And it reports no errors
     And it reports:
@@ -23,7 +23,7 @@ Feature: Handling different locales
     """
     puts 'こんにちは世界'
     """
-    When I run `reek -V konnichiwa.rb`
+    When I run reek -V konnichiwa.rb
     Then it succeeds
     And it reports no errors
     And it reports:

data/features/rake_task/rake_task.feature

@@ -10,7 +10,7 @@ Feature: Reek can be driven through its Task
 
       Reek::Rake::Task.new do |t|
         t.source_files = 'smelly.rb'
-        t.reek_opts = '--no-color'
+        t.reek_opts = '--no-color --no-documentation'
       end
       """
     When I run `rake reek`
@@ -18,8 +18,8 @@ Feature: Reek can be driven through its Task
     And it reports:
       """
       smelly.rb -- 2 warnings:
-        [4]:UncommunicativeMethodName: Smelly#x has the name 'x' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Method-Name.md]
-        [5]:UncommunicativeVariableName: Smelly#x has the variable name 'y' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+        [4]:UncommunicativeMethodName: Smelly#x has the name 'x'
+        [5]:UncommunicativeVariableName: Smelly#x has the variable name 'y'
       """
 
   Scenario: source_files using a FileList instead of a String
@@ -30,7 +30,7 @@ Feature: Reek can be driven through its Task
 
       Reek::Rake::Task.new do |t|
         t.source_files = FileList['smelly.*']
-        t.reek_opts = '--no-color'
+        t.reek_opts = '--no-color --no-documentation'
       end
       """
     When I run `rake reek`
@@ -38,8 +38,8 @@ Feature: Reek can be driven through its Task
     And it reports:
       """
       smelly.rb -- 2 warnings:
-        [4]:UncommunicativeMethodName: Smelly#x has the name 'x' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Method-Name.md]
-        [5]:UncommunicativeVariableName: Smelly#x has the variable name 'y' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+        [4]:UncommunicativeMethodName: Smelly#x has the name 'x'
+        [5]:UncommunicativeVariableName: Smelly#x has the variable name 'y'
       """
 
   Scenario: name changes the task name
@@ -50,7 +50,7 @@ Feature: Reek can be driven through its Task
 
       Reek::Rake::Task.new('silky') do |t|
         t.source_files = 'smelly.rb'
-        t.reek_opts = '--no-color'
+        t.reek_opts = '--no-color --no-documentation'
       end
       """
     When I run `rake silky`
@@ -58,8 +58,8 @@ Feature: Reek can be driven through its Task
     And it reports:
       """
       smelly.rb -- 2 warnings:
-        [4]:UncommunicativeMethodName: Smelly#x has the name 'x' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Method-Name.md]
-        [5]:UncommunicativeVariableName: Smelly#x has the variable name 'y' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+        [4]:UncommunicativeMethodName: Smelly#x has the name 'x'
+        [5]:UncommunicativeVariableName: Smelly#x has the variable name 'y'
       """
 
   Scenario: verbose prints the reek command
@@ -86,7 +86,7 @@ Feature: Reek can be driven through its Task
       Reek::Rake::Task.new do |t|
         t.fail_on_error = false
         t.source_files = 'smelly.rb'
-        t.reek_opts = '--no-color'
+        t.reek_opts = '--no-color --no-documentation'
       end
       """
     When I run `rake reek`
@@ -95,8 +95,8 @@ Feature: Reek can be driven through its Task
     And it reports:
       """
       smelly.rb -- 2 warnings:
-        [4]:UncommunicativeMethodName: Smelly#x has the name 'x' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Method-Name.md]
-        [5]:UncommunicativeVariableName: Smelly#x has the variable name 'y' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+        [4]:UncommunicativeMethodName: Smelly#x has the name 'x'
+        [5]:UncommunicativeVariableName: Smelly#x has the variable name 'y'
       """
 
   Scenario: can be configured with config_file
@@ -124,7 +124,7 @@ Feature: Reek can be driven through its Task
 
       Reek::Rake::Task.new do |t|
         t.source_files = 'clean.rb'
-        t.reek_opts = '--no-color'
+        t.reek_opts = '--no-color --no-documentation'
       end
       """
     When I set the environment variable "REEK_SRC" to "smelly.rb"
@@ -133,6 +133,6 @@ Feature: Reek can be driven through its Task
     And it reports:
       """
       smelly.rb -- 2 warnings:
-        [4]:UncommunicativeMethodName: Smelly#x has the name 'x' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Method-Name.md]
-        [5]:UncommunicativeVariableName: Smelly#x has the variable name 'y' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+        [4]:UncommunicativeMethodName: Smelly#x has the name 'x'
+        [5]:UncommunicativeVariableName: Smelly#x has the variable name 'y'
       """

data/features/reports/json.feature

@@ -24,7 +24,7 @@ Feature: Report smells using simple JSON layout
               "context": "Smelly#x",
               "lines": [ 4 ],
               "message": "has the name 'x'",
-              "wiki_link": "https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Method-Name.md",
+              "documentation_link": "https://github.com/troessner/reek/blob/v5.0.0/docs/Uncommunicative-Method-Name.md",
               "name": "x"
           },
           {
@@ -33,7 +33,7 @@ Feature: Report smells using simple JSON layout
               "context": "Smelly#x",
               "lines": [ 5 ],
               "message": "has the variable name 'y'",
-              "wiki_link": "https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md",
+              "documentation_link": "https://github.com/troessner/reek/blob/v5.0.0/docs/Uncommunicative-Variable-Name.md",
               "name": "y"
           }
       ]
@@ -53,7 +53,7 @@ Feature: Report smells using simple JSON layout
                   1
               ],
               "message": "has no descriptive comment",
-              "wiki_link": "https://github.com/troessner/reek/blob/master/docs/Irresponsible-Module.md"
+              "documentation_link": "https://github.com/troessner/reek/blob/v5.0.0/docs/Irresponsible-Module.md"
           }
       ]
       """