chutney 3.5.0 → 3.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 72a8913c71a8e2291fa14ef87b527e11481dfe45519370455fbddf2df8856836
-  data.tar.gz: cd4fded7ea629dd5ae5e5b74ac204c8b5d3dfee96e21f65db3c3355449dbb0c1
+  metadata.gz: 0badc9f82efb1e59bc755597c4ef3f7a4f973e37fec2fc25d4b93192319a75eb
+  data.tar.gz: 9459b41ac94999efc721d07f54e8e14b3c83c4f00786f003937a65dd2054fb9f
 SHA512:
-  metadata.gz: 2387b637bdb751865c8f6fbc97f61e90e25f7d6dc52f6da7dae598f5d156c06226f985df609b3a58597213f7e9b6e5a87be2fd5c67c78b57ae60c16603cde040
-  data.tar.gz: b7d67216485f10777e91250582facc4143e0f463360463f87aa8bf219e16b91e8e000a4e8ae94f442130f5246d0adf61e4df0b50b22bea1a05743145ea77e346
+  metadata.gz: 404c7bd04be47d2c2fb6c71e9d44bc317b5d0a9abe1a3362945ce2841a76f9cc25eb7703cc39134c2065dfcc8c83d5f615f24e035160cd10aab09178e8f8d6fe
+  data.tar.gz: 29150aae2c605d2ce489dd3a41b914ad7bc67dc53867122c7c11b67830d5fe82771a433d0b6250c926d0e7bd34c0fa8af53941c734e32d5d8db3f435153f8c0f

data/README.md

@@ -19,7 +19,7 @@
 
 </div>
 
-Read the documentation [here](https://billyruffian.github.io/chutney/) or try [Chutney Online](https://chutney.billy-ruffian.co.uk).
+Read the documentation [here](https://billyruffian.github.io/chutney/).
 
 See [this page](https://billyruffian.github.io/chutney/usage/rules.html) for a full list of the rules chutney encourages.
 

data/chutney.gemspec

@@ -50,7 +50,7 @@ Gem::Specification.new do |spec|
   
   spec.add_runtime_dependency 'amatch', '~> 0.4.0'
   spec.add_runtime_dependency 'cuke_modeler', '~> 3.3'
-  spec.add_runtime_dependency 'i18n', '>= 1.8.2', '< 1.13.0'
+  spec.add_runtime_dependency 'i18n', '>= 1.8.2', '< 1.15.0'
   spec.add_runtime_dependency 'pastel', '~> 0.7'
   spec.add_runtime_dependency 'tty-pie', '~> 0.3'
 
@@ -61,7 +61,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rake', '~> 13.0'
   spec.add_development_dependency 'rerun', '~> 0.13'
   spec.add_development_dependency 'rspec-expectations', '~> 3.0'
-  spec.add_development_dependency 'rubocop', '~> 1.45.1'
+  spec.add_development_dependency 'rubocop', '~> 1.50.2'
   spec.add_development_dependency 'rspec', '~> 3.8'
   
   spec.required_ruby_version = '>= 2.6'

data/config/chutney_defaults.yml

@@ -38,6 +38,8 @@ MissingTestAction:
     Enabled: true
 MissingVerification:
     Enabled: true
+InconsistentQuoting:
+    Enabled: true
 InvalidFileName:
     Enabled: true
 InvalidStepFlow:

data/docs/usage/rules.md

@@ -33,6 +33,8 @@ Chutney enforces its rules with the linters. These are:
 
 [GivensAfterBackground](https://github.com/BillyRuffian/chutney/blob/master/features/givens_after_background.feature): If you have a Background section and your scenario needs more preconditions then it should start immediately with an `And` step and not another `Given`.
 
+[InconsistentQuoting](https://github.com/BillyRuffian/chutney/blob/master/features/inconsistent_quoting.feature): Use either single quotation marks *or* double quotation marks around parameters but don't use a mix of the two styles.
+
 [InvalidFileName](https://github.com/BillyRuffian/chutney/blob/master/features/invalid_file_name.feature): Make sure your file name is in snake case, not mixed case or with spaces.
 
 [InvalidStepFlow](https://github.com/BillyRuffian/chutney/blob/master/features/invalid_step_flow.feature): Your scenarios should follow Given → When → Then, in that order.

data/lib/chutney/linter/inconsistent_quoting.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+module Chutney
+  # service class to lint for avoiding periods
+  class InconsistentQuoting < Linter
+    # regular expression to extract quoted string
+    # matching group 1: opening quote; 2: quoted text; 3: closing quote
+    # opening and closing quote must match (via backrefs)
+    # apostrophes, both singular and plural posessives, are accounted for
+    QUOTED_STRING = /(?!\b\b)(['"])(.*(?:\b'\b[^\1]*)*(?!\b[\1]\b))(\1)/.freeze
+    Parameter = Struct.new('Parameter', :quotation_mark, :name)
+
+    def lint
+      quoted_params = parameters.group_by(&:quotation_mark)
+      single_quoted = quoted_params[%(')] || []
+      double_quoted = quoted_params[%(")] || []
+      return unless single_quoted.count.positive? && double_quoted.count.positive?
+
+      add_issue(
+        I18n.t('linters.inconsistent_quoting',
+               count_single: single_quoted.count, count_double: double_quoted.count,
+               example_single: %('#{single_quoted.first.name}'), example_double: %("#{double_quoted.first.name}")),
+        feature
+      )
+    end
+
+    def parameters
+      parameters = []
+
+      steps do |_feature, _child, step|
+        step_parameters = step
+                          .text
+                          .scan(QUOTED_STRING)
+                          .map { |p| p.take(2) } # close quote will match open quote: drop it
+                          .map { |p| Parameter.new(*p) }
+        parameters.concat(step_parameters)
+      end
+
+      parameters
+    end
+  end
+end

data/lib/chutney/version.rb

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

data/lib/chutney.rb

@@ -16,6 +16,7 @@ require 'chutney/linter/bad_scenario_name'
 require 'chutney/linter/empty_feature_file'
 require 'chutney/linter/file_name_differs_feature_name'
 require 'chutney/linter/givens_after_background'
+require 'chutney/linter/inconsistent_quoting'
 require 'chutney/linter/invalid_file_name'
 require 'chutney/linter/invalid_step_flow'
 require 'chutney/linter/missing_example_name'

data/lib/config/locales/en.yml

@@ -33,6 +33,12 @@ en:
     givens_after_background: >-
       Avoid using the 'Given' keyword in scenarios if you have a background.
       Instead, setup steps begin your scenario and should start with an 'And' or 'But' keyword.
+    inconsistent_quoting: >-
+      Avoid using inconsistent quoting for your parameters.
+      You have %{count_single} single-quoted parameters (e.g. %{example_single})
+      and %{count_double} double-quoted parameters (e.g. %{example_double}).
+      You should adopt a single style to avoid confusion about the meaning and purpose
+      of the parameters.
     invalid_file_name: >-
       Filenames of feature files should be in snake case.
       You should name this file '%{recommended_name}'.

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: chutney
 version: !ruby/object:Gem::Version
-  version: 3.5.0
+  version: 3.6.0
 platform: ruby
 authors:
 - Nigel Brookes-Thomas
@@ -11,7 +11,7 @@ authors:
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-02-22 00:00:00.000000000 Z
+date: 2023-07-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: amatch
@@ -50,7 +50,7 @@ dependencies:
         version: 1.8.2
     - - "<"
       - !ruby/object:Gem::Version
-        version: 1.13.0
+        version: 1.15.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -60,7 +60,7 @@ dependencies:
         version: 1.8.2
     - - "<"
       - !ruby/object:Gem::Version
-        version: 1.13.0
+        version: 1.15.0
 - !ruby/object:Gem::Dependency
   name: pastel
   requirement: !ruby/object:Gem::Requirement
@@ -179,14 +179,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.45.1
+        version: 1.50.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.45.1
+        version: 1.50.2
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -262,6 +262,7 @@ files:
 - lib/chutney/linter/empty_feature_file.rb
 - lib/chutney/linter/file_name_differs_feature_name.rb
 - lib/chutney/linter/givens_after_background.rb
+- lib/chutney/linter/inconsistent_quoting.rb
 - lib/chutney/linter/invalid_file_name.rb
 - lib/chutney/linter/invalid_step_flow.rb
 - lib/chutney/linter/missing_example_name.rb
@@ -314,7 +315,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.1
+rubygems_version: 3.4.10
 signing_key:
 specification_version: 4
 summary: A linter for multi-lingual Gherkin