mutant 0.5.16 → 0.5.17

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9eae77be848852158b71548e280781bfb5a4dd55
-  data.tar.gz: 05c57b8131bbbc16f7284c6528a075f84e8efc21
+  metadata.gz: 5ed19319f31c14a3b2677f67cda130f8ae053af1
+  data.tar.gz: 6cb628e6e446ed9d1f0c14ad88cfd254d49ee216
 SHA512:
-  metadata.gz: 046f7c2d99e04971d914eba1761ef5e469b9fd3f62b1a3c61f1897ff4c5c38d6f851a9f72fc436d2a31418d8d64ef24fd2fcbcf97766d7a02e0be84600a2ed22
-  data.tar.gz: 87f6eec29d72085a441e1eb678b8287895021ecdfbacdc165b9c867a9d9816fe29a9d738a756bf0cff01fe5b23218eccefd6108db9c264a3bbaeb55ae1eec2e3
+  metadata.gz: 482eeee0d39d0e9b869b6a0625d71bf7b08bbdf8acff2fc9f29b44a9b74472cd038e166f053e17323dd41d46295f802245db69d7bb6dc7085336243bc448351b
+  data.tar.gz: f332afc9b4404fbcb8ee20e5e74fe6789df531c856d1fc4562d5cb92dee1d771d900405ba044f5360bad39f6400002786584da7f0f4c6efce0be2c5bd4ba5f64

data/.rubocop.yml

@@ -1,8 +1,8 @@
 AllCops:
-  Includes:
+  Include:
     - 'Gemfile'
-  Excludes:
+  Exclude:
     - 'Gemfile.devtools'
-    - 'vendor/**'
-    - 'tmp/**'
-    - 'benchmarks/**'
+    - 'vendor/**/*'
+    - 'tmp/**/*'
+    - 'benchmarks/**/*'

data/.travis.yml

@@ -6,10 +6,10 @@ rvm:
   - 1.9.3
   - 2.0.0
   - 2.1.1
-  - rbx
+  - rbx-2
 matrix:
   allow_failures:
-    - rvm: rbx
+    - rvm: rbx-2
 notifications:
   irc:
     channels:

data/Changelog.md

@@ -1,3 +1,11 @@
+# v0.5.17 2014-05-27
+
+Changes:
+
+* Report selected tests in progress runner
+* Fix scope of rspec selections to include meaningful parents.
+* Add short circuts on already dead mutations under multiple test selections.
+
 # v0.5.16 2014-05-27
 
 Changes:

data/Gemfile.devtools

@@ -1,34 +1,30 @@
 # encoding: utf-8
 
 group :development do
-  gem 'rake',       '~> 10.1.0'
-  gem 'rspec',      '~> 2.14.1'
-  gem 'rspec-core', '~> 2.14.8'
-  gem 'yard',       '~> 0.8.7'
+  gem 'rake',       '~> 10.3.1'
+  gem 'rspec',      '~> 2.99.0.beta2'
+  gem 'rspec-core', '~> 2.99.0.beta2'
+  gem 'yard',       '~> 0.8.7.4'
 
   platform :rbx do
     gem 'rubysl-singleton', '~> 2.0.0'
   end
 end
 
-group :yard do
-  gem 'kramdown', '~> 1.3.2'
-end
-
 group :guard do
-  gem 'guard',         '~> 2.4.0'
+  gem 'guard',         '~> 2.6.0'
   gem 'guard-bundler', '~> 2.0.0'
-  gem 'guard-rspec',   '~> 4.2.6'
-  gem 'guard-rubocop', '~> 1.0.2'
+  gem 'guard-rspec',   '~> 4.2.8'
+  gem 'guard-rubocop', '~> 1.1.0'
 
   # file system change event handling
-  gem 'listen',     '~> 2.5.0'
+  gem 'listen',     '~> 2.7.3'
   gem 'rb-fchange', '~> 0.0.6', require: false
-  gem 'rb-fsevent', '~> 0.9.3', require: false
-  gem 'rb-inotify', '~> 0.9.0', require: false
+  gem 'rb-fsevent', '~> 0.9.4', require: false
+  gem 'rb-inotify', '~> 0.9.4', require: false
 
   # notification handling
-  gem 'libnotify',               '~> 0.8.0', require: false
+  gem 'libnotify',               '~> 0.8.2', require: false
   gem 'rb-notifu',               '~> 0.0.4', require: false
   gem 'terminal-notifier-guard', '~> 1.5.3', require: false
 end
@@ -37,9 +33,9 @@ group :metrics do
   gem 'coveralls', '~> 0.7.0'
   gem 'flay',      '~> 2.4.0'
   gem 'flog',      '~> 4.2.0'
-  gem 'reek',      '~> 1.3.2'
-  gem 'rubocop',   '~> 0.18.1'
-  gem 'simplecov', '~> 0.8.2'
+  gem 'reek',      '~> 1.3.7'
+  gem 'rubocop',   '~> 0.21.0'
+  gem 'simplecov', '~> 0.7.1'
   gem 'yardstick', '~> 0.9.9'
 
   platforms :ruby_19, :ruby_20 do
