mutant 0.7.9 → 0.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a01c96b2fcfbb344d37ed40dc9773c36d0433bfc
-  data.tar.gz: f4ef32cec5591667c75a1e1e31200d51529a7b50
+  metadata.gz: fa063e8c2f6e2cf17858c47e1725f315e64c4c0b
+  data.tar.gz: b96166a5788f2de7088b36033ed7dc0198295b8f
 SHA512:
-  metadata.gz: 6aa3c28aeffad6471ced7ab755d34cab8b0c6d986c4d6739245b9706e3608fd18fbfed75287f94ac5d337e5078f4e711e94aee748ff9508b439689ad24417a2b
-  data.tar.gz: 36c0f9f971e94f602299f785a7d28a1f56b7e566a6ec3717f46905c7f7e306b574d028a7cd10e32f35b6455b742c0f5538b327fa4360e91c392a8dd76f6cc732
+  metadata.gz: a4f4038557d14c1da74122005c6dce8e510d3808cc01efec45f3bbcc6e53ece5430b2b1ce0b65d45f4e7234143881e14b21581a66a4f80c89c093b64c4c16dc9
+  data.tar.gz: f31d6f38605ba70960779c66ff5fa8767c508a9e87b2ed49c0ff3033635ad210a2a79040458bca86188e733d0e6cde77902e45f6f6b995e0ea6239e7f66ff5c3

data/Changelog.md

@@ -1,7 +1,17 @@
-# v0.7.9 pending
+# v0.8.0 2015-06-15
+
+* Drop support for ruby < 2.1
+* Remove broken `--code` option
+* Remove deprecated `--score` option
+* Add support for rspec-3.3
+* End support for rspec-3.{0,1}
+* Internal quality improvements
+
+# v0.7.9 2015-05-30
 
 * Deprecate `--score` flag replace with `--expected-coverage`
 * Set default job count to 4 under CI environments.
+* Relax parser dependency to ~>2.2.2
 
 # v0.7.8 2015-03-8
 

data/README.md

@@ -1,7 +1,7 @@
 mutant
 ======
 
