mutant 0.5.22 → 0.5.23

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 12466470fce5fafd680059de82205c093d93c503
-  data.tar.gz: c374d20379cb9c81068ce205a6d43d7ddd3505ed
+  metadata.gz: 19cc50cff5125bbeef98eb0511a7c02c7154b4d5
+  data.tar.gz: 153344653cd236a26f822ea809333a881b66491b
 SHA512:
-  metadata.gz: 3cf3581a8648769d72db4a2cee7522ebece339eef403af542c989cbac811cc3deb34fe03f25795ad80d3a9c5fa12fedc796dda3bc59565b282c1afba311302a3
-  data.tar.gz: 61deff160b2dca8288d53a615c8c4ff5a5005612d47383d6f1d1145d6b28eb1eae898d3634e6c16eb02b837b8549fe86834a96c8a015da98a1eaeaf91845a580
+  metadata.gz: 1b3b1a8e57fcfc7c5423e16b1c7a375ef26494e1d6df55c12e458efcf7cc4f26be1681c379b7ce110d91f3c7179f43f294d099b1ce41203a162043ce893ba505
+  data.tar.gz: 43b9c7b3c628cf529316433463083163ec8ae83911575df994cdc71d604b34a6e05872be557cef18150c78c20015973bd5db37fd81ec170b377f02b220264bc0

data/.travis.yml

@@ -7,6 +7,9 @@ rvm:
   - 2.0.0
   - 2.1.2
   - rbx-2
+matrix:
+  allow_failures:
+    - rvm: 2.1.2 # still segfaults on stack overflow on define_method
 notifications:
   irc:
     channels:

data/Changelog.md

@@ -1,3 +1,7 @@
+# v0.5.23 2014-06-15
+
+* Propagate exceptions from child-isolation-killforks to master
+
 # v0.5.22 2014-06-15
 
 * Fix invalid AST generation on operator method mutation with self as receiver.

data/README.md

