chutney 3.10.0 → 3.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6734d8a322d5394f40d363d0ad93def2424ea97be81f4a13cd7bb92c91ade597
-  data.tar.gz: 0146772a58c64cfa3aa8f7ab05861c94945d37abec3ff10c93bd3f653c9784b4
+  metadata.gz: c008b1c26d32fdc61e5971638623f09d6f34526e0482b31c9e6c98ca7770a5e6
+  data.tar.gz: 4868772b4ffea1157e0c443772d779dc5b693aa7ec693d3e37718d392ab6676b
 SHA512:
-  metadata.gz: 9f869fc17e30cee78412fab391049971c81316937148e73dcd6111afb6d55a5c60e9ac5db24d36917b9be63556dd8ee63ea4b1d082f8fd4a8b3edb3bae037ba4
-  data.tar.gz: 87f372ffc4bcef3f514aef73ae971d8a5dd8545d6bd7897770e7c69808c11fd7f0740afd0befe271ef49274720e91da5a92a4913cd108737bf2e1fbe99f1dd5e
+  metadata.gz: 03c10adcf4fe4c184ef0a867f7b735c1f39e9845ce691c6d6bb2679a3f81917d65e417dd32e80776a5db036afa11de056f35464fdf54db3caf55c82015803cce
+  data.tar.gz: 90fb24d6851765a01e014f1c838c51eed6d0190ce85265ab0344f481fded24ff9b632bdbc847263ddf2b20e80a81f61140cc452409785c52dcc000a105b4c8c4

data/docs/usage/rules.md

