chutney 2.1.1 → 3.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2da8d4d9f8b99fbedf8ed3b2407e5ec9024496258f86cd69fb76d35699b4ffa4
-  data.tar.gz: 1821ce09b96bc45203e1a65abac9d5c042bca5036933feda9c066c0ff852ede6
+  metadata.gz: da2d0aa4432eaa81b87b877c67dcada20758c521f193cfca6a91a83904ea209a
+  data.tar.gz: 1e2c22d44911f0816e0e28504c8a465105edf406848ea15305ef25af470c95d3
 SHA512:
-  metadata.gz: 3adad450052c2008c3a1db8f82e65893c39605592b72ce676e9b3973fd19f6f2812a949ac5fa375909277298c12115e94771b7dcee06b34dc09362f415af16da
-  data.tar.gz: ebb9e4d1ea6842131863666ea4362dfc3e94c50525b1444c75e5f8b662c3bd340481d4a8bb00b265991dec0546aab0017e3f86d3a76a850738fbf4407cc68d32
+  metadata.gz: f9f1821d7d97ab7fc2f67cdc6e33d4f5ff94d4a1b7e20017bfe71b3b784697e54c3493d91812412dcfda8b18f54a756a8232e36c53ac653a99b34d97eaaefa76
+  data.tar.gz: eebac54200ed820084c183ab48eeb538aacec5a93ff4eef09552fa4917ec65d76e0c12e266689ecb149c9b0ab0fd93d178a405822fa4010c904bb21a4ff9c4d1

data/.rubocop.yml

@@ -8,7 +8,7 @@
 
 # Offense count: 1
 Metrics/AbcSize:
-  Max: 16
+  Enabled: false
 
 # Offense count: 1
 # Configuration parameters: CountComments.
@@ -52,7 +52,13 @@ Layout/HeredocIndentation:
     - "**/*_spec.rb"
 
 Layout/TrailingWhitespace:
-  Enabled: false
+  Enabled: true
 
 Style/FrozenStringLiteralComment:
+  Enabled: true
+
+Style/StringConcatenation:
   Enabled: false
+
+AllCops:
+  NewCops: enable

data/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 gemspec

data/README.md

@@ -19,4 +19,8 @@
 
 </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
+
+Chutney 3+ (in beta) has replaced its direct dependency on Cucumber and instead uses the excellent [cuke_modeller](https://github.com/enkessler/cuke_modeler) to parse your feature files. 

data/Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rake/testtask'
 
 task default: :build

data/chutney.gemspec

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Disable rubocop checks for the .gemspec
 # I'll take the output from 'bundle gem new' to be authoritative
 # rubocop:disable all
@@ -12,9 +14,13 @@ Gem::Specification.new do |spec|
   spec.authors     = ['Nigel Brookes-Thomas', 'Stefan Rohe', 'Nishtha Argawal', 'John Gluck']
   spec.email       = ['nigel@brookes-thomas.co.uk']
 
-  spec.summary     = 'A linter for English language Gherkin'
-  spec.description = 'A linter for your Cucumber features. '  \
-                      'It supports any spoken language Cucumber v3 supports.'
+  spec.summary     = 'A linter for multi-lingual Gherkin'
+  spec.description = 'A linter for your Cucumber features. '                    \
+                     'Making sure you have nice, expressible Gherkin is '       \
+                     'essential is making sure you have a readable test-base. ' \
+                     'Chutney is designed to sniff out smells in your feature ' \
+                     'files. '                                                  \
+                     'It supports any spoken language Cucumber supports.'
 
   spec.homepage    = 'https://billyruffian.github.io/chutney/'
   spec.license     = 'MIT'
@@ -43,19 +49,20 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
   
   spec.add_runtime_dependency 'amatch', '~> 0.4.0'
-  spec.add_runtime_dependency 'gherkin', '~> 5.1.0'
+  spec.add_runtime_dependency 'cuke_modeler', '~> 3.3'
   spec.add_runtime_dependency 'i18n', '~> 1.8.2'
   spec.add_runtime_dependency 'pastel', '~> 0.7'
   spec.add_runtime_dependency 'tty-pie', '~> 0.3'
 
 
   spec.add_development_dependency 'coveralls', '~> 0.8'
-  spec.add_development_dependency 'cucumber', '~> 3.0'
+  spec.add_development_dependency 'cucumber', '~> 5.1'
   spec.add_development_dependency 'pry-byebug', '~> 3.0'
   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'
 end

data/config/{chutney.yml → chutney_defaults.yml}

@@ -12,6 +12,8 @@ BackgroundRequiresMultipleScenarios:
     Enabled: true
 BadScenarioName:
     Enabled: true
+EmptyFeatureFile:
+    Enabled: true
 FileNameDiffersFeatureName:
     Enabled: true
 GivensAfterBackground:

data/config/cucumber.yml

@@ -0,0 +1 @@
+default: --publish-quiet

data/docs/usage/rules.md

@@ -30,6 +30,9 @@ Chutney enforces its rules with the linters. These are:
 [BadScenarioName](https://github.com/BillyRuffian/chutney/blob/master/features/bad_scenario_name.feature)
 : You should avoid using words like 'test' or 'check' in your scenario names, instead you should define the behaviour of your system.
 
+[EmptyFeatureFile](https://github.com/BillyRuffian/chutney/blob/master/features/empty_feature_file.feature)
+: The feature should have content and should avoid committing empty features to repositories.
+
 [FileNameDiffersFeatureName](https://github.com/BillyRuffian/chutney/blob/master/features/file_name_differs_feature_name.feature)
 : The feature should have a name that follows the file name.
 

data/exe/chutney

@@ -1,4 +1,6 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
+
 require 'chutney'
 require 'chutney/formatter'
 require 'chutney/formatter/json_formatter'
@@ -10,19 +12,37 @@ 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
+    pastel = Pastel.new
+    chutney_config = Chutney::ChutneyLint.new.configuration
+    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
+        puts pastel.red('disabled')
+      end
+    end
+    exit
+  end
 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

@@ -1,4 +1,7 @@
+# frozen_string_literal: true
+
 require 'amatch'
+
 require 'chutney/configuration'
 require 'chutney/linter'
 require 'chutney/linter/avoid_full_stop'
@@ -8,6 +11,7 @@ require 'chutney/linter/avoid_typographers_quotes'
 require 'chutney/linter/background_does_more_than_setup'
 require 'chutney/linter/background_requires_multiple_scenarios'
 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/invalid_file_name'
@@ -32,9 +36,12 @@ 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'
-require 'gherkin/dialect'
-require 'gherkin/parser'
+# require 'gherkin/dialect'
+# require 'gherkin/parser'
 require 'i18n'
 require 'set'
 require 'yaml'
@@ -44,33 +51,36 @@ module Chutney
   class ChutneyLint
     extend Forwardable
     attr_accessor :verbose
-    attr_reader :files
-    attr_reader :results
-    
+    attr_reader :files, :results
+
     def_delegators :@files, :<<, :clear, :delete, :include?
 
     def initialize(*files)
       @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
       @config
     end
-    
+
     def configuration=(config)
       @config = config
     end
-    
+
     def analyse
       files.each do |f|
         lint(f)
@@ -80,25 +90,19 @@ module Chutney
     # alias for non-british English
     # https://dictionary.cambridge.org/dictionary/english/analyse
     alias analyze analyse
-    
+
     def linters
       @linters ||= Linter.descendants.filter { |l| configuration.dig(l.linter_name, 'Enabled') }
     end
-    
+
     def linters=(*linters)
       @linters = linters
     end
-    
+
     private
-    
-    def parse(text)
-      @parser ||= Gherkin::Parser.new
-      scanner = Gherkin::TokenScanner.new(text)
-      @parser.parse(scanner)
-    end
-    
+
     def lint(file)
-      parsed = parse(File.read(file))
+      parsed = CukeModeler::FeatureFile.new(file)
       linters.each do |linter_class|
         linter = linter_class.new(file, parsed, configuration[linter_class.linter_name])
         linter.lint

data/lib/chutney/configuration.rb

@@ -1,5 +1,8 @@
+# frozen_string_literal: true
+
 require 'delegate'
-module Chutney 
+
+module Chutney
   # gherkin_lint configuration object
   class Configuration < SimpleDelegator
     def initialize(path)
@@ -18,7 +21,11 @@ module Chutney
     end
 
     def load_user_configuration
-      config_file = Dir.glob(File.join(Dir.pwd, '**', '.chutney.yml')).first
+      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

@@ -1,19 +1,20 @@
+# frozen_string_literal: true
+
 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/json_formatter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Chutney
   # Plain old JSON formatter
   class JSONFormatter < Formatter

data/lib/chutney/formatter/pie_formatter.rb

@@ -1,13 +1,11 @@
+# frozen_string_literal: true
+
 require 'pastel'
 require 'tty/pie'
 
 module Chutney
   # format results as pie charts
   class PieFormatter < Formatter
-    def initialize
-      super
-    end
-    
     def format
       data = top_offences.map do |offence|
         {
@@ -19,14 +17,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|
@@ -36,9 +34,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
@@ -49,10 +47,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
@@ -62,7 +60,7 @@ module Chutney
       end
       @colour_looper.resume
     end
-    
+
     def put_summary
       pastel = Pastel.new
       print "#{files.count} features inspected, "
@@ -72,6 +70,5 @@ module Chutney
         puts pastel.red("#{files_with_issues.count} taste nasty")
       end
     end
-    
   end
 end

data/lib/chutney/formatter/rainbow_formatter.rb

@@ -1,39 +1,40 @@
+# frozen_string_literal: true
+
 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(i) }
+          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(issue)    
-      puts "    #{@pastel.dim(issue.dig(:location, :line))} #{issue[:message]}"
+
+    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?
@@ -42,6 +43,5 @@ module Chutney
         puts @pastel.red("#{files_with_issues.count} taste nasty")
       end
     end
-    
   end
 end