@@ -6,7 +6,6 @@ mutant
 [![Code Climate](https://codeclimate.com/github/mbj/mutant.png)](https://codeclimate.com/github/mbj/mutant)
 [![Inline docs](http://inch-ci.org/github/mbj/mutant.png)](http://inch-ci.org/github/mbj/mutant)
 [![Gem Version](https://img.shields.io/gem/v/mutant.svg)](https://rubygems.org/gems/mutant)
-[![Gittip](https://img.shields.io/gittip/mbj.svg)](https://gittip.com/mbj)
 
 Mutant is a mutation testing tool for ruby.
 
@@ -41,7 +40,7 @@ Blog-Posts
 Integrations
 ------------
 
-Only rspec2 and rspec-3 beta2 is supported currently.
+rspec2 and rspec-3 are supported currently. Minitest support is still *planned*.
 
 Projects using Mutant
 ---------------------
@@ -147,17 +146,6 @@ 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.
 
-Donations
----------
-
-I ([mbj](https://github.com/mbj)) build this and adjacent tools in my free time.
-LOTS of unaccounted time and fun already went into this project.
-
-You might consider to donate if you like this tool.
-
-* [![Gittip](https://img.shields.io/gittip/mbj.svg)](https://gittip.com/mbj)
-* [![Flattr this](http://api.flattr.com/button/flattr-badge-large.png)](http://flattr.com/thing/1823010/mbjmutant-on-GitHub)
-
 Support
 -------
 

data/config/flay.yml

@@ -1,3 +1,3 @@
 ---
 threshold: 18
-total_score: 906
+total_score: 899

data/lib/mutant.rb

@@ -16,6 +16,7 @@ require 'diff/lcs/hunk'
 require 'anima'
 require 'concord'
 require 'morpher'
+require 'parallel'
 
 # Library namespace
 module Mutant

data/lib/mutant/expression/method.rb

@@ -24,7 +24,7 @@ module Mutant
       def matcher(cache)
         methods_matcher = MATCHERS.fetch(scope_symbol).new(cache, scope)
         method = methods_matcher.methods.detect do |meth|
-          meth.name == method_name
+          meth.name.equal?(method_name)
         end or raise NameError, "Cannot find method #{identifier}"
         methods_matcher.matcher.build(cache, scope, method)
       end

data/lib/mutant/isolation.rb

@@ -16,50 +16,10 @@ module Mutant
     # @api private
     #
     def self.call(&block)
-      reader, writer = IO.pipe.each(&:binmode)
-
-      pid = fork
-
-      if pid.nil?
-        begin
-          reader.close
-          writer.write(Marshal.dump(block.call))
-          Kernel.exit!(0)
-        ensure
-          Kernel.exit!(1)
-        end
-      end
-
-      writer.close
-
-      read_result(reader, pid)
-    end
-
-    # Read result from child process
-    #
-    # @param [IO] reader
-    # @param [Fixnum] pid
-    #
-    # @return [Object]
-    #
-    # @raise [Error]
-    #
-    def self.read_result(reader, pid)
-      begin
-        data = Marshal.load(reader.read)
-      rescue ArgumentError, TypeError
-        raise Error, 'Childprocess wrote un-unmarshallable data'
-      end
-
-      status = Process.waitpid2(pid).last
-
-      unless status.exitstatus.zero?
-        raise Error, "Childprocess exited with nonzero exit status: #{status.exitstatus}"
-      end
-
-      data
+      Parallel.map([block], in_processes: 1) do |block|
+        block.call
+      end.first
     end
-    private_class_method :read_result
 
   end # Isolator
 end # Mutant

data/lib/mutant/matcher/chain.rb

@@ -33,7 +33,7 @@ module Mutant
       # @api private
       #
       def self.build(matchers)
-        if matchers.length == 1
+        if matchers.length.equal?(1)
           return matchers.first
         end
 

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

@@ -49,9 +49,9 @@ module Mutant
         def match?(node)
           location                  = node.location       || return
           expression                = location.expression || return
-          expression.line           == source_line &&
-          node.type                 == :def        &&
-          node.children[NAME_INDEX] == method_name
+          expression.line.equal?(source_line)           &&
+          node.type.equal?(:def)                        &&
+          node.children[NAME_INDEX].equal?(method_name)
         end
 
         # Matcher for memoized instance methods

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

@@ -45,7 +45,7 @@ module Mutant
         def line?(node)
           expression = node.location.expression
           return false unless expression
-          expression.line == source_line
+          expression.line.equal?(source_line)
         end
 
         # Test for name match
@@ -57,7 +57,7 @@ module Mutant
         # @api private
         #
         def name?(node)
-          node.children[NAME_INDEX] == method_name
+          node.children[NAME_INDEX].equal?(method_name)
         end
 
         # Test for receiver match
@@ -95,7 +95,7 @@ module Mutant
         #
         def receiver_name?(node)
           name = node.children[CONST_NAME_INDEX]
-          name.to_s == context.unqualified_name
+          name.to_s.eql?(context.unqualified_name)
         end
 
       end # Singleton

data/lib/mutant/matcher/methods.rb

@@ -43,7 +43,7 @@ module Mutant
       def methods
         candidate_names.each_with_object([]) do |name, methods|
           method = access(name)
-          methods << method if method.owner == candidate_scope
+          methods << method if method.owner.equal?(candidate_scope)
         end
       end
       memoize :methods

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

@@ -24,7 +24,7 @@ module Mutant
             emit_body_mutations
           end
           emit_body(nil)
-          emit_body(RAISE)
+          emit_body(N_RAISE)
         end
 
       end # Block

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

@@ -22,7 +22,7 @@ module Mutant
           emit_condition_mutations
           emit_body_mutations if body
           emit_body(nil)
-          emit_body(RAISE)
+          emit_body(N_RAISE)
         end
 
       end # While

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

@@ -14,7 +14,7 @@ module Mutant
         #
         def dispatch
           emit_arguments_mutations
-          emit_body(RAISE)
+          emit_body(N_RAISE)
           emit_body(nil)
           emit_body_mutations if body
         end

data/lib/mutant/mutator/node/literal/float.rb

@@ -22,9 +22,9 @@ module Mutant
           end
 
           SPECIAL = [
-            NodeHelpers::NAN,
-            NodeHelpers::NEGATIVE_INFINITY,
-            NodeHelpers::INFINITY
+            N_NAN,
+            N_NEGATIVE_INFINITY,
+            N_INFINITY
           ].freeze
 
           # Emit special cases
@@ -34,9 +34,7 @@ module Mutant
           # @api private
           #
           def emit_special_cases
-            SPECIAL.each do |value|
-              emit(value)
-            end
+            SPECIAL.each(&method(:emit))
           end
 
           # Return values to test against

data/lib/mutant/mutator/node/literal/range.rb

@@ -48,7 +48,7 @@ module Mutant
           #
           def emit_upper_bound_mutations
             emit__end_mutations
-            emit_type(NAN, _end)
+            emit_type(N_NAN, _end)
           end
 
           # Emit start mutations
@@ -59,8 +59,8 @@ module Mutant
           #
           def emit_lower_bound_mutations
             emit_start_mutations
-            emit_type(start, INFINITY)
-            emit_type(start, NAN)
+            emit_type(start, N_INFINITY)
+            emit_type(start, N_NAN)
           end
 
         end # Range

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

@@ -19,7 +19,7 @@ module Mutant
           if body
             mutate_body
           else
-            emit_child_update(body_index, RAISE)
+            emit_child_update(body_index, N_RAISE)
           end
           mutate_conditions
         end

data/lib/mutant/mutator/util.rb

@@ -35,7 +35,7 @@ module Mutant
       # @api private
       #
       def new?(generated)
-        input != generated
+        !input.eql?(generated)
       end
 
     end # Util

data/lib/mutant/node_helpers.rb

@@ -15,20 +15,15 @@ module Mutant
     end
     module_function :s
 
-    NAN =
-      s(:send, s(:float,  0.0), :/, s(:float, 0.0))
-    INFINITY =
-      s(:send, s(:float,  1.0), :/, s(:float, 0.0))
-    NEGATIVE_INFINITY =
-      s(:send, s(:float, -1.0), :/, s(:float, 0.0))
-
-    RAISE   = s(:send, nil, :raise)
-
-    N_TRUE  = s(:true)
-    N_FALSE = s(:false)
-    N_NIL   = s(:nil)
-    N_EMPTY = s(:empty)
-    N_SELF  = s(:self)
+    N_NAN               = s(:send, s(:float,  0.0), :/, s(:float, 0.0))
+    N_INFINITY          = s(:send, s(:float,  1.0), :/, s(:float, 0.0))
+    N_NEGATIVE_INFINITY = s(:send, s(:float, -1.0), :/, s(:float, 0.0))
+    N_RAISE             = s(:send, nil, :raise)
+    N_TRUE              = s(:true)
+    N_FALSE             = s(:false)
+    N_NIL               = s(:nil)
+    N_EMPTY             = s(:empty)
+    N_SELF              = s(:self)
 
     # Build a negated boolean node
     #

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

@@ -43,7 +43,7 @@ module Mutant
         #
         def lookup(subject)
           current = subject
-          until current == Object
+          until current.equal?(Object)
             if registry.key?(current)
               return registry.fetch(current)
             end

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

@@ -60,7 +60,7 @@ module Mutant
           def generic_stats
             subjects.each_with_object(Hash.new(0)) do |runner, stats|
               Walker.run(runner.subject.node) do |node|
-                if Mutator::Registry.lookup(node) == Mutator::Node::Generic
+                if Mutator::Registry.lookup(node).equal?(Mutator::Node::Generic)
                   stats[node.type] += 1
                 end
               end

data/lib/mutant/runner/config.rb

@@ -48,7 +48,7 @@ module Mutant
       # @api private
       #
       def success?
-        coverage.round(COVERAGE_PRECISION) == config.expected_coverage.round(COVERAGE_PRECISION)
+        coverage.round(COVERAGE_PRECISION).eql?(config.expected_coverage.round(COVERAGE_PRECISION))
       end
       memoize :success?
 

data/lib/mutant/version.rb

@@ -1,4 +1,4 @@
 module Mutant
   # The current mutant version
-  VERSION = '0.5.22'.freeze
+  VERSION = '0.5.23'.freeze
 end # Mutant

data/mutant.gemspec

@@ -37,7 +37,7 @@ Gem::Specification.new do |gem|
   gem.add_runtime_dependency('inflecto',      '~> 0.0.2')
   gem.add_runtime_dependency('anima',         '~> 0.2.0')
   gem.add_runtime_dependency('concord',       '~> 0.1.5')
+  gem.add_runtime_dependency('parallel',      '~> 1.0.0')
 
   gem.add_development_dependency('bundler', '~> 1.3', '>= 1.3.5')
-  gem.add_development_dependency('parallel', '~> 1.0.0')
 end

data/spec/unit/mutant/isolation_spec.rb

@@ -1,61 +1,16 @@
 require 'spec_helper'
 
 describe Mutant::Isolation do
-  describe '.isolate' do
+  describe '.run' do
     let(:object) { described_class }
 
-    before do
-      unless RUBY_VERSION.eql?('2.1.2')
-        skip 'Series of events is indeterministic cross ruby implementations. Skipping this test under non 2.1.2'
-      end
+    it 'isolates global effects from process' do
+      expect { object.call { ::Foo = 1 } }.not_to change { defined?(Foo) }.from(nil)
     end
 
-    let(:expected_return) { :foo }
-
-    subject { object.call(&block) }
-
-    def redirect_stderr
-      $stderr = File.open('/dev/null')
-    end
-
-    unless ENV['COVERAGE']
-      context 'when block returns mashallable data, and process exists zero' do
-        let(:block) do
-          lambda do
-            :data_from_child_process
-          end
-        end
-
-        it { should eql(:data_from_child_process) }
-      end
+    it 'return block value' do
+      expect(object.call { :foo }).to be(:foo)
     end
 
-    context 'when block does return marshallable data' do
-      let(:block) do
-        lambda do
-          redirect_stderr
-          $stderr # not mashallable, nothing written to pipe and raises exception in child
-        end
-      end
-
-      it 'raises an exception' do
-        expect { subject }.to raise_error(described_class::Error, 'Childprocess wrote un-unmarshallable data')
-      end
-    end
-
-    context 'when block causes the child to exit nonzero' do
-      let(:block) do
-        lambda do
-          method = Kernel.method(:exit!)
-          Kernel.define_singleton_method(:exit!) do |_status|
-            method.call(1)
-          end
-        end
-      end
-
-      it 'raises an exception' do
-        expect { subject }.to raise_error(described_class::Error, 'Childprocess exited with nonzero exit status: 1')
-      end
-    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mutant
 version: !ruby/object:Gem::Version
-  version: 0.5.22
+  version: 0.5.23
 platform: ruby
 authors:
 - Markus Schirp
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-15 00:00:00.000000000 Z
+date: 2014-06-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: parser
@@ -207,39 +207,39 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 0.1.5
 - !ruby/object:Gem::Dependency
-  name: bundler
+  name: parallel
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.3.5
-  type: :development
+        version: 1.0.0
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.3.5
+        version: 1.0.0
 - !ruby/object:Gem::Dependency
-  name: parallel
+  name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: '1.3'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.3.5
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: '1.3'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.3.5
 description: Mutation testing for ruby
 email:
 - mbj@schirp-dso.com
@@ -577,4 +577,3 @@ test_files:
 - spec/unit/mutant/warning_expectation.rb
 - spec/unit/mutant/warning_filter_spec.rb
 - spec/unit/mutant_spec.rb
-has_rdoc: