cuke_linter 0.12.1 → 0.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 670cec37df02d865f45baf856baab441942d1dd54f8b39089b88fd4657afc098
-  data.tar.gz: 840623e9e37f4f38ac3d5f1f28332b77f759b77a8dd3aa12a9554ede015ccddb
+  metadata.gz: 7669fb1b66c4e9c103aedb209961dbeca8b308f74bcc1159401fa21e1ffd5b05
+  data.tar.gz: 36ef3927bc68e7d52bde3c91f26ab0428d07b81c7163c0c31aaabf2847fa78db
 SHA512:
-  metadata.gz: 96beaf4a96234000319d61976ee7b19de8fe0c4d5a83583d0cd3fa33579078cbfce40eac96e52302423c69266db8cc0d3582263a1b913674a741142ee044f0bc
-  data.tar.gz: b21f8016215c971dcff8565232c3e078c9e2f0f6a4421c554e2479cb0a5d42d1abeb6b08ef5f6851d28d50b13b3a8362772d8ecbecfbcc6a66efa34681deaeb5
+  metadata.gz: 449a18606c23f574ffba7b5a2d80f9cc7170839daf9187e52404d31ca93e3fb20396860476d827125b6a85968d98985ba83e2ad92d127066a69322edee40459b
+  data.tar.gz: 407d387c3ce3c6eb3648179837757caa688b5b6bdab42b7cc684d3480054fadc0d7aa4c4ef555d241c77919ecb6e8ae7438736ba6b449e56c6ee76cfbdac07e4

data/CHANGELOG.md

@@ -8,6 +8,18 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 Nothing yet...
 
+## [0.13.0] - 2020-03-01
+
+### Added
+ - Added the ability to configure relevant keywords for linters whose behavior is impacted by the dialect in which feature files are written.
+   - BackgroundDoesMoreThanSetupLinter
+   - TestWithActionStepAsFinalStepLinter
+   - TestWithNoActionStepLinter
+   - TestWithNoVerificationStepLinter
+   - TestWithSetupStepAfterActionStepLinter
+   - TestWithSetupStepAfterVerificationStepLinter
+   - TestWithSetupStepAsFinalStepLinter
+
 
 ## [0.12.1] - 2020-02-15
 
@@ -135,7 +147,8 @@ Nothing yet...
 - Custom linters, formatters, and command line usability
 
 
-[Unreleased]: https://github.com/enkessler/cuke_linter/compare/v0.12.1...HEAD
+[Unreleased]: https://github.com/enkessler/cuke_linter/compare/v0.13.0...HEAD
+[0.13.0]: https://github.com/enkessler/cuke_linter/compare/v0.12.1...v0.13.0
 [0.12.1]: https://github.com/enkessler/cuke_linter/compare/v0.12.0...v0.12.1
 [0.12.0]: https://github.com/enkessler/cuke_linter/compare/v0.11.1...v0.12.0
 [0.11.1]: https://github.com/enkessler/cuke_linter/compare/v0.11.0...v0.11.1

data/lib/cuke_linter.rb

@@ -35,6 +35,10 @@ require 'cuke_linter/linters/test_with_too_many_steps_linter'
 
 module CukeLinter
 