-[![Build Status](https://secure.travis-ci.org/mbj/mutant.png?branch=master)](http://travis-ci.org/mbj/mutant)
+[![Build Status](https://circleci.com/gh/mbj/mutant.svg?style=shield&circle-token=1afd77e8f0f9d0a11fd8f15f5d7b10270f4665e2)](https://circleci.com/gh/mbj/mutant/tree/master)
 [![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)
@@ -13,7 +13,7 @@ Mutant is a mutation testing tool for Ruby.
 The idea is that if code can be changed and your tests do not notice, either that code isn't being covered
 or it does not have a speced side effect.
 
-Mutant supports MRI and RBX 2.0 and 2.1, while support for JRuby is planned.
+Mutant supports ruby >= 2.1, while support for JRuby is planned.
 It should also work under any Ruby engine that supports POSIX-fork(2) semantics.
 
 Mutant uses a pure Ruby [parser](https://github.com/whitequark/parser) and an [unparser](https://github.com/mbj/unparser)
@@ -48,20 +48,14 @@ Blog-Posts
 Installation
 ------------
 
-Install the gem `mutant` via your preferred method.
-
-```ruby
-gem install mutant
-```
-
-If you plan to use the RSpec integration you'll have to install `mutant-rspec` also.
-Please add an explicit dependency to `rspec-core` for the RSpec version you want to use.
+As mutant right now only supports rspec, install the gem `mutant-rspec` via your preferred method.
+It'll pull the `mutant` gem (in correct version), that contains the main engine.
 
 ```ruby
 gem install mutant-rspec
 ```
 
-The minitest integration is still in the works.
+The minitest integration is still in the [works](https://github.com/mbj/mutant/pull/330).
 
 The Crash / Stuck Problem (MRI)
 -------------------------------
@@ -131,7 +125,6 @@ Assuming you are using rspec, you can mutation test Rails models by adding the f
 
 ```ruby
 group :test do
-  gem 'mutant'
   gem 'mutant-rspec'
 end
 ```

data/bin/mutant

@@ -10,7 +10,23 @@ require 'mutant'
 namespace =
   if ARGV.include?('--zombie')
     $stderr.puts('Running mutant zombified!')
-    Mutant.zombify
+    Mutant::Zombifier.call(
+      namespace:        :Zombie,
+      load_path:        $LOAD_PATH,
+      kernel:           Kernel,
+      pathname:         Pathname,
+      require_highjack: Mutant::RequireHighjack.method(:call).to_proc.curry.call(Kernel),
+      root_require:     'mutant',
+      includes: %w[
+        mutant
+        unparser
+        morpher
+        adamantium
+        equalizer
+        anima
+        concord
+      ]
+    )
     Zombie::Mutant
   else
     Mutant

data/config/flay.yml

@@ -1,3 +1,3 @@
 ---
 threshold: 18
-total_score: 1241
+total_score: 1227

data/config/flog.yml

@@ -1,2 +1,2 @@
 ---
-threshold: 30.9
+threshold: 29.3

data/config/reek.yml

@@ -24,13 +24,14 @@ DuplicateMethodCall:
   allow_calls: []
 FeatureEnvy:
   enabled: false
+# Buggy smell detector
 IrresponsibleModule:
-  enabled: true
+  enabled: false
   exclude: []
 LongParameterList:
   enabled: true
   exclude:
-  - Mutant::Matcher::Method::Instance#self.build?
+  - Mutant::Matcher::Method::Instance#self.build
   max_params: 2
 LongYieldList:
   enabled: true
@@ -45,8 +46,7 @@ NestedIterators:
   - Mutant::Mutator::Util::Array::Element#dispatch
   - Mutant::Mutator::Node::Resbody#mutate_captures
   - Mutant::Mutator::Node::Arguments#emit_argument_mutations
-  - Mutant::RequireHighjack#infect
-  - Mutant::RequireHighjack#disinfect
+  - Mutant::RequireHighjack#self.call
   - Mutant::Selector::Expression#call
   - Mutant::Parallel::Master#run
   - Parser::Lexer#self.new

data/config/rubocop.yml

@@ -87,6 +87,10 @@ PercentLiteralDelimiters:
     '%W': '[]'
     '%x': ()
 
+# Use %i[...] for arrays of symbols
+SymbolArray:
+  Enabled: true
+
 # Align if/else blocks with the variable assignment
 EndAlignment:
   AlignWith: variable
@@ -106,11 +110,25 @@ UnneededPercentQ:
 
 # Allow a maximum ABC score
 Metrics/AbcSize:
-  Max: 18.00
+  Max: 21.02
 
 # Do not prefer lambda.call(...) over lambda.(...)
 LambdaCall:
   Enabled: false
 
-Style/RegexpLiteral:
-  MaxSlashes: 0
+# Buggy cop, returns false positive for our code base
+NonLocalExitFromIterator:
+  Enabled: false
+
+# To allow alignment of similar expressions we want to allow more than one
+# space around operators:
+#
+# let(:a) { bar    + something }
+# let(:b) { foobar + something }
+#
+SpaceAroundOperators:
+  Enabled: false
+
+# We use parallel assignments with great success
+ParallelAssignment:
+  Enabled: false

data/lib/mutant.rb

@@ -24,11 +24,8 @@ Thread.abort_on_exception = true
 
 # Library namespace
 module Mutant
-  # The frozen empty string used within mutant
-  EMPTY_STRING = ''.freeze
-  # The frozen empty array used within mutant
-  EMPTY_ARRAY = [].freeze
-
+  EMPTY_STRING   = ''.freeze
+  EMPTY_ARRAY    = [].freeze
   SCOPE_OPERATOR = '::'.freeze
 
   # Test if CI is detected via environment
@@ -40,58 +37,6 @@ module Mutant
   def self.ci?
     ENV.key?('CI')
   end
-
-  # Lookup constant for location
-  #
-  # @param [String] location
-  #
-  # @return [Object]
-  #
-  # @api private
-  #
-  def self.constant_lookup(location)
-    location.split(SCOPE_OPERATOR).reduce(Object) do |parent, name|
-      parent.const_get(name, nil)
-    end
-  end
-
-  # Perform self zombification
-  #
-  # @return [self]
-  #
-  # @api private
-  #
-  def self.zombify
-    Zombifier.run('mutant', :Zombie)
-    self
-  end
-
-  # Define instance of subclassed superclass as constant
-  #
-  # @param [Class] superclass
-  # @param [Symbol] name
-  #
-  # @return [self]
-  #
-  # @api private
-  #
-  # rubocop:disable MethodLength
-  #
-  def self.singleton_subclass_instance(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 # Mutant
 
 require 'mutant/version'
@@ -109,15 +54,15 @@ require 'mutant/actor/receiver'
 require 'mutant/actor/sender'
 require 'mutant/actor/mailbox'
 require 'mutant/actor/env'
+require 'mutant/cache'
+require 'mutant/delegator'
+require 'mutant/isolation'
 require 'mutant/parallel'
 require 'mutant/parallel/master'
 require 'mutant/parallel/worker'
 require 'mutant/parallel/source'
-require 'mutant/cache'
-require 'mutant/delegator'
 require 'mutant/warning_filter'
 require 'mutant/require_highjack'
-require 'mutant/isolation'
 require 'mutant/mutator'
 require 'mutant/mutation'
 require 'mutant/mutator/registry'
@@ -217,10 +162,19 @@ require 'mutant/reporter/null'
 require 'mutant/reporter/trace'
 require 'mutant/reporter/cli'
 require 'mutant/reporter/cli/printer'
+require 'mutant/reporter/cli/printer/config'
+require 'mutant/reporter/cli/printer/env_result'
+require 'mutant/reporter/cli/printer/env_progress'
+require 'mutant/reporter/cli/printer/mutation_result'
+require 'mutant/reporter/cli/printer/mutation_progress_result'
+require 'mutant/reporter/cli/printer/subject_progress'
+require 'mutant/reporter/cli/printer/subject_result'
+require 'mutant/reporter/cli/printer/status'
+require 'mutant/reporter/cli/printer/status_progressive'
+require 'mutant/reporter/cli/printer/test_result'
 require 'mutant/reporter/cli/tput'
 require 'mutant/reporter/cli/format'
 require 'mutant/zombifier'
-require 'mutant/zombifier/file'
 
 module Mutant
   # Reopen class to initialize constant to avoid dep circle

data/lib/mutant/cli.rb

@@ -149,10 +149,7 @@ module Mutant
       ) do |coverage|
         update(expected_coverage: Rational(coverage))
       end
-      opts.on('--score COVERAGE', 'Fail unless COVERAGE is not reached exactly [deprecated]') do |coverage|
-        update(expected_coverage: Rational(coverage, 100))
-      end
-      opts.on('--use STRATEGY', 'Use STRATEGY for killing mutations', &method(:setup_integration))
+      opts.on('--use INTEGRATION', 'Use INTEGRATION to kill mutations', &method(:setup_integration))
     end
 
     # Add filter options
@@ -167,9 +164,6 @@ module Mutant
       opts.on('--ignore-subject PATTERN', 'Ignore subjects that match PATTERN') do |pattern|
         add_matcher(:subject_ignores, Expression.parse(pattern))
       end
-      opts.on('--code CODE', 'Scope execution to subjects with CODE') do |code|
-        add_matcher(:subject_selects, [:code, code])
-      end
     end
 
     # Add debug options

data/lib/mutant/color.rb

@@ -15,7 +15,7 @@ module Mutant
       "\e[#{@code}m#{text}\e[0m"
     end
 
-    Mutant.singleton_subclass_instance('NONE', self) do
+    NONE = Class.new(self) do
 
       # Format null color
       #
@@ -41,7 +41,7 @@ module Mutant
       def initialize
       end
 
-    end
+    end.new
 
     RED   = Color.new(31)
     GREEN = Color.new(32)

data/lib/mutant/config.rb

@@ -15,7 +15,7 @@ module Mutant
       :expected_coverage
     )
 
-    [:fail_fast, :zombie, :debug].each do |name|
+    %i[fail_fast zombie debug].each do |name|
       define_method(:"#{name}?") { public_send(name) }
     end
 

data/lib/mutant/expression/method.rb

@@ -48,7 +48,7 @@ module Mutant
       # @api private
       #
       def scope
-        Mutant.constant_lookup(scope_name)
+        Object.const_get(scope_name)
       end
 
       # Return method name

data/lib/mutant/expression/methods.rb

@@ -60,7 +60,7 @@ module Mutant
       # @api private
       #
       def scope
-        Mutant.constant_lookup(scope_name)
+        Object.const_get(scope_name)
       end
 
       # Return scope symbol

data/lib/mutant/expression/namespace.rb

@@ -81,7 +81,7 @@ module Mutant
         # @api private
         #
         def matcher(env)
-          Matcher::Scope.new(env, Mutant.constant_lookup(namespace), self)
+          Matcher::Scope.new(env, Object.const_get(namespace), self)
         end
 
       end # Exact

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

@@ -11,24 +11,24 @@ module Mutant
         children :receiver, :selector
 
         SELECTOR_REPLACEMENTS = IceNine.deep_freeze(
-          reverse_map:   [:map, :each],
-          kind_of?:      [:instance_of?],
-          is_a?:         [:instance_of?],
-          reverse_each:  [:each],
-          reverse_merge: [:merge],
-          map:           [:each],
-          send:          [:public_send, :__send__],
-          __send__:      [:public_send],
-          gsub:          [:sub],
-          eql?:          [:equal?],
-          to_s:          [:to_str],
-          to_i:          [:to_int],
-          to_a:          [:to_ary],
-          :== =>         [:eql?, :equal?],
-          :>= =>         [:>, :==, :eql?, :equal?],
-          :<= =>         [:<, :==, :eql?, :equal?],
-          :> =>          [:==, :>=, :eql?, :equal?],
-          :< =>          [:==, :<=, :eql?, :equal?]
+          reverse_map:   %i[map each],
+          kind_of?:      %i[instance_of?],
+          is_a?:         %i[instance_of?],
+          reverse_each:  %i[each],
+          reverse_merge: %i[merge],
+          map:           %i[each],
+          send:          %i[public_send __send__],
+          __send__:      %i[public_send],
+          gsub:          %i[sub],
+          eql?:          %i[equal?],
+          to_s:          %i[to_str],
+          to_i:          %i[to_int],
+          to_a:          %i[to_ary],
+          :== =>         %i[eql? equal?],
+          :>= =>         %i[> == eql? equal?],
+          :<= =>         %i[< == eql? equal?],
+          :> =>          %i[== >= eql? equal?],
+          :< =>          %i[== <= eql? equal?]
         )
 
       private

data/lib/mutant/reporter/cli.rb

@@ -53,11 +53,6 @@ module Mutant
       #
       # TODO: Move this to a callback registration
       #
-      #   Reporters other than CLI that might exist in future
-      #   may only need the final report. So providing a noop callback
-      #   registration makes more sense for these.
-      #   As only CLI reporters exist currently I do not really care right now.
-      #
       # @return [Float]
       #
       # @api private
@@ -88,7 +83,7 @@ module Mutant
       # @api private
       #
       def report(env)
-        Printer::EnvResult.run(output, env)
+        Printer::EnvResult.call(output, env)
         self
       end