rutabaga 3.0.2 → 3.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3d6c51a32cd3abeb7753144e65c1166230b9e728d2ca2b34908b08b434454810
-  data.tar.gz: 3e0926f70210244f08f675b1e4fb4684b3553705a8b50e4acdf8812f87173955
+  metadata.gz: b6a38a9e6277b182b50fc33ded4f92eaec8b3d9b8888112af72e08f287e38e84
+  data.tar.gz: f9cbe896db240cb89be5454991ab06d0d9abf4072046bda5630ea794e018b9be
 SHA512:
-  metadata.gz: 3dd9b6d369a01336b575178686f32b92a2b3f99c2597f604427985278a080f650e80421966a7099a2606123ba576ef04feb0c069809e381c77803d001410dab1
-  data.tar.gz: 53bd5a7dfaef0c4af85e2dfa967c83a68916ab0048c7ec2444b5a0936e636c31bf91072e205cedadce122e77c17146c862e0213a4398ff08cfc67d1e4d7c7662
+  metadata.gz: 2e0aca4cd5ff1afdd6a0e7f800d424dd0a1bb1edd546e849134a8021ed0df47a5cfeb87af8b5243a32eddc4c6da157b01228ec92cc8b43ace0b9ba27ab174ea0
+  data.tar.gz: 5123b99de3be3fc63399b06e71f9baad2065e45bf1bca864cedd8fec50f27c69a8c7c6e879fdd0612bcdf899fdc1a3fb25ff7a69560e1e7c1cabf3609f6991e7

data/.github/workflows/codeql-analysis.yml

@@ -0,0 +1,70 @@
+# For most projects, this workflow file will not need changing; you simply need
+# to commit it to your repository.
+#
+# You may wish to alter this file to override the set of languages analyzed,
+# or to provide custom queries or build logic.
+#
+# ******** NOTE ********
+# We have attempted to detect the languages in your repository. Please check
+# the `language` matrix defined below to confirm you have the correct set of
+# supported CodeQL languages.
+#
+name: "CodeQL"
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    # The branches below must be a subset of the branches above
+    branches: [ master ]
+  schedule:
+    - cron: '24 10 * * 6'
+
+jobs:
+  analyze:
+    name: Analyze
+    runs-on: ubuntu-latest
+    permissions:
+      actions: read
+      contents: read
+      security-events: write
+
+    strategy:
+      fail-fast: false
+      matrix:
+        language: [ 'ruby' ]
+        # CodeQL supports [ 'cpp', 'csharp', 'go', 'java', 'javascript', 'python', 'ruby' ]
+        # Learn more about CodeQL language support at https://git.io/codeql-language-support
+
+    steps:
+    - name: Checkout repository
+      uses: actions/checkout@v2
+
+    # Initializes the CodeQL tools for scanning.
+    - name: Initialize CodeQL
+      uses: github/codeql-action/init@v1
+      with:
+        languages: ${{ matrix.language }}
+        # If you wish to specify custom queries, you can do so here or in a config file.
+        # By default, queries listed here will override any specified in a config file.
+        # Prefix the list here with "+" to use these queries and those in the config file.
+        # queries: ./path/to/local/query, your-org/your-repo/queries@main
+
+    # Autobuild attempts to build any compiled languages  (C/C++, C#, or Java).
+    # If this step fails, then you should remove it and run the build manually (see below)
+    - name: Autobuild
+      uses: github/codeql-action/autobuild@v1
+
+    # ℹ️ Command-line programs to run using the OS shell.
+    # 📚 https://git.io/JvXDl
+
+    # ✏️ If the Autobuild fails above, remove it and uncomment the following three lines
+    #    and modify them (or add more) to build your code if your project
+    #    uses a compiled language
+
+    #- run: |
+    #   make bootstrap
+    #   make release
+
+    - name: Perform CodeQL Analysis
+      uses: github/codeql-action/analyze@v1

data/.github/workflows/gempush.yml

@@ -24,6 +24,8 @@ jobs:
         mkdir -p $HOME/.gem
         touch $HOME/.gem/credentials
         chmod 0600 $HOME/.gem/credentials
+        git config --global user.email ${{ github.event.pusher.email }}
+        git config --global user.name  ${{ github.event.pusher.name }}
         printf -- "---\n:rubygems_api_key: ${GEM_HOST_API_KEY}\n" > $HOME/.gem/credentials
         gem release --tag --push
       env:

data/.github/workflows/project-board.yml

@@ -0,0 +1,17 @@
+name: Automatically add new Issues to Silversmith's GitHub project board
+# https://github.com/orgs/simplybusiness/projects/3
+# New issues in this repository will be added to the board.
+on:
+  issues:
+    types: [ opened, reopened ]
+
+jobs:
+  track_issues:
+    uses: simplybusiness/github-action-reusable-workflows/.github/workflows/project-board.yaml@master
+    with:
+      pr-id: ${{ github.event.issue.node_id }}
+      project-number: 3
+      org: simplybusiness
+      actor: ${{ github.actor }}
+    secrets:
+      github-token: ${{ secrets.SILVERSMITHS_PROJECT_MANAGEMENT }}

data/.github/workflows/rubocop-analysis.yml

@@ -0,0 +1,22 @@
+name: "Rubocop"
+
+on: [push]
+
+jobs:
+  rubocop_job:
+    runs-on: ubuntu-latest
+    name: Code Scanning job run
+    strategy:
+      fail-fast: false
+
+    steps:
+    - name: Checkout repository
+      uses: actions/checkout@v2
+
+    - name: Rubocop run
+      uses: arthurnn/code-scanning-rubocop/rubocop-action@master
+
+    - name: Upload Sarif output
+      uses: github/codeql-action/upload-sarif@v1
+      with:
+        sarif_file: rubocop.sarif

data/CODEOWNERS

@@ -0,0 +1,4 @@
+# Add your project owners info here
+# More information: https://help.github.com/articles/about-codeowners/
+* @simplybusiness/silversmiths 
+

data/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in rutabaga.gemspec
@@ -6,13 +8,8 @@ gemspec
 platform :ruby do
   gem 'pry-byebug'
   gem 'pry-doc'
-  gem 'pry-stack_explorer'
   gem 'pry-rescue'
+  gem 'pry-stack_explorer'
 end
 
-# Gem a dependency of Capybara, new version of rack 2.0 onwards require ruby 2.2 and above.
-if RUBY_VERSION.to_f < 2.2
-  gem 'rack', '~> 1.6'
-else
-  gem 'rack'
-end
+gem 'rack'

data/Rakefile

@@ -1,2 +1,4 @@
 #!/usr/bin/env rake
-require "bundler/gem_tasks"
+# frozen_string_literal: true
+
+require 'bundler/gem_tasks'

data/catalog-info.yaml

@@ -0,0 +1,8 @@
+apiVersion: backstage.io/v1alpha1
+kind: Component
+metadata:
+  name: rutabaga
+  description: Turnip extension to enable running turnip features from inside spec files, rather than outside.
+spec:
+  type: library
+  lifecycle: production

data/examples/compatibility/compatibility_spec.rb

@@ -1,14 +1,16 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 require 'capybara/rspec'
 
-describe "capaybara rails does not overwrite the feature command" do
+describe 'capaybara rails does not overwrite the feature command' do
   feature '../test2.feature' do
-    step "that :first + :second is calculated" do |first, second|
+    step 'that :first + :second is calculated' do |first, second|
       @first = first
       @second = second
     end
 
-    step "my result is :result" do |result|
+    step 'my result is :result' do |result|
       expect(@first.to_i + @second.to_i - 1).to eq(result.to_i)
     end
   end

data/examples/pattern/test_pattern_spec.rb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
-describe "it prints out the RSpec.configuration.pattern" do
-  it "examples/test2.feature" do
+describe 'it prints out the RSpec.configuration.pattern' do
+  it 'examples/test2.feature' do
     puts "RSpec.configuration.pattern: #{RSpec.configuration.pattern}"
   end
 end

data/examples/spec_helper.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 RSpec.configure do |config|
   config.expect_with :rspec do |c|
-    c.syntax = [:should, :expect]
+    c.syntax = %i[should expect]
   end
 end

data/examples/test_feature_example_group_spec.rb

@@ -1,109 +1,109 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
-describe "test feature argument" do
+describe 'test feature argument' do
   feature
 
-  step "that :first + :second is calculated" do |first, second|
+  step 'that :first + :second is calculated' do |first, second|
     @first = first
     @second = second
   end
 
-  step "my result is :result" do |result|
+  step 'my result is :result' do |result|
     expect(@first.to_i + @second.to_i).to eq(result.to_i)
   end
 end
 
-feature "feature block" do
-  step "that :first + :second is calculated" do |first, second|
+feature 'feature block' do
+  step 'that :first + :second is calculated' do |first, second|
     @first = first
     @second = second
   end
 
-  step "my result is :result" do |result|
+  step 'my result is :result' do |result|
     expect(@first.to_i + @second.to_i).to eq(result.to_i)
   end
 end
 
-describe "feature block inside a describe block" do
+describe 'feature block inside a describe block' do
   feature do
-
-    step "that :first + :second is calculated" do |first, second|
+    step 'that :first + :second is calculated' do |first, second|
       @first = first
       @second = second
     end
 
-    step "my result is :result" do |result|
+    step 'my result is :result' do |result|
       expect(@first.to_i + @second.to_i).to eq(result.to_i)
     end
   end
 end
 
-describe "should find the feature file using the root (and monkey patching the result)" do
-  feature "examples/test2.feature"
+describe 'should find feature file using root (and monkey patch result)' do
+  feature 'examples/test2.feature'
 
-  step "that :first + :second is calculated" do |first, second|
+  step 'that :first + :second is calculated' do |first, second|
     @first = first
     @second = second
   end
 
-  step "my result is :result" do |result|
+  step 'my result is :result' do |result|
     expect(@first.to_i + @second.to_i - 1).to eq(result.to_i)
   end
 end
 
-describe "causes a failing test" do
-  feature "examples/test2.feature"
+describe 'causes a failing test' do
+  feature 'examples/test2.feature'
 