@@ -48,10 +44,10 @@ group :metrics do
 
   platform :rbx do
     gem 'json',               '~> 1.8.1'
-    gem 'racc',               '~> 1.4'
+    gem 'racc',               '~> 1.4.11'
     gem 'rubysl-logger',      '~> 2.0.0'
     gem 'rubysl-open-uri',    '~> 2.0.0'
-    gem 'rubysl-prettyprint', '~> 2.0.2'
+    gem 'rubysl-prettyprint', '~> 2.0.3'
   end
 end
 
@@ -61,6 +57,6 @@ end
 
 platform :jruby do
   group :jruby do
-    gem 'jruby-openssl', '~> 0.8.5'
+    gem 'jruby-openssl', '~> 0.9.4'
   end
 end

data/config/flay.yml

@@ -1,3 +1,3 @@
 ---
 threshold: 18
-total_score: 794
+total_score: 801

data/config/rubocop.yml

@@ -7,7 +7,7 @@ inherit_from: ../.rubocop.yml
 
 
 AllCops:
-  Includes:
+  Include:
     - '../**/*.rake'
     - 'Gemfile'
     - 'Gemfile.triage'
@@ -30,6 +30,19 @@ CollectionMethods:
     find:     'detect'
     find_all: 'select'
 
+# Use square brackets for literal Array objects
+PercentLiteralDelimiters:
+  PreferredDelimiters:
+    '%':  ()
+    '%i': '[]'
+    '%q': ()
+    '%Q': ()
+    '%r': '{}'
+    '%s': ()
+    '%w': '[]'
+    '%W': '[]'
+    '%x': ()
+
 MethodLength:
   CountComments: false
   Max: 17 # TODO: Bring down to 10

data/lib/mutant/cli.rb

@@ -285,7 +285,7 @@ module Mutant
     # @api private
     #
     def use(name)
-      require "mutant-#{name}"
+      require "mutant/#{name}"
       @strategy = Strategy.lookup(name).new
     rescue LoadError
       $stderr.puts("Cannot load plugin: #{name.inspect}")
@@ -339,7 +339,7 @@ module Mutant
     def add_debug_options(opts)
       opts.on('--fail-fast', 'Fail fast') do
         @fail_fast = true
-      end.on('--version', 'Print mutants version') do |name|
+      end.on('--version', 'Print mutants version') do
         puts("mutant-#{Mutant::VERSION}")
         Kernel.exit(0)
       end.on('-d', '--debug', 'Enable debugging output') do

data/lib/mutant/cli/classifier.rb

@@ -132,7 +132,7 @@ module Mutant
       # @api private
       #
       abstract_method :matcher
-      private         :matcher
+      private :matcher
 
     end # Classifier
   end # CLI

data/lib/mutant/constants.rb

@@ -5,20 +5,20 @@ module Mutant
   symbolset = ->(strings) { strings.map(&:to_sym).to_set.freeze }
 
   # Set of nodes that cannot be on the LHS of an assignment
-  NOT_ASSIGNABLE = symbolset.(%w(int float str dstr class module self))
+  NOT_ASSIGNABLE = symbolset.(%w[int float str dstr class module self])
 
   # Set of op-assign types
-  OP_ASSIGN              = symbolset.call(%w(or_asgn and_asgn op_asgn))
+  OP_ASSIGN              = symbolset.call(%w[or_asgn and_asgn op_asgn])
   # Set of node types that are not valid when emitted standalone
-  NOT_STANDALONE         = symbolset.(%w( splat restarg block_pass))
-  INDEX_OPERATORS        = symbolset.(%w([] []=))
-  UNARY_METHOD_OPERATORS = symbolset.(%w(~@ +@ -@ !))
+  NOT_STANDALONE         = symbolset.(%w[ splat restarg block_pass])
+  INDEX_OPERATORS        = symbolset.(%w[[] []=])
+  UNARY_METHOD_OPERATORS = symbolset.(%w[~@ +@ -@ !])
 
   # Operators ruby implementeds as methods
