mutant 0.5.19 → 0.5.20

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b345e5eb8a65a538f350534528ca7c327bf2dde1
-  data.tar.gz: b516b255a58425b77916f19dc5640c2e4e849393
+  metadata.gz: 2423eb3dea806c37511ba8f9e23e20215330d9a5
+  data.tar.gz: b5c6e0e92cc025a1d70f61727567d3ccd64578c7
 SHA512:
-  metadata.gz: 6e1bcd22f9acf5400057b7dcac381f7c29ddf2dda558a422ed8b8d5d2c48189eeee17d3deee583cb0865ee3e2077f552b2451bc5743625f29739dca5bfb7021a
-  data.tar.gz: b242006807e001093b1f22ef974c22900987901263ef39fe4d0e6ef651fe3b54ee79c8fa6a0fb46367228b7cb8e45822c60c0de250f5cd0ef31a3d039d19c93e
+  metadata.gz: 0afc928d562d4f8adc32e1b1f7ba6c20544ee1c5e304c60772ab1fe5e7d648549a8e8b3c325ee7d828cc285f7083eb649d0344b90fb81c0b42f60cb78152034b
+  data.tar.gz: a7d05e7b132dc61673c683630de8a503496167521e57e9b7e0c3d86e7049bd979f7201772af5d9e1b3bfc8980731373480725193c6f93afc89aafd084cd023f8

data/.travis.yml

@@ -5,11 +5,8 @@ env:
 rvm:
   - 1.9.3
   - 2.0.0
-  - 2.1.1
+  - 2.1.2
   - rbx-2
-matrix:
-  allow_failures:
-    - rvm: rbx-2
 notifications:
   irc:
     channels:

data/Changelog.md

@@ -1,3 +1,12 @@
+# v0.5.20 2014-06-15
+
+* Remove support for matchers prefixed with ::
+* Fix cases where mutated source diff was empty #198
+* Fix mutate to simpler primitive violation break to next #203
+* Improve integration / corpus tests to spot highlevel regressions on CI level.
+
+* Remove support for matchers prefixed with ::
+
 # v0.5.19 2014-06-06
 
 Changes:

data/Gemfile.devtools

@@ -1,9 +1,9 @@
 # encoding: utf-8
 
 group :development do
-  gem 'rake',       '~> 10.3.1'
-  gem 'rspec',      '~> 2.99.0.beta2'
-  gem 'rspec-core', '~> 2.99.0.beta2'
+  gem 'rake',       '~> 10.3.2'
+  gem 'rspec',      '~> 2.99.0'
+  gem 'rspec-core', '~> 2.99.0'
   gem 'yard',       '~> 0.8.7.4'
 
   platform :rbx do
@@ -11,30 +11,34 @@ group :development do
   end
 end
 
+group :yard do
+  gem 'kramdown', '~> 1.3.3'
+end
+
 group :guard do
-  gem 'guard',         '~> 2.6.0'
+  gem 'guard',         '~> 2.6.1'
   gem 'guard-bundler', '~> 2.0.0'
-  gem 'guard-rspec',   '~> 4.2.8'
+  gem 'guard-rspec',   '~> 4.2.9'
   gem 'guard-rubocop', '~> 1.1.0'
 
   # file system change event handling
-  gem 'listen',     '~> 2.7.3'
+  gem 'listen',     '~> 2.7.7'
   gem 'rb-fchange', '~> 0.0.6', require: false
   gem 'rb-fsevent', '~> 0.9.4', require: false
-  gem 'rb-inotify', '~> 0.9.4', require: false
+  gem 'rb-inotify', '~> 0.9.5', require: false
 
   # notification handling
-  gem 'libnotify',               '~> 0.8.2', require: false
+  gem 'libnotify',               '~> 0.8.3', require: false
   gem 'rb-notifu',               '~> 0.0.4', require: false
   gem 'terminal-notifier-guard', '~> 1.5.3', require: false
 end
 
 group :metrics do
   gem 'coveralls', '~> 0.7.0'
