cuke_slicer 2.2.0 → 3.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c69098207d6d5580577ecad5469270b5cbec21138e66739d74815ed0a4442ab3
-  data.tar.gz: 5dd474604bdddeddf0f02ffebec61b6af2a9319f48bc8b8e10f37a94cb7459d0
+  metadata.gz: fa79b66cbc3e85e4d8a532123fe92ed2431da5435fc28617c171b891ba850b2e
+  data.tar.gz: 58106b978a4a83ac08e4f16b1c99f3c3c230170bb4273d2d6e634cefa2a40b8d
 SHA512:
-  metadata.gz: 213337d4665b2c5b329c9335fde6afc0af9dbe4031a626eb45eba6acce31924f1df00cbb83a55cfd756dabea379b06c515a4633ff5ee3657170f8e4520acce5d
-  data.tar.gz: cd6e817e1a30de5700bb11f47435a08af612cc7fb218bf07e1f8522d21a3f00eb1578423cb56fbeff136fd38cc36cec310774cbf2a8a223a27ebef202026fe4a
+  metadata.gz: 3bd5f29074a87dcc620347623feaca14023b1399ea56dc71bdebc1067e2b063a4a705ed3711b4c3f881acf33ce793a51eb7ef72902ce48d25a39a30097da2fcd
+  data.tar.gz: b69b1759f3aeab9c16bc9521280e6e12b9757ccfcc6960b60b46dec88352a51b2b14e729265f05f11597e14513dd1d1c3a1b56b42e93d978f085b5858f1b37d9

data/CHANGELOG.md

@@ -8,6 +8,23 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 Nothing yet...
 
+
+## [3.0.0] - 2024-03-17
+
+### Added
+ - Tests inside of `Rule` elements are now taken into account when slicing.
+
+### Removed
+  - Removed support for some older versions of the gem's dependencies:
+    - CukeModeler `0.x`, CukeModeler `1.x`, and CukeModeler `2.x`. Slicing features with `Rule` elements requires at
+      least CukeModeler `3.x`, because that is the major version that first modeled rules.
+    - Ruby `1.x`. Testing against older Rubies in the current CI ecosystem has become too burdensome and Ruby `1.x` is 
+      long past end-of-life. Additionally, CukeModeler `3.x` requires at least Ruby `2.x`.
+    
+### Changed
+  - No longer including every file in the Git repository as part of the gem. Only the files needed for using the 
+    gem (and the informative ones like the README) will be packaged into the released gem.
+
 ## [2.2.0] - 2021-01-07
 
 ### Added
@@ -57,7 +74,8 @@ Nothing yet...
 
  - Initial release
 
-[Unreleased]: https://github.com/enkessler/cuke_slicer/compare/v2.2.0...HEAD
+[Unreleased]: https://github.com/enkessler/cuke_slicer/compare/v3.0.0...HEAD
+[3.0.0]: https://github.com/enkessler/cuke_slicer/compare/v2.2.0...v3.0.0
 [2.2.0]: https://github.com/enkessler/cuke_slicer/compare/v2.1.0...v2.2.0
 [2.1.0]: https://github.com/enkessler/cuke_slicer/compare/v2.0.3...v2.1.0
 [2.0.3]: https://github.com/enkessler/cuke_slicer/compare/v2.0.2...v2.0.3

data/LICENSE.txt

@@ -1,5 +1,5 @@
 Copyright (c) 2015-2019 Grange Insurance
-Copyright (c) 2020 Eric Kessler
+Copyright (c) 2020-2024 Eric Kessler
 
 MIT License
 

data/README.md