-  step "that :first + :second is calculated" do |first, second|
+  step 'that :first + :second is calculated' do |first, second|
     @first = first
     @second = second
   end
 
-  step "my result is :result" do |result|
+  step 'my result is :result' do |result|
     expect(@first.to_i + @second.to_i).to eq(result.to_i)
   end
 end
 
 describe "finds a feature file given as parameter to the 'feature' method" do
-  feature "examples/test3.feature"
+  feature 'examples/test3.feature'
 
-  step "that :first * :second is calculated" do |first, second|
+  step 'that :first * :second is calculated' do |first, second|
     @first = first
     @second = second
   end
 
-  step "my result is :result" do |result|
+  step 'my result is :result' do |result|
     expect(@first.to_i * @second.to_i).to eq(result.to_i)
   end
 end
 
-describe "finds a feature file with a different name in the same directory" do
-  feature "test3.feature"
+describe 'finds a feature file with a different name in the same directory' do
+  feature 'test3.feature'
 
-  step "that :first * :second is calculated" do |first, second|
+  step 'that :first * :second is calculated' do |first, second|
     @first = first
     @second = second
   end
 
-  step "my result is :result" do |result|
+  step 'my result is :result' do |result|
     expect(@first.to_i * @second.to_i).to eq(result.to_i)
   end
 end
 
-describe "backgrounds are properly called" do
-  feature "examples/test_background.feature"
+describe 'backgrounds are properly called' do
+  feature 'examples/test_background.feature'
 
-  step "we add :initial" do |initial|
+  step 'we add :initial' do |_initial|
     @initial = 10
   end
 
-  step "that :first * :second is calculated" do |first, second|
+  step 'that :first * :second is calculated' do |first, second|
     @first = first
     @second = second
   end
 
-  step "my result is :result" do |result|
+  step 'my result is :result' do |result|
     expect(@initial.to_i + @first.to_i * @second.to_i).to eq(result.to_i)
   end
-
 end

data/examples/test_spec.rb

@@ -1,15 +1,16 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
-describe "should find the feature file using the root (and monkey patching the result)" do
-  feature "examples/test2.feature"
+describe 'should find feature file using root (and monkey patch result)' do
+  feature 'examples/test2.feature'
 
-  step "that :first + :second is calculated" do |first, second|
+  step 'that :first + :second is calculated' do |first, second|
     @first = first
     @second = second
   end
 
-  step "my result is :result" do |result|
+  step 'my result is :result' do |result|
     expect(@first.to_i + @second.to_i - 1).to eq(result.to_i)
   end
-
 end

data/lib/rspec/core/example_group_patch.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # This file needs to be in this folder structure otherwise rspec
 # will not filter it out of the caller chain, and therefore report
 # incorrect file locations for example groups. This especially
@@ -5,10 +7,11 @@
 
 # Monkey patch rspec to block capybara from using feature
 class RSpec::Core::Configuration
-  alias_method :orig_alias_example_group_to, :alias_example_group_to
+  alias orig_alias_example_group_to alias_example_group_to
 
   def alias_example_group_to(new_name, *args)
-    return if [:feature, :xfeature, :ffeature].include?(new_name)
+    return if %i[feature xfeature ffeature].include?(new_name)
+
     orig_alias_example_group_to(new_name, *args)
   end
 end
@@ -16,16 +19,18 @@ end
 # Monkey patch RSpec to add the feature method in example groups
 class RSpec::Core::ExampleGroup
   class << self
-    alias_method :orig_subclass, :subclass
+    alias orig_subclass subclass
 
     def subclass(parent, description, *all_args, &example_group_block)
-      rutabaga = all_args.first.any? { |arg| arg.kind_of?(Hash) && arg[:rutabaga] }
+      rutabaga = all_args.first.any? { |arg| arg.is_a?(Hash) && arg[:rutabaga] }
 
-      self.orig_subclass(parent, description, *all_args, &example_group_block).tap do |describe|
-        Rutabaga::ExampleGroup::Feature.feature(describe, description, all_args.last) if rutabaga
+      orig_subclass(parent, description, *all_args, &example_group_block).tap do |describe|
+        if rutabaga
+          Rutabaga::ExampleGroup::Feature.feature(describe, description, all_args.last)
+        end
       end
     end
   end
 
-  define_example_group_method :feature, :rutabaga => true
+  define_example_group_method :feature, rutabaga: true
 end

data/lib/rutabaga/example_group/feature.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'turnip/rspec'
 require 'rspec'
 
@@ -5,7 +7,7 @@ module Rutabaga
   module ExampleGroup
     module Feature
       class << self
-        def feature(example_group_class, description, args)
+        def feature(example_group_class, description, _args)
           Util.require_if_exists 'turnip_helper'
 
           Turnip::RSpec.rutabaga_run(Util.find_feature(description), example_group_class)

data/lib/rutabaga/no_turnip.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Rutabaga
   NO_TURNIP = true
 end
@@ -12,7 +14,7 @@ module Turnip
     module Loader
       def load(*a, &b)
         if a.first.end_with?('.feature')
