mutant 0.3.6 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f59906eaae74d48bd002bf30178571853d2af117
-  data.tar.gz: 08efab444be18fa267bdfd7fd7e0b644fc40e2ab
+  metadata.gz: e603e77505aa94ac731c26cf64c5b3f85453746a
+  data.tar.gz: 44f1fbd18eefabeb1770af343343fa5e0ab2a56c
 SHA512:
-  metadata.gz: fa62d1908fc639d8ba4cc713c44374db670d92f8aa5f9b6cf011efcce0d78c98f92fc9c799c0e397887aeb228e926240dfe446d9d6b24ecac3f886a6913b0971
-  data.tar.gz: 406669d0fa589a75ee96d530dddef31dcafd54b67c16ff5faaea4c6f476f107e392ca666bbbd965c039d64cbc4babe58527b874afdb1718ecf0056bb91884f7d
+  metadata.gz: 9a189701a6e9fee5573dfcd7ba638e9bf04219f38738a33a143d61cc6d58c26f0cf3369339b27716f71269b58f45a9f3328c14239431642b0c28d6997f2034fe
+  data.tar.gz: 71629135ffe50088a0ea524034df7af56ec43bb9a7cae9ab4ef900fa30877fe2aa5c403f86d62c7cee4607cce28b7ee16cebba5417ea0aab514073f19ea9504a

data/.travis.yml

@@ -8,6 +8,7 @@ rvm:
 matrix:
   allow_failures:
     - rvm: rbx
+    - rvm: 2.1.0
 notifications:
   irc:
     channels:

data/Changelog.md

@@ -1,3 +1,16 @@
+# v0.5.0 2014-02-16
+
+Changes:
+
+* Add configurable coverage expectation via --coverage (default 100%)
+* rspec integration was moved into a gem 'mutant-rspec'
+* Replace filters implementation with morpher predicates
+* Drop --rspec option use: --use rspec instead.
+
+# v0.4.0 2014-02-16
+
+Status: Yanked because of broken dependencies.
+
 # v0.3.4 2014-01-11
 
 Changes:

data/Gemfile

@@ -4,7 +4,7 @@ source 'https://rubygems.org'
 
 gem 'mutant', path: '.'
 
-gemspec
+gemspec name: 'mutant'
 
 gem 'devtools', git: 'https://github.com/rom-rb/devtools.git'
 eval_gemfile 'Gemfile.devtools'

data/Guardfile

@@ -2,33 +2,15 @@
 
 guard :bundler do
   watch('Gemfile')
-  watch('Gemfile.lock')
-  watch(%w{.+.gemspec\z})
 end
 
-guard :rspec, cli: File.read('.rspec').split.push('--fail-fast').join(' '), keep_failed: false do
-  # Run all specs if configuration is modified
-  watch('.rspec')              { 'spec' }
-  watch('Guardfile')           { 'spec' }
-  watch('Gemfile.lock')        { 'spec' }
-  watch('spec/spec_helper.rb') { 'spec' }
+guard :rspec, :all_after_pass => false, :all_on_start => false, :cmd => 'bundle exec rspec --fail-fast --seed 0' do
+  # run all specs if the spec_helper or supporting files files are modified
+  watch('spec/spec_helper.rb')                      { 'spec/unit' }
+  watch(%r{\Aspec/(?:lib|support|shared)/.+\.rb\z}) { 'spec/unit' }
 
-  # Run all specs if supporting files files are modified
-  watch(%r{\Aspec/(?:fixtures|lib|support|shared)/.+\.rb\z}) { 'spec' }
+  watch(%r{lib/.*.rb})                              { 'spec/unit' }
 
-  # Run unit specs if associated lib code is modified
-  watch(%r{\Alib/(.+)\.rb\z})                                         { |m| Dir["spec/unit/#{m[1]}*"]         }
-  watch(%r{\Alib/(.+)/support/(.+)\.rb\z})                            { |m| Dir["spec/unit/#{m[1]}/#{m[2]}*"] }
-  watch("lib/#{File.basename(File.expand_path('../', __FILE__))}.rb") { 'spec'                                }
-
-  # Run a spec if it is modified
-  watch(%r{\Aspec/(?:unit|integration)/.+_spec\.rb\z})
+  # run a spec if it is modified
+  watch(%r{\Aspec/.+_spec\.rb\z})
 end
