mutant 0.5.26 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 40c90d9f752a1b8e3f120deb4c47f6a4a3683369
-  data.tar.gz: 833c87b79967039486f66c6e295992642243f2e9
+  metadata.gz: b4c622c635f57a57ea3d7463d71da71c84450a8e
+  data.tar.gz: 4147047e1dd38d1115f415e5cb8e2eebba932c6d
 SHA512:
-  metadata.gz: 243b80a0db1e4c41a0d47fbc2056cf48d10e268a6c5af1cc9728a5645022a0ae042287242a8bef1fed6ec2a3df19b24cc8c589083aa4940a104fdef9a3e0a079
-  data.tar.gz: 113d8551fcc6102f8cc459b0e793e80abd77f685f80eb2895d5cf989b12eb65ffbc5d642bc762a5d750faeef90dde6a5c95a5c6d47365f3f6a30897f2cbf48a7
+  metadata.gz: fd80ff4f0469e2bf182c5060fe4696ce48373c014dd76dc083fc1a210812908c201c344afa6f34ec066c5714e0ad1a6f94ca436eb5c31fb9cf92d76b072c8467
+  data.tar.gz: 54779c5e90ebc0f5f909ae8b485eecb92da095e0abaf9f28d214d500ee2f0ea4d8e836b8cc17d9abe8a9b034b89de5651487ceeb25dd347c1e0a9864d5c444cb

data/.rspec

@@ -1,4 +1,5 @@
 --color
+--require spec_helper
 --format progress
 --warnings
 --order random

data/.travis.yml

@@ -10,6 +10,7 @@ rvm:
 matrix:
   allow_failures:
     - rvm: 2.1.2 # still segfaults on stack overflow on define_method
+    - rvm: rbx-2 # Travis does not take care about RBX
 notifications:
   irc:
     channels:

data/Changelog.md

@@ -1,3 +1,16 @@
+# v0.6.0 2014-07-17
+
+* Parallel execution / reporting.
+* Add -j, --jobs flag to control concurrency.
+* Fix blind spots on send with block.
+* Add mutation from `foo { bar }` to `bar`
+* Add mutation from `#reverse_merge` to `#merge`
+* Add mutation from `#<=` to `#<`, `#==`, `#eql?`, `#equal?`
+* Add mutation from `#>=` to `#>`, `#==`, `#eql?`, `#equal?`
+* Add mutation from `#>` to `#==`, `#eql?`, `#equal?`
+* Add mutation from `#<` to `#==`, `#eql?`, `#equal?`
+* Fix reporting of diff errors to include context [tjchambers]
+
 # v0.5.26 2014-07-07
 
 * Fix exceptions generation matcher errors
@@ -63,7 +76,7 @@ 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.
+* Add short circuits on already dead mutations under multiple test selections.
 
 # v0.5.16 2014-05-27
 
@@ -90,7 +103,7 @@ Changes:
 
 Changes:
 
-* Imporve reporting of isolation problems
+* Improve reporting of isolation problems
 * Centralize test selection
 * Report selected tests
 * Report rspec output on noop failures
@@ -109,7 +122,7 @@ Changes:
 * Fix crash on while and until without body
 * Better require highjack based zombifier
 * Do not mutate nthref $1 to gvar $0
-* Use faster duplicate guarding hashing AST::Node intances
+* Use faster duplicate guarding hashing AST::Node instances
 * Fix lots of shadowed invalid ASTs
 * Fix undefine initialize warnings, Closes #175
 

data/Gemfile

@@ -2,8 +2,6 @@
 
 source 'https://rubygems.org'
 
-gem 'mutant', path: '.'
-
 gemspec name: 'mutant'
 
 gem 'devtools', git: 'https://github.com/rom-rb/devtools.git'

data/Gemfile.devtools

@@ -2,8 +2,8 @@
 
 group :development do
   gem 'rake',       '~> 10.3.2'
-  gem 'rspec',      '~> 2.99.0'
-  gem 'rspec-core', '~> 2.99.0'
+  gem 'rspec',      '~> 3.0.0'
+  gem 'rspec-its',  '~> 1.0.1'
   gem 'yard',       '~> 0.8.7.4'
 
   platform :rbx do

data/README.md

@@ -12,9 +12,8 @@ Mutant is a mutation testing tool for ruby.
 The idea is that if code can be changed and your tests do not notice, either that code isn't being covered
 or it does not have a speced side effect.
 
-Mutant supports MRI and RBX 1.9 and 2.0, while support for jruby is planned.
+Mutant supports MRI and RBX 1.9, 2.0 and 2.1, while support for jruby is planned.
 It should also work under any ruby engine that supports POSIX-fork(2) semantics.
