chutney 3.0.0.beta.2 → 3.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c99f32f7cba016b8d419d1867c5005b448e05939565afff74db238c68222f2d9
-  data.tar.gz: 9f21a905656a69d4bbfc7288b7abcec37e36e3fb5950c852aa8c6e4183e52cd9
+  metadata.gz: a489eb526fae6a79ff386986463cf473aa4f5bc353448e99338781556f7fd194
+  data.tar.gz: 5cd7a3264aba055c8fe738147edfa661f57e2c0e258d5991b42adbfb2af45f05
 SHA512:
-  metadata.gz: 3a4339e31214900e809bdbdaf968a2d21af9bc7526365cd2f00a17745b10272947114c018e6b182c6679769636ffc127d6880d69dbbbb8dc66f0d5f5762a410d
-  data.tar.gz: b755781258f1781e4586a77f639682b7ebb0a584d50aa0c001a7eca5c5e8b0c16487b2d1dd1978e97744c5f7ec163ea169ae8d9c6114e1ed22cda716e3173a34
+  metadata.gz: 9fb8325c116764fadd900b0b98961eca7cf95f49ac171ef64e77e29f1806d6ff78bcca90ea9e2119320d33d81b13e67749fb3bf8e916df13e154871b54602b35
+  data.tar.gz: 8290dd430b9b55c900179717c3b3fcd83ea805e9142751dec8c69ae4c1e2233a3d5392d8951705a3fed0678b3a9fd768941889b5e9086465fa54ced8c43b1811

data/.rubocop.yml

@@ -52,11 +52,11 @@ Layout/HeredocIndentation:
     - "**/*_spec.rb"
 
 Layout/TrailingWhitespace:
-  Enabled: false
+  Enabled: true
 
 Style/FrozenStringLiteralComment:
   Enabled: true
-  
+
 Style/StringConcatenation:
   Enabled: false
 

data/README.md

@@ -19,7 +19,7 @@
 
 </div>
 
-Read the documentation [here](https://billyruffian.github.io/chutney/).
+Read the documentation [here](https://billyruffian.github.io/chutney/) or try [Chutney Online](https://chutney.billy-ruffian.co.uk).
 
 ## Notes
 

data/chutney.gemspec

@@ -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', '~> 0.89.0'
+  spec.add_development_dependency 'rubocop', '~> 0.90.0'
   spec.add_development_dependency 'rspec', '~> 3.8'
   
   spec.required_ruby_version = '~> 2.6'

data/exe/chutney

@@ -12,14 +12,14 @@ formatters = Set.new
 
 OptionParser.new do |opts|
   opts.banner = 'Usage: chutney [files]'
-  opts.on('-f', 
-          '--format [formatter]', 
+  opts.on('-f',
+          '--format [formatter]',
           'One of JSONFormatter, PieFormatter or RainbowFormatter (default).') do |formatter|
     raise 'No Such Formatter' unless %w[JSONFormatter PieFormatter RainbowFormatter].include? formatter
 
     formatters << formatter
   end
-  
+
   opts.on('-l',
           '--linters',
           'List the linter status by this configuration and exit') do
@@ -28,7 +28,7 @@ OptionParser.new do |opts|
     max_name_length = chutney_config.keys.map(&:length).max + 1
     chutney_config.each do |linter, value|
       print pastel.cyan(linter.ljust(max_name_length))
-      
+
       if value['Enabled']
         puts pastel.green('enabled')
       else
@@ -42,7 +42,7 @@ end.parse!
 formatters << 'RainbowFormatter' if formatters.empty?
 
 files = ARGV.map { |pattern| Dir.glob(pattern) }.flatten
-files = Dir.glob('features/**/*.feature') if ARGV.empty? 
+files = Dir.glob('features/**/*.feature') if ARGV.empty?
 
 linter = Chutney::ChutneyLint.new(*files)
 report = linter.analyse

data/lib/chutney.rb

@@ -36,6 +36,7 @@ require 'chutney/linter/unknown_variable'
 require 'chutney/linter/unused_variable'
 require 'chutney/linter/use_background'
 require 'chutney/linter/use_outline'
+require 'chutney/version'
 
 require 'cuke_modeler'
 require 'forwardable'
@@ -58,14 +59,18 @@ module Chutney
       @files = files
       @results = Hash.new { |h, k| h[k] = [] }
       i18n_paths = Dir[File.expand_path(File.join(__dir__, 'config/locales')) + '/*.yml']
-      return if I18n.load_path.include?(i18n_paths)
 
-      I18n.load_path += i18n_paths
+      i18n_paths.each do |path|
+        next if I18n.load_path.include?(path)
+
+        I18n.load_path << path
+        I18n.backend.reload!
+      end
     end
 
     def configuration
       unless @config
-        default_file = [File.expand_path('..', __dir__), '**/config', 'chutney.yml']
+        default_file = [File.expand_path('..', __dir__), '**/config', 'chutney_defaults.yml']
         config_file = Dir.glob(File.join(default_file)).first.freeze
         @config = Configuration.new(config_file)
       end

data/lib/chutney/configuration.rb

@@ -1,7 +1,8 @@
 # frozen_string_literal: true
+
 require 'delegate'
 
-module Chutney 
+module Chutney
   # gherkin_lint configuration object
   class Configuration < SimpleDelegator
     def initialize(path)
@@ -23,7 +24,7 @@ module Chutney
       config_files = ['chutney.yml', '.chutney.yml'].map do |fname|
         Dir.glob(File.join(Dir.pwd, '**', fname))
       end.flatten
-      
+
       config_file = config_files.first
       merge_config(config_file) if !config_file.nil? && File.exist?(config_file)
     end

data/lib/chutney/formatter.rb

@@ -4,18 +4,17 @@ module Chutney
   # base class for all formatters
   class Formatter
     attr_accessor :results
-    
+
     def initialize
       @results = {}
-    end  
-        
+    end
+
     def files
       results.map { |k, _v| k }
     end
-    
+
     def files_with_issues
       results.filter { |_k, v| v.any? { |r| r[:issues].count.positive? } }
     end
-    
   end
 end

data/lib/chutney/formatter/pie_formatter.rb

@@ -9,7 +9,7 @@ module Chutney
     def initialize
       super
     end
-    
+
     def format
       data = top_offences.map do |offence|
         {
@@ -21,14 +21,14 @@ module Chutney
       end
       print_report(data)
     end
-    
+
     def print_report(data)
       return if data.empty?
 
       print TTY::Pie.new(data: data, radius: 8, legend: { format: '%<label>s %<name>s %<value>i' })
       puts
     end
-    
+
     def top_offences
       offence = Hash.new(0)
       files_with_issues.each do |_file, linter|
@@ -38,9 +38,9 @@ module Chutney
       end
       offence.reject { |_k, v| v.zero? }.sort_by { |_linter, count| -count }
     end
-    
+
     def char_loop
-      @char_looper ||= Fiber.new do 
+      @char_looper ||= Fiber.new do
         chars = %w[• x + @ * / -]
         current = 0
         loop do
@@ -51,10 +51,10 @@ module Chutney
       end
       @char_looper.resume
     end
-    
+
     def colour_loop
-      @colour_looper ||= Fiber.new do 
-        colours = %i[bright_cyan bright_magenta bright_yellow bright_green] 
+      @colour_looper ||= Fiber.new do
+        colours = %i[bright_cyan bright_magenta bright_yellow bright_green]
         current = 0
         loop do
           current = 0 if current >= colours.count
@@ -64,7 +64,7 @@ module Chutney
       end
       @colour_looper.resume
     end
-    
+
     def put_summary
       pastel = Pastel.new
       print "#{files.count} features inspected, "
@@ -74,6 +74,5 @@ module Chutney
         puts pastel.red("#{files_with_issues.count} taste nasty")
       end
     end
-    
   end
 end

data/lib/chutney/formatter/rainbow_formatter.rb

@@ -5,38 +5,36 @@ require 'pastel'
 module Chutney
   # pretty formatter
   class RainbowFormatter < Formatter
-  
     def initialize
       super
-      
+
       @pastel = Pastel.new
     end
-  
-    def format      
+
+    def format
       files_with_issues.each do |file, linter|
         put_file(file)
         linter.filter { |l| !l[:issues].empty? }.each do |linter_with_issues|
-          
           put_linter(linter_with_issues)
           linter_with_issues[:issues].each { |i| put_issue(file, i) }
         end
       end
       put_summary
     end
-    
+
     def put_file(file)
       puts @pastel.cyan(file.to_s)
     end
-    
+
     def put_linter(linter)
       puts @pastel.red("  #{linter[:linter]}")
     end
-    
-    def put_issue(file, issue)    
+
+    def put_issue(file, issue)
       puts "    #{issue[:message]}"
       puts "    #{@pastel.dim file.to_s}:#{@pastel.dim(issue.dig(:location, :line))}"
     end
-    
+
     def put_summary
       print "#{files.count} features inspected, "
       if files_with_issues.count.zero?
@@ -45,6 +43,5 @@ module Chutney
         puts @pastel.red("#{files_with_issues.count} taste nasty")
       end
     end
-    
   end
 end

data/lib/chutney/linter.rb

@@ -7,7 +7,7 @@ module Chutney
   class Linter
     attr_accessor :issues
     attr_reader :filename, :configuration
-  
+
     Lint = Struct.new(:message, :gherkin_type, :location, :feature, :scenario, :step, keyword_init: true)
 
     def self.descendants
@@ -26,56 +26,56 @@ module Chutney
     def lint
       raise 'not implemented'
     end
-    
+
     def and_word?(word)
       dialect_word(:and).include?(word)
     end
-    
+
     def background_word?(word)
       dialect_word(:background).include?(word)
     end
-    
+
     def but_word?(word)
       dialect_word(:but).include?(word)
     end
-    
+
     def examples_word?(word)
       dialect_word(:examples).include?(word)
     end
-    
+
     def feature_word?(word)
       dialect_word(:feature).include?(word)
     end
-    
+
     def given_word?(word)
       dialect_word(:given).include?(word)
     end
-    
+
     def scenario_outline_word?(word)
       dialect_word(:scenarioOutline).include?(word)
     end
-    
+
     def then_word?(word)
       dialect_word(:then).include?(word)
     end
-    
+
     def when_word?(word)
       dialect_word(:when).include?(word)
     end
-    
+
     def dialect_word(word)
       CukeModeler::Parsing.dialects[dialect][word.to_s].map(&:strip)
     end
-    
+
     def dialect
       @content.feature&.parsing_data&.dig(:language) || 'en'
     end
-  
+
     def tags_for(element)
       element.tags.map { |tag| tag.name[1..-1] }
     end
-    
-    def add_issue(message, feature = nil, scenario = nil, item = nil)      
+
+    def add_issue(message, feature = nil, scenario = nil, item = nil)
       issues << Lint.new(
         message: message,
         gherkin_type: type(feature, scenario, item),
@@ -85,17 +85,17 @@ module Chutney
         step: item&.parsing_data&.dig(:name)
       ).to_h
     end
-    
+
     def location(feature, scenario, step)
       if step
         step.parsing_data[:location]
       elsif scenario
         scenario.parsing_data.dig(:scenario, :location) || scenario.parsing_data.dig(:background, :location)
-      else 
-        feature ? feature.parsing_data[:location] : 0
+      else
+        feature ? feature.parsing_data[:location] : { line: 0, column: 0 }
       end
     end
-    
+
     def type(_feature, scenario, step)
       if step
         :step
@@ -111,14 +111,14 @@ module Chutney
         @content.feature
       end
     end
-    
-    def elements 
+
+    def elements
       return [] unless feature
-      
-      if block_given? 
+
+      if block_given?
         feature.children.each do |child|
           next if off_switch?(child)
-        
+
           yield(feature, child)
         end
       else
@@ -136,38 +136,38 @@ module Chutney
       off_switch ||= off_switch?(feature) unless element == feature
       off_switch
     end
-    
+
     def background
-      if block_given?          
+      if block_given?
         yield(feature, feature&.background)
       else
         feature&.background
       end
     end
-    
+
     def scenarios
       if block_given?
         feature&.tests&.each do |test|
           yield(feature, test)
         end
-        
+
       else
         feature&.tests
       end
     end
-    
+
     def filled_scenarios
       if block_given?
         scenarios do |feature, scenario|
           next if scenario.steps.empty?
-        
+
           yield(feature, scenario)
         end
       else
         scenarios ? scenarios.filter { |s| !s.steps.empty? } : []
       end
     end
-    
+
     def steps
       feature&.tests&.each do |t|
         t.steps.each { |s| yield(feature, t, s) }
@@ -177,7 +177,7 @@ module Chutney
     def self.linter_name
       name.split('::').last
     end
-    
+
     def linter_name
       self.class.linter_name
     end
@@ -187,10 +187,10 @@ module Chutney
       value += render_step_argument(step.block) if step.block
       value
     end
-    
+
     def render_step_argument(argument)
       return "\n#{argument.content}" if argument.is_a?(CukeModeler::DocString)
-      
+
       result = argument.rows.map do |row|
         "|#{row.cells.map(&:value).join '|'}|"
       end.join "\n"

data/lib/chutney/linter/avoid_full_stop.rb

@@ -2,12 +2,10 @@
 
 module Chutney
   # service class to lint for avoiding periods
-  class AvoidFullStop < Linter    
+  class AvoidFullStop < Linter
     def lint
       steps do |feature, child, step|
-        
         add_issue(I18n.t('linters.avoid_full_stop'), feature, child, step) if step.text.strip.end_with? '.'
-        
       end
     end
   end

data/lib/chutney/linter/avoid_outline_for_single_example.rb

@@ -4,13 +4,13 @@ module Chutney
   # service class to lint for avoiding outline for single example
   class AvoidOutlineForSingleExample < Linter
     def lint
-      scenarios do |feature, scenario|      
+      scenarios do |feature, scenario|
         next unless scenario.is_a? CukeModeler::Outline
         next unless scenario.examples
-        
+
         next if scenario.examples.length > 1
         next if scenario.examples.first.rows.length > 2 # first row is the header
-        
+
         add_issue(I18n.t('linters.avoid_outline_for_single_example'), feature, scenario)
       end
     end

data/lib/chutney/linter/avoid_scripting.rb

@@ -2,7 +2,7 @@
 
 module Chutney
   # service class to lint for avoid scripting
-  class AvoidScripting < Linter  
+  class AvoidScripting < Linter
     def lint
       scenarios do |feature, scenario|
         when_steps = filter_when_steps(scenario.steps)
@@ -10,7 +10,7 @@ module Chutney
         add_issue(I18n.t('linters.avoid_scripting', count: whens), feature, scenario, when_steps.last) if whens > 1
       end
     end
-    
+
     def filter_when_steps(steps)
       steps
         .drop_while { |step| !when_word?(step.keyword) }

data/lib/chutney/linter/background_does_more_than_setup.rb

@@ -2,14 +2,13 @@
 
 module Chutney
   # service class to lint for background that does more than setup
-  class BackgroundDoesMoreThanSetup < Linter   
+  class BackgroundDoesMoreThanSetup < Linter
     def lint
       background do |feature, background|
-      
-        invalid_steps = background&.steps&.select do |step| 
+        invalid_steps = background&.steps&.select do |step|
           when_word?(step.keyword) || then_word?(step.keyword)
         end
-        
+
         next if invalid_steps.nil? || invalid_steps.empty?
 
         add_issue(I18n.t('linters.background_does_more_than_setup'), feature, background, invalid_steps.first)

data/lib/chutney/linter/background_requires_multiple_scenarios.rb

@@ -6,7 +6,7 @@ module Chutney
   # service class for check that there are multiple scenarios once a background is used
   class BackgroundRequiresMultipleScenarios < Linter
     MESSAGE = 'Avoid using Background steps for just one scenario'
-  
+
     def lint
       background do |feature, background|
         next unless background

data/lib/chutney/linter/bad_scenario_name.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
 module Chutney
-  # service class to lint for bad scenario names  
+  # service class to lint for bad scenario names
   class BadScenarioName < Linter
     def lint
       scenarios do |feature, scenario|
         next if scenario.name.empty?
-        
+
         bad = /\w*(test|verif|check)\w*/i
         match = scenario.name.match(bad).to_a.first
         add_issue(I18n.t('linters.bad_scenario_name', word: match), feature, scenario) if match

data/lib/chutney/linter/file_name_differs_feature_name.rb

@@ -5,11 +5,11 @@ module Chutney
   class FileNameDiffersFeatureName < Linter
     def lint
       return unless feature
-        
+
       expected_feature_name = title_case(filename)
       return if ignore_whitespaces(feature.name).casecmp(ignore_whitespaces(expected_feature_name)) == 0
-        
-      add_issue(I18n.t('linters.file_name_differs_feature_name', expected: expected_feature_name), feature) 
+
+      add_issue(I18n.t('linters.file_name_differs_feature_name', expected: expected_feature_name), feature)
     end
 
     def title_case(value)

data/lib/chutney/linter/givens_after_background.rb

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
 module Chutney
-  # service class to lint for bad scenario names  
+  # service class to lint for bad scenario names
   class GivensAfterBackground < Linter
     def lint
       return unless background
-      
+
       filled_scenarios do |feature, scenario|
         scenario.steps.each do |step|
           add_issue(I18n.t('linters.givens_after_background'), feature, scenario, step) if given_word?(step.keyword)

data/lib/chutney/linter/invalid_file_name.rb

@@ -13,7 +13,7 @@ module Chutney
         end
       end
     end
-    
+
     def recommend(filename)
       File.basename(filename, '.*').gsub(/::/, '/')
           .gsub(/([A-Z]+)([A-Z][a-z])/, '\1_\2')

data/lib/chutney/linter/invalid_step_flow.rb

@@ -7,7 +7,7 @@ module Chutney
       filled_scenarios do |feature, scenario|
         steps = scenario.steps.select { |step| !and_word?(step.keyword) && !but_word?(step.keyword) }
         next if steps.empty?
-        
+
         last_step_is_an_action(feature, scenario, steps)
         given_after_non_given(feature, scenario, steps)
         verification_before_action(feature, scenario, steps)
@@ -33,7 +33,7 @@ module Chutney
     def verification_before_action(feature, scenario, steps)
       steps.each do |step|
         break if when_word?(step.keyword)
-        
+
         add_issue(I18n.t('linters.invalid_step_flow.missing_action'), feature, scenario) if then_word?(step.keyword)
       end
     end

data/lib/chutney/linter/missing_example_name.rb

@@ -3,7 +3,6 @@
 module Chutney
   # service class to lint for missing example names
   class MissingExampleName < Linter
-  
     def lint
       scenarios do |_feature, scenario|
         next unless scenario.is_a? CukeModeler::Outline
@@ -11,17 +10,16 @@ module Chutney
         scenario.examples.each do |example|
           example_count = scenario.examples&.length || 0
           next unless example_count > 1
-          
+
           check_example(scenario, example)
         end
       end
-    end               
+    end
 
     def check_example(scenario, example)
       name = example.name.strip
       duplicate_name_count = scenario.examples.filter { |e| e.name == name }.count
       add_issue(I18n.t('linters.missing_example_name'), feature, scenario, example) if duplicate_name_count >= 2
     end
-    
   end
 end

data/lib/chutney/linter/missing_feature_description.rb

@@ -6,7 +6,7 @@ module Chutney
     MESSAGE = 'Features should have a description so that its purpose is clear'
     def lint
       return unless feature
-      
+
       add_issue(I18n.t('linters.missing_feature_description'), feature) if feature.description.empty?
     end
   end

data/lib/chutney/linter/missing_feature_name.rb

@@ -2,10 +2,10 @@
 
 module Chutney
   # service class to lint for missing feature names
-  class MissingFeatureName < Linter    
+  class MissingFeatureName < Linter
     def lint
       return unless feature
-      
+
       add_issue(I18n.t('linters.missing_feature_name'), feature) if feature.name.empty?
     end
   end

data/lib/chutney/linter/missing_scenario_name.rb

@@ -2,8 +2,7 @@
 
 module Chutney
   # service class to lint for missing scenario names
-  class MissingScenarioName < Linter  
-  
+  class MissingScenarioName < Linter
     def lint
       scenarios do |feature, scenario|
         add_issue(I18n.t('linters.missing_scenario_name'), feature, scenario) if scenario.name.empty?

data/lib/chutney/linter/missing_test_action.rb

@@ -7,7 +7,7 @@ module Chutney
       filled_scenarios do |feature, scenario|
         when_steps = scenario.steps.select { |step| when_word?(step.keyword) }
         next unless when_steps.empty?
-        
+
         add_issue(I18n.t('linters.missing_test_action'), feature, scenario)
       end
     end

data/lib/chutney/linter/missing_verification.rb

@@ -9,7 +9,7 @@ module Chutney
       filled_scenarios do |feature, scenario|
         then_steps = scenario.steps.select { |step| then_word?(step.keyword) }
         next unless then_steps.empty?
-        
+
         add_issue(I18n.t('linters.missing_test_verification'), feature, scenario)
       end
     end

data/lib/chutney/linter/required_tags_starts_with.rb

@@ -3,26 +3,25 @@
 module Chutney
   # service class to lint for tags used multiple times
   class RequiredTagsStartsWith < Linter
-        
     def lint
       return unless pattern
 
       scenarios do |feature, scenario|
         next if match_pattern? tags_for(feature)
         next if match_pattern? tags_for(scenario)
-        
+
         add_issue(
-          I18n.t('linters.required_tags_starts_with', 
-                 allowed: pattern.join(', ')), 
+          I18n.t('linters.required_tags_starts_with',
+                 allowed: pattern.join(', ')),
           feature, scenario
         )
       end
     end
-    
+
     def pattern
       configuration['Matcher'] || nil
     end
-  
+
     def match_pattern?(target)
       target.each do |t|
         return true if t.start_with?(*pattern)

data/lib/chutney/linter/same_tag_for_all_scenarios.rb

@@ -14,13 +14,13 @@ module Chutney
       tags = scenario_tags
       return if tags.nil? || tags.empty?
       return unless feature.tests.length > 1
-            
+
       tags.each do |tag|
         next if tag == 'skip'
 
         add_issue(
-          I18n.t('linters.same_tag_for_all_scenarios.feature_level', 
-                 tag: tag), 
+          I18n.t('linters.same_tag_for_all_scenarios.feature_level',
+                 tag: tag),
           feature
         )
       end
@@ -32,12 +32,12 @@ module Chutney
         next if tags.nil? || tags.empty?
         next unless scenario.is_a? CukeModeler::Outline
         next unless scenario.examples.length > 1
-        
+
         tags.each do |tag|
           next if tag == 'skip'
 
-          add_issue(I18n.t('linters.same_tag_for_all_scenarios.example_level', 
-                           tag: tag), feature, scenario)        
+          add_issue(I18n.t('linters.same_tag_for_all_scenarios.example_level',
+                           tag: tag), feature, scenario)
         end
       end
     end
@@ -55,13 +55,13 @@ module Chutney
     def example_tags(scenario)
       result = nil
       return result unless scenario.is_a?(CukeModeler::Outline) && scenario.examples
-      
+
       scenario.examples.each do |example|
         return nil unless example.tags
-        
+
         tags = tags_for(example)
         result = tags if result.nil?
-        
+
         result &= tags
       end
       result

data/lib/chutney/linter/scenario_names_match.rb

@@ -6,14 +6,13 @@ module Chutney
   # service class to lint for tags used multiple times
   class ScenarioNamesMatch < Linter
     MESSAGE = 'Scenario Name does not match pattern'
-  
 
     def lint
       scenarios do |feature, scenario|
         next unless (scenario.name =~ /#{configuration['Matcher']}/).nil?
-        
+
         add_issue(
-          I18n.t('linters.scenario_names_match', 
+          I18n.t('linters.scenario_names_match',
                  pattern: configuration['Matcher']), feature, scenario
         )
       end

data/lib/chutney/linter/tag_used_multiple_times.rb

@@ -8,7 +8,7 @@ module Chutney
         total_tags = tags_for(feature) + tags_for(scenario)
         double_used_tags = total_tags.find_all { |a| total_tags.count(a) > 1 }.uniq!
         next if double_used_tags.nil?
-                
+
         add_issue(
           I18n.t('linters.tag_used_multiple_times', tags: double_used_tags.join(',')), feature
         )

data/lib/chutney/linter/too_clumsy.rb

@@ -9,7 +9,7 @@ module Chutney
       filled_scenarios do |feature, scenario|
         characters = scenario.steps.map { |step| step.text.length }.inject(0, :+)
         next if characters < 400
-        
+
         add_issue(
           I18n.t('linters.too_clumsy', length: characters), feature, scenario
         )

data/lib/chutney/linter/too_long_step.rb

@@ -6,14 +6,14 @@ module Chutney
     def lint
       steps do |feature, scenario, step|
         next if step.text.length <= maxlength
-        
+
         add_issue(
-          I18n.t('linters.too_long_step', length: step.text.length, max: maxlength), 
+          I18n.t('linters.too_long_step', length: step.text.length, max: maxlength),
           feature, scenario
         )
       end
     end
-    
+
     def maxlength
       configuration['MaxLength'] || '120'
     end

data/lib/chutney/linter/too_many_different_tags.rb

@@ -6,20 +6,20 @@ module Chutney
     def lint
       tags = all_tags
       return if tags.length <= maxcount
-      
+
       add_issue(
-        I18n.t('linters.too_many_different_tags', count: tags.length, max: maxcount), 
+        I18n.t('linters.too_many_different_tags', count: tags.length, max: maxcount),
         feature
       )
     end
-    
-    def maxcount 
+
+    def maxcount
       configuration['MaxCount']&.to_i || 3
     end
-    
+
     def all_tags
       return [] unless feature&.scenarios
-      
+
       tags_for(feature) + feature.scenarios.map { |scenario| tags_for(scenario) }.flatten
     end
   end

data/lib/chutney/linter/too_many_steps.rb

@@ -6,14 +6,14 @@ module Chutney
     def lint
       filled_scenarios do |feature, scenario|
         next if scenario.steps.length <= maxcount
-        
+
         add_issue(
-          I18n.t('linters.too_many_steps', count: scenario.steps.length, max: maxcount), 
+          I18n.t('linters.too_many_steps', count: scenario.steps.length, max: maxcount),
           feature
         )
       end
     end
-    
+
     def maxcount
       configuration['MaxCount']&.to_i || 10
     end

data/lib/chutney/linter/too_many_tags.rb

@@ -7,14 +7,14 @@ module Chutney
       scenarios do |feature, scenario|
         tags = tags_for(feature) + tags_for(scenario)
         next unless tags.length > maxcount
-        
+
         add_issue(
-          I18n.t('linters.too_many_tags', count: tags.length, max: maxcount), 
+          I18n.t('linters.too_many_tags', count: tags.length, max: maxcount),
           feature
         )
       end
     end
-    
+
     def maxcount
       configuration['MaxCount']&.to_i || 3
     end

data/lib/chutney/linter/unique_scenario_names.rb

@@ -14,13 +14,13 @@ module Chutney
         end
       end
     end
-    
+
     def issue(name, first_location, scenario)
       add_issue(
         I18n.t('linters.unique_scenario_names',
                name: name,
                line: first_location[:line],
-               column: first_location[:column]), 
+               column: first_location[:column]),
         feature, scenario
       )
     end

data/lib/chutney/linter/unknown_variable.rb

@@ -9,7 +9,7 @@ module Chutney
         scenario.steps.each do |step|
           step_vars(step).each do |used_var|
             next if known_vars.include? used_var
-            
+
             add_issue(
               I18n.t('linters.unknown_variable', variable: used_var), feature, scenario
             )
@@ -21,7 +21,7 @@ module Chutney
     def step_vars(step)
       vars = gather_vars step.text
       return vars unless step.block
-      
+
       vars + gather_vars_from_argument(step.block)
     end
 
@@ -39,7 +39,7 @@ module Chutney
 
     def known_variables(scenario)
       return [] unless scenario.is_a? CukeModeler::Outline
-    
+
       scenario.examples.map { |ex| ex.rows.first.cells.map(&:value) }.flatten
     end
   end

data/lib/chutney/linter/unused_variable.rb

@@ -6,9 +6,8 @@ module Chutney
     def lint
       scenarios do |feature, scenario|
         next unless scenario.is_a? CukeModeler::Outline
-      
+
         scenario.examples.each do |example|
-          
           example.rows.first.cells.map(&:value).each do |variable|
             next if used?(variable, scenario)
 
@@ -20,7 +19,7 @@ module Chutney
 
     def used?(variable, scenario)
       variable = "<#{variable}>"
-      
+
       scenario.steps.each do |step|
         return true if step.text.include? variable
         next unless step.block
@@ -36,7 +35,7 @@ module Chutney
 
     def used_in_table?(variable, step)
       return false unless step.block.is_a?(CukeModeler::Table)
-      
+
       step.block.rows.each do |row|
         row.cells.each { |cell| return true if cell.value.include?(variable) }
       end

data/lib/chutney/linter/use_background.rb

@@ -16,11 +16,11 @@ module Chutney
 
     def gather_givens
       return unless feature.children
-      
+
       has_non_given_step = false
       scenarios do |_feature, scenario|
         next unless scenario.steps
-        
+
         has_non_given_step = true unless given_word?(scenario.steps.first.keyword)
       end
       return if has_non_given_step
@@ -33,7 +33,7 @@ module Chutney
     def expanded_steps(&block)
       scenarios do |_feature, scenario|
         next unless scenario.steps
-        
+
         prototypes = [render_step(scenario.steps.first)]
         prototypes = expand_examples(scenario.examples, prototypes) if scenario.is_a? CukeModeler::Outline
         prototypes.each(&block)
@@ -52,7 +52,7 @@ module Chutney
       headers = example.rows.first.cells.map(&:value)
       example.rows.each_with_index do |row, idx|
         next if idx.zero? # skip the header
-        
+
         modified_sentence = sentence.dup
         headers.zip(row.cells.map(&:value)).map do |key, value|
           modified_sentence.gsub!("<#{key}>", value)

data/lib/chutney/linter/use_outline.rb

@@ -11,18 +11,18 @@ module Chutney
       scenarios.product(scenarios) do |lhs, rhs|
         next if lhs == rhs
         next if lhs[:reference][:line] > rhs[:reference][:line]
-        
+
         similarity = determine_similarity(lhs[:text], rhs[:text])
         next unless similarity >= 0.95
-        
+
         similarity_pct = similarity.round(3) * 100
-        
+
         add_issue(lhs, rhs, similarity_pct)
       end
     end
-    
+
     def add_issue(lhs, rhs, pct)
-      super(I18n.t('linters.use_outline', 
+      super(I18n.t('linters.use_outline',
                    pct: pct,
                    lhs_name: lhs[:name],
                    lhs_line: lhs[:reference][:line],
@@ -39,11 +39,11 @@ module Chutney
     def gather_scenarios(feature)
       scenarios = []
       return scenarios if feature.nil? || !feature.tests
-      
+
       scenarios do |_feature, scenario|
         next unless scenario.steps
         next if scenario.steps.empty?
-        
+
         scenarios.push generate_reference(feature, scenario)
       end
       scenarios

data/lib/chutney/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Chutney
-  VERSION = '3.0.0.beta.2'
+  VERSION = '3.0.0'
 end

data/spec/chutney_spec.rb

@@ -8,25 +8,25 @@ describe Chutney::ChutneyLint do
   it 'has a version number' do
     expect(Chutney::VERSION).not_to be nil
   end
-  
+
   describe '#initialize' do
     it 'creates an instance' do
       expect(subject).not_to be_nil
     end
-    
+
     it 'has an empty list of files if none are given' do
       expect(subject.files).to eq []
     end
-    
+
     it 'has a list of files given on initialization' do
       alt_subject = Chutney::ChutneyLint.new('a', 'b')
       expect(alt_subject.files).to eq %w[a b]
     end
-    
+
     it 'initializes a results hash' do
       expect(subject.results).to eq({})
     end
-    
+
     it 'sets the load path for I18n' do
       expect(I18n.load_path).not_to eq []
     end
@@ -36,25 +36,25 @@ describe Chutney::ChutneyLint do
         expect(subject.configuration).not_to be_nil
         expect(subject.configuration).to respond_to :[]
       end
-      
+
       it 'allows the configuration to be set explicitly' do
         config = { 'BackgroundDoesMoreThanSetup' => { 'Enabled' => true } }
         subject.configuration = config
         expect(subject.configuration).to be config
       end
-      
+
       it 'controls the available linters' do
         subject.configuration = {}
         expect(subject.linters).to be_empty
       end
-      
+
       it 'enables linters to be activated' do
         config = { 'BackgroundDoesMoreThanSetup' => { 'Enabled' => true } }
         subject.configuration = config
         expect(subject.linters).to eq [Chutney::BackgroundDoesMoreThanSetup]
       end
     end
-    
+
     context 'linting' do
       it 'aliases analyse and analyze' do
         expect(subject).to respond_to :analyse

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: chutney
 version: !ruby/object:Gem::Version
-  version: 3.0.0.beta.2
+  version: 3.0.0
 platform: ruby
 authors:
 - Nigel Brookes-Thomas
@@ -11,7 +11,7 @@ authors:
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-09-17 00:00:00.000000000 Z
+date: 2020-09-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: amatch
@@ -173,14 +173,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.89.0
+        version: 0.90.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.89.0
+        version: 0.90.0
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -217,7 +217,7 @@ files:
 - README.md
 - Rakefile
 - chutney.gemspec
-- config/chutney.yml
+- config/chutney_defaults.yml
 - config/cucumber.yml
 - docs/.keep
 - docs/_config.yml
@@ -297,9 +297,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '2.6'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubygems_version: 3.1.2
 signing_key: