mutant 0.5.6 → 0.5.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 54ae8c3475a320f0414ab619b85fbb4bb2965471
-  data.tar.gz: 8d7fb1c5f84d6da82c4ea00ef4bae0f08c1b4354
+  metadata.gz: 6ed226d5dffa5ae1f7427abfa3d0906c4795aeb9
+  data.tar.gz: 683e7fb70196af33a3fa62bd208c1944f59c33c0
 SHA512:
-  metadata.gz: becb5fa79ed00c804b3b04f3f4cdffb99bfe0c9fc206ec8fb1e6b4f4ba9acc16eee2cd7363e9d5e23e0f03663870c5482e3682fe6d9eac29ba020725e5936ecd
-  data.tar.gz: ed0f26b5694e3b909c0de51db291c66149415dbadc6834f1bcdeef1e48a9c39acdd16233c5edf76c436433c9bc49baf3698fbab0a1c317c9df6d065ac18fff9a
+  metadata.gz: 484d060c5dbeeb7287a91d8e85d5146d628fface21960803e77d3a5d36d3eae5dfedcb2abd83dab0c5d926fde2cd1d5637bc3b4a0cbacb61c384d58a034b5095
+  data.tar.gz: 1aa578d23500f3be6ac4f885c4ef0dfeaa29047df8a869a158de38da36eef9728c3e1fe6b319a5bdffa3e97a71e1f93eb0904815b2c8e6bf6639f1018374f451

data/Changelog.md

@@ -1,3 +1,9 @@
+# v0.5.7 2014-03-23
+
+Changes:
+
+* Fix crash on invalid partial AST unparsing, closes: #164
+
 # v0.5.6 2014-03-09
 
 Changes:

data/lib/mutant.rb

@@ -87,6 +87,7 @@ require 'mutant/mutator/node/if'
 require 'mutant/mutator/node/case'
 require 'mutant/mutator/node/splat'
 require 'mutant/mutator/node/resbody'
+require 'mutant/mutator/node/rescue'
 require 'mutant/config'
 require 'mutant/loader'
 require 'mutant/context'

data/lib/mutant/loader.rb

@@ -3,21 +3,7 @@
 module Mutant
   # Base class for code loaders
   class Loader
-    include AbstractType
-    include Procto.call
-
-    # Initialize and insert mutation into vm
-    #
-    # @param [Parser::AST::Node] root
-    # @param [Subject] subject
-    #
-    # @return [undefined]
-    #
-    # @api private
-    #
-    def initialize(root, subject)
-      @root, @subject = root, subject
-    end
+    include AbstractType, Concord.new(:root, :subject), Procto.call
 
     # Eval based loader
     class Eval < self
@@ -29,11 +15,12 @@ module Mutant
       # @api private
       #
       def call
+        subject.prepare
         eval(
           source,
           TOPLEVEL_BINDING,
-          @subject.source_path.to_s,
-          @subject.source_line
+          subject.source_path.to_s,
+          subject.source_line
         )
         nil
       end
@@ -47,7 +34,7 @@ module Mutant
       # @api private
       #
       def source
-        Unparser.unparse(@root)
+        Unparser.unparse(root)
       end
 
     end # Eval

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