-
-# Deactivated for now. Somehow it disables the rspec guard.
-#
-# guard :rubocop, cli: %w[--config config/rubocop.yml] do
-#   watch(%r{.+\.(?:rb|rake)\z})
-#   watch(%r{\Aconfig/rubocop\.yml\z})  { |m| File.dirname(m[0]) }
-#   watch(%r{(?:.+/)?\.rubocop\.yml\z}) { |m| File.dirname(m[0]) }
-# end

data/LICENSE

@@ -1,4 +1,4 @@
-Copyright (c) 2012 Markus Schirp
+Copyright (c) 2012-2014 Markus Schirp
 
 Permission is hereby granted, free of charge, to any person obtaining
 a copy of this software and associated documentation files (the

data/README.md

@@ -12,12 +12,14 @@ or it does not have a speced side effect.
 
 A more readable introduction can be found under: http://solnic.eu/2013/01/23/mutation-testing-with-mutant.html
 
-Mutant supports MRI and RBX 1.9 and 2.0, while support for jruby is planned. It should also work under
-any ruby engine that supports POSIX-fork(2) semantics.
+Mutant supports MRI and RBX 1.9 and 2.0, while support for jruby is planned.
+It should also work under any ruby engine that supports POSIX-fork(2) semantics.
+Support for MRI 2.1 is unstable, because this MRI release segfaults on basic metaprogramming mutants dependencies do.
 
-Only rspec2 is supported currently. This is subject to change.
+Integrations
+------------
 
-It is easy to write a mutation killer/strategy for other test/spec frameworks than rspec2.
+Only rspec2 is supported currently.
 
 Projects using Mutant
 ---------------------
@@ -44,10 +46,14 @@ Installation
 
 Install the gem `mutant` via your preferred method.
 
-The 0.2 series is stable but has outdated dependencies. The 0.3 series is in beta phase currently.
+```ruby
+gem install mutant
+```
+
+If you plan to use the rspec integration you'll have to install `mutant-rspec` also.
 
 ```ruby
-gem install mutant --pre
+gem install mutant-rspec
 ```
 
 Mutations
@@ -86,13 +92,13 @@ Examples
 ```
 cd virtus
 # Run mutant on virtus namespace
-mutant --include lib --require virtus --rspec ::Virtus*
+mutant --include lib --require virtus --use rspec ::Virtus*
 # Run mutant on specific virtus class
-mutant --include lib --require virtus --rspec ::Virtus::Attribute
+mutant --include lib --require virtus --use rspec ::Virtus::Attribute
 # Run mutant on specific virtus class method
-mutant --include lib --require virtus --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 --rspec ::Virtus::Attribute#type
+mutant --include lib --require virtus --use rspec ::Virtus::Attribute#type
 ```
 
 Presentations:

data/config/flay.yml

@@ -1,3 +1,3 @@
 ---
 threshold: 18
-total_score: 790
+total_score: 811

data/config/reek.yml

@@ -24,14 +24,15 @@ DuplicateMethodCall:
 FeatureEnvy:
   enabled: true
   exclude:
-  - Mutant::CLI#parse
-  - Mutant::CLI#add_strategies
   - Mutant::Matcher::Method::Instance#match?
   - Mutant::Matcher::Method::Singleton#receiver?
   - Mutant::Mutation::Evil#success?
   - Mutant::Mutation::Neutral#success?
   - Mutant::Reporter::CLI#subject_results
-  - Mutant::Strategy::Rspec#options
+  - Mutant::Rspec::Strategy#options
+  # Nature of OptionParser :(
+  - Mutant::CLI#add_environmental_options
+  - Mutant::CLI#parse
 IrresponsibleModule:
   enabled: true
   exclude: []
@@ -80,14 +81,14 @@ TooManyStatements:
   enabled: true
   exclude:
   - Mutant#self.singleton_subclass_instance
-  - Mutant::CLI#parse
-  - Mutant::CLI#add_options
-  - Mutant::CLI#add_strategies
-  - Mutant::Killer::Rspec#run
+  - Mutant::Rspec::Killer#run
   - Mutant::Reporter::CLI#colorized_diff
   - Mutant::Reporter::CLI::Printer::Config::Runner#run
   - Mutant::Runner#dispatch
   - Mutant::Zombifier::File#self.find_uncached
+  # How mutant does CLI parsing is shit
+  - Mutant::CLI#parse
+  - Mutant::CLI#initialize
   max_statements: 6
 UncommunicativeMethodName:
   enabled: true
@@ -132,8 +133,7 @@ UtilityFunction:
   - Mutant::Mutation::Evil#success?
   - Mutant::Mutation::Neutral#success?
   - Mutant::NodeHelpers#s
-  - Mutant::Strategy::Rspec#configuration
-  - Mutant::Strategy::Rspec#options
-  - Mutant::Strategy::Rspec#world
-
+  - Mutant::Rspec::Strategy#configuration
+  - Mutant::Rspec::Strategy#options
+  - Mutant::Rspec::Strategy#world
   max_helper_calls: 0

data/config/rubocop.yml

@@ -89,7 +89,7 @@ MultilineBlockChain:
   Enabled: false
 
 ClassLength:
-  Max: 105
+  Max: 119
 
 # I align private keywords with class body
 IndentationWidth:

data/lib/mutant.rb

@@ -17,10 +17,9 @@ require 'unparser'
 require 'ice_nine'
 require 'diff/lcs'
 require 'diff/lcs/hunk'
-require 'rspec'
 require 'anima'
 require 'concord'
-require 'rspec'
+require 'morpher'
 
 # Library namespace
 module Mutant
@@ -34,11 +33,7 @@ require 'mutant/node_helpers'
 require 'mutant/singleton_methods'
 require 'mutant/constants'
 require 'mutant/random'
-require 'mutant/predicate'
-require 'mutant/predicate/attribute'
-require 'mutant/predicate/whitelist'
-require 'mutant/predicate/blacklist'
-require 'mutant/predicate/matcher'
+require 'mutant/walker'
 require 'mutant/mutator'
 require 'mutant/mutation'
 require 'mutant/mutation/evil'
@@ -111,13 +106,12 @@ require 'mutant/matcher/methods'
 require 'mutant/matcher/namespace'
 require 'mutant/matcher/scope'
 require 'mutant/matcher/filter'
+require 'mutant/matcher/null'
 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/rspec'
 require 'mutant/runner'
 require 'mutant/runner/config'
 require 'mutant/runner/subject'
@@ -126,7 +120,6 @@ require 'mutant/cli'
 require 'mutant/cli/classifier'
 require 'mutant/cli/classifier/namespace'
 require 'mutant/cli/classifier/method'
-require 'mutant/cli/builder'
 require 'mutant/color'
 require 'mutant/differ'
 require 'mutant/reporter'

data/lib/mutant/cli.rb

@@ -6,7 +6,7 @@ module Mutant
 
   # Comandline parser
   class CLI
-    include Adamantium::Flat, Equalizer.new(:config)
+    include Adamantium::Flat, Equalizer.new(:config), NodeHelpers
 
     # Error raised when CLI argv is invalid
     Error = Class.new(RuntimeError)
@@ -32,6 +32,132 @@ module Mutant
       EXIT_FAILURE
     end
 
+    # Builder for configuration components
+    class Builder
+      include NodeHelpers
+
+      # Initalize object
+      #
+      # @return [undefined]
+      #
+      # @api private
+      #
+      def initialize
+        @matchers          = []
+        @subject_ignores   = []
+        @subject_selectors = []
+      end
+
+      # Add a subject ignore
+      #
+      # @param [Matcher]
+      #
+      # @return [self]
+      #
+      # @api private
+      #
+      def add_subject_ignore(matcher)
+        @subject_ignores << matcher
+        self
+      end
+
+      # Add a subject selector
+      #
+      # @param [#call] selector
+      #
+      # @return [self]
+      def add_subject_selector(selector)
+        @subject_selectors << selector
+        self
+      end
+
+      # Add a subject matcher
+      #
+      # @param [#call] selector
+      #
+      # @return [self]
+      #
+      # @api private
+      #
+      def add_matcher(matcher)
+        @matchers << matcher
+        self
+      end
+
+      def matcher
+        if @matchers.empty?
+          raise(Error, 'No patterns given')
+        end
+
+        matcher = Matcher::Chain.build(@matchers)
+
+        if predicate
+          Matcher::Filter.new(matcher, predicate)
+        else
+          matcher
+        end
+      end
+
+    private
+
+      # Return subject selector
+      #
+      # @return [#call]
+      #   if selector is present
+      #
+      # @return [nil]
+      #   otherwise
+      #
+      # @api private
+      #
+      def subject_selector
+        if @subject_selectors.any?
+          Morpher::Evaluator::Predicate::Or.new(@subject_selectors)
+        end
+      end
+
+      # Return predicate
+      #
+      # @return [#call]
+      #   if filter is needed
+      #
+      # @return [nil]
+      #   othrwise
+      #
+      # @api private
+      #
+      def predicate
+        if subject_selector && subject_rejector
+          Morpher::Evaluator::Predicate::And.new([
+            subject_selector,
+            Morpher::Evaluator::Predicate::Negation.new(subject_rejector)
+          ])
+        elsif subject_selector
+          subject_selector
+        elsif subject_rejector
+          Morpher::Evaluator::Predicate::Negation.new(subject_rejector)
+        else
+          nil
+        end
+      end
+
+      # Return subject rejector
+      #
+      # @return [#call]
+      #
+      # @api private
+      #
+      def subject_rejector
+        rejectors = @subject_ignores.flat_map(&:to_a).map do |subject|
+          Morpher.evaluator(s(:eql, s(:attribute, :identification), s(:static, subject.identification)))
+        end
+
+        if rejectors.any?
+          Morpher::Evaluator::Predicate::Or.new(rejectors)
+        end
+      end
+    end
+
     # Initialize objecct
     #
     # @param [Array<String>]
@@ -41,10 +167,11 @@ module Mutant
     # @api private
     #
     def initialize(arguments = [])
-      @filters, @matchers = [], []
+      @builder = Builder.new
       @debug = @fail_fast = @zombie = false
+      @expect_coverage = 100.0
+      @strategy = Strategy::Null.new
       @cache = Mutant::Cache.new
-      @strategy_builder = nil
       parse(arguments)
       config # trigger lazyness now
     end
@@ -60,56 +187,17 @@ module Mutant
         cache:             @cache,
         zombie:            @zombie,
         debug:             @debug,
-        matcher:           matcher,
-        subject_predicate: @subject_predicate.output,
-        strategy:          @strategy.output,
+        matcher:           @builder.matcher,
+        strategy:          @strategy,
         fail_fast:         @fail_fast,
-        reporter:          reporter
+        reporter:          Reporter::CLI.new($stdout),
+        expected_coverage: @expect_coverage
       )
     end
     memoize :config
 
   private
 
-    # Return reporter
-    #
-    # @return [Mutant::Reporter::CLI]
-    #
-    # @api private
-    #
-    def reporter
-      Reporter::CLI.new($stdout)
-    end
-
-    # Return matcher
-    #
-    # @return [Mutant::Matcher]
-    #
-    # @raise [CLI::Error]
-    #   raises error when matcher is not given
-    #
-    # @api private
-    #
-    def matcher
-      if @matchers.empty?
-        raise(Error, 'No matchers given')
-      end
-
-      Matcher::Chain.build(@matchers)
-    end
-
-    # Add mutation filter
-    #
-    # @param [Class<Predicate>] klass
-    #
-    # @return [undefined]
-    #
-    # @api private
-    #
-    def add_filter(klass, *arguments)
-      @filters << klass.new(*arguments)
-    end
-
     # Parse the command-line options
     #
     # @param [Array<String>] arguments
@@ -124,11 +212,12 @@ module Mutant
     #
     def parse(arguments)
       opts = OptionParser.new do |builder|
-        builder.banner = 'usage: mutant STRATEGY [options] MATCHERS ...'
+        builder.banner = 'usage: mutant STRATEGY [options] PATTERN ...'
         builder.separator('')
-        add_strategies(builder)
         add_environmental_options(builder)
