mutant 0.9.8 → 0.9.13

data/lib/mutant/parallel/worker.rb

@@ -39,20 +39,12 @@ module Mutant
 
     private
 
-      # Next job, if any
-      #
-      # @return [Job, nil]
       def next_job
         var_source.with do |source|
           source.next if source.next?
         end
       end
 
-      # Add result
-      #
-      # @param [Object] result
-      #
-      # @return [Boolean]
       def add_result(result)
         var_sink.with do |sink|
           sink.result(result)
@@ -60,32 +52,18 @@ module Mutant
         end
       end
 
-      # Register job to be started
-      #
-      # @param [Job] job
-      #
-      # @return [undefined]
       def job_start(job)
         var_active_jobs.with do |active_jobs|
           active_jobs << job
         end
       end
 
-      # Register job to be done
-      #
-      # @param [Job] job
-      # @param [Object] result
-      #
-      # @return [undefined]
       def job_done(job)
         var_active_jobs.with do |active_jobs|
           active_jobs.delete(job)
         end
       end
 
-      # Finalize worker
-      #
-      # @return [undefined]
       def finalize
         var_final.put(nil) if var_running.modify(&:pred).zero?
       end

data/lib/mutant/reporter/cli.rb

@@ -69,11 +69,6 @@ module Mutant
 
     private
 
-      # Write output frame
-      #
-      # @param [String] frame
-      #
-      # @return [undefined]
       def write(frame)
         output.write(frame)
       end

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

@@ -49,12 +49,6 @@ module Mutant
 
       private
 
-        # Format object with printer
-        #
-        # @param [Class:Printer] printer
-        # @param [Object] object
-        #
-        # @return [String]
         def format(printer, object)
           buffer = new_buffer
           printer.call(Output.new(tty, buffer), object)
@@ -84,9 +78,6 @@ module Mutant
 
         private
 
-          # New buffer
-          #
-          # @return [StringIO]
           def new_buffer
             StringIO.new
           end

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

@@ -44,81 +44,41 @@ module Mutant
 
       private
 
-        # Status color
-        #
-        # @return [Color]
         def status_color
-          success? ? Color::GREEN : Color::RED
+          success? ? Unparser::Color::GREEN : Unparser::Color::RED
         end
 
-        # Visit a collection of objects
-        #
-        # @return [Class::Printer] printer
-        # @return [Enumerable<Object>] collection
-        #
-        # @return [undefined]
         def visit_collection(printer, collection)
           collection.each do |object|
             visit(printer, object)
           end
         end
 
-        # Visit object
-        #
-        # @param [Class::Printer] printer
-        # @param [Object] object
-        #
-        # @return [undefined]
         def visit(printer, object)
           printer.call(output, object)
         end
 
-        # Print an info line to output
-        #
-        # @return [undefined]
         def info(string, *arguments)
           puts(string % arguments)
         end
 
-        # Print a status line to output
-        #
-        # @return [undefined]
         def status(string, *arguments)
           puts(colorize(status_color, string % arguments))
         end
 
-        # Print a line to output
-        #
-        # @return [undefined]
         def puts(string)
           output.puts(string)
         end
 
-        # Colorize message
-        #
-        # @param [Color] color
-        # @param [String] message
-        #
-        # @return [String]
-        #   if color is enabled
-        #   unmodified message otherwise
         def colorize(color, message)
-          color = Color::NONE unless tty?
+          color = Unparser::Color::NONE unless tty?
           color.format(message)
         end
 
-        # Test if output is a tty
-        #
-        # @return [Boolean]
         def tty?
           output.tty?
         end
 
-        # Test if output can be colored
-        #
-        # @return [Boolean]
-        #
-        # @api private
         alias_method :color?, :tty?
       end # Printer
     end # CLI

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

@@ -40,29 +40,14 @@ module Mutant
 
         private
 
-          # Mutations processed per second
-          #
-          # @return [Float]
-          #
-          # @api private
           def mutations_per_second
             amount_mutation_results / runtime
           end
 
-          # Coverage in percent
-          #
-          # @return [Float]
-          #
-          # @api private
           def coverage_percent
             coverage * 100
           end
 
-          # Overhead in percent
-          #
-          # @return [Float]
-          #
-          # @api private
           def overhead_percent
             (overhead / killtime) * 100
           end

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

@@ -28,7 +28,7 @@ module Mutant
             * Bug in your test suite
             * Bug in your test suite under concurrency
 