-Support for MRI 2.1 is unstable, because this MRI release segfaults on basic metaprogramming mutants dependencies do.
 
 Mutant uses a pure ruby [parser](https://github.com/whitequark/parser) and an [unparser](https://github.com/mbj/unparser)
 to do its magic.
@@ -37,11 +36,6 @@ Blog-Posts
 * http://www.sitepoint.com/mutation-testing-mutant/
 * http://solnic.eu/2013/01/23/mutation-testing-with-mutant.html
 
-Integrations
-------------
-
-rspec2 and rspec-3 are supported currently. Minitest support is still *planned*.
-
 Projects using Mutant
 ---------------------
 
@@ -78,6 +72,8 @@ Please add an explicit dependency to `rspec-core` for the rspec version you want
 gem install mutant-rspec
 ```
 
+The minitest integration is still in the works.
+
 Mutations
 ---------
 
@@ -143,20 +139,18 @@ Example for a subject like `Foo::Bar#baz` it will run all example groups with de
 `Foo::Bar#baz`, `Foo::Bar` and `Foo`. The order is important, so if mutant finds example groups in the
 current prefix level, these example groups *must* kill the mutation.
 
-This test selection strategy is compatible with the old `--rspec-dm2` and `--rspec-unit` strategy.
-The old flags were removed.  It allows to define very fine grained specs, or coarse grained - as you like.
-
 Support
 -------
 
-I'm very happy to receive/answer feedback/questions and critism.
+I'm very happy to receive/answer feedback/questions and criticism.
 
 Your options:
 
-* GitHub Issues https://github.com/mbj/mutant/issues
-* Ping me on https://twitter.com/_m_b_j_
-* #mutant channel on freenode, I hang around on CET daytimes. (nick mbj)
-  You'll also find others [ROM](https://github.com/rom-rb) team members here that can answer questions.
+* [GitHub Issues](https://github.com/mbj/mutant/issues)
+* Ping me on [twitter](https://twitter.com/_m_b_j_)
+
+There is also the [#mutant] channel on freenode. As my OSS time budged is very limited I cannot
+join it often. Please prefer to use github issues with a 'Question: ' prefix in title.
 
 Credits
 -------

data/bin/mutant

@@ -1,5 +1,4 @@
 #!/usr/bin/env ruby
-# encoding: utf-8
 
 trap('INT') do |status|
   exit! 128 + status

data/config/flay.yml

@@ -1,3 +1,3 @@
 ---
 threshold: 18
-total_score: 1040
+total_score: 1069

data/config/flog.yml

@@ -1,2 +1,2 @@
 ---
-threshold: 29.3
+threshold: 30.6

data/config/mutant.yml

@@ -5,7 +5,7 @@ expect_coverage: 62.08
 ignore_subjects:
 # Mutation causes infinite runtime
 - Mutant::Runner.lookup
-# Suboptimal test selection stragegy (will be fixed soon) causes timeouts on CI
+# Suboptimal test selection strategy (will be fixed soon) causes timeouts on CI
 - Mutant::Zombifier*
 - Mutant::Reporter*
 - Mutant::CLI*

data/config/reek.yml

@@ -12,6 +12,7 @@ ControlParameter:
   enabled: true
   exclude:
   - Mutant::Expression#match_length
+  - Mutant::Reporter::CLI::Printer::SubjectProgress#print_mutation_result
 DataClump:
   enabled: true
   exclude: []
@@ -28,9 +29,10 @@ FeatureEnvy:
   # Nature of OptionParser :(
   - Mutant::CLI#add_environment_options
   - Mutant::Env#scope_name
+  - Mutant::Diff#minimized_hunks
   - Mutant::Integration::Rspec#run
-  - Mutant::Integration::Rspec2#full_description
-  - Mutant::Integration::Rspec3#full_description
+  - Mutant::Integration::Rspec::Rspec2#full_description
+  - Mutant::Integration::Rspec::Rspec3#full_description
   - Mutant::Matcher::Method::Instance#match?
   - Mutant::Matcher::Method::Singleton#receiver?
   - Mutant::Mutation::Evil#success?
@@ -39,6 +41,8 @@ FeatureEnvy:
   - Mutant::Meta::Example::Verification#format_mutation
   - Mutant::Reporter::CLI#subject_results
   - Mutant::Runner#run_mutation_test
+  - Mutant::Runner#kill_mutation
+  - Mutant::Runner#finish
 IrresponsibleModule:
   enabled: true
   exclude: []
@@ -46,6 +50,7 @@ LongParameterList:
   enabled: true
   exclude:
   - Mutant::Matcher::Method::Instance#self.build?
+  - Mutant::Runner#finish # API client of parallel, one gets _ignored.
   - Mutant::Runner#self.run
   max_params: 2
 LongYieldList:
@@ -61,10 +66,8 @@ NestedIterators:
   - Mutant::Mutator::Util::Array::Element#dispatch
   - Mutant::Mutator::Node::Resbody#mutate_captures
   - Mutant::Mutator::Node::Arguments#emit_argument_mutations
-  - Mutant::Reporter::CLI::Report::Env#generic_stats
   - Mutant::RequireHighjack#infect
   - Mutant::RequireHighjack#desinfect
-  - Mutant::Reporter::CLI::Registry#included
   - Mutant::Subject#tests
   - Parser::Lexer#self.new
   max_allowed_nesting: 1
@@ -81,9 +84,8 @@ RepeatedConditional:
 TooManyInstanceVariables:
   enabled: true
   exclude:
-  - Mutant::CLI     # 4 vars
-  - Mutant::Killer  # 4 vars
   - Mutant::Mutator # 4 vars
+  - Mutant::Runner  # 4 vars
   max_instance_variables: 3
 TooManyMethods:
   enabled: true
@@ -99,9 +101,9 @@ TooManyStatements:
   exclude:
   - Mutant#self.singleton_subclass_instance
   - Mutant::Integration::Rspec#run
-  - Mutant::Reporter::CLI::Report::Env#run
-  - Mutant::Reporter::CLI::Registry#included
   - Mutant::Reporter::CLI#colorized_diff
+  - Mutant::Reporter::CLI::Printer::EnvProgress#run
+  - Mutant::Reporter::CLI::Printer::Config#run
   - Mutant::RequireHighjack#infect
   - Mutant::Rspec::Killer#run
   - Mutant::Runner#visit_collection
@@ -156,11 +158,12 @@ UtilityFunction:
   - Mutant::CLI#reporter
   - Mutant::Integration::Rspec#configuration
   - Mutant::Integration::Rspec#options
-  - Mutant::Integration::Rspec2#full_description
-  - Mutant::Integration::Rspec2#new_reporter
-  - Mutant::Integration::Rspec3#full_description
+  - Mutant::Integration::Rspec::Rspec2#full_description
+  - Mutant::Integration::Rspec::Rspec2#new_reporter
+  - Mutant::Integration::Rspec::Rspec3#full_description
   - Mutant::Meta::Example::Verification#format_mutation
   - Mutant::Mutation::Evil#success?
   - Mutant::Mutation::Neutral#success?
+  - Mutant::Reporter::CLI::Format::Progressive#new_buffer
   - Mutant::Runner#run_mutation_test
   max_helper_calls: 0

data/config/rubocop.yml

@@ -89,7 +89,7 @@ ConstantName:
 TrivialAccessors:
   Enabled: false
 
-# And also have a differend opinion here
+# And also have a different opinion here
 AndOr:
   Enabled: false
 

data/lib/mutant.rb

@@ -17,6 +17,7 @@ require 'anima'
 require 'concord'
 require 'morpher'
 require 'parallel'
+require 'open3'
 
 # Library namespace
 module Mutant
@@ -27,6 +28,16 @@ module Mutant
 
   SCOPE_OPERATOR = '::'.freeze
 
+  # Test if CI is detected via environment
+  #
+  # @return [Boolean]
+  #
+  # @api private
+  #
+  def self.ci?
+    ENV.key?('CI')
+  end
+
   # Lookup constant for location
   #
   # @param [String] location
@@ -91,7 +102,6 @@ require 'mutant/cache'
 require 'mutant/delegator'
 require 'mutant/warning_filter'
 require 'mutant/warning_expectation'
-require 'mutant/walker'
 require 'mutant/require_highjack'
 require 'mutant/isolation'
 require 'mutant/mutator'
@@ -167,7 +177,6 @@ require 'mutant/matcher/config'
 require 'mutant/matcher/compiler'
 require 'mutant/matcher/chain'
 require 'mutant/matcher/method'
-require 'mutant/matcher/method/finder'
 require 'mutant/matcher/method/singleton'
 require 'mutant/matcher/method/instance'
 require 'mutant/matcher/methods'
@@ -184,43 +193,35 @@ require 'mutant/integration'
 require 'mutant/cli'
 require 'mutant/color'
 require 'mutant/diff'
+require 'mutant/runner'
+require 'mutant/runner/collector'
 require 'mutant/result'
 require 'mutant/reporter'
 require 'mutant/reporter/null'
 require 'mutant/reporter/trace'
 require 'mutant/reporter/cli'
-require 'mutant/reporter/cli/registry'
 require 'mutant/reporter/cli/printer'
-require 'mutant/reporter/cli/report'
-require 'mutant/reporter/cli/report/env'
-require 'mutant/reporter/cli/report/subject'
-require 'mutant/reporter/cli/report/mutation'
-require 'mutant/reporter/cli/report/test'
-require 'mutant/reporter/cli/progress'
-require 'mutant/reporter/cli/progress/env'
-require 'mutant/reporter/cli/progress/config'
-require 'mutant/reporter/cli/progress/subject'
-require 'mutant/reporter/cli/progress/noop'
-require 'mutant/reporter/cli/progress/result'
-require 'mutant/reporter/cli/progress/result/mutation'
-require 'mutant/reporter/cli/progress/result/subject'
-require 'mutant/runner'
+require 'mutant/reporter/cli/tput'
+require 'mutant/reporter/cli/format'
 require 'mutant/zombifier'
 require 'mutant/zombifier/file'
 
 module Mutant
-  # Repoen class to initialize constant to avoid dep circle
+  # Reopen class to initialize constant to avoid dep circle
   class Config
+    CI_DEFAULT_PROCESSOR_COUNT = 2
+
     DEFAULT = new(
       debug:             false,
       fail_fast:         false,
       integration:       Integration::Null.new,
       matcher_config:    Matcher::Config::DEFAULT,
-      includes:          [],
-      requires:          [],
+      includes:          EMPTY_ARRAY,
+      requires:          EMPTY_ARRAY,
       isolation:         Mutant::Isolation::Fork,
-      reporter:          Reporter::CLI.new($stdout),
+      reporter:          Reporter::CLI.build($stdout),
       zombie:            false,
+      processes:         Mutant.ci? ? CI_DEFAULT_PROCESSOR_COUNT : Parallel.processor_count,
       expected_coverage: 100.0
     )
   end # Config

data/lib/mutant/ast.rb

@@ -1,5 +1,52 @@
 module Mutant
   # AST helpers
   module AST
+
+    # Walk all ast nodes
+    #
+    # @param [Parser::AST::Node]
+    #
+    # @yield [Parser::AST::Node]
+    #   all nodes recursively including root
+    #
+    # @return [self]
+    #
+    # @api private
+    #
+    def self.walk(node, &block)
+      raise ArgumentError, 'block expected' unless block_given?
+
+      block.call(node)
+      node.children.grep(Parser::AST::Node).each do |child|
+        walk(child, &block)
+      end
+
+      self
+    end
+
+    # Find last node satisfing predicate (as block)
+    #
+    # @return [Parser::AST::Node]
+    #   if satisfing node is found
+    #
+    # @yield [Parser::AST::Node]
+    #
+    # @yieldreturn [Boolean]
+    #   true in case node satisfies predicate
+    #
+    # @return [nil]
+    #   otherwise
+    #
+    # @api private
+    #
+    def self.find_last(node, &predicate)
+      raise ArgumentError, 'block expected' unless block_given?
+      neddle = nil
+      walk(node) do |candidate|
+        neddle = candidate if predicate.call(candidate, &predicate)
+      end
+      neddle
+    end
+
   end # AST
 end # Mutant

data/lib/mutant/cli.rb

@@ -2,11 +2,11 @@ require 'optparse'
 
 module Mutant
 
-  # Comandline parser
+  # Commandline parser
   class CLI
     include Adamantium::Flat, Equalizer.new(:config), Procto.call(:config)
 
-    # Error faild when CLI argv is invalid
+    # Error failed when CLI argv is invalid
     Error = Class.new(RuntimeError)
 
     EXIT_FAILURE = 1
@@ -79,7 +79,7 @@ module Mutant
 
     # Parse matchers
     #
-    # @param [Array<String>] patterns
+    # @param [Array<String>] expressions
     #
     # @return [undefined]
     #
@@ -112,6 +112,9 @@ module Mutant
       opts.on('-r', '--require NAME', 'Require file with NAME') do |name|
         add(:requires, name)
       end
+      opts.on('-j', '--jobs NUMBER', 'Number of kill processes. Defaults to number of processors.') do |number|
+        update(processes: Integer(number))
+      end
     end
 
     # Use integration
@@ -124,7 +127,7 @@ module Mutant
     #
     def setup_integration(name)
       require "mutant/integration/#{name}"
-      update(integration: Integration.lookup(name).new)
+      update(integration: Integration.lookup(name))
     end
 
     # Add options