-        add_options(builder)
+        add_mutation_options(builder)
+        add_filter_options(builder)
+        add_debug_options(builder)
       end
 
       patterns =
@@ -152,28 +241,7 @@ module Mutant
     def parse_matchers(patterns)
       patterns.each do |pattern|
         matcher = Classifier.run(@cache, pattern)
-        @matchers << matcher if matcher
-      end
-    end
-
-    # Add strategies
-    #
-    # @param [OptionParser] parser
-    #
-    # @return [undefined]
-    #
-    # @api private
-    #
-    def add_strategies(parser)
-      parser.separator(EMPTY_STRING)
-      parser.separator('Strategies:')
-
-      {
-        Builder::Rspec              => :@strategy,
-        Builder::Predicate::Subject => :@subject_predicate,
-      }.each do |builder, instance_variable_name|
-        builder = builder.new(@cache, parser)
-        instance_variable_set(instance_variable_name, builder)
+        @builder.add_matcher(matcher)
       end
     end
 
@@ -186,6 +254,8 @@ module Mutant
     # @api private
     #
     def add_environmental_options(opts)
+      opts.separator('')
+      opts.separator('Environment:')
       opts.on('--zombie', 'Run mutant zombified') do
         @zombie = true
       end.on('-I', '--include DIRECTORY', 'Add DIRECTORY to $LOAD_PATH') do |directory|
@@ -195,25 +265,74 @@ module Mutant
       end
     end
 
+    # Use plugin
+    #
+    # FIXME: For now all plugins are strategies. Later they could be anything that allows "late integration".
+    #
+    # @param [String] name
+    #
+    # @return [undefined]
+    #
+    # @api private
+    #
+    def use(name)
+      require "mutant-#{name}"
+      @strategy = Strategy.lookup(name).new
+    rescue LoadError
+      $stderr.puts("Cannot load plugin: #{name.inspect}")
+      raise
+    end
+
     # Add options
     #
-    # @param [Object] opts
+    # @param [OptionParser] opts
     #
     # @return [undefined]
     #
     # @api private
     #
-    def add_options(opts)
-      opts.separator ''
-      opts.separator 'Options:'
+    def add_mutation_options(opts)
+      opts.separator(EMPTY_STRING)
+      opts.separator('Options:')
+
+      opts.on('--score COVERAGE', 'Fail unless COVERAGE is not reached exactly') do |coverage|
+        @expected_coverage = Float(coverage)
+      end.on('--use STRATEGY', 'Use STRATEGY for killing mutations') do |runner|
+        use(runner)
+      end
+    end
 
-      opts.on('--version', 'Print mutants version') do |name|
+    # Add filter options
+    #
+    # @param [OptionParser] opts
+    #
+    # @return [undefined]
+    #
+    # @api private
+    #
+    def add_filter_options(opts)
+      opts.on('--ignore-subject PATTERN', 'Ignore subjects that match PATTERN') do |pattern|
+        @builder.add_subject_ignore(Classifier.run(@cache, pattern))
+      end
+      opts.on('--code CODE', 'Scope execution to subjects with CODE') do |code|
+        @builder.add_subject_selector(Morpher.evaluator(s(:eql, s(:attribute, :code), s(:static, code))))
+      end
+    end
+
+    # Add debug options
+    #
+    # @param [OptionParser] opts
+    #
+    # @return [undefined]
+    #
+    # @api private
+    #
+    def add_debug_options(opts)
+      opts.on('--fail-fast', 'Fail fast') do
+        @fail_fast = true
+      end.on('--version', 'Print mutants version') do |name|
         puts("mutant-#{Mutant::VERSION}")
         Kernel.exit(0)
-      end.on('--code FILTER', 'Adds a code filter') do |filter|
-        add_filter(Predicate::Attribute, :code, filter)
-      end.on('--fail-fast', 'Fail fast') do
-        @fail_fast = true
       end.on('-d', '--debug', 'Enable debugging output') do
         @debug = true
       end.on_tail('-h', '--help', 'Show this message') do