-            The following exception was raised:
+            The following exception was raised while reading the killfork result:
 
             ```
             %s
@@ -60,45 +60,36 @@ module Mutant
           #
           # @return [undefined]
           def run
-            __send__(MAP.fetch(object.class))
             print_log_messages
+            __send__(MAP.fetch(object.class))
           end
 
         private
 
-          # Visit successful isolation result
-          #
-          # @return [undefined]
           def visit_success
             visit(TestResult, object.value)
           end
 
-          # Print log messages
-          #
-          # @return [undefined]
           def print_log_messages
             log = object.log
 
-            puts(LOG_MESSAGES % log) unless log.empty?
+            return if log.empty?
+
+            puts('Log messages (combined stderr and stdout):')
+
+            log.each_line do |line|
+              puts('[killfork] %<line>s' % { line: line })
+            end
           end
 
-          # Visit child error isolation result
-          #
-          # @return [undefined]
           def visit_child_error
             puts(CHILD_ERROR_MESSAGE % object.value.inspect)
           end
 
-          # Visit fork error isolation result
-          #
-          # @return [undefined]
           def visit_fork_error
             puts(FORK_ERROR_MESSAGE)
           end
 
-          # Visit exception isolation result
-          #
-          # @return [undefined]
           def visit_exception
             exception = object.value
 
@@ -110,9 +101,6 @@ module Mutant
             )
           end
 
-          # Visit chain
-          #
-          # @return [undefined]
           def visit_chain
             printer = self.class
 

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

@@ -18,11 +18,6 @@ module Mutant
 
         private
 
-          # Write colorized char
-          #
-          # @param [String] char
-          #
-          # @return [undefined]
           def char(char)
             output.write(colorize(status_color, char))
           end

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

@@ -59,9 +59,6 @@ module Mutant
 
         private
 
-          # Print mutation details
-          #
-          # @return [undefined]
           def print_details
             __send__(MAP.fetch(mutation.class))
 
@@ -69,11 +66,8 @@ module Mutant
             visit_isolation_result
           end
 
-          # Evil mutation details
-          #
-          # @return [String]
           def evil_details
-            diff = Diff.build(mutation.original_source, mutation.source)
+            diff = Unparser::Diff.build(mutation.original_source, mutation.source)
             diff = color? ? diff.colorized_diff : diff.diff
             if diff
               output.write(diff)
@@ -82,9 +76,6 @@ module Mutant
             end
           end
 
-          # Print no diff message
-          #
-          # @return [undefined]
           def print_no_diff_message
             info(
               NO_DIFF_MESSAGE,
@@ -95,30 +86,18 @@ module Mutant
             )
           end
 
-          # Noop details
-          #
-          # @return [String]
           def noop_details
             info(NOOP_MESSAGE)
           end
 
-          # Neutral details
-          #
-          # @return [String]
           def neutral_details
             info(NEUTRAL_MESSAGE, original_node.inspect, mutation.source)
           end
 
-          # Visit failed test results
-          #
-          # @return [undefined]
           def visit_isolation_result
             visit(IsolationResult, isolation_result)
           end
 
-          # Original node
-          #
-          # @return [Parser::AST::Node]
           def original_node
             mutation.subject.node
           end

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

@@ -35,18 +35,10 @@ module Mutant
 
         private
 
-          # Object being printed
-          #
-          # @return [Result::Env]
           def object
             super().payload
           end
 
-          # Mutations processed per second
-          #
-          # @return [Float]
-          #
-          # @api private
           def mutations_per_second
             amount_mutation_results / runtime
           end

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

@@ -32,9 +32,6 @@ module Mutant
 
         private
 
-          # Print stats
-          #
-          # @return [undefined]
           def print_stats
             status(
               FORMAT,
@@ -47,16 +44,10 @@ module Mutant
             )
           end
 
-          # Print progress bar finish
-          #
-          # @return [undefined]
           def print_progress_bar_finish
             puts(nil) unless amount_mutation_results.zero?
           end
 
-          # Print mutation results
-          #
-          # @return [undefined]
           def print_mutation_results
             visit_collection(MutationProgressResult, object.mutation_results)
           end

data/lib/mutant/repository/diff.rb

@@ -29,10 +29,6 @@ module Mutant
 
     private
 
-      # Touched paths
-      #
-      # @return [Hash{Pathname => Path}]
-      #
       # rubocop:disable Metrics/MethodLength
       def touched_paths
         pathname = world.pathname
@@ -49,13 +45,8 @@ module Mutant
           .to_h
       end
       memoize :touched_paths
+      # rubocop:enable Metrics/MethodLength
 
-      # Parse path
-      #
-      # @param [Pathname] work_dir
-      # @param [String] line
-      #
-      # @return [Path]
       def parse_line(work_dir, line)
         match = FORMAT.match(line) or fail Error, "Invalid git diff-index line: #{line}"
 
@@ -88,9 +79,6 @@ module Mutant
 
       private
 
-        # Ranges of hunks in the diff
-        #
-        # @return [Array<Range<Integer>>]
         def diff_ranges
           world
             .capture_stdout(%W[git diff --unified=0 #{to} -- #{path}])

data/lib/mutant/repository/diff/ranges.rb

@@ -18,11 +18,6 @@ module Mutant
           diff.lines.flat_map(&method(:parse_ranges)).to_set
         end
 
-        # Parse ranges from line
-        #
-        # @param [String] line
-        #
-        # @return [Array<Range<Integer>>]
         def self.parse_ranges(line)
           match = REGEXP.match(line) or return EMPTY_ARRAY
 
@@ -34,12 +29,6 @@ module Mutant
         end
         private_class_method :parse_ranges
 
-        # Construct a range from start point and offset
-        #
-        # @param [String] start
-        # @param [String, nil] offset
-        #
-        # @return [Range<Integer>]
         def self.mk_range(start, offset)
           start = Integer(start)