@@ -4,13 +4,13 @@ Basic stuff:
 [![Downloads](https://img.shields.io/gem/dt/cuke_slicer.svg)](https://rubygems.org/gems/cuke_slicer)
 
 User stuff:
+[![Cucumber Docs](http://img.shields.io/badge/Documentation-Features-green.svg)](https://github.com/enkessler/cuke_slicer/tree/master/testing/cucumber/features)
 [![Yard Docs](http://img.shields.io/badge/Documentation-API-blue.svg)](https://www.rubydoc.info/gems/cuke_slicer)
 
 Developer stuff:
-[![Build Status](https://travis-ci.org/enkessler/cuke_slicer.svg)](https://travis-ci.org/enkessler/cuke_slicer)
-[![Build status](https://ci.appveyor.com/api/projects/status/706c8x5ug897wq3x?svg=true)](https://ci.appveyor.com/project/enkessler/cuke-slicer)
-[![Coverage Status](https://coveralls.io/repos/github/enkessler/cuke_slicer/badge.svg)](https://coveralls.io/github/enkessler/cuke_slicer)
-[![Maintainability](https://codeclimate.com/github/enkessler/cuke_slicer/badges/gpa.svg)](https://codeclimate.com/github/enkessler/cuke_slicer/maintainability)
+[![Build Status](https://github.com/enkessler/cuke_slicer/actions/workflows/ci.yml/badge.svg?branch=master)](https://github.com/enkessler/cuke_slicer/actions/workflows/ci.yml?query=branch%3Amaster)
+[![Coverage Status](https://coveralls.io/repos/github/enkessler/cuke_slicer/badge.svg?branch=master)](https://coveralls.io/github/enkessler/cuke_slicer?branch=master)
+[![Maintainability](https://api.codeclimate.com/v1/badges/14c5ad2f8583be273418/maintainability)](https://codeclimate.com/github/enkessler/cuke_slicer/maintainability)
 
 ---
 
@@ -60,11 +60,19 @@ Or install it yourself as:
 
     system('cucumber @tests_to_run.txt')
 
+In addition to tag and path filters, a block can be used for filtering tests by inspecting the 
+underlying CukeModeler models that represent test cases and filtering on whatever criteria you can think up.
 
-## Contributing
+    CukeSlicer::Slicer.new.slice(test_directory, {}, :file_line) do |test_case|
+      test_case.is_a?(CukeModeler::Scenario) &&
+      test_case.get_ancestor(:feature).name =~ /first/
+    end
 
-1. Fork it ( https://github.com/[my-github-username]/cuke_slicer/fork )
-2. Create your feature branch (`git checkout -b my-new-feature`)
-3. Commit your changes (`git commit -am 'Add some feature'`)
-4. Push to the branch (`git push origin my-new-feature`)
-5. Create a new Pull Request
+
+## Development and Contributing
+
+See [CONTRIBUTING.md](https://github.com/enkessler/cuke_slicer/blob/master/CONTRIBUTING.md)
+
+## License
+
+The gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).

data/cuke_slicer.gemspec

@@ -1,34 +1,49 @@
-# coding: utf-8
 lib = File.expand_path('../lib', __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'cuke_slicer/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = "cuke_slicer"
+  spec.name          = 'cuke_slicer'
   spec.version       = CukeSlicer::VERSION
-  spec.authors       = ["Eric Kessler"]
-  spec.email         = ["morrow748@gmail.com"]
-  spec.summary       = %q{A gem for extracting test cases from a Cucumber test suite.}
-  spec.description   = %q{Slices a Cucumber test suite into the smallest possible executable pieces (i.e. scenarios and individual outline example rows. These pieces can then be more easily used for running tests in parallel.}
-  spec.homepage      = "https://github.com/enkessler/cuke_slicer"
-  spec.license       = "MIT"
+  spec.authors       = ['Eric Kessler']
+  spec.email         = ['morrow748@gmail.com']
+  spec.summary       = 'A gem for extracting test cases from a Cucumber test suite.'
+  spec.description   = ['Slices a Cucumber test suite into the smallest possible executable pieces (i.e. scenarios ',
+                        'and individual outline example rows. These pieces can then be more easily used for running ',
+                        'tests in parallel.'].join
+  spec.homepage      = 'https://github.com/enkessler/cuke_slicer'
+  spec.license       = 'MIT'
 
-  spec.files         = `git ls-files -z`.split("\x0")
-  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
-  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
-  spec.require_paths = ["lib"]
+  spec.metadata = {
+    'bug_tracker_uri'   => 'https://github.com/enkessler/cuke_slicer/issues',
+    'changelog_uri'     => 'https://github.com/enkessler/cuke_slicer/blob/master/CHANGELOG.md',
+    'documentation_uri' => 'https://www.rubydoc.info/gems/cuke_slicer',
+    'homepage_uri'      => 'https://github.com/enkessler/cuke_slicer',
+    'source_code_uri'   => 'https://github.com/enkessler/cuke_slicer'
+  }
 
-  spec.required_ruby_version = '>= 1.9.3', '< 4.0'
+  # Specify which files should be added to the gem when it is released.
+  # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
+  spec.files = Dir.chdir(File.expand_path('', __dir__)) do
+    source_controlled_files = `git ls-files -z`.split("\x0")
+    source_controlled_files.keep_if { |file| file =~ %r{^(lib|testing/cucumber/features)} }
+    source_controlled_files + ['README.md', 'LICENSE.txt', 'CHANGELOG.md', 'cuke_slicer.gemspec']
+  end
+  spec.require_paths = ['lib']
 
-  spec.add_runtime_dependency "cuke_modeler", "< 4.0"
+  spec.required_ruby_version = '>= 2.3', '< 4.0'
 
+  spec.add_runtime_dependency 'cuke_modeler', '>= 3.2', '< 4.0'
+
+  spec.add_development_dependency 'childprocess', '< 5.0'
+  spec.add_development_dependency 'ffi', '< 2.0' # This is an invisible dependency for the `childprocess` gem on Windows
   spec.add_development_dependency 'bundler', '< 3.0'
   spec.add_development_dependency 'rake', '< 13.0.0'
-  spec.add_development_dependency "rspec", '~> 3.0'
+  spec.add_development_dependency 'rspec', '~> 3.0'
   spec.add_development_dependency 'cucumber', '< 5.0.0'
-  spec.add_development_dependency 'simplecov', '<= 0.16.1' # Coveralls gem does not support any newer version than this
-  spec.add_development_dependency "racatt", '~> 1.0'
-  spec.add_development_dependency 'coveralls', '< 1.0.0'
+  spec.add_development_dependency 'simplecov', '< 1.0.0'
+  spec.add_development_dependency 'simplecov-lcov', '< 1.0'
   spec.add_development_dependency 'rainbow', '< 4.0.0'
-
+  spec.add_development_dependency 'rubocop', '<= 0.50.0' # RuboCop can not lint against Ruby 2.0 after this version
+  spec.add_development_dependency 'yard', '< 1.0'
 end

data/lib/cuke_slicer/collections/nested_tag_collection.rb

@@ -1,22 +1,24 @@
-require "cuke_slicer/helpers/helpers"
+require 'cuke_slicer/helpers/helpers'
 
 
 # Internal helper module that is not part of the public API. Subject to change at any time.
 # :nodoc: all
 module CukeSlicer
+  # private
   class NestedTagCollection
 
     include Helpers
 
 
-    def initialize collection
+    def initialize(collection)
       self.nested_collection = collection
     end
 
+    # private
     def validate
       nested_collection.each do |element|
-        raise(ArgumentError, "Tag filters cannot be nested more than one level deep.") if element.is_a?(Array)
-        raise(ArgumentError, "Filter '#{element}' must be a String or Regexp. Got #{element.class}") unless str_regex?(element)
+        raise(ArgumentError, 'Tag filters cannot be nested more than one level deep.') if element.is_a?(Array)
+        raise(ArgumentError, "Filter '#{element}' must be a String or Regexp. Got #{element.class}") unless str_regex?(element) # rubocop:disable Metrics/LineLength
       end
     end
 

data/lib/cuke_slicer/collections/path_collection.rb

@@ -1,18 +1,20 @@
-require "cuke_slicer/helpers/helpers"
+require 'cuke_slicer/helpers/helpers'
 
 
 # Internal helper module that is not part of the public API. Subject to change at any time.
 # :nodoc: all
 module CukeSlicer
+  # private
   class PathCollection
 
     include Helpers
 
 
-    def initialize filter_values
+    def initialize(filter_values)
       self.filter_values = filter_values
     end
 
+    # private
     def validate
       filter_values.each do |val|
         raise(ArgumentError, "Filter '#{val}' must be a String or Regexp. Got #{val.class}") unless str_regex?(val)

data/lib/cuke_slicer/collections/tag_collection.rb

@@ -1,22 +1,24 @@
-require "cuke_slicer/helpers/helpers"
-require "cuke_slicer/collections/nested_tag_collection"
+require 'cuke_slicer/helpers/helpers'
+require 'cuke_slicer/collections/nested_tag_collection'
 
 
 # Internal helper module that is not part of the public API. Subject to change at any time.
 # :nodoc: all
 module CukeSlicer
+  # private
   class TagCollection
 
     include Helpers
 
 
-    def initialize parameters
+    def initialize(parameters)
       self.filter_values = parameters
     end
 
+    # private
     def validate
       filter_values.each do |val|
-        raise(ArgumentError, "Filter '#{val}' must be a String, Regexp, or Array. Got #{val.class}") unless str_regex_arr?(val)
+        raise(ArgumentError, "Filter '#{val}' must be a String, Regexp, or Array. Got #{val.class}") unless str_regex_arr?(val) # rubocop:disable Metrics/LineLength
         NestedTagCollection.new(val).validate if val.is_a?(Array)
       end
     end

data/lib/cuke_slicer/extractors/directory_extractor.rb

@@ -1,10 +1,12 @@
 # Internal helper module that is not part of the public API. Subject to change at any time.
 # :nodoc: all
 module CukeSlicer
+  # private
   class DirectoryExtractor
 
+    # private
     def extract(target, filters, format, &block)
-      Array.new.tap do |test_cases|
+      [].tap do |test_cases|
         target.feature_files.each do |feature_file|
           test_cases.concat(FileExtractor.new.extract(feature_file, filters, format, &block))
         end

data/lib/cuke_slicer/extractors/file_extractor.rb

@@ -1,20 +1,25 @@
-require "cuke_slicer/helpers/matching_helpers"
-require "cuke_slicer/helpers/filter_helpers"
-require "cuke_slicer/helpers/extraction_helpers"
+require 'cuke_slicer/helpers/matching_helpers'
+require 'cuke_slicer/helpers/filter_helpers'
+require 'cuke_slicer/helpers/extraction_helpers'
 
+# TODO: get rid of this class
+# rubocop:disable Metrics/AbcSize
 
 # Internal helper module that is not part of the public API. Subject to change at any time.
 # :nodoc: all
 module CukeSlicer
+  # private
   class FileExtractor
 
     include ExtractionHelpers
 
 
+    # private
     def extract(target, filters, format, &block)
-      Array.new.tap do |test_cases|
+      [].tap do |test_cases|
         unless target.feature.nil?
           tests = target.feature.tests
+          tests += target.feature.rules.flat_map(&:tests)
 
           runnable_elements = extract_runnable_elements(extract_runnable_block_elements(tests, filters, &block))
 
@@ -32,3 +37,5 @@ module CukeSlicer
 
   end
 end
+
+# rubocop:enable Metrics/AbcSize

data/lib/cuke_slicer/filters/filter_set.rb

@@ -1,29 +1,31 @@
-require "cuke_slicer/helpers/helpers"
-require "cuke_slicer/collections/tag_collection"
-require "cuke_slicer/collections/path_collection"
+require 'cuke_slicer/helpers/helpers'
+require 'cuke_slicer/collections/tag_collection'
+require 'cuke_slicer/collections/path_collection'
 
 
 # Internal helper module that is not part of the public API. Subject to change at any time.
 # :nodoc: all
 module CukeSlicer
+  # private
   class FilterSet
 
     include Helpers
 
 
-    def initialize filter_type, filter_value
+    def initialize(filter_type, filter_value)
       self.filter_type = filter_type
       self.filter_value = filter_value
     end
 
+    # private
     def validate
       block_unknown
       block_invalid
 
-      if filter_value.is_a?(Array)
-        TagCollection.new(filter_value).validate if is_tag?(filter_type)
-        PathCollection.new(filter_value).validate if is_path?(filter_type)
-      end
+      return unless filter_value.is_a?(Array)
+
+      TagCollection.new(filter_value).validate if tag?(filter_type)
+      PathCollection.new(filter_value).validate if path?(filter_type)
     end
 
 
@@ -31,11 +33,11 @@ module CukeSlicer
 
 
     def block_unknown
-      raise(ArgumentError, "Unknown filter '#{filter_type}'") unless CukeSlicer::Slicer.known_filters.include?(filter_type)
+      raise(ArgumentError, "Unknown filter '#{filter_type}'") unless CukeSlicer::Slicer.known_filters.include?(filter_type) # rubocop:disable Metrics/LineLength
     end
 
     def block_invalid
-      raise(ArgumentError, "Invalid filter '#{filter_value}'. Must be a String, Regexp, or Array thereof. Got #{filter_value.class}") unless str_regex_arr?(filter_value)
+      raise(ArgumentError, "Invalid filter '#{filter_value}'. Must be a String, Regexp, or Array thereof. Got #{filter_value.class}") unless str_regex_arr?(filter_value) # rubocop:disable Metrics/LineLength
     end
 
     attr_accessor :filter_type, :filter_value

data/lib/cuke_slicer/helpers/extraction_helpers.rb

@@ -1,13 +1,19 @@
+# TODO: get rid of this module
+# rubocop:disable Metrics/AbcSize
+
+
 # Internal helper module that is not part of the public API. Subject to change at any time.
 # :nodoc: all
 module CukeSlicer
+  # private
   module ExtractionHelpers
 
     include FilterHelpers
 
 
+    # private
     def extract_runnable_block_elements(things, filters, &block)
-      Array.new.tap do |elements|
+      [].tap do |elements|
         things.each do |thing|
           if thing.is_a?(CukeModeler::Outline)
             elements.concat(thing.examples)
@@ -25,14 +31,13 @@ module CukeSlicer
       end
     end
 
+    # private
     def extract_runnable_elements(things)
-      method_for_row_models = Gem.loaded_specs['cuke_modeler'].version.version[/^0/] ? :row_elements : :rows
-
-      Array.new.tap do |elements|
+      [].tap do |elements|
         things.each do |thing|
           if thing.is_a?(CukeModeler::Example)
             # Slicing in order to remove the parameter row element
-            elements.concat(thing.send(method_for_row_models).slice(1, thing.rows.count))
+            elements.concat(thing.rows.slice(1, thing.rows.count))
           else
             elements << thing
           end
@@ -42,3 +47,4 @@ module CukeSlicer
 
   end
 end
+# rubocop:enable Metrics/AbcSize

data/lib/cuke_slicer/helpers/filter_helpers.rb

@@ -1,60 +1,66 @@
 # Internal helper module that is not part of the public API. Subject to change at any time.
 # :nodoc: all
 module CukeSlicer
+  # private
   module FilterHelpers
 
     include MatchingHelpers
 
 
+    # private
     def apply_custom_filter(elements, &block)
-      if block
-        elements.reject! do |element|
-          block.call(element)
-        end
+      return unless block
+
+      elements.reject! do |element|
+        yield(element)
       end
     end
 
+    # private
     def filter_excluded_tags(elements, filters)
-      if filters
-        filters = [filters] unless filters.is_a?(Array)
-
-        unless filters.empty?
-          elements.reject! do |element|
-            matching_tag?(element, filters)
-          end
-        end
+      return unless filters
+
+      filters = [filters] unless filters.is_a?(Array)
+
+      return if filters.empty?
+
+      elements.reject! do |element|
+        matching_tag?(element, filters)
       end
     end
 
+    # private
     def filter_included_tags(elements, filters)
-      if filters
-        filters = [filters] unless filters.is_a?(Array)
+      return unless filters
+
+      filters = [filters] unless filters.is_a?(Array)
 
-        elements.keep_if do |element|
-          matching_tag?(element, filters)
-        end
+      elements.keep_if do |element|
+        matching_tag?(element, filters)
       end
     end
 
+    # private
     def filter_excluded_paths(elements, filters)
-      if filters
-        filters = [filters] unless filters.is_a?(Array)
+      return unless filters
 
-        elements.reject! do |element|
-          matching_path?(element, filters)
-        end
+      filters = [filters] unless filters.is_a?(Array)
+
+      elements.reject! do |element|
+        matching_path?(element, filters)
       end
     end
 
+    # private
     def filter_included_paths(elements, filters)
-      if filters
-        filters = [filters] unless filters.is_a?(Array)
-
-        unless filters.empty?
-          elements.keep_if do |element|
-            matching_path?(element, filters)
-          end
-        end
+      return unless filters
+
+      filters = [filters] unless filters.is_a?(Array)
+
+      return if filters.empty?
+
+      elements.keep_if do |element|
+        matching_path?(element, filters)
       end
     end
 

data/lib/cuke_slicer/helpers/helpers.rb

@@ -1,21 +1,22 @@
 # Internal helper module that is not part of the public API. Subject to change at any time.
 # :nodoc: all
 module CukeSlicer
+  # private
   module Helpers
 
     def str_regex?(parameter)
-      parameter.is_a?(String) or parameter.is_a?(Regexp)
+      parameter.is_a?(String) || parameter.is_a?(Regexp)
     end
 
     def str_regex_arr?(parameter)
-      parameter.is_a?(String) or parameter.is_a?(Regexp) or parameter.is_a?(Array)
+      parameter.is_a?(String) || parameter.is_a?(Regexp) || parameter.is_a?(Array)
     end
 
-    def is_tag?(parameter)
+    def tag?(parameter)
       parameter.to_s =~ /tag/
     end
 
-    def is_path?(parameter)
+    def path?(parameter)
       parameter.to_s =~ /path/
     end
 

data/lib/cuke_slicer/helpers/matching_helpers.rb

@@ -1,15 +1,17 @@
 # Internal helper module that is not part of the public API. Subject to change at any time.
 # :nodoc: all
 module CukeSlicer
+  # private
   module MatchingHelpers
 
+    # private
     def matching_tag?(element, filters)
       filters.each do |filter|
-        if filter.is_a?(Array)
-          filter_match = or_filter_match(element, filter)
-        else
-          filter_match = and_filter_match(element, filter)
-        end
+        filter_match = if filter.is_a?(Array)
+                         or_filter_match(element, filter)
+                       else
+                         and_filter_match(element, filter)
+                       end
 
         return false unless filter_match
       end
@@ -17,19 +19,21 @@ module CukeSlicer
       true
     end
 
+    # private
     def and_filter_match(element, filter)
       filter_match(element, filter)
     end
 
+    # private
     def or_filter_match(element, filters)
       filters.any? do |filter|
         filter_match(element, filter)
       end
     end
 
+    # private
     def filter_match(element, filter)
-      tag_values = element.all_tags
-      tag_values = tag_values.collect { |tag| tag.name } unless Gem.loaded_specs['cuke_modeler'].version.version[/^[0]/]
+      tag_values = element.all_tags.map(&:name)
 
       if filter.is_a?(Regexp)
         tag_values.any? { |tag| tag =~ filter }
@@ -38,6 +42,7 @@ module CukeSlicer
       end
     end
 
+    # private
     def matching_path?(element, filters)
       filters.any? do |filtered_path|
         if filtered_path.is_a?(Regexp)

data/lib/cuke_slicer/slicer.rb

@@ -1,8 +1,9 @@
-require "cuke_slicer/extractors/directory_extractor"
-require "cuke_slicer/extractors/file_extractor"
-require "cuke_slicer/filters/filter_set"
-
+require 'cuke_slicer/extractors/directory_extractor'
+require 'cuke_slicer/extractors/file_extractor'
+require 'cuke_slicer/filters/filter_set'
 
+# TODO: refactor all of this
+# rubocop:disable Metrics/AbcSize, Style/OptionalArguments
 module CukeSlicer
 
   # The object responsible for slicing up a Cucumber test suite into discrete test cases.
@@ -32,29 +33,23 @@ module CukeSlicer
       begin
         target = File.directory?(target) ? CukeModeler::Directory.new(target) : CukeModeler::FeatureFile.new(target)
       rescue => e
-        if e.message =~ /lexing|parsing/i
-          raise(ArgumentError, "A syntax or lexing problem was encountered while trying to parse #{target}")
-        else
-          raise e
-        end
-
+        raise e unless e.message =~ /lexing|parsing/i
+        raise(ArgumentError, "A syntax or lexing problem was encountered while trying to parse #{target}")
       end
 
       if target.is_a?(CukeModeler::Directory)
-        sliced_tests = DirectoryExtractor.new.extract(target, filters, format, &block)
+        DirectoryExtractor.new.extract(target, filters, format, &block)
       else
-        sliced_tests = FileExtractor.new.extract(target, filters, format, &block)
+        FileExtractor.new.extract(target, filters, format, &block)
       end
-
-      sliced_tests
     end
 
     # The filtering options that are currently supported by the slicer.
     def self.known_filters
-      [:excluded_tags,
-       :included_tags,
-       :excluded_paths,
-       :included_paths]
+      %i[excluded_tags
+         included_tags
+         excluded_paths
+         included_paths]
     end
 
 
@@ -62,7 +57,7 @@ module CukeSlicer
 
 
     def validate_target(target)
-      raise(ArgumentError, "File or directory '#{target}' does not exist") unless File.exists?(target.to_s)
+      raise(ArgumentError, "File or directory '#{target}' does not exist") unless File.exist?(target.to_s)
     end
 
     def validate_filters(filters)
@@ -71,8 +66,9 @@ module CukeSlicer
     end
 
     def validate_format(format)
-      raise(ArgumentError, "Invalid Output Format: #{format}") unless [:test_object, :file_line].include?(format)
+      raise(ArgumentError, "Invalid Output Format: #{format}") unless %i[test_object file_line].include?(format)
     end
 
   end
 end
+# rubocop:enable Metrics/AbcSize, Style/OptionalArguments