mutant 0.2.12 → 0.2.13

data/Changelog.md

@@ -1,3 +1,7 @@
+# v0.2.13 2013-01-05
+
+* [fixed] Capture failures that occur in the window between mutation insertion and spec run as kills
+
 # v0.2.12 2013-01-03
 
 * [fixed] Do not crash when trying to load methods from precompiled ruby under rbx

data/Gemfile

@@ -2,5 +2,5 @@ source 'https://rubygems.org'
 
 gemspec
 
-gem 'devtools', :git => 'https://github.com/mbj/devtools.git'
+gem 'devtools', :git => 'https://github.com/datamapper/devtools.git'
 eval(File.read(File.join(File.dirname(__FILE__),'Gemfile.devtools')))

data/Gemfile.devtools

@@ -1,30 +1,36 @@
+# encoding: utf-8
+
 group :development do
-  gem 'rake',    '~> 10.0'
-  gem 'rspec',   '~> 2.12.0'
-  gem 'yard',    '~> 0.8.3'
+  gem 'rake',  '~> 10.0.3'
+  gem 'rspec', '~> 2.12.0'
+  gem 'yard',  '~> 0.8.3'
+end
+
+group :yard do
+  gem 'redcarpet', '~> 2.2.2', :platforms => [ :mri, :rbx ]
 end
 
 group :guard do
-  gem 'guard',         '~> 1.5.4'
+  gem 'guard',         '~> 1.6.1'
   gem 'guard-bundler', '~> 1.0.0'
-  gem 'guard-rspec',   '~> 2.1.1'
-  gem 'rb-inotify', :git => 'https://github.com/mbj/rb-inotify'
-end
+  gem 'guard-rspec',   '~> 2.3.3'
 
-group :benchmarks do
-  gem 'rbench', '~> 0.2.3'
-end
+  # file system change event handling
+  gem 'rb-fchange', '~> 0.0.6', :require => false
+  gem 'rb-fsevent', '~> 0.9.3', :require => false
+  gem 'rb-inotify', '~> 0.8.8', :require => false, :git => 'https://github.com/nex3/rb-inotify'
 
-platform :jruby do
-  group :jruby do
-    gem 'jruby-openssl', '~> 0.7.4'
-  end
+  # notification handling
+  gem 'libnotify',               '~> 0.8.0', :require => false
+  gem 'rb-notifu',               '~> 0.0.4', :require => false
+  gem 'terminal-notifier-guard', '~> 1.5.3', :require => false
 end
 
 group :metrics do
-  gem 'flay',      '~> 1.4.2'
-  gem 'flog',      '~> 2.5.1'
-  gem 'reek',      '~> 1.2.8', :git => 'https://github.com/dkubb/reek.git'
+  gem 'backports', '~> 2.6.5'
+  gem 'flay',      '~> 1.4.3'
+  gem 'flog',      '~> 2.5.3'
+  gem 'reek',      '~> 1.2.13', :git => 'https://github.com/dkubb/reek.git', :branch => 'fix-redundant-irresponsible-module-warnings'
   gem 'roodi',     '~> 2.1.0'
   gem 'yardstick', '~> 0.8.0'
 
@@ -33,11 +39,25 @@ group :metrics do
     gem 'yard-spellcheck', '~> 0.1.5'
   end
 
+  platforms :mri_18 do
+    gem 'rcov', '~> 1.0.0'
+  end
+
   platforms :mri_19 do
-    gem 'simplecov',   '~> 0.7'
+    gem 'simplecov', '~> 0.7.1'
   end
 
   platforms :rbx do
-    gem 'pelusa', '~> 0.2.1'
+    gem 'pelusa', '~> 0.2.2'
+  end
+end
+
+group :benchmarks do
+  gem 'rbench', '~> 0.2.3'
+end
+
+platform :jruby do
+  group :jruby do
+    gem 'jruby-openssl', '~> 0.8.2'
   end
 end

data/README.md

@@ -64,8 +64,8 @@ The following specs are executed to kill a mutation on:
 ```
 Public instance  methods: spec/unit/#{namespace}/#{class_name}/#{method_name}_spec.rb
 Public singleton methods: spec/unit/#{namespace}/#{class_name}/class_methods/#{method_name}_spec.rb
-Public instance  methods: spec/unit/#{namespace}/#{class_name}/
-Public singleton methods: spec/unit/#{namespace}/#{class_name}/class_methods
+Private instance  methods: spec/unit/#{namespace}/#{class_name}/*_spec.rb
+Private singleton methods: spec/unit/#{namespace}/#{class_name}/class_methods/*_spec.rb
 ```
 
 ### --rspec-unit
