mutant 0.2.20 → 0.3.0.beta2

data/lib/mutant/subject/method.rb

@@ -0,0 +1,100 @@
+module Mutant
+  class Subject
+    # Abstract base class for method subjects
+    class Method < self
+
+      # Test if method is public
+      #
+      # @return [true]
+      #   if method is public
+      #
+      # @return [false]
+      #   otherwise
+      #
+      # @api private
+      #
+      abstract_method :public?
+
+      # Return method name
+      #
+      # @return [Symbol]
+      #
+      # @api private
+      #
+      def name
+        node.children[self.class::NAME_INDEX]
+      end
+
+    private
+
+      # Return scope
+      #
+      # @return [Class, Module]
+      #
+      # @api private
+      #
+      def scope
+        context.scope
+      end
+
+      # Return subtype identifier
+      #
+      # @return [String]
+      #
+      # @api private
+      #
+      def subtype
+        "#{context.identification}#{self.class::SYMBOL}#{name}"
+      end
+
+      # Instance method subjects
+      class Instance < self
+
+        NAME_INDEX = 0
+        SYMBOL = '#'.freeze
+
+        # Test if method is public
+        #
+        # @return [true]
+        #   if method is public
+        #
+        # @return [false]
+        #   otherwise
+        #
+        # @api private
+        #
+        def public?
+          scope.public_method_defined?(name)
+        end
+        memoize :public?
+
+      private
+
+      end # Instance
+
+      # Singleton method subjects
+      class Singleton < self
+
+        NAME_INDEX = 1
+        SYMBOL = '.'.freeze
+
+        # Test if method is public
+        #
+        # @return [true]
+        #   if method is public
+        #
+        # @return [false]
+        #   otherwise
+        #
+        # @api private
+        #
+        def public?
+          scope.singleton_class.public_method_defined?(name)
+        end
+        memoize :public?
+
+      end # Singleton
+
+    end # Method
+  end # Subject
+end # Mutant

data/lib/mutant/subject.rb

@@ -1,31 +1,7 @@
 module Mutant
   # Subject of a mutation
   class Subject
-    include Adamantium::Flat, Enumerable, Equalizer.new(:context, :matcher, :node)
-
-    # Return context
-    #
-    # @return [Context]
-    #
-    # @api private
-    #
-    attr_reader :context
-
-    # Return matcher
-    #
-    # @return [Matcher]
-    #
-    # @api private
-    #
-    attr_reader :matcher
-
-    # Return AST node
-    #
-    # @return [Rubinius::AST::Node]
-    #
-    # @api private
-    #
-    attr_reader :node
+    include AbstractType, Adamantium::Flat, Enumerable, Concord::Public.new(:context, :node)
 
     # Enumerate possible mutations
     #
@@ -40,7 +16,7 @@ module Mutant
     def each
       return to_enum unless block_given?
       Mutator.each(node) do |mutant|
-        yield Mutation.new(self, mutant)
+        yield Mutation::Evil.new(self, mutant)
       end
 
       self
@@ -53,7 +29,7 @@ module Mutant
     # @api private
     #
     def noop
-      Mutation::Noop.new(self)
+      Mutation::Neutral.new(self, node)
     end
     memoize :noop
 
@@ -74,36 +50,36 @@ module Mutant
     # @api private
     #
     def source_line
-      node.line
+      node.location.expression.line
     end
 
-    # Return subject identicication
+    # Return subject identification
     #
     # @return [String]
     #
     # @api private
     #
     def identification
-      "#{matcher.identification}:#{source_path}:#{source_line}"
+      "#{subtype}:#{source_path}:#{source_line}"
     end
     memoize :identification
 
     # Return source representation of ast
     #
-    # @return [Source]
+    # @return [String]
     #
     # @api private
-    # 
+    #
     def source
-      ToSource.to_source(node)
+      Unparser.unparse(node)
     end
     memoize :source
 
     # Return root AST for node
     #
-    # @param [Rubinius::AST::Node] node
+    # @param [Parser::AST::Node] node
     #
-    # @return [Rubinius::AST::Node]
+    # @return [Parser::AST::Node]
     #
     # @api private
     #
@@ -113,7 +89,7 @@ module Mutant
 
     # Return root AST node for original AST ndoe
     #
-    # @return [Rubinius::AST::Node]
+    # @return [Parser::AST::Node]
     #
     # @api private
     #
@@ -124,21 +100,14 @@ module Mutant
 
   private
 
-    # Initialize subject
-    #
-    # @param [Matcher] matcher
-    #   the context of mutations
-    #
-    # @param [Rubinius::AST::Node] node
-    #   the node to be mutated
+    # Return subtype identifier
     #
-    # @return [unkown]
+    # @return [String]
     #
     # @api private
     #
-    def initialize(matcher, context, node)
-      @matcher, @context, @node = matcher, context, node
-    end
+    abstract_method :subtype
+    private :subtype
 
-  end
-end
+  end # Subject
+end # Mutant

data/lib/mutant/support/method_object.rb

@@ -1,6 +1,7 @@
 module Mutant
   # A mixing to create method object semantics
   module MethodObject
+
     # Hook called when descendant is extended
     #
     # @param [Module|Class] descendant
@@ -27,5 +28,6 @@ module Mutant
     def run(*args)
       new(*args)
     end
-  end
-end
+
+  end # MethodObject
+end # Mutant

data/lib/mutant.rb

@@ -1,4 +1,4 @@
-require 'backports'
+require 'benchmark'
 require 'set'
 require 'adamantium'
 require 'ice_nine'
@@ -8,29 +8,21 @@ require 'securerandom'
 require 'equalizer'
 require 'digest/sha1'
 require 'inflecto'
-require 'to_source'
+require 'parser'
+require 'parser/current'
+require 'unparser'
 require 'ice_nine'
 require 'diff/lcs'
 require 'diff/lcs/hunk'
 require 'rspec'
-
-# Patch ice none to freeze nodes correctly
-class IceNine::Freezer
-  # Rubinius namsepace
-  class Rubinius 
-    # AST namespace
-    class AST < IceNine::Freezer::Object
-      # Node configuration
-      class Node < IceNine::Freezer::Object
-      end
-    end
-  end
-end
+require 'anima'
+require 'concord'
 
 # Library namespace
 module Mutant
 end
 
+require 'mutant/node_helpers'
 require 'mutant/singleton_methods'
 require 'mutant/constants'
 require 'mutant/support/method_object'
@@ -38,6 +30,8 @@ require 'mutant/helper'
 require 'mutant/random'
 require 'mutant/mutator'
 require 'mutant/mutation'
+require 'mutant/mutation/evil'
+require 'mutant/mutation/neutral'
 require 'mutant/mutation/filter'
 require 'mutant/mutation/filter/code'
 require 'mutant/mutation/filter/whitelist'
@@ -55,55 +49,66 @@ require 'mutant/mutator/node/literal/string'
 require 'mutant/mutator/node/literal/fixnum'
 require 'mutant/mutator/node/literal/float'
 require 'mutant/mutator/node/literal/array'
-require 'mutant/mutator/node/literal/empty_array'
 require 'mutant/mutator/node/literal/hash'
 require 'mutant/mutator/node/literal/regex'
 require 'mutant/mutator/node/literal/nil'
-require 'mutant/mutator/node/block'
+require 'mutant/mutator/node/assignment'
+require 'mutant/mutator/node/argument'
+require 'mutant/mutator/node/arguments'
+require 'mutant/mutator/node/begin'
 require 'mutant/mutator/node/while'
 require 'mutant/mutator/node/super'
 require 'mutant/mutator/node/send'
-require 'mutant/mutator/node/send/with_arguments'
-require 'mutant/mutator/node/send/binary_operator_method'
+require 'mutant/mutator/node/send/binary'
 require 'mutant/mutator/node/when'
 require 'mutant/mutator/node/assignment'
 require 'mutant/mutator/node/define'
-require 'mutant/mutator/node/formal_arguments_19'
-require 'mutant/mutator/node/formal_arguments_19/default_mutations'
-require 'mutant/mutator/node/formal_arguments_19/require_defaults'
-require 'mutant/mutator/node/formal_arguments_19/pattern_argument_expansion'
-require 'mutant/mutator/node/actual_arguments'
-require 'mutant/mutator/node/pattern_arguments'
-require 'mutant/mutator/node/pattern_variable'
-require 'mutant/mutator/node/default_arguments'
+require 'mutant/mutator/node/mlhs'
 require 'mutant/mutator/node/return'
-require 'mutant/mutator/node/iter_19'
+require 'mutant/mutator/node/block'
 require 'mutant/mutator/node/if'
-require 'mutant/mutator/node/receiver_case'
-require 'mutant/loader' 
+require 'mutant/mutator/node/case'
+require 'mutant/config'
+require 'mutant/loader'
 require 'mutant/context'
 require 'mutant/context/scope'
 require 'mutant/subject'
+require 'mutant/subject/method'
 require 'mutant/matcher'
 require 'mutant/matcher/chain'
-require 'mutant/matcher/object_space'
 require 'mutant/matcher/method'
 require 'mutant/matcher/method/singleton'
 require 'mutant/matcher/method/instance'
-require 'mutant/matcher/scope_methods'
-require 'mutant/matcher/method/classifier'
+require 'mutant/matcher/methods'
+require 'mutant/matcher/namespace'
+require 'mutant/matcher/scope'
 require 'mutant/killer'
 require 'mutant/killer/static'
 require 'mutant/killer/rspec'
 require 'mutant/killer/forking'
+require 'mutant/killer/forked'
 require 'mutant/strategy'
+require 'mutant/strategy/static'
+require 'mutant/strategy/method_expansion'
 require 'mutant/strategy/rspec'
-require 'mutant/strategy/rspec/example_lookup'
+require 'mutant/strategy/rspec/dm2'
+require 'mutant/strategy/rspec/dm2/lookup'
+require 'mutant/strategy/rspec/dm2/lookup/method'
 require 'mutant/runner'
+require 'mutant/runner/config'
+require 'mutant/runner/subject'
+require 'mutant/runner/mutation'
 require 'mutant/cli'
+require 'mutant/cli/classifier'
+require 'mutant/cli/classifier/namespace'
+require 'mutant/cli/classifier/method'
 require 'mutant/color'
 require 'mutant/differ'
 require 'mutant/reporter'
-require 'mutant/reporter/stats'
 require 'mutant/reporter/null'
 require 'mutant/reporter/cli'
+require 'mutant/reporter/cli/printer'
+require 'mutant/reporter/cli/printer/config'
+require 'mutant/reporter/cli/printer/subject'
+require 'mutant/reporter/cli/printer/killer'
+require 'mutant/reporter/cli/printer/mutation'

data/mutant.gemspec

@@ -2,9 +2,10 @@
 
 Gem::Specification.new do |gem|
   gem.name        = 'mutant'
-  gem.version     = '0.2.20'
+  gem.version     = '0.3.0.beta2'
   gem.authors     = [ 'Markus Schirp' ]
-  gem.email       = [ 'mbj@seonic.net' ]
+  gem.email       = [ 'mbj@schirp-dso.com' ]
+
   gem.description = 'Mutation testing for ruby'
   gem.summary     = 'Mutation testing tool for ruby under MRI and Rubinius'
   gem.homepage    = 'https://github.com/mbj/mutant'
@@ -15,14 +16,14 @@ Gem::Specification.new do |gem|
   gem.extra_rdoc_files = %w[TODO LICENSE]
   gem.executables       = [ 'mutant' ]
 
-  gem.add_runtime_dependency('backports',           [ '~> 3.0', '>= 3.0.3' ])
-  gem.add_runtime_dependency('to_source',           '~> 0.2.19')
-  gem.add_runtime_dependency('ice_nine',            '~> 0.7.0')
+  gem.add_runtime_dependency('parser',              '~> 2.0.beta7')
+  gem.add_runtime_dependency('unparser',            '~> 0.0.4')
+  gem.add_runtime_dependency('ice_nine',            '~> 0.8.0')
   gem.add_runtime_dependency('descendants_tracker', '~> 0.0.1')
-  gem.add_runtime_dependency('adamantium',          '~> 0.0.7')
+  gem.add_runtime_dependency('adamantium',          '~> 0.0.8')
   gem.add_runtime_dependency('equalizer',           '~> 0.0.5')
   gem.add_runtime_dependency('inflecto',            '~> 0.0.2')
-  gem.add_runtime_dependency('abstract_type',       '~> 0.0.3')
-  gem.add_runtime_dependency('diff-lcs',            '~> 1.2.1')
+  gem.add_runtime_dependency('anima',               '~> 0.0.6')
+  gem.add_runtime_dependency('concord',             '~> 0.1.1')
   gem.add_runtime_dependency('rspec',               '~> 2.13.0')
 end

data/spec/integration/mutant/rspec_killer_spec.rb

@@ -11,14 +11,14 @@ describe Mutant,'rspec integration' do
   let(:strategy) { Mutant::Strategy::Rspec::DM2 }
 
   specify 'allows to kill mutations' do
-    Kernel.system("bundle exec mutant -I lib -r test_app --rspec-dm2 ::TestApp::Literal#string").should be(true)
+    Kernel.system('bundle exec mutant --rspec-dm2 ::TestApp::Literal#string').should be(true)
   end
 
   specify 'fails to kill mutations when they are not covered' do
-    Kernel.system("bundle exec mutant -I lib -r test_app --rspec-dm2 ::TestApp::Literal#uncovered_string").should be(false)
+    Kernel.system('bundle exec mutant --rspec-dm2 ::TestApp::Literal#uncovered_string').should be(false)
   end
 
   specify 'fails when some mutations when are not covered' do
-    Kernel.system("bundle exec mutant -I lib -r test_app --rspec-dm2 ::TestApp::Literal").should be(false)
+    Kernel.system('bundle exec mutant --rspec-dm2 ::TestApp::Literal').should be(false)
   end
 end

data/spec/integration/mutant/test_mutator_handles_types_spec.rb

@@ -0,0 +1,9 @@
+require 'spec_helper'
+
+describe Mutant do
+  specify 'mutant should not crash for any node parser can generate' do
+    Mutant::NODE_TYPES.each do |type|
+      Mutant::Mutator::Node::Registry.lookup(Mutant::NodeHelpers.s(type))
+    end
+  end
+end

data/spec/integration/mutant/zombie_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe Mutant, 'as a zombie' do
-  specify 'allows to create zombie from mutant' do
+  pending 'allows to create zombie from mutant' do
     Zombie.setup
     Zombie::Runner
   end

data/spec/shared/method_matcher_behavior.rb

@@ -0,0 +1,35 @@
+shared_examples_for 'a method matcher' do
+  before do
+    subject
+  end
+
+  let(:node)              { mutation_subject.node    }
+  let(:context)           { mutation_subject.context }
+  let(:mutation_subject)  { yields.first   }
+
+  it 'should return one subject' do
+    yields.size.should be(1)
+  end
+
+  it_should_behave_like 'an #each method'
+
+  it 'should have correct method name' do
+    name.should eql(method_name)
+  end
+
+  it 'should have correct line number' do
+    (node.location.expression.line - base).should eql(method_line)
+  end
+
+  it 'should have correct arity' do
+    arguments.children.length.should eql(method_arity)
+  end
+
+  it 'should have correct scope in context' do
+    context.send(:scope).should eql(scope)
+  end
+
+  it 'should have the correct node type' do
+    node.type.should be(type)
+  end
+end

data/spec/shared/mutator_behavior.rb

@@ -1,11 +1,51 @@
+class Subject
+
+  include Equalizer.new(:source)
+
+  Undefined = Object.new.freeze
+
+  attr_reader :source
+
+  def self.coerce(input)
+    case input
+    when Parser::AST::Node
+      new(input)
+    when String
+      new(Parser::CurrentRuby.parse(input))
+    else
+      raise
+    end
+  end
+
+  def to_s
+    "#{@node.inspect}\n#{@source}"
+  end
+
+  def initialize(node)
+    source = Unparser.unparse(node)
+    @node, @source = node, source
+  end
+
+  def assert_transitive!
+    generated = Unparser.generate(@node)
+    parsed    = Parser::CurrentRuby.parse(generated)
+    again     = Unparser.generate(parsed)
+    unless generated == again
+      # mostly an unparser bug!
+      fail "Untransitive:\n%s\n---\n%s" % [generated, again]
+    end
+    self
+  end
+end
+
 shared_examples_for 'a mutator' do
   subject { object.each(node) { |item| yields << item } }
 
-  let(:yields)        { []              }
-  let(:object)        { described_class }
+  let(:yields) { []              }
+  let(:object) { described_class }
 
   unless instance_methods.map(&:to_s).include?('node')
-    let(:node)          { source.to_ast   }
+    let(:node) { parse(source) }
   end
 
   it_should_behave_like 'a command method'
@@ -15,44 +55,35 @@ shared_examples_for 'a mutator' do
 
     it { should be_instance_of(to_enum.class) }
 
-    def assert_transitive(ast)
-      generated = ToSource.to_source(ast)
-      parsed    = generated.to_ast
-      again     = ToSource.to_source(parsed)
-      unless generated == again
-        fail "Untransitive:\n%s\n---\n%s" % [generated, again]
+    let(:expected_mutations) do
+      mutations.map do |mutation|
+        Subject.coerce(mutation)
       end
     end
 
-    unless instance_methods.include?(:expected_mutations)
-      let(:expected_mutations)  do
-        mutations.map do |mutation|
-          case mutation
-          when String
-            ast = mutation.to_ast
-            assert_transitive(ast)
-            ast
-          when Rubinius::AST::Node
-            assert_transitive(mutation)
-            mutation
-          else
-            raise
-          end
-        end.map do |node|
-          ToSource.to_source(node)
-        end.to_set
-      end
+    let(:generated_mutations) do
     end
 
     it 'generates the expected mutations' do
-      generated  = self.subject.map { |mutation| ToSource.to_source(mutation) }.to_set
 
-      missing    = (expected_mutations - generated).to_a
-      unexpected = (generated - expected_mutations).to_a
+      generated  = subject.map { |node| Subject.new(node) }
 
-      unless generated == expected_mutations
-        fail "Missing mutations:\n%s\nUnexpected mutations:\n%s" % [missing.join("\n----\n"), unexpected.join("\n----\n")]
+      missing    = expected_mutations - generated
+      unexpected = generated - expected_mutations
+
+      message = []
+
+      if missing.any?
+        message << 'Missing mutations (%i):' % missing.length
+        message.concat(missing)
       end
+
+      if unexpected.any?
+        message << 'Unexpected mutations (%i):' % unexpected.length
+        message.concat(unexpected)
+      end
+
+      fail "Original:\n#{generate(node)}\n-----\n#{message.join("\n-----\n")}" if message.any?
     end
   end
 end

data/spec/spec_helper.rb

@@ -1,13 +1,23 @@
-# encoding: utf-8
-
+require 'mutant'
 require 'devtools'
 Devtools.init_spec_helper
 
 $: << File.join(TestApp.root,'lib')
 
 require 'test_app'
-require 'mutant'
+
+module ParserHelper
+  def generate(node)
+    Unparser.unparse(node)
+  end
+
+  def parse(string)
+    Parser::CurrentRuby.parse(string)
+  end
+end
 
 RSpec.configure do |config|
   config.include(CompressHelper)
+  config.include(ParserHelper)
+  config.include(Mutant::NodeHelpers)
 end

data/spec/support/ice_nine_config.rb

@@ -0,0 +1,8 @@
+require 'ice_nine'
+
+module IceNine
+  class Freezer
+    class RSpec < NoFreeze
+    end
+  end
+end

data/spec/support/rspec.rb

@@ -8,7 +8,7 @@ module Rspec
   # @api private
   #
   def self.nest
-    original_world, original_configuration = 
+    original_world, original_configuration =
       ::RSpec.instance_variable_get(:@world),
       ::RSpec.instance_variable_get(:@configuration)
 

data/spec/support/zombie.rb

@@ -151,7 +151,7 @@ module Zombie
   #
   # @api private
   #
-  # FIXME: 
+  # FIXME:
   #  Yeah looks very ugly but im currently to exited to do a cleanup.
   #
   def self.files