@@ -57,6 +57,8 @@ Chutney enforces its rules with the linters. These are:
 
 [RequiredTagsStartWith](https://github.com/BillyRuffian/chutney/blob/master/features/required_tags_starts_with.feature): Chutney can enforce a configurable naming prefix for your tags.
 
+[SameSecenario](https://github.com/BillyRuffian/chutney/blob/master/features/same_scenario.feature): Makes sure you don't acidentally duplicate the actions of your scenario. *Since v3.11.0*
+
 [SameTagForDifferentCase](https://github.com/BillyRuffian/chutney/blob/master/features/same_tag_different_case.feature): Don't mix the case of tags which have already appeared. Cucumber tags are case sensitive. *Since 3.1.0*
 
 [SameTagForAllScenarios](https://github.com/BillyRuffian/chutney/blob/master/features/same_tag_for_all_scenarios.feature): You have the same tag for all you scenarios; move the tag to the feature level instead.

data/lib/chutney/linter/same_scenario.rb

@@ -0,0 +1,52 @@
+module Chutney
+
+  class SameScenario < Linter
+    def lint
+      dictionary = Hash.new { |hash, key| hash[key] = [] }
+
+      scenarios do |feature, scenario|
+        text = scenario
+                 .steps
+                 .map { |step| "#{step.text} #{child_text(step)}" }
+                 .join("\n") + example_text(scenario)
+
+        digest = Digest::SHA2.hexdigest(text)
+        dictionary[digest] << { scenario:, feature: }
+      end
+
+      dictionary.filter { |k, v| v.size > 1 }
+                .each_value do |v|
+        v[1...].each { |problem|
+          add_issue(I18n.t('linters.same_scenario',
+                           feature: problem[:feature].name,
+                           scenario: problem[:scenario].name,
+                           original_feature: v.first[:feature].name,
+                           original_scenario: v.first[:scenario].name),
+                    problem[:feature], problem[:scenario], nil) }
+      end
+    end
+
+    def child_text(step)
+      step.children.map do |child|
+        if child.is_a?(CukeModeler::Table)
+          child.rows.flat_map(&:cells).map(&:value).join(' | ')
+        elsif child.is_a?(CukeModeler::DocString)
+          child.content
+        else
+          ''
+        end
+      end
+    end
+
+    def example_text(scenario)
+      if scenario.is_a?(CukeModeler::Outline)
+        "\n" + scenario.examples
+                       .flat_map(&:rows)
+                       .flat_map(&:cells)
+                       .map(&:value).join(' | ')
+      else
+        ''
+      end
+    end
+  end
+end

data/lib/chutney/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Chutney
-  VERSION = '3.10.0'
+  VERSION = '3.11.0'
 end

data/lib/chutney.rb

@@ -30,6 +30,7 @@ require 'chutney/linter/missing_scenario_outline'
 require 'chutney/linter/missing_test_action'
 require 'chutney/linter/missing_verification'
 require 'chutney/linter/required_tags_starts_with'
+require 'chutney/linter/same_scenario'
 require 'chutney/linter/same_tag_different_case'
 require 'chutney/linter/same_tag_for_all_scenarios'
 require 'chutney/linter/scenario_names_match'

data/lib/config/locales/en.yml

@@ -78,6 +78,8 @@ en:
       the assertion being tested.
     required_tags_starts_with: >-
       You have tags which do not match the required format. Allowed prefixes are '%{allowed}'.
+    same_scenario: >-
+      Duplicate scenario detected: %{feature} / %{scenario} is a duplicate of %{original_feature} / %{original_scenario}
     scenario_names_match: >-
       You have a scenario name which does not match the required format.
       Allowed patterns are '%{pattern}'.

data/usechutney.com/_data/navigation.yml

@@ -81,6 +81,8 @@ sidebar:
         url: /docs/rules/missing-test-verification/
       - title: "Required tags start with"
         url: /docs/rules/required-tags-start-with/
+      - title: "Same Scenario"
+        url:  /docs/rules/same-scenario/
       - title: "Same tag different case"
         url: /docs/rules/same-tag-different-case/
       - title: "Same tag for all scenarios"

data/usechutney.com/docs/rules/same-scenario/index.md

@@ -0,0 +1,24 @@
+---
+title: "Same Scenario"
+layout: single
+permalink: /docs/rules/same-scenario/
+---
+
+You have a `Scenario`, or a `Scenario Outline` which appears to be duplicated. This probably means you have a redundant test. You should probably make sure you only have one copy of your steps to be consitent and efficient.
+
+
+This rule ignores the names of your scenarios but takes into account data tables, examples and doc strings.
+
+## Examples
+
+```gherkin
+Feature: Test
+  Scenario: Scenario A
+      When I do a thing
+      Then A thing happens
+
+  Scenario: Scenario B
+      When I do a thing
+      Then A thing happens
+```
+

metadata

@@ -1,17 +1,16 @@
 --- !ruby/object:Gem::Specification
 name: chutney
 version: !ruby/object:Gem::Version
-  version: 3.10.0
+  version: 3.11.0
 platform: ruby
 authors:
 - Nigel Brookes-Thomas
 - Stefan Rohe
 - Nishtha Argawal
 - John Gluck
-autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-11-27 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: amatch
@@ -181,6 +180,7 @@ files:
 - lib/chutney/linter/missing_test_action.rb
 - lib/chutney/linter/missing_verification.rb
 - lib/chutney/linter/required_tags_starts_with.rb
+- lib/chutney/linter/same_scenario.rb
 - lib/chutney/linter/same_tag_different_case.rb
 - lib/chutney/linter/same_tag_for_all_scenarios.rb
 - lib/chutney/linter/scenario_names_match.rb
@@ -241,6 +241,7 @@ files:
 - usechutney.com/docs/rules/missing-test-action/index.md
 - usechutney.com/docs/rules/missing-test-verification/index.md
 - usechutney.com/docs/rules/required-tag-starts-with/index.md
+- usechutney.com/docs/rules/same-scenario/index.md
 - usechutney.com/docs/rules/same-tag-different-case/index.md
 - usechutney.com/docs/rules/same-tag-for-all-scenarios/index.md
 - usechutney.com/docs/rules/scenario-names-match/index.md
@@ -266,7 +267,6 @@ metadata:
   homepage_uri: https://www.usechutney.com/
   source_code_uri: https://github.com/BillyRuffian/chutney
   changelog_uri: https://github.com/BillyRuffian/chutney/releases
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -281,8 +281,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.23
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: A linter for multi-lingual Gherkin
 test_files: []