@@ -91,6 +91,7 @@ Alternatives
 Credits
 -------
 
+* [Markus Schirp (mbj)](https://github.com/mbj)
 * A [gist](https://gist.github.com/1065789) from [dkubb](https://github.com/dkubb) showing ideas.
 * Older abandoned [mutant](https://github.com/txus/mutant). For motivating me doing this one.
 * [heckle](https://github.com/seattlerb/heckle). For getting me into mutation testing.

data/TODO

@@ -2,6 +2,7 @@ Code:
   * Test mutant with dynamically created zombie.
   * Fix ugly code within default parameter mutations
   * Break up lib/mutant/mutator/node/send.rb in class specific files
+  * Log all warnings throug reporter, so remove random $stderr.puts calls
 
 AST:
   * Fix the rubinius AST to allow setting @vcall_style variable in Rubinius::AST::Send nodes.
@@ -27,11 +28,12 @@ Loader:
   * Make sure loader does not change visibility of injected mutants
 
 Killers:
-  * Aggregate warnings on missing spec files
+  * Move test framework specific stuff to strategy
   * Add a general master <=> killer IPC interface. So different strategies of isolation 
     (fork, vs jruby runtime creation) will work without big impact.
 
 Strategy:
+  * Aggregate warnings on missing spec files
   * Provide "expicit files to kill with" strategy
   * Automatically load ./spec/spec_helper.rb for rspec strategies (No need to specify -I and -r anymore)
 

data/config/flay.yml

@@ -1,3 +1,3 @@
 ---
 threshold: 25 # Todo bring down to ~20
-total_score: 902
+total_score: 914

data/config/site.reek

@@ -10,13 +10,14 @@ UncommunicativeParameterName:
 LargeClass:
   max_methods: 10
   exclude: 
-  #- "Mutant::Matcher::Method" # 13 methods
   - "Mutant::Reporter::CLI" # 16 methods TODO Reduce!
+  - "Mutant::Reporter::Stats" # 6 ivars TODO Reduce!
   - "Mutant::CLI" # 19 methods and 7 ivars, TODO Reduce!
   enabled: true
   max_instance_variables: 3
 UncommunicativeMethodName:
-  accept: []
+  accept: 
+  - sha1
   exclude: []
   enabled: true
   reject:
@@ -26,7 +27,9 @@ UncommunicativeMethodName:
 LongParameterList:
   max_params: 2
   exclude: 
+  - "Mutant::Mutator::Node#new_send_with_arguments" # 3 params
   - "Mutant::Context::Constant#initialize" # 3 params
+  - "Mutant::Subject#initialize" # 3 params
   enabled: true
   overrides: {}
 FeatureEnvy:
@@ -43,6 +46,7 @@ IrresponsibleModule:
   enabled: true
 UncommunicativeModuleName:
   accept: 
+  - Mutant::Mutator::Node::Iter19
   - Mutant::Strategy::Rspec::DM2
   exclude: []
   enabled: true
@@ -53,6 +57,8 @@ NestedIterators:
   ignore_iterators: []
   exclude: 
   - Mutant#self.define_singleton_subclass
+  - Mutant::Mutator::Util::Array::Element#dispatch
+  - Mutant::Mutator::Node::ReceiverCase#emit_when_branch_mutation
   enabled: true
   max_allowed_nesting: 1
 LongMethod:
@@ -61,7 +67,8 @@ LongMethod:
   enabled: true
 Duplication:
   allow_calls: []
-  exclude: []
+  exclude: 
+  - Mutant::Mutator::Node::Literal::Hash#emit_element_presence
   enabled: true
   max_calls: 1
 UtilityFunction:

data/lib/mutant.rb

@@ -29,80 +29,10 @@ end
 
 # Library namespace
 module Mutant
-
-  # The list of ruby kewords from http://ruby-doc.org/docs/keywords/1.9/
-  KEYWORDS = %w(
-    BEGIN END __ENCODING__ __END__ __FILE__
-    __LINE__ alias and begin break case class
-    def define do else elsif end ensure false
-    for if in module next nil not or redo
-    rescue retry return self super then true
-    undef unless until when while yield
-  ).map(&:to_sym).to_set.freeze
-
-  BINARY_METHOD_OPERATOR_EXPANSIONS = {
-    :<=>  => :spaceship_operator,
-    :===  => :case_equality_operator,
-    :[]=  => :element_writer,
-    :[]   => :element_reader,
-    :<=   => :less_than_or_equal_to_operator,
-    :>=   => :greater_than_or_equal_to_operator,
-    :==   => :equality_operator,
-    :'!~' => :nomatch_operator,
-    :'!=' => :inequality_operator,
-    :=~   => :match_operator,
-    :<<   => :left_shift_operator,
-    :>>   => :right_shift_operator,
-    :**   => :exponentation_operator,
-    :*    => :multiplication_operator,
-    :%    => :modulo_operator,
-    :/    => :division_operator,
-    :|    => :bitwise_or_operator,
-    :^    => :bitwise_xor_operator,
-    :&    => :bitwise_and_operator,
-    :<    => :less_than_operator,
-    :>    => :greater_than_operator,
-    :+    => :addition_operator,
-    :-    => :substraction_operator
-  }.freeze
-
-  UNARY_METHOD_OPERATOR_EXPANSIONS = {
-    :~@   => :unary_match_operator,
-    :+@   => :unary_addition_operator,
-    :-@   => :unary_substraction_operator,
-    :'!'  => :negation_operator
-  }.freeze
-
-  BINARY_METHOD_OPERATORS = BINARY_METHOD_OPERATOR_EXPANSIONS.keys.to_set.freeze
-
-  OPERATOR_EXPANSIONS = BINARY_METHOD_OPERATOR_EXPANSIONS.merge(UNARY_METHOD_OPERATOR_EXPANSIONS).freeze
-
-  # Define instance of subclassed superclass as constant
-  #
-  # @param [Class] superclass
-  # @param [Symbol] name
-  #
-  # @return [self]
-  #
-  # @api private
-  #
-  def self.define_singleton_subclass(name, superclass, &block)
-    klass = Class.new(superclass) do
-
-      def inspect; self.class.name; end
-
-      define_singleton_method(:name) do
-        "#{superclass.name}::#{name}".freeze
-      end
-
-    end
-    klass.class_eval(&block)
-    superclass.const_set(name, klass.new)
-    self
-  end
-
 end
 
+require 'mutant/singleton_methods'
+require 'mutant/constants'
 require 'mutant/support/method_object'
 require 'mutant/helper'
 require 'mutant/random'
@@ -133,6 +63,9 @@ require 'mutant/mutator/node/block'
 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/when'
 require 'mutant/mutator/node/assignment'
 require 'mutant/mutator/node/define'
 require 'mutant/mutator/node/formal_arguments_19'
@@ -145,7 +78,7 @@ require 'mutant/mutator/node/pattern_variable'
 require 'mutant/mutator/node/default_arguments'
 require 'mutant/mutator/node/return'
 require 'mutant/mutator/node/iter_19'
-require 'mutant/mutator/node/if_statement'
+require 'mutant/mutator/node/if'
 require 'mutant/mutator/node/receiver_case'
 require 'mutant/loader' 
 require 'mutant/context'

data/lib/mutant/constants.rb

@@ -0,0 +1,49 @@
+module Mutant
+
+  # The list of ruby kewords from http://ruby-doc.org/docs/keywords/1.9/
+  KEYWORDS = %w(
+    BEGIN END __ENCODING__ __END__ __FILE__
+    __LINE__ alias and begin break case class
+    def define do else elsif end ensure false
+    for if in module next nil not or redo
+    rescue retry return self super then true
+    undef unless until when while yield
+  ).map(&:to_sym).to_set.freeze
+
+  BINARY_METHOD_OPERATOR_EXPANSIONS = {
+    :<=>  => :spaceship_operator,
+    :===  => :case_equality_operator,
+    :[]=  => :element_writer,
+    :[]   => :element_reader,
+    :<=   => :less_than_or_equal_to_operator,
+    :>=   => :greater_than_or_equal_to_operator,
+    :==   => :equality_operator,
+    :'!~' => :nomatch_operator,
+    :'!=' => :inequality_operator,
+    :=~   => :match_operator,
+    :<<   => :left_shift_operator,
+    :>>   => :right_shift_operator,
+    :**   => :exponentation_operator,
+    :*    => :multiplication_operator,
+    :%    => :modulo_operator,
+    :/    => :division_operator,
+    :|    => :bitwise_or_operator,
+    :^    => :bitwise_xor_operator,
+    :&    => :bitwise_and_operator,
+    :<    => :less_than_operator,
+    :>    => :greater_than_operator,
+    :+    => :addition_operator,
+    :-    => :substraction_operator
+  }.freeze
+
+  UNARY_METHOD_OPERATOR_EXPANSIONS = {
+    :~@   => :unary_match_operator,
+    :+@   => :unary_addition_operator,
+    :-@   => :unary_substraction_operator,
+    :'!'  => :negation_operator
+  }.freeze
+
+  BINARY_METHOD_OPERATORS = BINARY_METHOD_OPERATOR_EXPANSIONS.keys.to_set.freeze
+
+  OPERATOR_EXPANSIONS = BINARY_METHOD_OPERATOR_EXPANSIONS.merge(UNARY_METHOD_OPERATOR_EXPANSIONS).freeze
+end

data/lib/mutant/killer/forking.rb

@@ -41,8 +41,12 @@ module Mutant
       #
       def run
         fork do
-          killer = @killer.new(strategy, mutation)
-          Kernel.exit(killer.fail? ? 1 : 0)
+          begin
+            killer = @killer.new(strategy, mutation)
+            Kernel.exit(killer.fail? ? 1 : 0)
+          rescue
+            Kernel.exit(1)
+          end
         end
 
         status = Process.wait2.last

data/lib/mutant/loader.rb

@@ -17,21 +17,21 @@ module Mutant
     # Initialize and insert mutation into vm
     #
     # @param [Rubinius::AST::Script] root
-    # @param [String] file
-    # @param [Fixnum] line
+    # @param [Subject] subject
     #
     # @return [undefined]
     #
     # @api private
     #
-    def initialize(root, file, line)
-      @root, @file, @line = root, file, line
+    def initialize(root, subject)
+      @root, @subject = root, subject
       run
     end
 
     # Eval based loader
     class Eval < self
-      private
+
+    private
 
       # Run loader
       #
@@ -40,7 +40,7 @@ module Mutant
       # @api private
       #
       def run
-        eval(source, TOPLEVEL_BINDING, @file, @line)
+        Kernel.eval(source, TOPLEVEL_BINDING, @subject.source_path, @subject.source_line)
       end
 
       # Return source
@@ -54,78 +54,5 @@ module Mutant
       end
     end
 
-    # Rubinius script node based loaded
-    class Rubinius < self
-      private
-
-      # Run loader
-      #
-      # @return [undefined]
-      #
-      # @api private
-      #
-      def run(root)
-        Rubinius.run_script(compiled_code)
-      end
-
-      # Return compiled code
-      #
-      # @return [Rubinius::CompiledCode]
-      #
-      # @api private
-      #
-      # FIXME: rbx on travis is older than on my devbox.
-      #
-      def compiled_code
-        _script = script
-        _script.respond_to?(:compiled_code) ? _script.compiled_code : _script.compiled_method
-      end
-
-      # Return code script
-      #
-      # @return [Rubinius::CompiledCode::Script]
-      #
-      # @api private
-      #
-      def script
-        compiled_code_raw.create_script
-      end
-
-      # Return compiled code for node
-      #
-      # @return [Rubinius::CompiledCode]
-      #
-      # @api private
-      #
-      def compiled_code_raw
-        compiler.run
-      end
-
-      # Return compiler loaded with mutated ast
-      #
-      # @return [Rubinius::Compiler]
-      #
-      # @api private
-      #
-      def compiler
-        Rubinius::Compiler.new(:bytecode, :compiled_method).tap do |compiler|
-          compiler.generator.input(@root)
-        end
-      end
-
-      # Return script node
-      #
-      # @param [Rubinius::AST::Node] node
-      #
-      # @return [Rubinius::AST::Script]
-      #
-      # @api private
-      #
-      def script(node)
-        Rubinius::AST::Script.new(node).tap do |script|
-          script.file = source_path
-        end
-      end
-    end
   end
 end