mutant 0.8.8 → 0.8.9

data/lib/mutant/runner/sink.rb

@@ -1,107 +1,64 @@
 module Mutant
   class Runner
-    # Abstract base class for computation sinks
     class Sink
-      include AbstractType
+      include Concord.new(:env)
 
-      # sink status
+      # Initialize object
       #
-      # @return [Object]
+      # @return [undefined]
+      def initialize(*)
+        super
+        @start           = Time.now
+        @subject_results = {}
+      end
+
+      # Runner status
       #
-      # @api private
-      abstract_method :status
+      # @return [Result::Env]
+      def status
+        Result::Env.new(
+          env:             env,
+          runtime:         Time.now - @start,
+          subject_results: @subject_results.values
+        )
+      end
 
-      # Test if computation should be stopped
+      # Test if scheduling stopped
       #
       # @return [Boolean]
-      #
-      # @api private
-      abstract_method :stop?
+      def stop?
+        env.config.fail_fast && !status.subject_results.all?(&:success?)
+      end
 
-      # Consume result
+      # Handle mutation finish
       #
-      # @param [Object] result
+      # @param [Result::Mutation] mutation_result
       #
       # @return [self]
-      #
-      # @api private
-      abstract_method :result
-
-      # Mutation result sink
-      class Mutation < self
-        include Concord.new(:env)
-
-        # Initialize object
-        #
-        # @return [undefined]
-        #
-        # @api private
-        def initialize(*)
-          super
-          @start           = Time.now
-          @subject_results = Hash.new do |_hash, subject|
-            Result::Subject.new(
-              subject:          subject,
-              tests:            [],
-              mutation_results: []
-            )
-          end
-        end
-
-        # Runner status
-        #
-        # @return [Status]
-        #
-        # @api private
-        def status
-          env_result
-        end
+      def result(mutation_result)
+        subject = mutation_result.mutation.subject
 
-        # Test if scheduling stopped
-        #
-        # @return [Boolean]
-        #
-        # @api private
-        def stop?
-          env.config.fail_fast && !env_result.subject_results.all?(&:success?)
-        end
+        @subject_results[subject] = Result::Subject.new(
+          subject:          subject,
+          mutation_results: previous_mutation_results(subject) + [mutation_result],
+          tests:            mutation_result.test_result.tests
+        )
 
-        # Handle mutation finish
-        #
-        # @param [Result::Mutation] mutation_result
-        #
-        # @return [self]
-        #
-        # @api private
-        def result(mutation_result)
-          mutation = mutation_result.mutation
+        self
+      end
 
-          original = @subject_results[mutation.subject]
+    private
 
-          @subject_results[mutation.subject] = original.with(
-            mutation_results: (original.mutation_results + [mutation_result]),
-            tests:            mutation_result.test_result.tests
-          )
-
-          self
-        end
-
-      private
-
-        # Current result
-        #
-        # @return [Result::Env]
-        #
-        # @api private
-        def env_result
-          Result::Env.new(
-            env:             env,
-            runtime:         Time.now - @start,
-            subject_results: @subject_results.values
-          )
-        end
+      # Return previous results
+      #
+      # @param [Subject]
+      #
+      # @return [Array<Result::Mutation>]
+      def previous_mutation_results(subject)
+        subject_result = @subject_results.fetch(subject) { return EMPTY_ARRAY }
+        subject_result.mutation_results
+      end
 
-      end # Mutation
     end # Sink
   end # Runner
 end # Mutant

data/lib/mutant/selector.rb

@@ -8,8 +8,6 @@ module Mutant
     # @param [Subject] subjecto
     #
     # @return [Enumerable<Test>]
-    #
-    # @api private
     abstract_method :call
 
   end # Selector

data/lib/mutant/selector/expression.rb

@@ -9,8 +9,6 @@ module Mutant
       # @param [Subject] subject
       #
       # @return [Enumerable<Test>]
-      #
-      # @api private
       def call(subject)
         subject.match_expressions.each do |match_expression|
           subject_tests = integration.all_tests.select do |test|

data/lib/mutant/subject.rb

@@ -8,8 +8,6 @@ module Mutant
     #
     # @return [Enumerable<Mutation>]
     # @return [undefined]
-    #
-    # @api private
     def mutations
       mutations = [neutral_mutation]
       Mutator.each(node) do |mutant|
@@ -22,8 +20,6 @@ module Mutant
     # Source path
     #
     # @return [Pathname]
-    #
-    # @api private
     def source_path
       context.source_path
     end
@@ -31,8 +27,6 @@ module Mutant
     # Prepare subject for insertion of mutation
     #
     # @return [self]
-    #
-    # @api private
     def prepare
       self
     end
@@ -40,8 +34,6 @@ module Mutant
     # Source line range
     #
     # @return [Range<Fixnum>]
-    #
-    # @api private
     def source_lines
       expression = node.location.expression
       expression.line..expression.source_buffer.decompose_position(expression.end_pos).first
@@ -51,8 +43,6 @@ module Mutant
     # First source line
     #
     # @return [Fixnum]
-    #
-    # @api private
     def source_line
       source_lines.begin
     end
@@ -60,8 +50,6 @@ module Mutant
     # Identification string
     #
     # @return [String]
-    #
-    # @api private
     def identification
       "#{expression.syntax}:#{source_path}:#{source_line}"
     end
@@ -70,8 +58,6 @@ module Mutant
     # Source representation of AST
     #
     # @return [String]
-    #
-    # @api private
     def source
       Unparser.unparse(wrap_node(node))
     end
@@ -80,15 +66,11 @@ module Mutant
     # Match expression
     #
     # @return [Expression]
-    #
-    # @api private
     abstract_method :expression
 
     # Match expressions
     #
     # @return [Enumerable<Expression>]
-    #
-    # @api private
     abstract_method :match_expressions
 
   private
@@ -96,8 +78,6 @@ module Mutant
     # Neutral mutation
     #
     # @return [Mutation::Neutral]
-    #
-    # @api private
     def neutral_mutation
       Mutation::Neutral.new(self, wrap_node(node))
     end
@@ -107,8 +87,6 @@ module Mutant
     # @param [Parser::AST::Node] node
     #
     # @return [Parser::AST::Node]
-    #
-    # @api private
     def wrap_node(node)
       node
     end

data/lib/mutant/subject/method.rb

@@ -6,8 +6,6 @@ module Mutant
       # Method name
       #
       # @return [Expression]
-      #
-      # @api private
       def name
         node.children.fetch(self.class::NAME_INDEX)
       end
@@ -15,13 +13,11 @@ module Mutant
       # Match expression
       #
       # @return [String]
-      #
-      # @api private
       def expression
         Expression::Method.new(
+          method_name:  name.to_s,
           scope_symbol: self.class::SYMBOL,
-          scope_name:   scope.name,
-          method_name:  name.to_s
+          scope_name:   scope.name
         )
       end
       memoize :expression
@@ -29,8 +25,6 @@ module Mutant
       # Match expressions
       #
       # @return [Array<Expression>]
-      #
-      # @api private
       def match_expressions
         [expression].concat(context.match_expressions)
       end
@@ -41,8 +35,6 @@ module Mutant
       # The scope
       #
       # @return [Class, Module]
-      #
-      # @api private
       def scope
         context.scope
       end

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

@@ -10,8 +10,6 @@ module Mutant
         # Prepare subject for mutation insertion
         #
         # @return [self]
-        #
-        # @api private
         def prepare
           scope.__send__(:undef_method, name)
           self
@@ -24,8 +22,6 @@ module Mutant
           # Prepare subject for mutation insertion
           #
           # @return [self]
-          #
-          # @api private
           def prepare
             scope.__send__(:memoized_methods).instance_variable_get(:@memory).delete(name)
             super
@@ -39,8 +35,6 @@ module Mutant
           # @param [Parser::AST::Node] mutant
           #
           # @return [Parser::AST::Node]
-          #
-          # @api private
           def wrap_node(mutant)
             s(:begin, mutant, s(:send, nil, :memoize, s(:args, s(:sym, name))))
           end

data/lib/mutant/subject/method/singleton.rb

@@ -10,8 +10,6 @@ module Mutant
         # Prepare subject for mutation insertion
         #
         # @return [self]
-        #
-        # @api private
         def prepare
           scope.singleton_class.__send__(:undef_method, name)
           self

data/lib/mutant/test.rb

@@ -1,13 +1,14 @@
 module Mutant
   # Abstract base class for test that might kill a mutation
   class Test
-    include Adamantium::Flat, Anima.new(:id, :expression)
+    include Adamantium::Flat, Anima.new(
+      :expression,
+      :id
+    )
 
     # Identification string
     #
     # @return [String]
-    #
-    # @api private
     alias_method :identification, :id
 
   end # Test

data/lib/mutant/version.rb

@@ -1,4 +1,4 @@
 module Mutant
   # Current mutant version
-  VERSION = '0.8.8'.freeze
+  VERSION = '0.8.9'.freeze
 end # Mutant

data/lib/mutant/warning_filter.rb

@@ -10,8 +10,6 @@ module Mutant
     # @param [#write] target
     #
     # @return [undefined]
-    #
-    # @api private
     def initialize(target)
       @target   = target
       @warnings = []
@@ -20,8 +18,6 @@ module Mutant
     # Warnings captured by filter
     #
     # @return [Array<String>]
-    #
-    # @api private
     attr_reader :warnings
 
     # Target stream to capture warnings on
@@ -29,8 +25,6 @@ module Mutant
     # @return [#write] target
     #
     # @return [undefined]
-    #
-    # @api private
     attr_reader :target
     protected :target
 
@@ -39,8 +33,6 @@ module Mutant
     # @param [String] message
     #
     # @return [self]
-    #
-    # @api private
     def write(message)
       if WARNING_PATTERN =~ message
         warnings << message
@@ -54,8 +46,6 @@ module Mutant
     # Use warning filter during block execution
     #
     # @return [Array<String>]
-    #
-    # @api private
     def self.use
       original = $stderr
       $stderr = filter = new(original)

data/lib/mutant/zombifier.rb

@@ -3,13 +3,14 @@ module Mutant
   class Zombifier
     include Anima.new(
       :includes,
-      :namespace,
       :load_path,
       :kernel,
+      :namespace,
+      :pathname,
       :require_highjack,
-      :root_require,
-      :pathname
+      :root_require
     )
+
     private(*anima.attribute_names)
 
     include AST::Sexp
@@ -21,8 +22,6 @@ module Mutant
     # @param [Symbol] namespace
     #
     # @return [undefined]
-    #
-    # @api private
     def initialize(*)
       super
       @includes = %r{\A#{Regexp.union(includes)}(?:/.*)?\z}
@@ -32,8 +31,6 @@ module Mutant
     # Call zombifier
     #
     # @return [self]
-    #
-    # @api private
     def self.call(*args)
       new(*args).__send__(:call)
       self
@@ -44,8 +41,6 @@ module Mutant
     # Run zombifier
     #
     # @return [undefined]
-    #
-    # @api private
     def call
       @original = require_highjack.call(method(:require))
       require(root_require)
@@ -54,8 +49,6 @@ module Mutant
     # Test if logical name is subjected to zombification
     #
     # @param [String]
-    #
-    # @api private
     def include?(logical_name)
       !@zombified.include?(logical_name) && includes =~ logical_name
     end
@@ -65,8 +58,6 @@ module Mutant
     # @param [#to_s] logical_name
     #
     # @return [undefined]
-    #
-    # @api private
     def require(logical_name)
       logical_name = logical_name.to_s
       @original.call(logical_name)
@@ -83,8 +74,6 @@ module Mutant
     #
     # @raise [LoadError]
     #   otherwise
-    #
-    # @api private
     def find(logical_name)
       file_name = "#{logical_name}.rb"
 
@@ -103,8 +92,6 @@ module Mutant
     # @param [Pathname] source_path
     #
     # @return [undefined]
-    #
-    # @api private
     def zombify(source_path)
       kernel.eval(
         Unparser.unparse(namespaced_node(source_path)),
@@ -118,10 +105,8 @@ module Mutant
     # @param [Pathname] source_path
     #
     # @return [Parser::AST::Node]
-    #
-    # @api private
     def namespaced_node(source_path)
-      s(:module, s(:const, nil, namespace), Parser::CurrentRuby.parse(source_path.read))
+      s(:module, s(:const, nil, namespace), ::Parser::CurrentRuby.parse(source_path.read))
     end
 
   end # Zombifier