+  DEFAULT_GIVEN_KEYWORD = 'Given'.freeze
+  DEFAULT_WHEN_KEYWORD = 'When'.freeze
+  DEFAULT_THEN_KEYWORD = 'Then'.freeze
+
   @original_linters = { 'BackgroundDoesMoreThanSetupLinter'            => BackgroundDoesMoreThanSetupLinter.new,
                         'ElementWithCommonTagsLinter'                  => ElementWithCommonTagsLinter.new,
                         'ElementWithDuplicateTagsLinter'               => ElementWithDuplicateTagsLinter.new,

data/lib/cuke_linter/linters/background_does_more_than_setup_linter.rb

@@ -4,11 +4,17 @@ module CukeLinter
 
   class BackgroundDoesMoreThanSetupLinter < Linter
 
+    # Changes the linting settings on the linter using the provided configuration
+    def configure(options)
+      @when_keywords = options['When']
+      @then_keywords = options['Then']
+    end
+
     # The rule used to determine if a model has a problem
     def rule(model)
       return false unless model.is_a?(CukeModeler::Background)
 
-      model.steps.collect(&:keyword).any? { |keyword| (keyword == 'When') || (keyword == 'Then') }
+      model.steps.collect(&:keyword).any? { |keyword| when_keywords.include?(keyword) || then_keywords.include?(keyword) }
     end
 
     # The message used to describe the problem that has been found
@@ -16,5 +22,15 @@ module CukeLinter
       'Background has non-setup steps'
     end
 
+    private
+
+    def when_keywords
+      @when_keywords || [DEFAULT_WHEN_KEYWORD]
+    end
+
+    def then_keywords
+      @then_keywords || [DEFAULT_THEN_KEYWORD]
+    end
+
   end
 end

data/lib/cuke_linter/linters/test_with_action_step_as_final_step_linter.rb

@@ -4,6 +4,11 @@ module CukeLinter
 
   class TestWithActionStepAsFinalStepLinter < Linter
 
+    # Changes the linting settings on the linter using the provided configuration
+    def configure(options)
+      @when_keywords = options['When']
+    end
+
     # The rule used to determine if a model has a problem
     def rule(model)
       return false unless model.is_a?(CukeModeler::Scenario) || model.is_a?(CukeModeler::Outline)
@@ -11,7 +16,7 @@ module CukeLinter
       model_steps = model.steps || []
       return false unless model_steps.last
 
-      model_steps.last.keyword == 'When'
+      when_keywords.include?(model_steps.last.keyword)
     end
 
     # The message used to describe the problem that has been found
@@ -19,5 +24,11 @@ module CukeLinter
       "Test has 'When' as the final step."
     end
 
+    private
+
+    def when_keywords
+      @when_keywords || [DEFAULT_WHEN_KEYWORD]
+    end
+
   end
 end

data/lib/cuke_linter/linters/test_with_no_action_step_linter.rb

@@ -4,6 +4,11 @@ module CukeLinter
 
   class TestWithNoActionStepLinter < Linter
 
+    # Changes the linting settings on the linter using the provided configuration
+    def configure(options)
+      @when_keywords = options['When']
+    end
+
     # The rule used to determine if a model has a problem
     def rule(model)
       return false unless model.is_a?(CukeModeler::Scenario) || model.is_a?(CukeModeler::Outline)
@@ -11,7 +16,7 @@ module CukeLinter
       model_steps      = model.steps || []
       background_steps = model.parent_model.has_background? ? model.parent_model.background.steps || [] : []
       all_steps        = model_steps + background_steps
-      all_steps.none? { |step| step.keyword == 'When' }
+      all_steps.none? { |step| when_keywords.include?(step.keyword) }
     end
 
     # The message used to describe the problem that has been found
@@ -19,5 +24,11 @@ module CukeLinter
       "Test does not have a 'When' step."
     end
 
+    private
+
+    def when_keywords
+      @when_keywords || [DEFAULT_WHEN_KEYWORD]
+    end
+
   end
 end

data/lib/cuke_linter/linters/test_with_no_verification_step_linter.rb

@@ -4,6 +4,11 @@ module CukeLinter
 
   class TestWithNoVerificationStepLinter < Linter
 
+    # Changes the linting settings on the linter using the provided configuration
+    def configure(options)
+      @then_keywords = options['Then']
+    end
+
     # The rule used to determine if a model has a problem
     def rule(model)
       return false unless model.is_a?(CukeModeler::Scenario) || model.is_a?(CukeModeler::Outline)
@@ -11,7 +16,7 @@ module CukeLinter
       model_steps      = model.steps || []
       background_steps = model.parent_model.has_background? ? model.parent_model.background.steps || [] : []
       all_steps        = model_steps + background_steps
-      all_steps.none? { |step| step.keyword == 'Then' }
+      all_steps.none? { |step| then_keywords.include?(step.keyword) }
     end
 
     # The message used to describe the problem that has been found
@@ -19,5 +24,11 @@ module CukeLinter
       "Test does not have a 'Then' step."
     end
 
+    private
+
+    def then_keywords
+      @then_keywords || [DEFAULT_THEN_KEYWORD]
+    end
+
   end
 end

data/lib/cuke_linter/linters/test_with_setup_step_after_action_step_linter.rb

@@ -4,6 +4,12 @@ module CukeLinter
 
   class TestWithSetupStepAfterActionStepLinter < Linter
 
+    # Changes the linting settings on the linter using the provided configuration
+    def configure(options)
+      @given_keywords = options['Given']
+      @when_keywords  = options['When']
+    end
+
     # The rule used to determine if a model has a problem
     def rule(model)
       return false unless model.is_a?(CukeModeler::Scenario) || model.is_a?(CukeModeler::Outline)
@@ -13,9 +19,9 @@ module CukeLinter
 
       model_steps.each do |step|
         if action_step_found
-          return true if step.keyword == 'Given'
+          return true if given_keywords.include?(step.keyword)
         else
-          action_step_found = step.keyword == 'When'
+          action_step_found = when_keywords.include?(step.keyword)
         end
       end
 
@@ -27,5 +33,15 @@ module CukeLinter
       "Test has 'Given' step after 'When' step."
     end
 
+    private
+
+    def given_keywords
+      @given_keywords || [DEFAULT_GIVEN_KEYWORD]
+    end
+
+    def when_keywords
+      @when_keywords || [DEFAULT_WHEN_KEYWORD]
+    end
+
   end
 end

data/lib/cuke_linter/linters/test_with_setup_step_after_verification_step_linter.rb

@@ -4,6 +4,12 @@ module CukeLinter
 
   class TestWithSetupStepAfterVerificationStepLinter < Linter
 
+    # Changes the linting settings on the linter using the provided configuration
+    def configure(options)
+      @given_keywords = options['Given']
+      @then_keywords  = options['Then']
+    end
+
     # The rule used to determine if a model has a problem
     def rule(model)
       return false unless model.is_a?(CukeModeler::Scenario) || model.is_a?(CukeModeler::Outline)
@@ -13,9 +19,9 @@ module CukeLinter
 
       model_steps.each do |step|
         if verification_step_found
-          return true if step.keyword == 'Given'
+          return true if given_keywords.include?(step.keyword)
         else
-          verification_step_found = step.keyword == 'Then'
+          verification_step_found = then_keywords.include?(step.keyword)
         end
       end
 
@@ -27,5 +33,15 @@ module CukeLinter
       "Test has 'Given' step after 'Then' step."
     end
 
+    private
+
+    def given_keywords
+      @given_keywords || [DEFAULT_GIVEN_KEYWORD]
+    end
+
+    def then_keywords
+      @then_keywords || [DEFAULT_THEN_KEYWORD]
+    end
+
   end
 end

data/lib/cuke_linter/linters/test_with_setup_step_as_final_step_linter.rb

@@ -4,6 +4,11 @@ module CukeLinter
 
   class TestWithSetupStepAsFinalStepLinter < Linter
 
+    # Changes the linting settings on the linter using the provided configuration
+    def configure(options)
+      @given_keywords = options['Given']
+    end
+
     # The rule used to determine if a model has a problem
     def rule(model)
       return false unless model.is_a?(CukeModeler::Scenario) || model.is_a?(CukeModeler::Outline)
@@ -11,7 +16,7 @@ module CukeLinter
       model_steps = model.steps || []
       return false unless model_steps.last
 
-      model_steps.last.keyword == 'Given'
+      given_keywords.include?(model_steps.last.keyword)
     end
 
     # The message used to describe the problem that has been found
@@ -19,5 +24,11 @@ module CukeLinter
       "Test has 'Given' as the final step."
     end
 
+    private
+
+    def given_keywords
+       @given_keywords || [DEFAULT_GIVEN_KEYWORD]
+    end
+
   end
 end

data/lib/cuke_linter/version.rb

@@ -1,4 +1,4 @@
 module CukeLinter
   # The release version of this gem
-  VERSION = '0.12.1'
+  VERSION = '0.13.0'
 end

data/testing/cucumber/features/linters/background_does_more_than_setup.feature

@@ -48,3 +48,37 @@ Feature: Background does more than setup linter
       """
     When it is linted
     Then no error is reported
+
+  Scenario Outline: Configuration of keywords for different dialect
+    Given a linter for backgrounds that do more than setup has been registered
+    And the following configuration file:
+      """
+      BackgroundDoesMoreThanSetupLinter:
+        Given:
+          - Dado
+        When:
+          - Quando
+        Then:
+          - Então
+          - Entao
+      """
+    And the following feature:
+      """
+      # language:pt
+      Funcionalidade: Feature name
+
+        Contexto: A Background in pt dialect
+          Dado some setup in pt dialect
+          <step>
+      """
+    When the configuration file is loaded
+    And it is linted
+    Then an error is reported:
+      | linter                            | problem                        | location         |
+      | BackgroundDoesMoreThanSetupLinter | Background has non-setup steps | <path_to_file>:4 |
+
+    Examples:
+      | step                                          |
+      | Quando this is an action in pt dialect        |
+      | Então this is a validation in pt dialect      |
+      | Entao this is also a validation in pt dialect |

data/testing/cucumber/features/linters/test_with_action_as_final_step.feature

@@ -22,7 +22,29 @@ Feature: Test with action step as final step linter
       | linter                              | problem                            | location         |
       | TestWithActionStepAsFinalStepLinter | Test has 'When' as the final step. | <path_to_file>:3 |
 
-  @wip
-  Scenario: Configuration
+  Scenario: Configuration of keywords for different dialect
+    Given a linter for tests with an action step as the final step has been registered
+    And the following configuration file:
+      """
+      TestWithActionStepAsFinalStepLinter:
+        Given:
+          - Dado
+        When:
+          - Quando
+          - '*'
+        Then:
+          - Então
+      """
+    And the following feature:
+      """
+      # language:pt
+      Funcionalidade: Feature name
 
-  Configure the keyword(s) that count as action steps?
+        Cenário: scenario name
+          Quando this is an action in pt dialect
+      """
+    When the configuration file is loaded
+    And it is linted
+    Then an error is reported:
+      | linter                              | problem                            | location         |
+      | TestWithActionStepAsFinalStepLinter | Test has 'When' as the final step. | <path_to_file>:4 |

data/testing/cucumber/features/linters/test_with_no_action_step.feature

@@ -23,8 +23,34 @@ Feature: Test with no action step linter
       | linter                     | problem                           | location         |
       | TestWithNoActionStepLinter | Test does not have a 'When' step. | <path_to_file>:3 |
 
+  Scenario: Configuration of keywords for different dialect
+    Given a linter for tests with no action step has been registered
+    And the following configuration file:
+      """
+      TestWithNoActionStepLinter:
+        Given:
+          - Dado
+        When:
+          - Quando
+        Then:
+          - Então
+      """
+    And the following feature:
+      """
+      # language:pt
+      Funcionalidade: Feature name
+
+        Cenário: scenario name
+          Dado some setup in pt dialect
+          Então this is an validation in pt dialect
+      """
+    When the configuration file is loaded
+    And it is linted
+    Then an error is reported:
+      | linter                     | problem                           | location         |
+      | TestWithNoActionStepLinter | Test does not have a 'When' step. | <path_to_file>:4 |
+
   @wip
   Scenario: Configuration
 
   Ideas: Configure whether or not the linter triggers on tests with no steps at all?
-  Configure the keyword(s) that count as an action step?

data/testing/cucumber/features/linters/test_with_no_verification_step.feature

@@ -24,8 +24,35 @@ Feature: Test with no verification step linter
       | linter                           | problem                           | location         |
       | TestWithNoVerificationStepLinter | Test does not have a 'Then' step. | <path_to_file>:3 |
 
+  Scenario: Configuration of keywords for different dialect
+    Given a linter for tests with no verification step has been registered
+    And the following configuration file:
+      """
+      TestWithNoVerificationStepLinter:
+        Given:
+          - Dado
+        When:
+          - Quando
+        Then:
+          - Então
+      """
+    And the following feature:
+      """
+      # language:pt
+      Funcionalidade: Feature name
+
+        Cenário: scenario name
+          Dado some setup in pt dialect
+          Quando this is an action in pt dialect
+          E this is and AND in pt dialect
+      """
+    When the configuration file is loaded
+    And it is linted
+    Then an error is reported:
+      | linter                           | problem                           | location         |
+      | TestWithNoVerificationStepLinter | Test does not have a 'Then' step. | <path_to_file>:4 |
+
   @wip
   Scenario: Configuration
 
   Ideas: Configure whether or not the linter triggers on tests with no steps at all?
-  Configure the keyword(s) that count as a verification step?