@@ -10,9 +10,7 @@ module Mutant
         # These nodes still need a dedicated mutator,
         # your contribution is that close!
         handle(
-          :ensure,
-          :rescue, :redo, :defined?,
-          :regopt, :retry, :arg_expr,
+          :ensure, :redo, :defined?, :regopt, :retry, :arg_expr,
           :kwrestarg, :kwoptarg, :kwarg, :undef, :module, :empty,
           :alias, :for, :xstr, :back_ref, :class,
           :sclass, :match_with_lvasgn, :match_current_line, :while_post,

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

@@ -36,8 +36,6 @@ module Mutant
           Util::Array.each(captures.children, self) do |matchers|
             next if matchers.empty?
             emit_captures(s(:array, *matchers))
-            # p capture
-            # emit_captures(s(:array, *capture))
           end
         end
 

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

@@ -0,0 +1,26 @@
+# encoding: utf-8
+
+module Mutant
+  class Mutator
+    class Node
+      # Mutator for rescue nodes
+      class Rescue < Generic
+
+        handle :rescue
+
+        # Return identity
+        #
+        # @param [Parser::AST::Node] node
+        #
+        # @return [String]
+        #
+        # @api private
+        #
+        def self.identity(node)
+          super(NodeHelpers.s(:kwbegin, node))
+        end
+
+      end # Rescue
+    end # Node
+  end # Mutator
+end # Mutant

data/lib/mutant/subject.rb

@@ -29,6 +29,16 @@ module Mutant
       context.source_path
     end
 
+    # Prepare the subject for the insertion of mutation
+    #
+    # @return [self]
+    #
+    # @api private
+    #
+    def prepare
+      self
+    end
+
     # Return source line
     #
     # @return [Fixnum]

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

@@ -24,6 +24,17 @@ module Mutant
         end
         memoize :public?
 
+        # Prepare subject for mutation insertion
+        #
+        # @return [self]
+        #
+        # @api private
+        #
+        def prepare
+          scope.send(:undef_method, name)
+          self
+        end
+
       private
 
         # Mutator for memoized instance methods

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

@@ -24,6 +24,17 @@ module Mutant
         end
         memoize :public?
 
+        # Prepare subject for mutation insertion
+        #
+        # @return [self]
+        #
+        # @api private
+        #
+        def prepare
+          scope.singleton_class.send(:undef_method, name)
+          self
+        end
+
       end # Singleton
     end # Method
   end # Subject

data/lib/mutant/version.rb

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

data/spec/unit/mutant/loader/eval_spec.rb

@@ -14,6 +14,10 @@ describe Mutant::Loader::Eval, '.call' do
     double('Subject', source_path: path, source_line: line)
   end
 
+  before do
+    expect(mutation_subject).to receive(:prepare).and_return(mutation_subject)
+  end
+
   let(:source) do
     <<-RUBY
       class SomeNamespace

data/spec/unit/mutant/mutator/node/define_spec.rb

@@ -14,6 +14,19 @@ describe Mutant::Mutator, 'def' do
     it_should_behave_like 'a mutator'
   end
 
+  context 'empty rescue body' do
+    let(:source) { "def foo\nfoo\nrescue\nend" }
+
+    let(:mutations) do
+      mutations = []
+      mutations << 'def foo; raise; end'
+      mutations << 'def foo; nil; rescue; end'
+      mutations << 'def foo; end'
+    end
+
+    it_should_behave_like 'a mutator'
+  end
+
   context 'with no arguments' do
     let(:source) { 'def foo; true; false; end' }
 

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

@@ -12,6 +12,28 @@ describe Mutant::Subject::Method::Instance do
     s(:def, :foo, s(:args))
   end
 
+  describe '#prepare' do
+
+    let(:context) do
+      Mutant::Context::Scope.new(scope, double('Source Path'))
+    end
+
+    let(:scope) do
+      Class.new do
+        def foo
+        end
+      end
+    end
+
+    subject { object.prepare }
+
+    it 'undefines method on scope' do
+      expect { subject }.to change { scope.instance_methods.include?(:foo) }.from(true).to(false)
+    end
+
+    it_should_behave_like 'a command method'
+  end
+
   describe '#source' do
     subject { object.source }
 

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

@@ -0,0 +1,42 @@
+# encoding: UTF-8
+
+require 'spec_helper'
+
+describe Mutant::Subject::Method::Singleton do
+  include Mutant::NodeHelpers
+
+  let(:object)  { described_class.new(context, node) }
+  let(:context) { double }
+
+  let(:node) do
+    s(:defs, s(:self), :foo, s(:args))
+  end
+
+  describe '#prepare' do
+
+    let(:context) do
+      Mutant::Context::Scope.new(scope, double('Source Path'))
+    end
+
+    let(:scope) do
+      Class.new do
+        def self.foo
+        end
+      end
+    end
+
+    subject { object.prepare }
+
+    it 'undefines method on scope' do
+      expect { subject }.to change { scope.methods.include?(:foo) }.from(true).to(false)
+    end
+
+    it_should_behave_like 'a command method'
+  end
+
+  describe '#source' do
+    subject { object.source }
+
+    it { should eql("def self.foo\nend") }
+  end
+end

data/test_app/Gemfile.rspec2

@@ -1,4 +1,5 @@
 source 'https://rubygems.org'
 
-gem 'rspec', '~> 2.14.1'
-gem 'mutant', path: '../'
+gem 'rspec',  '~> 2.14.1'
+gem 'mutant',       path: '../'
+gem 'mutant-rspec', path: '../'

data/test_app/Gemfile.rspec3

@@ -1,3 +1,4 @@
 source 'https://rubygems.org'
 gem 'rspec', '~> 3.0.0.beta2'
-gem 'mutant', path: '../'
+gem 'mutant',       path: '../'
+gem 'mutant-rspec', path: '../'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mutant
 version: !ruby/object:Gem::Version
-  version: 0.5.6
+  version: 0.5.7
 platform: ruby
 authors:
 - Markus Schirp
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-03-12 00:00:00.000000000 Z
+date: 2014-03-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: parser
@@ -315,6 +315,7 @@ files:
 - lib/mutant/mutator/node/nthref.rb
 - lib/mutant/mutator/node/op_asgn.rb
 - lib/mutant/mutator/node/resbody.rb
+- lib/mutant/mutator/node/rescue.rb
 - lib/mutant/mutator/node/restarg.rb
 - lib/mutant/mutator/node/return.rb
 - lib/mutant/mutator/node/send.rb
@@ -435,6 +436,7 @@ files:
 - spec/unit/mutant/strategy_spec.rb
 - spec/unit/mutant/subject/context_spec.rb
 - spec/unit/mutant/subject/method/instance_spec.rb
+- spec/unit/mutant/subject/method/singleton_spec.rb
 - spec/unit/mutant/subject/mutations_spec.rb
 - spec/unit/mutant/subject/node_spec.rb
 - spec/unit/mutant/subject_spec.rb
@@ -475,7 +477,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.0
+rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
 summary: Mutation testing tool for ruby under MRI and Rubinius
@@ -554,6 +556,7 @@ test_files:
 - spec/unit/mutant/strategy_spec.rb
 - spec/unit/mutant/subject/context_spec.rb
 - spec/unit/mutant/subject/method/instance_spec.rb
+- spec/unit/mutant/subject/method/singleton_spec.rb
 - spec/unit/mutant/subject/mutations_spec.rb
 - spec/unit/mutant/subject/node_spec.rb
 - spec/unit/mutant/subject_spec.rb