-  gem 'flay',      '~> 2.4.0'
-  gem 'flog',      '~> 4.2.0'
+  gem 'flay',      '~> 2.5.0'
+  gem 'flog',      '~> 4.2.1'
   gem 'reek',      '~> 1.3.7'
-  gem 'rubocop',   '~> 0.21.0'
+  gem 'rubocop',   '~> 0.23.0'
   gem 'simplecov', '~> 0.7.1'
   gem 'yardstick', '~> 0.9.9'
 

data/README.md

@@ -5,6 +5,8 @@ mutant
 [![Dependency Status](https://gemnasium.com/mbj/mutant.png)](https://gemnasium.com/mbj/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.
 
@@ -115,13 +117,13 @@ Examples
 ```
 cd virtus
 # Run mutant on virtus namespace
-mutant --include lib --require virtus --use rspec ::Virtus*
+mutant --include lib --require virtus --use rspec Virtus*
 # Run mutant on specific virtus class
-mutant --include lib --require virtus --use rspec ::Virtus::Attribute
+mutant --include lib --require virtus --use rspec Virtus::Attribute
 # Run mutant on specific virtus class method
-mutant --include lib --require virtus --use rspec ::Virtus::Attribute.build
+mutant --include lib --require virtus --use rspec Virtus::Attribute.build
 # Run mutant on specific virtus instance method
-mutant --include lib --require virtus --use rspec ::Virtus::Attribute#type
+mutant --include lib --require virtus --use rspec Virtus::Attribute#type
 ```
 
 Subjects
@@ -151,8 +153,10 @@ 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. I have a [gittip](https://gittip.com/mbj) account.
-Also I'm looking for sponsors. Email me if you are interested.
+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: 883
+total_score: 903

data/config/reek.yml

@@ -56,6 +56,7 @@ NestedIterators:
   - Mutant::CLI#parse
   - Mutant::Mutator::Util::Array::Element#dispatch
   - Mutant::Mutator::Node::Resbody#mutate_captures
+  - Mutant::Mutator::Node::Arguments#emit_argument_mutations
   - Mutant::Reporter::CLI::Report::Config#generic_stats
   - Mutant::RequireHighjack#infect
   - Mutant::RequireHighjack#desinfect
@@ -88,12 +89,13 @@ TooManyMethods:
   - Mutant::CLI
   - Mutant::Mutator::Node
   - Mutant::Reporter::CLI
+  - Mutant::Meta::Example::Verification
   max_methods: 10
 TooManyStatements:
   enabled: true
   exclude:
   - Mutant#self.singleton_subclass_instance
-  - Mutant#self.isolate
+  - Mutant::Isolation#self.call
   - Mutant::Rspec::Killer#run
   - Mutant::Reporter::CLI#colorized_diff
   - Mutant::Reporter::CLI::Report::Config#run

data/lib/mutant.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 require 'stringio'
 require 'set'
 require 'adamantium'
@@ -26,6 +24,45 @@ module Mutant
   # The frozen empty array used within mutant
   EMPTY_ARRAY = [].freeze
 
+  symbolset = ->(strings) { strings.map(&:to_sym).to_set.freeze }
+
+  SCOPE_OPERATOR = '::'.freeze
+
+  # Set of nodes that cannot be on the LHS of an assignment
+  NOT_ASSIGNABLE         = symbolset.(%w[int float str dstr class module self nil])
+  # Set of op-assign types
+  OP_ASSIGN              = symbolset.call(%w[or_asgn and_asgn op_asgn])
+  # Set of node types that are not valid when emitted standalone
+  NOT_STANDALONE         = symbolset.(%w[splat restarg block_pass])
+  INDEX_OPERATORS        = symbolset.(%w[[] []=])
+  UNARY_METHOD_OPERATORS = symbolset.(%w[~@ +@ -@ !])
+
+  # Operators ruby implementeds as methods
+  METHOD_OPERATORS = symbolset.(%w[
+    <=> === []= [] <= >= == !~ != =~ <<
+    >> ** * % / | ^ & < > + - ~@ +@ -@ !
+  ])
+
+  BINARY_METHOD_OPERATORS = (
+    METHOD_OPERATORS - (INDEX_OPERATORS + UNARY_METHOD_OPERATORS)
+  ).to_set.freeze
+
+  OPERATOR_METHODS = (
+    METHOD_OPERATORS + INDEX_OPERATORS + UNARY_METHOD_OPERATORS
+  ).to_set.freeze
+
+  # Nodes that are NOT handled by mutant.
+  #
+  # not - 1.8 only, mutant does not support 1.8
+  #
+  NODE_BLACKLIST = symbolset.(%w[not])
+
+  # Nodes that are NOT generated by parser but used by mutant / unparser.
+  NODE_EXTRA = symbolset.(%w[empty])
+
+  # All node types mutant handles
+  NODE_TYPES = ((Parser::Meta::NODE_TYPES + NODE_EXTRA) - NODE_BLACKLIST).to_set.freeze
+
   # Lookup constant for location
   #
   # @param [String] location
@@ -35,7 +72,7 @@ module Mutant
   # @api private
   #
   def self.constant_lookup(location)
-    location.sub(CBASE_PATTERN, EMPTY_STRING).split(SCOPE_OPERATOR).reduce(Object) do |parent, name|
+    location.split(SCOPE_OPERATOR).reduce(Object) do |parent, name|
       parent.const_get(name, nil)
     end
   end
@@ -51,43 +88,6 @@ module Mutant
     self
   end
 
-  IsolationError = Class.new(RuntimeError)
-
-  # Call block in isolation
-  #
-  # This isolation implements the fork strategy.
-  # Future strategies will probably use a process pool that can
-  # handle multiple mutation kills, in-isolation at once.
-  #
-  # @return [Object]
-  #
-  # @api private
-  #
-  def self.isolate(&block)
-    reader, writer = IO.pipe.each(&:binmode)
-
-    pid = fork do
-      reader.close
-      writer.write(Marshal.dump(block.call))
-    end
-
-    writer.close
-
-    begin
-      data = Marshal.load(reader.read)
-    rescue ArgumentError
-      raise IsolationError, 'Childprocess wrote un-unmarshallable data'
-    end
-
-    status = Process.waitpid2(pid).last
-
-    unless status.exitstatus.zero?
-      raise IsolationError, "Childprocess exited with nonzero exit status: #{status.exitstatus}"
-    end
-
-    data
-  end
-
   # Define instance of subclassed superclass as constant
   #
   # @param [Class] superclass
@@ -120,9 +120,9 @@ require 'mutant/delegator'
 require 'mutant/node_helpers'
 require 'mutant/warning_filter'
 require 'mutant/warning_expectation'
-require 'mutant/constants'
 require 'mutant/walker'
 require 'mutant/require_highjack'
+require 'mutant/isolation'
 require 'mutant/mutator'
 require 'mutant/mutation'
 require 'mutant/mutation/evil'

data/lib/mutant/cache.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   # An AST cache
   class Cache

data/lib/mutant/cli.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 require 'optparse'
 
 module Mutant
@@ -120,9 +118,7 @@ module Mutant
       # @api private
       #
       def subject_selector
-        if @subject_selectors.any?
-          Morpher::Evaluator::Predicate::Boolean::Or.new(@subject_selectors)
-        end
+        Morpher::Evaluator::Predicate::Boolean::Or.new(@subject_selectors) if @subject_selectors.any?
       end
 
       # Return predicate
@@ -153,6 +149,10 @@ module Mutant
       # Return subject rejector
       #
       # @return [#call]
+      #   if there is a subject rejector
+      #
+      # @return [nil]
+      #   otherwise
       #
       # @api private
       #
@@ -161,9 +161,7 @@ module Mutant
           Morpher.compile(s(:eql, s(:attribute, :identification), s(:static, subject.identification)))
         end
 
-        if rejectors.any?
-          Morpher::Evaluator::Predicate::Boolean::Or.new(rejectors)
-        end
+        Morpher::Evaluator::Predicate::Boolean::Or.new(rejectors) if rejectors.any?
       end
     end
 

data/lib/mutant/color.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   # Class to colorize strings
   class Color

data/lib/mutant/config.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   # The configuration of a mutator run
   class Config

data/lib/mutant/context.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   # An abstract context where mutations can be appied to.
   class Context

data/lib/mutant/context/scope.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   class Context
     # Scope context for mutation (Class or Module)

data/lib/mutant/delegator.rb

@@ -1,5 +1,7 @@
 module Mutant
+  # A simple delegator with opinions
   module Delegator
+    # Class level mixins
     module ClassMethods
 
     private

data/lib/mutant/diff.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Mutant
   # Class to create diffs from source code
   class Diff
@@ -8,7 +6,7 @@ module Mutant
     # Return source diff
     #
     # @return [String]
-    #   if there is a diff
+    #   if there is exactly one diff
     #
     # @return [nil]
     #   otherwise
@@ -16,19 +14,9 @@ module Mutant
     # @api private
     #
     def diff
-      case diffs.length
-      when 0
-        nil
-      when 1
-        ::Diff::LCS::Hunk.new(old, new, diffs.first, max_length, 0)
-          .diff(:unified) << "\n"
-      else
-        $stderr.puts(
-          'Mutation resulted in more than one diff, should not happen! ' \
-          'PLS report a bug!'
-        )
-        nil
-      end
+      return unless diffs.length.equal?(1)
+      ::Diff::LCS::Hunk.new(old, new, diffs.first, max_length, 0)
+        .diff(:unified) << "\n"
     end
     memoize :diff
 

data/lib/mutant/expression.rb

@@ -13,16 +13,16 @@ module Mutant
       *OPERATOR_METHODS.map(&:to_s)
     ).freeze
 
-    SCOPE_PATTERN = /
-      (?:#{SCOPE_OPERATOR})?#{SCOPE_NAME_PATTERN}
-      (?:#{SCOPE_OPERATOR}#{SCOPE_NAME_PATTERN})*
-    /x.freeze
+    SCOPE_PATTERN = /#{SCOPE_NAME_PATTERN}(?:#{SCOPE_OPERATOR}#{SCOPE_NAME_PATTERN})*/.freeze
 
     REGISTRY = {}
 
     # Initialize expression
     #
     # @param [MatchData] match
+    #
+    # @api private
+    #
     def initialize(*)
       super
       @syntax = match.to_s
@@ -42,6 +42,8 @@ module Mutant
     #
     # @return [Fixnum]
     #
+    # @api private
+    #
     def match_length(neddle)
       if eql?(neddle)
         syntax.length
@@ -99,9 +101,8 @@ module Mutant
     def self.expressions(input)
       REGISTRY.each_with_object([]) do |(regexp, klass), expressions|
         match = regexp.match(input)
-        if match
-          expressions << klass.new(match)
-        end
+        next unless match
+        expressions << klass.new(match)
       end
     end
     private_class_method :expressions

data/lib/mutant/expression/method.rb

@@ -9,15 +9,9 @@ module Mutant
         '#' => Matcher::Methods::Instance
       }.freeze
 
-      regexp = /
-        \A
-          (?<scope_name>#{SCOPE_PATTERN})
-          (?<scope_symbol>[.#])
-          (?<method_name>#{METHOD_NAME_PATTERN})
-        \z
-      /x.freeze
-
-      register(regexp)
+      register(
+        /\A(?<scope_name>#{SCOPE_PATTERN})(?<scope_symbol>[.#])(?<method_name>#{METHOD_NAME_PATTERN})\z/
+      )
 
       # Return method matcher
       #