mutant 0.3.6 → 0.5.0

data/lib/mutant/config.rb

@@ -8,10 +8,10 @@ module Mutant
       :debug,
       :strategy,
       :matcher,
-      :subject_predicate,
       :reporter,
       :fail_fast,
-      :zombie
+      :zombie,
+      :expected_coverage
     )
 
     # Enumerate subjects
@@ -28,7 +28,7 @@ module Mutant
     #
     def subjects(&block)
       return to_enum(__method__) unless block_given?
-      Matcher::Filter.new(matcher, subject_predicate).each(&block)
+      matcher.each(&block)
       self
     end
 

data/lib/mutant/killer.rb

@@ -99,5 +99,25 @@ module Mutant
     #
     abstract_method :run
 
+    # Null killer that never kills a mutation
+    class Null < self
+
+    private
+
+      # Run killer
+      #
+      # @return [true]
+      #   when mutant was killed
+      #
+      # @return [false]
+      #   otherwise
+      #
+      # @api private
+      #
+      def run
+        false
+      end
+
+    end
   end # Killer
 end # Mutant

data/lib/mutant/matcher/filter.rb

@@ -4,7 +4,7 @@ module Mutant
   class Matcher
     # Matcher filter
     class Filter < self
-      include Concord.new(:matcher, :filter)
+      include Concord.new(:matcher, :predicate)
 
       # Enumerate matches
       #
@@ -16,14 +16,9 @@ module Mutant
       #
       # @api private
       #
-      def each
+      def each(&block)
         return to_enum unless block_given?
-
-        matcher.each do |subject|
-          next if filter.match?(subject)
-          yield subject
-        end
-
+        matcher.select(&predicate.method(:call)).each(&block)
         self
       end
 

data/lib/mutant/matcher/method/instance.rb

@@ -73,7 +73,7 @@ module Mutant
           # @api private
           #
           def source_location
-            scope.original_instance_method(method.name).source_location
+            scope.unmemoized_instance_method(method.name).source_location
           end
 
         end # Memoized

data/lib/mutant/matcher/namespace.rb

@@ -54,6 +54,29 @@ module Mutant
         end
       end
 
+      # Return scope name
+      #
+      # @param [Class,Module] scope
+      #
+      # @return [String]
+      #   if scope has a name and does not raise exceptions optaining it
+      #
+      # @return [nil]
+      #   otherwise
+      #
+      # @api private
+      #
+      def self.scope_name(scope)
+        scope.name
+      rescue => exception
+        $stderr.puts <<-MESSAGE
+WARNING:
+While optaining #{scope.class}#name from: #{scope.inspect}
+It raised an error: #{exception.inspect} fix your lib!
+        MESSAGE
+        nil
+      end
+
       # Yield scope if name matches pattern
       #
       # @param [Module,Class] scope
@@ -63,9 +86,15 @@ module Mutant
       # @api private
       #
       def emit_scope(scope)
-        name = scope.name
+        name = self.class.scope_name(scope)
         # FIXME: Fix nokogiri to return a string here
-        return unless name.kind_of?(String)
+        unless name.nil? or name.kind_of?(String)
+          $stderr.puts <<-MESSAGE
+WARNING:
+#{scope.class}#name did not return a string or nil.
+Fix your lib!
+          MESSAGE
+        end
         if pattern =~ name
           yield scope
         end

data/lib/mutant/matcher/null.rb

@@ -0,0 +1,26 @@
+# encoding: utf-8
+
+module Mutant
+  class Matcher
+    # A null matcher, that does not match any subjects
+    class Null < self
+      include Equalizer.new
+
+      # Enumerate subjects
+      #
+      # @return [Enumerator<Subject]
+      #   if no block given
+      #
+      # @return [self]
+      #   otherwise
+      #
+      # @api private
+      #
+      def each(&block)
+        return to_enum unless block_given?
+        self
+      end
+
+    end # Null
+  end # Matcher
+end # Mutant