-          ::RSpec.warning "Calling features directly has been disabled by rutabaga. To re-enable, do not require rutabaga/no_turnip."
+          ::RSpec.warning 'Calling features directly has been disabled by rutabaga. To re-enable, do not require rutabaga/no_turnip.'
         else
           super
         end
@@ -23,5 +25,5 @@ end
 
 ::RSpec.configure do |c|
   # Blow away rutabaga's pattern if still there
-  c.pattern.gsub!(",features/**/*.feature", "")
+  c.pattern.gsub!(',features/**/*.feature', '')
 end

data/lib/rutabaga/turnip.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rutabaga'
 ##
 #
@@ -8,8 +10,8 @@ module Turnip
     module Loader
       def load(*a, &b)
         if a.first.end_with?('.feature')
-          if legal_directories.none? {|d| a.first.end_with? d }
-            ::RSpec.warning "Features can only be called from turnip enable directories. These are configured " \
+          if legal_directories.none? { |d| a.first.end_with? d }
+            ::RSpec.warning 'Features can only be called from turnip enable directories. These are configured ' \
                             "in RSpec.configuration.pattern which is currently '#{::RSpec.configuration.pattern}'"
           else
             require_if_exists 'turnip_helper'
@@ -26,9 +28,9 @@ module Turnip
 
       def legal_directories
         @legal_directories ||= ::RSpec.configuration.pattern.split(',')
-          .select {|p| /\.feature\Z/ =~ p}
-          .map {|d| Dir.glob(File.join(::RSpec.configuration.default_path,d)) }
-          .flatten
+                                      .select { |p| /\.feature\Z/ =~ p }
+                                      .map { |d| Dir.glob(File.join(::RSpec.configuration.default_path, d)) }
+                                      .flatten
       end
     end
   end

data/lib/rutabaga/util.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Utils and monkey patches for both versions of feature
 
 # Monkey patch for Turnip to not have to copy loads of code
@@ -24,16 +26,18 @@ module Rutabaga
         tried = []
 
         if description =~ /.*\.(feature|rutabaga)\Z/
-          return description if File.exists?(description)
+          return description if File.exist?(description)
+
           tried << description
 
           candidate = File.join(extract_directory, description)
-          return candidate if File.exists?(candidate)
+          return candidate if File.exist?(candidate)
+
           tried << candidate
         else
           feature_files = extract_features
           feature_files.each do |feature_file|
-            return feature_file if File.exists?(feature_file)
+            return feature_file if File.exist?(feature_file)
           end
           tried += feature_files
         end
@@ -57,14 +61,14 @@ module Rutabaga
       def extract_directory
         caller(0).find do |call|
           call =~ /_spec.rb:/
-        end.gsub(/\/[^\/]+_spec.rb:.*\Z/, '')
+        end.gsub(%r{/[^/]+_spec.rb:.*\Z}, '')
       end
 
       def extract_features
         base = caller(0).find do |call|
           call =~ /_spec.rb:/
         end.gsub(/_spec.rb:.*\Z/, '')
-        [base+'.feature', base+'.rutabaga']
+        [base + '.feature', base + '.rutabaga']
       end
     end
   end
@@ -73,8 +77,8 @@ end
 ::RSpec.configure do |c|
   # Blow away turnip's pattern, and focus just on features directory
   if defined?(Rutabaga::NO_TURNIP)
-    c.pattern.gsub!(",**/*.feature", "")
+    c.pattern.gsub!(',**/*.feature', '')
   else
-    c.pattern.gsub!(",**/*.feature", ",features/**/*.feature")
+    c.pattern.gsub!(',**/*.feature', ',features/**/*.feature')
   end
 end

data/lib/rutabaga/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Rutabaga
-  VERSION = '3.0.2'
+  VERSION = '3.1.0'
 end

data/lib/rutabaga.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rutabaga/version'
 require 'turnip'
 require 'rutabaga/example_group/feature'

data/rutabaga.gemspec

@@ -1,14 +1,15 @@
-# coding: utf-8
-require File.expand_path('../lib/rutabaga/version', __FILE__)
+# frozen_string_literal: true
+
+require File.expand_path('lib/rutabaga/version', __dir__)
 
 Gem::Specification.new do |gem|
   gem.authors       = ['Lukas Oberhuber']
   gem.email         = ['lukas.oberhuber@simplybusiness.co.uk']
-  gem.description   = %q{Allows using feature from within RSpec and is built on top of Turnip}
-  gem.summary       = %q{Calling Turnip feature files from RSpec, which allows encapsulating a feature inside a describe block}
+  gem.description   = 'Allows using feature from within RSpec and is built on top of Turnip'
+  gem.summary       = 'Calling Turnip feature files from RSpec, which allows encapsulating a feature inside a describe block'
   gem.homepage      = 'https://github.com/simplybusiness/rutabaga'
 
-  gem.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
+  gem.executables   = `git ls-files -- bin/*`.split("\n").map { |f| File.basename(f) }
   gem.files         = `git ls-files`.split("\n")
   gem.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
   gem.name          = 'rutabaga'
@@ -16,8 +17,8 @@ Gem::Specification.new do |gem|
   gem.version       = Rutabaga::VERSION
   gem.license       = 'MIT'
 