-  METHOD_OPERATORS = symbolset.(%w(
+  METHOD_OPERATORS = symbolset.(%w[
     <=> === []= [] <= >= == !~ != =~ <<
     >> ** * % / | ^ & < > + - ~@ +@ -@ !
-  ))
+  ])
 
   BINARY_METHOD_OPERATORS = (
     METHOD_OPERATORS - (INDEX_OPERATORS + UNARY_METHOD_OPERATORS)
@@ -32,10 +32,10 @@ module Mutant
   #
   # not - 1.8 only, mutant does not support 1.8
   #
-  NODE_BLACKLIST = symbolset.(%w(not))
+  NODE_BLACKLIST = symbolset.(%w[not])
 
   # Nodes that are NOT generated by parser but used by mutant / unparser.
-  NODE_EXTRA = symbolset.(%w(empty))
+  NODE_EXTRA = symbolset.(%w[empty])
 
   NODE_TYPES = ((Parser::Meta::NODE_TYPES + NODE_EXTRA) - NODE_BLACKLIST).to_set.freeze
 

data/lib/mutant/matcher/method.rb

@@ -46,7 +46,7 @@ module Mutant
       def skip?
         location = source_location
         if location.nil? || BLACKLIST.match(location.first)
-          message = sprintf(
+          message = format(
             '%s does not have valid source location unable to emit matcher',
             method.inspect
           )

data/lib/mutant/matcher/method/singleton.rb

@@ -94,7 +94,7 @@ module Mutant
           when :const
             receiver_name?(receiver)
           else
-            message = sprintf(
+            message = format(
               'Can only match :defs on :self or :const got %s unable to match',
               receiver.type.inspect
             )

data/lib/mutant/matcher/null.rb

@@ -16,7 +16,7 @@ module Mutant
       #
       # @api private
       #
-      def each(&block)
+      def each
         return to_enum unless block_given?
         self
       end

data/lib/mutant/mutator/node/arguments.rb

@@ -43,7 +43,7 @@ module Mutant
         # @api private
         #
         def mlhs_childs_with_index
-          children.each_with_index.select do |child, index|
+          children.each_with_index.select do |child, _index|
             child.type == :mlhs
           end
         end

data/lib/mutant/mutator/node/send.rb

@@ -145,7 +145,7 @@ module Mutant
         #
         def mutate_arguments
           emit_self(receiver, selector)
-          remaining_children_with_index.each do |node, index|
+          remaining_children_with_index.each do |_node, index|
             mutate_child(index)
             delete_child(index)
           end

data/lib/mutant/reporter/cli/printer.rb

@@ -60,7 +60,7 @@ module Mutant
         # @api private
         #
         def info(string, *arguments)
-          puts(sprintf(string, *arguments))
+          puts(format(string, *arguments))
         end
 
         # Print a status line to output
@@ -70,7 +70,7 @@ module Mutant
         # @api private
         #
         def status(string, *arguments)
-          puts(colorize(status_color, sprintf(string, *arguments)))
+          puts(colorize(status_color, format(string, *arguments)))
         end
 
         # Print a line to output

data/lib/mutant/reporter/cli/progress.rb

@@ -6,6 +6,8 @@ module Mutant
       # Abstract base class for process printers
       class Progress < Printer
         include AbstractType, Registry.new
+
+        delegate :running?
       end # Progress
     end # CLI
   end # Reporter

data/lib/mutant/reporter/cli/progress/config.rb

@@ -5,9 +5,9 @@ module Mutant
         # Progress printer for configuration
         class Config < self
 
-          handle(Mutant::Config)
+          handle(Mutant::Runner::Config)
 
-          delegate :matcher, :strategy, :expected_coverage
+          delegate :config
 
           # Report configuration
           #
@@ -18,10 +18,12 @@ module Mutant
           # @api private
           #
           def run
-            info 'Mutant configuration:'
-            info 'Matcher:         %s',     matcher.inspect
-            info 'Strategy:        %s',     strategy.inspect
-            info 'Expect Coverage: %02f%%', expected_coverage.inspect
+            if running?
+              info 'Mutant configuration:'
+              info 'Matcher:         %s',     config.matcher.inspect
+              info 'Strategy:        %s',     config.strategy.inspect
+              info 'Expect Coverage: %02f%%', config.expected_coverage.inspect
+            end
             self
           end
 

data/lib/mutant/reporter/cli/progress/mutation.rb

@@ -13,12 +13,15 @@ module Mutant
 
           # Run printer
           #
-          # @return [undefined]
+          # @return [self]
           #
           # @api private
           #
           def run
-            char(success? ? SUCCESS : FAILURE)
+            unless running?
+              char(success? ? SUCCESS : FAILURE)
+            end
+            self
           end
 
         private

data/lib/mutant/reporter/cli/progress/noop.rb

@@ -5,7 +5,7 @@ module Mutant
         # Noop CLI progress reporter
         class Noop < self
 
-          handle(Mutant::Mutation)
+          handle(Mutant::Runner::Killer)
 
           # Noop progress report
           #

data/lib/mutant/reporter/cli/progress/subject.rb

@@ -2,30 +2,15 @@ module Mutant
   class Reporter
     class CLI
       class Progress
-        # Subject results printer
-        class Subject < self
-
-          handle(Mutant::Subject)
-
-          # Run subject results printer
-          #
-          # @return [undefined]
-          #
-          # @api private
-          #
-          def run
-            puts(object.identification)
-          end
-
-        end # Subject
-
         # Reporter for subject runners
-        class SubjectRunner < self
+        class Subject < self
 
           FORMAT = '(%02d/%02d) %3d%% - %0.02fs'.freeze
 
           handle(Mutant::Runner::Subject)
 
+          delegate :running?, :tests, :subject
+
           # Run printer
           #
           # @return [undefined]
@@ -33,8 +18,15 @@ module Mutant
           # @api private
           #
           def run
-            print_progress_bar_finish
-            print_stats
+            if running?
+              puts(subject.identification)
+              tests.each do |test|
+                puts "- #{test.identification}"
+              end
+            else
+              print_progress_bar_finish
+              print_stats
+            end
             self
           end
 

data/lib/mutant/reporter/cli/report/config.rb

@@ -23,13 +23,13 @@ module Mutant
           #
           def run
             failed_subjects.each(&method(:visit))
-            info   'Subjects:  %s',      amount_subjects
-            info   'Mutations: %s',      amount_mutations
-            info   'Kills:     %s',      amount_kills
-            info   'Alive:     %s',      amount_alive
-            info   'Runtime:   %0.2fs',  runtime
-            info   'Killtime:  %0.2fs',  killtime
-            info   'Overhead:  %0.2f%%', overhead
+            info 'Subjects:  %s',      amount_subjects
+            info 'Mutations: %s',      amount_mutations
+            info 'Kills:     %s',      amount_kills
+            info 'Alive:     %s',      amount_alive
+            info 'Runtime:   %0.2fs',  runtime
+            info 'Killtime:  %0.2fs',  killtime
+            info 'Overhead:  %0.2f%%', overhead
             status 'Coverage:  %0.2f%%', coverage
             status 'Expected:  %0.2f%%', object.config.expected_coverage
             print_generic_stats

data/lib/mutant/runner.rb

@@ -53,6 +53,16 @@ module Mutant
       handler.new(config, object, *arguments)
     end
 
+    # Test if runner is running
+    #
+    # Yeah this is evil. Should be refactored away
+    #
+    # @return [Boolean]
+    #
+    def running?
+      @running
+    end
+
     # Return config
     #
     # @return [Mutant::Config]
@@ -73,7 +83,11 @@ module Mutant
       @config = config
       @stop   = false
       @start  = Time.now
+      @running = true
+      progress(self)
       run
+      @running = false
+      progress(self)
       @end = Time.now
     end
 

data/lib/mutant/runner/config.rb

@@ -134,7 +134,6 @@ module Mutant
       # @api private
       #
       def run
-        progress(config)
         run_subjects
         @end = Time.now
         reporter.report(self)

data/lib/mutant/runner/killer.rb

@@ -32,6 +32,15 @@ module Mutant
         @report.success?
       end
 
+      # Test if this killer ALONE can determine if mutation is dead
+      #
+      # @return [Boolean]
+      #
+      def mutation_dead?
+        test_report = report.test_report
+        killer.mutation.should_fail? && test_report.failed?
+      end
+
       # Initialize object
       #
       # @param [Config] config
@@ -46,6 +55,8 @@ module Mutant
         super(config)
       end
 
+    private
+
       # Run killer
       #
       # @api private

data/lib/mutant/runner/mutation.rb

@@ -63,14 +63,20 @@ module Mutant
       # @api private
       #
       def run
-        progress(mutation)
-        @killers = @tests.map do |test|
+        @killers = []
+
+        killers = @tests.map do |test|
           Mutant::Killer.new(
             mutation: mutation,
             test:     test
           )
-        end.map(&method(:visit))
-        progress(self)
+        end
+
+        killers.each do |killer|
+          runner = visit(killer)
+          @killers << runner
+          return if runner.mutation_dead?
+        end
       end
 
     end # Mutation

data/lib/mutant/runner/subject.rb

@@ -83,9 +83,7 @@ module Mutant
       # @api private
       #
       def run
-        progress(subject)
         @mutations = visit_collection(subject.mutations, tests)
-        progress(self)
       end
 
     end # Subject

data/lib/mutant/strategy.rb

@@ -83,8 +83,6 @@ module Mutant
       []
     end
 
-  private
-
     # Null strategy that never kills a mutation
     class Null < self
 

data/lib/mutant/subject/method/instance.rb

@@ -43,8 +43,6 @@ module Mutant
           self
         end
 
-      private
-
         # Mutator for memoized instance methods
         class Memoized < self
           include NodeHelpers

data/lib/mutant/version.rb

@@ -2,5 +2,5 @@
 
 module Mutant
   # The current mutant version
-  VERSION = '0.5.16'.freeze
+  VERSION = '0.5.17'.freeze
 end # Mutant

data/lib/mutant/zombifier.rb

@@ -6,16 +6,16 @@ module Mutant
     include Adamantium::Flat, Concord.new(:namespace)
 
     # Excluded into zombification
-    includes = %w(
+    includes = %w[
       mutant
       morpher
       adamantium
       equalizer
       anima
       concord
-    )
+    ]
 
-    INCLUDES = %r(\A#{Regexp.union(includes)}(?:/.*)?\z).freeze
+    INCLUDES = %r{\A#{Regexp.union(includes)}(?:/.*)?\z}.freeze
 
     # Initialize object
     #

data/lib/mutant/zombifier/file.rb

@@ -11,7 +11,7 @@ module Mutant
       # @api private
       #
       def zombify(namespace)
-        $stderr.puts("Zombifying #{path.to_s}")
+        $stderr.puts("Zombifying #{path}")
         eval(
           Unparser.unparse(namespaced_node(namespace)),
           TOPLEVEL_BINDING,

data/mutant-rspec.gemspec

@@ -13,7 +13,7 @@ Gem::Specification.new do |gem|
   gem.license     = 'MIT'
 
   gem.require_paths    = %w[lib]
-  gem.files            = `git ls-files -- lib/mutant{-,/}rspec.rb lib/mutant/rspec`.split("\n")
+  gem.files            = `git ls-files -- lib/mutant/rspec.rb lib/mutant/rspec`.split("\n")
   gem.test_files       = `git ls-files -- spec/{unit/mutant/rspec,integration/rspec}`.split("\n")
   gem.extra_rdoc_files = %w[TODO LICENSE]
 

data/spec/integration/mutant/corpus_spec.rb

@@ -23,13 +23,14 @@ describe 'Mutant on ruby corpus' do
     # @raise [Exception]
     #   otherwise
     #
+    # rubocop:disable MethodLength
     def verify
       checkout
       total = 0
       parse_errors = []
       start = Time.now
       Pathname.glob(repo_path.join('**/*.rb')).sort.each do |path|
-        puts "Generating mutations for: #{path.to_s}"
+        puts "Generating mutations for: #{path}"
         begin
           node = Parser::CurrentRuby.parse(path.read)
         # Ignore known parser bugs
@@ -39,7 +40,7 @@ describe 'Mutant on ruby corpus' do
         end
         next if node.nil?
         count = 0
-        Mutant::Mutator::Node.each(node) do |mutant|
+        Mutant::Mutator::Node.each(node) do
           count += 1
           if (count % 1000).zero?
             puts count
@@ -49,9 +50,13 @@ describe 'Mutant on ruby corpus' do
         total += count
       end
       took = Time.now - start
-      puts "Total Mutations/Time/Parse-Errors: %s/%0.2fs/%i - %0.2f/s" % [
-        total, took, parse_errors.size, total / took
-      ]
+      puts format(
+        'Total Mutations/Time/Parse-Errors: %s/%0.2fs/%i - %0.2f/s',
+        total,
+        took,
+        parse_errors.size,
+        total / took
+      )
       if parse_errors.any?
         puts 'Files with parse errors:'
         parse_errors.each(&method(:puts))
@@ -69,11 +74,11 @@ describe 'Mutant on ruby corpus' do
       TMP.mkdir unless TMP.directory?
       if repo_path.exist?
         Dir.chdir(repo_path) do
-          system(%w(git pull origin master))
-          system(%w(git clean -f -d -x))
+          system(%w[git pull origin master])
+          system(%w[git clean -f -d -x])
         end
       else
-        system(%W(git clone #{repo_uri} #{repo_path}))
+        system(%W[git clone #{repo_uri} #{repo_path}])
       end
       self
     end

data/spec/integration/mutant/zombie_spec.rb

@@ -4,7 +4,7 @@ require 'spec_helper'
 
 describe 'as a zombie' do
   specify 'it allows to create zombie from mutant' do
-    expect { Mutant.zombify }.to change { !!defined?(Zombie) }.from(false).to(true)
+    expect { Mutant.zombify }.to change { defined?(Zombie) }.from(nil).to('constant')
     expect(Zombie.constants).to include(:Mutant)
   end
 end

data/spec/unit/mutant/cli_new_spec.rb

@@ -15,7 +15,7 @@ shared_examples_for 'a cli parser' do
 
   it { expect(subject.strategy).to eql(expected_strategy) }
   it { expect(subject.reporter).to eql(expected_reporter) }
-  it { expect(subject.matcher).to  eql(expected_matcher)  }
+  it { expect(subject.matcher).to eql(expected_matcher)   }
 end
 
 describe Mutant::CLI, '.new' do
@@ -39,7 +39,7 @@ describe Mutant::CLI, '.new' do
   subject { cli }
 
   context 'with unknown flag' do
-    let(:arguments) { %w(--invalid) }
+    let(:arguments) { %w[--invalid] }
 
     let(:expected_message) { 'invalid option: --invalid' }
 
@@ -47,7 +47,7 @@ describe Mutant::CLI, '.new' do
   end
 
   context 'with unknown option' do
-    let(:arguments) { %w(--invalid Foo) }
+    let(:arguments) { %w[--invalid Foo] }
 
     let(:expected_message) { 'invalid option: --invalid' }
 
@@ -63,14 +63,14 @@ describe Mutant::CLI, '.new' do
   end
 
   context 'with code filter and missing argument' do
-    let(:arguments)        { %w(--code)    }
+    let(:arguments)        { %w[--code]                 }
     let(:expected_message) { 'missing argument: --code' }
 
     it_should_behave_like 'an invalid cli run'
   end
 
   context 'with explicit method pattern' do
-    let(:arguments)        { %w(TestApp::Literal#float) }
+    let(:arguments)        { %w[TestApp::Literal#float] }
 
     let(:expected_matcher) do
       ns::Method::Instance.new(cache, TestApp::Literal, TestApp::Literal.instance_method(:float))
@@ -81,7 +81,7 @@ describe Mutant::CLI, '.new' do
 
   context 'with debug flag' do
     let(:pattern)          { '::TestApp*'           }
-    let(:arguments)        { %W(--debug #{pattern}) }
+    let(:arguments)        { %W[--debug #{pattern}] }
     let(:expected_matcher) { ns::Namespace.new(cache, TestApp) }
 
     it_should_behave_like 'a cli parser'
@@ -93,7 +93,7 @@ describe Mutant::CLI, '.new' do
 
   context 'with zombie flag' do
     let(:pattern)          { '::TestApp*'            }
-    let(:arguments)        { %W(--zombie #{pattern}) }
+    let(:arguments)        { %W[--zombie #{pattern}] }
     let(:expected_matcher) { ns::Namespace.new(cache, TestApp) }
 
     it_should_behave_like 'a cli parser'
@@ -104,8 +104,8 @@ describe Mutant::CLI, '.new' do
   end
 
   context 'with namespace pattern' do
-    let(:pattern)          { '::TestApp*'   }
-    let(:arguments)        { %W(#{pattern}) }
+    let(:pattern)          { '::TestApp*' }
+    let(:arguments)        { [pattern]    }
     let(:expected_matcher) { ns::Namespace.new(cache, TestApp) }
 
     it_should_behave_like 'a cli parser'
@@ -113,7 +113,7 @@ describe Mutant::CLI, '.new' do
 
   context 'with subject code filter' do
     let(:pattern)   { 'TestApp::Literal#float' }
-    let(:arguments) { %W(--code faa --code bbb #{pattern}) }
+    let(:arguments) { %W[--code faa --code bbb #{pattern}] }
 
     let(:expected_filter) do
       Morpher.evaluator(

data/spec/unit/mutant/diff_spec.rb

@@ -12,7 +12,7 @@ describe Mutant::Diff do
     let(:old_string) { "foo\nbar" }
     let(:new_string) { "bar\nbaz" }
 
-    it { should eql(Mutant::Diff.new(%w(foo bar), %w(bar baz))) }
+    it { should eql(Mutant::Diff.new(%w[foo bar], %w[bar baz])) }
 
   end
 
@@ -46,8 +46,8 @@ describe Mutant::Diff do
     subject { object.diff }
 
     context 'when there is a diff at begin of hunk' do
-      let(:old) { %w(foo bar) }
-      let(:new) { %w(baz bar) }
+      let(:old) { %w[foo bar] }
+      let(:new) { %w[baz bar] }
 
       let(:expectation) do
         strip_indent(<<-STR)
@@ -64,8 +64,8 @@ describe Mutant::Diff do
     end
 
     context 'when there is a diff NOT at begin of hunk' do
-      let(:old) { %w(foo bar)     }
-      let(:new) { %w(foo baz bar) }
+      let(:old) { %w[foo bar]     }
+      let(:new) { %w[foo baz bar] }
 
       let(:expectation) do
         strip_indent(<<-STR)
@@ -82,8 +82,8 @@ describe Mutant::Diff do
     end
 
     context 'when the diff has a long context at begin' do
-      let(:old) { %w(foo bar baz boz a b c)       }
-      let(:new) { %w(foo bar baz boz a b c other) }
+      let(:old) { %w[foo bar baz boz a b c]       }
+      let(:new) { %w[foo bar baz boz a b c other] }
 
       let(:expectation) do
         strip_indent(<<-STR)
@@ -105,8 +105,8 @@ describe Mutant::Diff do
     end
 
     context 'when the diff has a long context at end, deleting' do
-      let(:old) { %w(other foo bar baz boz a b c) }
-      let(:new) { %w(foo bar baz boz a b c)       }
+      let(:old) { %w[other foo bar baz boz a b c] }
+      let(:new) { %w[foo bar baz boz a b c]       }
 
       let(:expectation) do
         strip_indent(<<-STR)
@@ -128,8 +128,8 @@ describe Mutant::Diff do
     end
 
     context 'when the diff has a long context at end, inserting' do
-      let(:old) { %w(foo bar baz boz a b c)       }
-      let(:new) { %w(other foo bar baz boz a b c) }
+      let(:old) { %w[foo bar baz boz a b c]       }
+      let(:new) { %w[other foo bar baz boz a b c] }
 
       let(:expectation) do
         strip_indent(<<-STR)

data/spec/unit/mutant/matcher/method/instance_spec.rb

@@ -2,6 +2,7 @@
 
 require 'spec_helper'
 
+# rubocop:disable ClassAndModuleChildren
 describe Mutant::Matcher::Method::Instance do
 
   let(:cache) { Fixtures::AST_CACHE }
@@ -44,7 +45,7 @@ describe Mutant::Matcher::Method::Instance do
           def bar
           end
 
-          def bar(arg)
+          def bar(_arg)
           end
         end
 
@@ -57,7 +58,7 @@ describe Mutant::Matcher::Method::Instance do
       context 'on the same line' do
         let(:base) { __LINE__ }
         class self::Foo
-          def bar; end; def bar(arg); end
+          def bar; end; def bar(_arg); end
         end
 
         let(:method_line)  { 2 }
@@ -69,7 +70,7 @@ describe Mutant::Matcher::Method::Instance do
       context 'on the same line with differend scope' do
         let(:base) { __LINE__ }
         class self::Foo
-          def self.bar; end; def bar(arg); end
+          def self.bar; end; def bar(_arg); end
         end
 
         let(:method_line)  { 2 }

data/spec/unit/mutant/matcher/method/singleton_spec.rb

@@ -2,6 +2,7 @@
 
 require 'spec_helper'
 
+# rubocop:disable ClassAndModuleChildren
 describe Mutant::Matcher::Method::Singleton, '#each' do
   subject { object.each { |subject| yields << subject } }
 
@@ -80,7 +81,7 @@ describe Mutant::Matcher::Method::Singleton, '#each' do
           module Bar
             def self.baz
             end
-            def Foo.baz(arg)
+            def Foo.baz(_arg)
             end
           end
         end

data/spec/unit/mutant/mutator/node/literal/boolean_spec.rb

@@ -7,7 +7,7 @@ describe Mutant::Mutator::Node::Literal, 'boolean' do
     let(:source) { 'true' }
 
     let(:mutations) do
-      %w(nil false)
+      %w[nil false]
     end
 
     it_should_behave_like 'a mutator'
@@ -17,7 +17,7 @@ describe Mutant::Mutator::Node::Literal, 'boolean' do
     let(:source) { 'false' }
 
     let(:mutations) do
-      %w(nil true)
+      %w[nil true]
     end
 
     it_should_behave_like 'a mutator'

data/spec/unit/mutant/mutator/node/literal/fixnum_spec.rb

@@ -6,7 +6,7 @@ describe Mutant::Mutator::Node::Literal, 'fixnum' do
   let(:source) { '10' }
 
   let(:mutations) do
-    %W(nil 0 1 -10 9 11)
+    %w[nil 0 1 -10 9 11]
   end
 
   it_should_behave_like 'a mutator'

data/spec/unit/mutant/mutator/node/literal/string_spec.rb

@@ -8,7 +8,7 @@ describe Mutant::Mutator::Node::Literal, 'string' do
   let(:source) { '"foo"' }
 
   let(:mutations) do
-    %W(nil)
+    %w[nil]
   end
 
   it_should_behave_like 'a mutator'

data/spec/unit/mutant/mutator/node/literal/symbol_spec.rb

@@ -8,7 +8,7 @@ describe Mutant::Mutator::Node::Literal, 'symbol' do
   let(:source) { ':foo' }
 
   let(:mutations) do
-    %w(nil) << ':foo__mutant__'
+    %w[nil :foo__mutant__]
   end
 
   it_should_behave_like 'a mutator'

data/spec/unit/mutant/runner/mutation_spec.rb

@@ -5,23 +5,25 @@ require 'spec_helper'
 describe Mutant::Runner::Mutation do
   let(:object) { described_class.new(config, mutation, tests) }
 
-  let(:reporter)  { double('Reporter')                                     }
-  let(:mutation)  { double('Mutation', class: Mutant::Mutation)            }
-  let(:strategy)  { double('Strategy')                                     }
-  let(:killer_a)  { Mutant::Killer.new(test: test_a, mutation: mutation)   }
-  let(:killer_b)  { Mutant::Killer.new(test: test_b, mutation: mutation)   }
-  let(:runner_a)  { double('Runner A', success?: success_a, stop?: stop_a) }
-  let(:runner_b)  { double('Runner B', success?: success_b, stop?: stop_b) }
-  let(:runners)   { [runner_a, runner_b]                                   }
-  let(:killers)   { [killer_a, killer_b]                                   }
-  let(:fail_fast) { false                                                  }
-  let(:success_a) { true                                                   }
-  let(:success_b) { true                                                   }
-  let(:stop_a)    { false                                                  }
-  let(:stop_b)    { false                                                  }
-  let(:test_a)    { double('test a')                                       }
-  let(:test_b)    { double('test b')                                       }
-  let(:tests)     { [test_a, test_b]                                       }
+  let(:reporter)  { double('Reporter')                                                             }
+  let(:mutation)  { double('Mutation', class: Mutant::Mutation)                                    }
+  let(:strategy)  { double('Strategy')                                                             }
+  let(:killer_a)  { Mutant::Killer.new(test: test_a, mutation: mutation)                           }
+  let(:killer_b)  { Mutant::Killer.new(test: test_b, mutation: mutation)                           }
+  let(:runner_a)  { double('Runner A', success?: success_a, stop?: stop_a, mutation_dead?: dead_a) }
+  let(:runner_b)  { double('Runner B', success?: success_b, stop?: stop_b, mutation_dead?: dead_b) }
+  let(:runners)   { [runner_a, runner_b]                                                           }
+  let(:killers)   { [killer_a, killer_b]                                                           }
+  let(:fail_fast) { false                                                                          }
+  let(:success_a) { true                                                                           }
+  let(:success_b) { true                                                                           }
+  let(:stop_a)    { false                                                                          }
+  let(:stop_b)    { false                                                                          }
+  let(:dead_a)    { false                                                                          }
+  let(:dead_b)    { false                                                                          }
+  let(:test_a)    { double('test a')                                                               }
+  let(:test_b)    { double('test b')                                                               }
+  let(:tests)     { [test_a, test_b]                                                               }
 
   before do
     expect(Mutant::Runner).to receive(:run).with(config, killer_a).and_return(runner_a)

data/test_app/Gemfile.devtools

@@ -1,34 +1,30 @@
 # encoding: utf-8
 
 group :development do
-  gem 'rake',       '~> 10.1.0'
-  gem 'rspec',      '~> 2.14.1'
-  gem 'rspec-core', '~> 2.14.8'
-  gem 'yard',       '~> 0.8.7'
+  gem 'rake',       '~> 10.3.1'
+  gem 'rspec',      '~> 2.99.0.beta2'
+  gem 'rspec-core', '~> 2.99.0.beta2'
+  gem 'yard',       '~> 0.8.7.4'
 
   platform :rbx do
     gem 'rubysl-singleton', '~> 2.0.0'
   end
 end
 
-group :yard do
-  gem 'kramdown', '~> 1.3.2'
-end
-
 group :guard do
-  gem 'guard',         '~> 2.4.0'
+  gem 'guard',         '~> 2.6.0'
   gem 'guard-bundler', '~> 2.0.0'
-  gem 'guard-rspec',   '~> 4.2.6'
-  gem 'guard-rubocop', '~> 1.0.2'
+  gem 'guard-rspec',   '~> 4.2.8'
+  gem 'guard-rubocop', '~> 1.1.0'
 
   # file system change event handling
-  gem 'listen',     '~> 2.5.0'
+  gem 'listen',     '~> 2.7.3'
   gem 'rb-fchange', '~> 0.0.6', require: false
-  gem 'rb-fsevent', '~> 0.9.3', require: false
-  gem 'rb-inotify', '~> 0.9.0', require: false
+  gem 'rb-fsevent', '~> 0.9.4', require: false
+  gem 'rb-inotify', '~> 0.9.4', require: false
 
   # notification handling
-  gem 'libnotify',               '~> 0.8.0', require: false
+  gem 'libnotify',               '~> 0.8.2', require: false
   gem 'rb-notifu',               '~> 0.0.4', require: false
   gem 'terminal-notifier-guard', '~> 1.5.3', require: false
 end
@@ -37,9 +33,9 @@ group :metrics do
   gem 'coveralls', '~> 0.7.0'
   gem 'flay',      '~> 2.4.0'
   gem 'flog',      '~> 4.2.0'
-  gem 'reek',      '~> 1.3.2'
-  gem 'rubocop',   '~> 0.18.1'
-  gem 'simplecov', '~> 0.8.2'
+  gem 'reek',      '~> 1.3.7'
+  gem 'rubocop',   '~> 0.21.0'
+  gem 'simplecov', '~> 0.7.1'
   gem 'yardstick', '~> 0.9.9'
 
   platforms :ruby_19, :ruby_20 do
@@ -48,10 +44,10 @@ group :metrics do
 
   platform :rbx do
     gem 'json',               '~> 1.8.1'
-    gem 'racc',               '~> 1.4'
+    gem 'racc',               '~> 1.4.11'
     gem 'rubysl-logger',      '~> 2.0.0'
     gem 'rubysl-open-uri',    '~> 2.0.0'
-    gem 'rubysl-prettyprint', '~> 2.0.2'
+    gem 'rubysl-prettyprint', '~> 2.0.3'
   end
 end
 
@@ -61,6 +57,6 @@ end
 
 platform :jruby do
   group :jruby do
-    gem 'jruby-openssl', '~> 0.8.5'
+    gem 'jruby-openssl', '~> 0.9.4'
   end
 end

data/test_app/lib/test_app/literal.rb

@@ -7,7 +7,7 @@ module TestApp
       true
     end
 
-    def command(foo)
+    def command(_foo)
       self
     end
 

data/test_app/spec/unit/test_app/literal/command_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-RSpec.describe TestApp::Literal, '#string' do
+RSpec.describe TestApp::Literal, '#command' do
   subject { object.command(double) }
 
   let(:object) { described_class.new }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mutant
 version: !ruby/object:Gem::Version
-  version: 0.5.16
+  version: 0.5.17
 platform: ruby
 authors:
 - Markus Schirp