data/lib/mutant/mutation.rb

@@ -56,7 +56,6 @@ module Mutant
     def identification
       "#{subject.identification}:#{code}"
     end
-    memoize :identification
 
     # Return mutation code
     #

data/lib/mutant/reporter/cli/printer.rb

@@ -10,6 +10,35 @@ module Mutant
 
         REGISTRY = {}
 
+        # Create delegators to object
+        #
+        # @return [undefined]
+        #
+        # @api private
+        #
+        def self.delegate(*names)
+          names.each do |name|
+            define_delegator(name)
+          end
+        end
+        private_class_method :delegate
+
+        # Create delegator to object
+        #
+        # @param [Symbol] name
+        #
+        # @return [undefined]
+        #
+        # @api private
+        #
+        def self.define_delegator(name)
+          define_method(name) do
+            object.public_send(name)
+          end
+          private name
+        end
+        private_class_method :define_delegator
+
         # Registre handler for class
         #
         # @param [Class] klass

data/lib/mutant/reporter/cli/printer/config.rb

@@ -10,6 +10,8 @@ module Mutant
 
           handle(Mutant::Config)
 
+          delegate :matcher, :strategy, :expected_coverage
+
           # Report configuration
           #
           # @param [Mutant::Config] config
@@ -20,9 +22,9 @@ module Mutant
           #
           def run
             info 'Mutant configuration:'
-            info 'Matcher:        %s', object.matcher.inspect
-            info 'Subject Filter: %s', object.subject_predicate.inspect
-            info 'Strategy:       %s', object.strategy.inspect
+            info 'Matcher:         %s',     matcher.inspect
+            info 'Strategy:        %s',     strategy.inspect
+            info 'Expect Coverage: %02f%%', expected_coverage.inspect
             self
           end
 
@@ -31,6 +33,11 @@ module Mutant
 
             handle(Mutant::Runner::Config)
 
+            delegate(
+              :amount_kills, :amount_mutations, :amount_kils,
+              :coverage, :subjects, :failed_subjects, :runtime, :mutations
+            )
+
             # Run printer
             #
             # @return [self]
@@ -42,73 +49,18 @@ module Mutant
               info   'Subjects:  %s',      amount_subjects
               info   'Mutations: %s',      amount_mutations
               info   'Kills:     %s',      amount_kills
+              info   'Alive:     %s',      amount_alive
               info   'Runtime:   %0.2fs',  runtime
               info   'Killtime:  %0.2fs',  killtime
               info   'Overhead:  %0.2f%%', overhead
               status 'Coverage:  %0.2f%%', coverage
-              status 'Alive:     %s',      amount_alive
+              status 'Expected:  %0.2f%%', object.config.expected_coverage
               print_generic_stats
               self
             end
 
           private
 
-            # Return subjects
-            #
-            # @return [Array<Subject>]
-            #
-            # @api private
-            #
-            def subjects
-              object.subjects
-            end
-
-            # Walker for all ast nodes
-            class Walker
-
-              # Run walkter
-              #
-              # @param [Parser::AST::Node] root
-              #
-              # @return [self]
-              #
-              # @api private
-              #
-              def self.run(root, &block)
-                new(root, block)
-                self
-              end
-
-              private_class_method :new
-
-              # Initialize and run walker
-              #
-              # @param [Parser::AST::Node] root
-              # @param [#call(node)] block
-              #
-              # @return [undefined]
-              #
-              # @api private
-              #
-              def initialize(root, block)
-                @root, @block = root, block
-                dispatch(root)
-              end
-
-            private
-
-              # Perform dispatch
-              #
-              # @return [undefined]
-              #
-              # @api private
-              #
-              def dispatch(node)
-                @block.call(node)
-                node.children.grep(Parser::AST::Node).each(&method(:dispatch))
-              end
-            end
-
             # Print generic stats
             #
             # @return [undefined]
@@ -131,7 +83,7 @@ module Mutant
             # @api private
             #
             def generic_stats
-              object.subjects.each_with_object(Hash.new(0)) do |runner, stats|
+              subjects.each_with_object(Hash.new(0)) do |runner, stats|
                 Walker.run(runner.subject.node) do |node|
                   if Mutator::Registry.lookup(node) == Mutator::Node::Generic
                     stats[node.type] += 1
@@ -157,32 +109,11 @@ module Mutant
             # @api private
             #
             def print_mutations
-              object.failed_subjects.each do |subject|
+              failed_subjects.each do |subject|
                 Subject::Runner::Details.run(subject, output)
               end
             end
 
-            # Return mutations
-            #
-            # @return [Array<Mutation>]
-            #
-            # @api private
-            #
-            def mutations
-              subjects.map(&:mutations).flatten
-            end
-            memoize :mutations
-
-            # Return amount of mutations
-            #
-            # @return [Fixnum]
-            #
-            # @api private
-            #
-            def amount_mutations
-              mutations.length
-            end
-
             # Return amount of time in killers
             #
             # @return [Float]
@@ -194,16 +125,6 @@ module Mutant
             end
             memoize :killtime
 
-            # Return amount of kills
-            #
-            # @return [Fixnum]
-            #
-            # @api private
-            #
-            def amount_kills
-              mutations.select(&:success?).length
-            end
-
             # Return mutant overhead
             #
             # @return [Float]
@@ -215,27 +136,6 @@ module Mutant
               Rational(runtime - killtime, runtime) * 100
             end
 
-            # Return runtime
-            #
-            # @return [Float]
-            #
-            # @api private
-            #
-            def runtime
-              object.runtime
-            end
-
-            # Return coverage
-            #
-            # @return [Float]
-            #
-            # @api private
-            #
-            def coverage
-              return 0 if amount_mutations.zero?
-              Rational(amount_kills, amount_mutations) * 100
-            end
-
             # Return amount of alive mutations
             #
             # @return [Fixnum]
@@ -243,12 +143,12 @@ module Mutant
             # @api private
             #
             def amount_alive
-              amount_mutations - amount_kills
+              object.amount_mutations - amount_kills
             end
 
           end # Runner
         end # Config
       end # Printer
-    end # Cli
+    end # CLI
   end # Reporter
 end # Mutant

data/lib/mutant/runner/config.rb

@@ -40,6 +40,8 @@ module Mutant
       end
       memoize :failed_subjects
 
+      COVERAGE_PRECISION = 1
+
       # Test if run was successful
       #
       # @return [true]
@@ -51,7 +53,7 @@ module Mutant
       # @api private
       #
       def success?
-        failed_subjects.empty?
+        coverage.round(COVERAGE_PRECISION) == config.expected_coverage.round(COVERAGE_PRECISION)
       end
       memoize :success?
 
@@ -65,6 +67,50 @@ module Mutant
         config.strategy
       end
 
+      # Return coverage
+      #
+      # @return [Float]
+      #
+      # @api private
+      #
+      def coverage
+        return 0.0 if amount_mutations.zero?
+        Rational(amount_kills, amount_mutations) * 100
+      end
+      memoize :coverage
+
+      # Return amount of kills
+      #
+      # @return [Fixnum]
+      #
+      # @api private
+      #
+      def amount_kills
+        mutations.select(&:success?).length
+      end
+      memoize :amount_kills
+
+      # Return mutations
+      #
+      # @return [Array<Mutation>]
+      #
+      # @api private
+      #
+      def mutations
+        subjects.map(&:mutations).flatten
+      end
+      memoize :mutations
+
+      # Return amount of mutations
+      #
+      # @return [Fixnum]
+      #
+      # @api private
+      #
+      def amount_mutations
+        mutations.length
+      end
+
     private
 
       # Run config