-  gem.add_runtime_dependency 'turnip', ['>= 3.1.0', '< 5.0']
   gem.add_runtime_dependency 'rspec', ['~> 3.0']
+  gem.add_runtime_dependency 'turnip', ['>= 3.1.0', '< 4.4']
 
   gem.add_development_dependency 'capybara'
   gem.add_development_dependency 'pry', '~> 0'

data/spec/feature_spec.rb

@@ -1,39 +1,41 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
-describe 'integration', :type => :integration do
+describe 'integration', type: :integration do
   describe 'functionality' do
     before(:all) do
-      @result = %x(rspec -r rutabaga -fd examples/*_spec.rb 2>&1)
+      @result = `rspec -r rutabaga -fd examples/*_spec.rb 2>&1`
     end
 
-    it "shows the correct description" do
+    it 'shows the correct description' do
       expect(@result).to include('ensures the feature is called')
       expect(@result).to include('that 2 + 2 is calculated')
       expect(@result).to include('my result is 4')
     end
 
-    it "executes features as an argument" do
+    it 'executes features as an argument' do
       expect(@result).to include('test feature argument')
     end
 
-    it "executes features as blocks/example groups" do
+    it 'executes features as blocks/example groups' do
       expect(@result).to include('feature block')
     end
 
-    it "executes features as blocks inside example groups" do
+    it 'executes features as blocks inside example groups' do
       expect(@result).to include('feature block inside a describe block')
     end
 
-    it "should not show any pending steps" do
+    it 'should not show any pending steps' do
       expect(@result).not_to include('PENDING')
       expect(@result).not_to include('No such step')
     end
 
-    it "prints out failures and successes" do
+    it 'prints out failures and successes' do
       expect(@result).to include('18 examples, 4 failures')
     end
 
-    it "should find features relative to the root" do
+    it 'should find features relative to the root' do
       expect(@result).not_to include('Feature file not found')
     end
 
@@ -42,26 +44,26 @@ describe 'integration', :type => :integration do
       expect(@result).to include('my result is 8')
     end
 
-    it "should scope steps to describe blocks" do
+    it 'should scope steps to describe blocks' do
       expect(@result).not_to include('Turnip::Ambiguous')
     end
 
-    it "should provide failure messages that allow a specific scenario to be run" do
-      expect(@result).to match(/rspec '{0,1}.\/examples\/test_feature_example_group_spec.rb\[1:1:1:4:1\]'{0,1}/)
+    it 'should provide failure messages that allow a specific scenario to be run' do
+      expect(@result).to match(%r{rspec '{0,1}./examples/test_feature_example_group_spec.rb\[1:1:1:4:1\]'{0,1}})
     end
   end
 
   describe 'compatibility' do
     before(:all) do
-      @result = %x(rspec -r rutabaga -fd examples/compatibility/*_spec.rb 2>&1)
+      @result = `rspec -r rutabaga -fd examples/compatibility/*_spec.rb 2>&1`
     end
 
-    it "passes all tests" do
+    it 'passes all tests' do
       expect(@result).to include('1 example, 0 failures')
     end
 
-    it "runs feature blocks even if capybara/rspec is installed" do
-      expect(@result).to include("capaybara rails does not overwrite the feature command")
+    it 'runs feature blocks even if capybara/rspec is installed' do
+      expect(@result).to include('capaybara rails does not overwrite the feature command')
     end
   end
 end

data/spec/formatter_spec.rb

@@ -1,15 +1,17 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
-describe 'formatter', :type => :integration do
+describe 'formatter', type: :integration do
   before(:all) do
-    @result = %x(rspec -r rutabaga -r rspec_formatter/formatter.rb --format RspecFormatter::Formatter spec/features/fixture.feature examples/test_spec.rb 2>&1)
+    @result = `rspec -r rutabaga -r rspec_formatter/formatter.rb --format RspecFormatter::Formatter spec/features/fixture.feature examples/test_spec.rb 2>&1`
   end
 
-  it "has the feature location" do
-    expect(@result).to include("rspec_core_formatter:file_path: ./spec/features/fixture.feature")
+  it 'has the feature location' do
+    expect(@result).to include('rspec_core_formatter:file_path: ./spec/features/fixture.feature')
   end
 
-  it "has the rutabaga test location" do
-    expect(@result).to include("rspec_core_formatter:file_path: ./examples/test_spec.rb")
+  it 'has the rutabaga test location' do
+    expect(@result).to include('rspec_core_formatter:file_path: ./examples/test_spec.rb')
   end
 end

data/spec/no_turnip_spec.rb

@@ -1,57 +1,59 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
-describe 'no_turnip', :type => :integration do
+describe 'no_turnip', type: :integration do
   describe 'turnip disabled' do
     before(:all) do
-      @result = %x(rspec -r rutabaga -r rutabaga/no_turnip examples/test.feature 2>&1)
+      @result = `rspec -r rutabaga -r rutabaga/no_turnip examples/test.feature 2>&1`
     end
 
-    it "should raise an error when trying to call turnip features directly" do
-      expect(@result).to include("Calling features directly has been disabled by rutabaga")
-      expect(@result).to include("rutabaga/no_turnip")
+    it 'should raise an error when trying to call turnip features directly' do
+      expect(@result).to include('Calling features directly has been disabled by rutabaga')
+      expect(@result).to include('rutabaga/no_turnip')
     end
 
-    it "has the correct file pattern" do
-      result = %x(rspec -r rutabaga/no_turnip -r rutabaga examples/pattern/test_pattern_spec.rb)
+    it 'has the correct file pattern' do
+      result = `rspec -r rutabaga/no_turnip -r rutabaga examples/pattern/test_pattern_spec.rb`
       expect(result).to include("RSpec.configuration.pattern: **{,/*/**}/*_spec.rb\n")
-      result = %x(rspec -r rutabaga -r rutabaga/no_turnip -r rutabaga examples/pattern/test_pattern_spec.rb)
+      result = `rspec -r rutabaga -r rutabaga/no_turnip -r rutabaga examples/pattern/test_pattern_spec.rb`
       expect(result).to include("RSpec.configuration.pattern: **{,/*/**}/*_spec.rb\n")
-      result = %x(rspec -r rutabaga/no_turnip examples/pattern/test_pattern_spec.rb)
+      result = `rspec -r rutabaga/no_turnip examples/pattern/test_pattern_spec.rb`
       expect(result).to include("RSpec.configuration.pattern: **{,/*/**}/*_spec.rb\n")
     end
   end
 
   describe 'turnip enabled' do
-    describe "in correct directory" do
-      it "should be able to call features directly" do
-        @result = %x(rspec -r rutabaga/turnip spec/features/fixture.feature 2>&1)
-        expect(@result).to include("No such step")
+    describe 'in correct directory' do
+      it 'should be able to call features directly' do
+        @result = `rspec -r rutabaga/turnip spec/features/fixture.feature 2>&1`
+        expect(@result).to include('No such step')
       end
 
-      it "rutabaga turnip is the default" do
-        @result = %x(rspec -r rutabaga spec/features/fixture.feature 2>&1)
-        expect(@result).to include("No such step")
+      it 'rutabaga turnip is the default' do
+        @result = `rspec -r rutabaga spec/features/fixture.feature 2>&1`
+        expect(@result).to include('No such step')
       end
     end
 
-    describe "outside of the correct directory" do
+    describe 'outside of the correct directory' do
       before(:all) do
-        @result = %x(rspec -r rutabaga examples/test.feature 2>&1)
+        @result = `rspec -r rutabaga examples/test.feature 2>&1`
       end
 
-      it "warns about the directory" do
-        expect(@result).to include("WARNING: Features can only be called from turnip enable directories")
+      it 'warns about the directory' do
+        expect(@result).to include('WARNING: Features can only be called from turnip enable directories')
       end
     end
 
-    it "has the correct file pattern" do
-      result = %x(rspec -r rutabaga examples/pattern/test_pattern_spec.rb)
+    it 'has the correct file pattern' do
+      result = `rspec -r rutabaga examples/pattern/test_pattern_spec.rb`
       expect(result).to include("RSpec.configuration.pattern: **{,/*/**}/*_spec.rb,features/**/*.feature\n")
-      result = %x(rspec -r rutabaga/turnip -r rutabaga examples/pattern/test_pattern_spec.rb)
+      result = `rspec -r rutabaga/turnip -r rutabaga examples/pattern/test_pattern_spec.rb`
       expect(result).to include("RSpec.configuration.pattern: **{,/*/**}/*_spec.rb,features/**/*.feature\n")
-      result = %x(rspec -r rutabaga -r rutabaga/turnip -r rutabaga examples/pattern/test_pattern_spec.rb)
+      result = `rspec -r rutabaga -r rutabaga/turnip -r rutabaga examples/pattern/test_pattern_spec.rb`
       expect(result).to include("RSpec.configuration.pattern: **{,/*/**}/*_spec.rb,features/**/*.feature\n")
-      result = %x(rspec -r rutabaga/turnip examples/pattern/test_pattern_spec.rb)
+      result = `rspec -r rutabaga/turnip examples/pattern/test_pattern_spec.rb`
       expect(result).to include("RSpec.configuration.pattern: **{,/*/**}/*_spec.rb,features/**/*.feature\n")
     end
   end

data/spec/rspec_formatter/formatter.rb

@@ -1,12 +1,14 @@
+# frozen_string_literal: true
+
 module RspecFormatter
   class Formatter
     RSpec::Core::Formatters.register self, :example_group_started, :example_group_finished
 
-    def initialize(output)
+    def initialize(_output)
       @group_nesting = 0
     end
 
-    def example_group_started(example_group)
+    def example_group_started(_example_group)
       @group_nesting += 1
     end
 

data/spec/rutabaga/util_spec.rb

@@ -1,129 +1,141 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 require 'rutabaga'
 
 describe Rutabaga::Util do
-  describe "location of test from stack track" do
-    it "finds the directory" do
-      expect(subject.class.send(:extract_directory)).to match(/\/rutabaga\/spec\/rutabaga\Z/)
+  describe 'location of test from stack track' do
+    it 'finds the directory' do
+      expect(subject.class.send(:extract_directory)).to match(%r{/rutabaga/spec/rutabaga\Z})
     end
 
-    it "finds the feature" do
+    it 'finds the feature' do
       features = subject.class.send(:extract_features)
-      expect(features[0]).to match(/\/rutabaga\/spec\/rutabaga\/util\.feature\Z/)
-      expect(features[1]).to match(/\/rutabaga\/spec\/rutabaga\/util\.rutabaga\Z/)
+      expect(features[0]).to match(%r{/rutabaga/spec/rutabaga/util\.feature\Z})
+      expect(features[1]).to match(%r{/rutabaga/spec/rutabaga/util\.rutabaga\Z})
     end
   end
 
-  describe ".find_feature" do
+  describe '.find_feature' do
     let(:subject) { Rutabaga::Util.find_feature(@description) }
     before do
-      allow(File).to receive(:exists?).with('spec/rutabaga/existing.feature').and_return(true)
-      allow(File).to receive(:exists?).with('spec/rutabaga/missing.feature').and_return(false)
-      allow(File).to receive(:exists?).with(nil).and_return(false)
+      allow(File).to receive(:exist?).with('spec/rutabaga/existing.feature').and_return(true)
+      allow(File).to receive(:exist?).with('spec/rutabaga/missing.feature').and_return(false)
+      allow(File).to receive(:exist?).with(nil).and_return(false)
     end
 
-    it "returns the file if it exists" do
+    it 'returns the file if it exists' do
       @description = 'spec/rutabaga/existing.feature'
       expect(subject).to eq('spec/rutabaga/existing.feature')
     end
 
     describe "looks for the feature in the spec's directory" do
       before do
-        allow(File).to receive(:exists?).with('different.feature').and_return(false)
+        allow(File).to receive(:exist?).with('different.feature').and_return(false)
       end
 
-      it "looks directly in the directory" do
+      it 'looks directly in the directory' do
         @description = 'different.feature'
-        expect(File).to receive(:exists?).with(/spec\/rutabaga\/different\.feature\Z/).and_return(true)
+        expect(File).to receive(:exist?)
+          .with(%r{spec/rutabaga/different\.feature\Z})
+          .and_return(true)
 
-        expect(subject).to match(/spec\/rutabaga\/different\.feature\Z/)
+        expect(subject).to match(%r{spec/rutabaga/different\.feature\Z})
       end
 
-      it "allows sub-directories" do
-        allow(File).to receive(:exists?).with('subdirectory/different.feature').and_return(false)
+      it 'allows sub-directories' do
+        allow(File).to receive(:exist?).with('subdirectory/different.feature')
+                                       .and_return(false)
 
         @description = 'subdirectory/different.feature'
-        expect(File).to receive(:exists?).with(/spec\/rutabaga\/subdirectory\/different\.feature\Z/).and_return(true)
+        expect(File).to receive(:exist?)
+          .with(%r{spec/rutabaga/subdirectory/different\.feature\Z})
+          .and_return(true)
 
-        expect(subject).to match(/spec\/rutabaga\/subdirectory\/different\.feature\Z/)
+        expect(subject).to match(%r{spec/rutabaga/subdirectory/different\.feature\Z})
       end
     end
 
-    describe "figures out the feature name from the spec name" do
-      it "description is nil" do
+    describe 'figures out the feature name from the spec name' do
+      it 'description is nil' do
         @description = nil
-        allow(File).to receive(:exists?).with(/spec\/rutabaga\/util\.feature\Z/).and_return(true)
-        expect(subject).to match(/spec\/rutabaga\/util\.feature\Z/)
+        allow(File).to receive(:exist?).with(%r{spec/rutabaga/util\.feature\Z})
+                                       .and_return(true)
+        expect(subject).to match(%r{spec/rutabaga/util\.feature\Z})
       end
 
-      it "description does not match a feature file" do
+      it 'description does not match a feature file' do
         @description = 'this is not a feature file'
-        allow(File).to receive(:exists?).with(/this is not a feature file/).and_return(false)
+        allow(File).to receive(:exist?).with(/this is not a feature file/)
+                                       .and_return(false)
 
-        allow(File).to receive(:exists?).with(/spec\/rutabaga\/util\.feature\Z/).and_return(true)
-        expect(subject).to match(/spec\/rutabaga\/util\.feature\Z/)
+        allow(File).to receive(:exist?).with(%r{spec/rutabaga/util\.feature\Z})
+                                       .and_return(true)
+        expect(subject).to match(%r{spec/rutabaga/util\.feature\Z})
       end
 
-      it "description does not match a feature file" do
+      it 'description does not match a feature file' do
         @description = 'this is not a feature file'
-        allow(File).to receive(:exists?).with(/this is not a feature file/).and_return(false)
+        allow(File).to receive(:exist?).with(/this is not a feature file/).and_return(false)
+
+        allow(File).to receive(:exist?).with(%r{spec/rutabaga/util\.feature\Z})
+                                       .and_return(false)
+        allow(File).to receive(:exist?).with(%r{spec/rutabaga/util\.rutabaga\Z})
+                                       .and_return(true)
 
-        allow(File).to receive(:exists?).with(/spec\/rutabaga\/util\.feature\Z/).and_return(false)
-        allow(File).to receive(:exists?).with(/spec\/rutabaga\/util\.rutabaga\Z/).and_return(true)
-        expect(subject).to match(/spec\/rutabaga\/util\.rutabaga\Z/)
+        expect(subject).to match(%r{spec/rutabaga/util\.rutabaga\Z})
       end
 
-      it "handles paths with spaces" do
+      it 'handles paths with spaces' do
         @description = '/User/person/Internet plugins/feature.feature'
-        allow(File).to receive(:exists?).with(@description).and_return(true)
+        allow(File).to receive(:exist?).with(@description).and_return(true)
 
         expect(subject).to eq(@description)
       end
 
-      it "allows the .feature extension" do
-        @description = "example.feature"
-        allow(File).to receive(:exists?).with(@description).and_return(true)
+      it 'allows the .feature extension' do
+        @description = 'example.feature'
+        allow(File).to receive(:exist?).with(@description).and_return(true)
 
         expect(subject).to include(@description)
       end
 
-      it "allows the .rutabaga extension" do
-        @description = "example.rutabaga"
-        allow(File).to receive(:exists?).with(@description).and_return(true)
+      it 'allows the .rutabaga extension' do
+        @description = 'example.rutabaga'
+        allow(File).to receive(:exist?).with(@description).and_return(true)
 
         expect(subject).to include(@description)
       end
-
     end
 
-    describe "raises an error if the feature cannot be found" do
+    describe 'raises an error if the feature cannot be found' do
       before do
-        allow(File).to receive(:exists?).and_return(false)
+        allow(File).to receive(:exist?).and_return(false)
       end
 
-      it "has a nil description" do
+      it 'has a nil description' do
         @description = nil
 
-        expect{subject}.to raise_error(/Feature file not found\. Tried: .*\/spec\/rutabaga\/util\.feature/)
+        expect { subject }.to raise_error(%r{Feature file not found\. Tried: .*/spec/rutabaga/util\.feature})
       end
 
-      it "has a sentance description" do
-        @description = "my life as a dog"
+      it 'has a sentance description' do
+        @description = 'my life as a dog'
 
-        expect{subject}.to raise_error(/Feature file not found\. Tried: .*\/spec\/rutabaga\/util\.feature/)
+        expect { subject }.to raise_error(%r{Feature file not found\. Tried: .*/spec/rutabaga/util\.feature})
       end
 
       it "has a filename description but the file doesn't exist" do
-        @description = "example.feature"
+        @description = 'example.feature'
 
-        expect{subject}.to raise_error(/Feature file not found\. Tried: example\.feature, .*example\.feature/)
+        expect { subject }.to raise_error(/Feature file not found\. Tried: example\.feature, .*example\.feature/)
       end
 
-      it "raises an error if the filename does not end in feature" do
-        @description = "example.other"
-        allow(File).to receive(:exists?).with(@description).and_return(true)
+      it 'raises an error if the filename does not end in feature' do
+        @description = 'example.other'
+        allow(File).to receive(:exist?).with(@description).and_return(true)
 
-        expect{subject}.to raise_error(/Feature file not found\. Tried: .*\/spec\/rutabaga\/util\.feature/)
+        expect { subject }.to raise_error(%r{Feature file not found\. Tried: .*/spec/rutabaga/util\.feature})
       end
     end
   end

data/spec/spec_helper.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 RSpec.configure do |config|
   config.expect_with :rspec do |c|
-    c.syntax = [:should, :expect]
+    c.syntax = %i[should expect]
   end
 end

metadata

@@ -1,49 +1,49 @@
 --- !ruby/object:Gem::Specification
 name: rutabaga
 version: !ruby/object:Gem::Version
-  version: 3.0.2
+  version: 3.1.0
 platform: ruby
 authors:
 - Lukas Oberhuber
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-05-08 00:00:00.000000000 Z
+date: 2022-11-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: turnip
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 3.1.0
-    - - "<"
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '3.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 3.1.0
-    - - "<"
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '3.0'
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: turnip
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: 3.1.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '4.4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: 3.1.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '4.4'
 - !ruby/object:Gem::Dependency
   name: capybara
   requirement: !ruby/object:Gem::Requirement
@@ -80,14 +80,19 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".github/workflows/ci.yml"
+- ".github/workflows/codeql-analysis.yml"
 - ".github/workflows/gempush.yml"
+- ".github/workflows/project-board.yml"
+- ".github/workflows/rubocop-analysis.yml"
 - ".gitignore"
 - CHANGELOG.md
+- CODEOWNERS
 - Gemfile
 - Gemfile.turnip3
 - LICENSE
 - README.md
 - Rakefile
+- catalog-info.yaml
 - examples/compatibility/compatibility_spec.rb
 - examples/pattern/test_pattern_spec.rb
 - examples/spec_helper.rb
@@ -133,7 +138,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.3.7
 signing_key: 
 specification_version: 4
 summary: Calling Turnip feature files from RSpec, which allows encapsulating a feature