minitest-bender 0.0.3 → 1.0.0

data/lib/minitest-bender/recorders/grouped_icons.rb

@@ -0,0 +1,36 @@
+module MinitestBender
+  module Recorders
+    class GroupedIcons
+      def initialize(io)
+        @printer = Printers::Plain.new(io)
+      end
+
+      def print_context(result_context)
+        printer.print_line
+
+        context_path = result_context.path
+        context_separator = result_context.separator
+        prefix = result_context.prefix
+
+        path = context_path[0...-1].join(context_separator)
+        path << context_separator unless path.empty?
+        klass = context_path.last
+
+        printer.print("#{prefix}#{path}#{Colorizer.colorize(klass, :normal, :bold)} ")
+      end
+
+      def print_result(result)
+        printer.print(result.to_icon)
+        printer.advance
+      end
+
+      def print_context_with_results(_result_context, _results)
+        # do_nothing
+      end
+
+      private
+
+      attr_reader :printer
+    end
+  end
+end

data/lib/minitest-bender/recorders/icons.rb

@@ -0,0 +1,26 @@
+module MinitestBender
+  module Recorders
+    class Icons
+      def initialize(io)
+        @printer = Printers::Plain.new(io)
+      end
+
+      def print_context(_result_context)
+        # do nothing
+      end
+
+      def print_result(result)
+        printer.print(result.to_icon)
+        printer.advance
+      end
+
+      def print_context_with_results(_result_context, _results)
+        # do_nothing
+      end
+
+      private
+
+      attr_reader :printer
+    end
+  end
+end

data/lib/minitest-bender/recorders/none.rb

@@ -0,0 +1,17 @@
+module MinitestBender
+  module Recorders
+    class None
+      def print_context(_result_context)
+        # do nothing
+      end
+
+      def print_result(_result)
+        # do nothing
+      end
+
+      def print_context_with_results(_result_context, _results)
+        # do_nothing
+      end
+    end
+  end
+end

data/lib/minitest-bender/recorders/progress.rb

@@ -0,0 +1,25 @@
+module MinitestBender
+  module Recorders
+    class Progress
+      def initialize(io, total_tests_count)
+        @printer = Printers::WithProgressBar.new(io, total_tests_count)
+      end
+
+      def print_context(_result_context)
+        # do nothing
+      end
+
+      def print_result(result)
+        printer.advance
+      end
+
+      def print_context_with_results(_result_context, _results)
+        # do_nothing
+      end
+
+      private
+
+      attr_reader :printer
+    end
+  end
+end

data/lib/minitest-bender/recorders/progress_groups.rb

@@ -0,0 +1,48 @@
+module MinitestBender
+  module Recorders
+    class ProgressGroups
+      def initialize(io, total_tests_count)
+        @printer = Printers::WithProgressBar.new(io, total_tests_count)
+        @total_tests_count = total_tests_count
+      end
+
+      def print_context(_result_context)
+        # do nothing
+      end
+
+      def print_result(result)
+        printer.advance
+      end
+
+      def print_context_with_results(result_context, results)
+        context_path = result_context.path
+        context_separator = result_context.separator
+        prefix = result_context.prefix
+
+        path = context_path[0...-1].join(context_separator)
+        path << context_separator unless path.empty?
+        klass = context_path.last
+
+        printer.print_line("#{prefix}#{counters(result_context)} #{path}#{Colorizer.colorize(klass, :normal, :bold)}")
+      end
+
+      private
+
+      attr_reader :printer, :total_tests_count
+
+      def counters(result_context)
+        states.map do |state|
+          state.colored_icon_with_context_count(result_context, counters_padding_right)
+        end.join('  ')
+      end
+
+      def states
+        @states ||= MinitestBender.states.values
+      end
+
+      def counters_padding_right
+        @counters_padding_right ||= total_tests_count.to_s.size + 1
+      end
+    end
+  end
+end

data/lib/minitest-bender/recorders/progress_groups_and_issues.rb

@@ -0,0 +1,55 @@
+module MinitestBender
+  module Recorders
+    class ProgressGroupsAndIssues
+      def initialize(io, total_tests_count)
+        @printer = Printers::WithProgressBar.new(io, total_tests_count)
+        @total_tests_count = total_tests_count
+      end
+
+      def print_context(_result_context)
+        # do nothing
+      end
+
+      def print_result(result)
+        printer.print_line(result_line(result)) unless result.passed?
+        lines = result.state.detail_lines_without_header(result)
+        printer.print_lines(lines)
+        printer.advance
+      end
+
+      def print_context_with_results(result_context, results)
+        context_path = result_context.path
+        context_separator = result_context.separator
+        prefix = result_context.prefix
+
+        path = context_path[0...-1].join(context_separator)
+        path << context_separator unless path.empty?
+        klass = context_path.last
+
+        printer.print_line("#{prefix}#{counters(result_context)} #{path}#{Colorizer.colorize(klass, :normal, :bold)}")
+      end
+
+      private
+
+      attr_reader :printer, :total_tests_count
+
+      def result_line(result)
+        "  #{result.formatted_label_and_time}#{result.formatted_number} #{result.formatted_name_with_context}"
+      end
+
+      def counters(result_context)
+        states.map do |state|
+          state.colored_icon_with_context_count(result_context, counters_padding_right)
+        end.join('  ')
+      end
+
+      def states
+        @states ||= MinitestBender.states.values
+      end
+
+      def counters_padding_right
+        @counters_padding_right ||= total_tests_count.to_s.size + 1
+      end
+    end
+  end
+end

data/lib/minitest-bender/recorders/progress_issues.rb

@@ -0,0 +1,32 @@
+module MinitestBender
+  module Recorders
+    class ProgressIssues
+      def initialize(io, total_tests_count)
+        @printer = Printers::WithProgressBar.new(io, total_tests_count)
+      end
+
+      def print_context(_result_context)
+        # do nothing
+      end
+
+      def print_result(result)
+        printer.print_line(result_line(result)) unless result.passed?
+        lines = result.state.detail_lines_without_header(result)
+        printer.print_lines(lines)
+        printer.advance
+      end
+
+      def print_context_with_results(_result_context, _results)
+        # do_nothing
+      end
+
+      private
+
+      attr_reader :printer
+
+      def result_line(result)
+        "  #{result.formatted_label_and_time}#{result.formatted_number} #{result.formatted_name_with_context}"
+      end
+    end
+  end
+end

data/lib/minitest-bender/recorders/progress_verbose.rb

@@ -0,0 +1,34 @@
+module MinitestBender
+  module Recorders
+    class ProgressVerbose
+      def initialize(io, total_tests_count)
+        @printer = Printers::WithProgressBar.new(io, total_tests_count)
+      end
+
+      def print_context(result_context)
+        printer.print_line
+        printer.print_line(Colorizer.colorize(result_context.with_prefix, :normal, :bold))
+      end
+
+      def print_result(result)
+        printer.print_line(result_line(result))
+        lines = result.state.detail_lines_without_header(result)
+        padded_lines = lines.map { |line| "  #{line}" }
+        printer.print_lines(padded_lines)
+        printer.advance
+      end
+
+      def print_context_with_results(_result_context, _results)
+        # do_nothing
+      end
+
+      private
+
+      attr_reader :printer
+
+      def result_line(result)
+        "    #{result.formatted_label_and_time}#{result.formatted_number} #{result.name}"
+      end
+    end
+  end
+end

data/lib/minitest-bender/result_context.rb

@@ -0,0 +1,39 @@
+# coding: utf-8
+# frozen_string_literal: true
+
+module MinitestBender
+  class ResultContext < String
+    PREFIX = '• '
+    SEPARATOR = ' ▸ '
+    CLASS_SEPARATOR = '::'
+
+    def initialize(class_name)
+      @class_name = class_name
+      super(path.join(separator))
+    end
+
+    def path
+      class_name.split(class_separator)
+    end
+
+    def with_prefix
+      prefix + self
+    end
+
+    def separator
+      SEPARATOR
+    end
+
+    def prefix
+      PREFIX
+    end
+
+    private
+
+    attr_reader :class_name
+
+    def class_separator
+      CLASS_SEPARATOR
+    end
+  end
+end

data/lib/minitest-bender/result_factory.rb

@@ -26,6 +26,11 @@ module MinitestBender
     end
 
     def parsed_name(minitest_result)
+      if minitest_result.name.is_a?(Class)
+        # something went wrong inside minitest (infinite loop?)
+        raise minitest_result.failures[0].error
+      end
+
       minitest_result.name.match(RESULT_NAME_REGEXP)
     end
   end

data/lib/minitest-bender/results/base.rb

@@ -1,77 +1,133 @@
+# coding: utf-8
+# frozen_string_literal: true
+
+require 'forwardable'
+
 module MinitestBender
   module Results
     class Base
       extend Forwardable
       def_delegators :@minitest_result, :passed?, :skipped?, :assertions, :failures, :time
+      attr_reader :state, :execution_order
+
+      NAME_PREFIX = '♦'
 
       def initialize(minitest_result)
         @minitest_result = minitest_result
         @state = MinitestBender.states.fetch(minitest_result.result_code)
+        @execution_order = state.add_result(self).size
       end
 
       def context
-        @context ||=
-          if minitest_result.respond_to?(:klass) # minitest >= 5.11
-            minitest_result.klass
-          else
-            minitest_result.class.name
-          end.gsub('::', ' > ')
+        @context ||= ResultContext.new(adjusted_class_name)
+      end
+
+      def to_icon
+        state.colored_icon
       end
 
-      def header
-        Colorin.white("• #{context}").bold
+      def name_sort_key
+        name
       end
 
-      def details_header(number)
-        "    #{number}#{Colorin.white(context)} > #{name}"
+      def formatted_name_with_context
+        "#{Colorizer.colorize(context, :normal)} #{name_prefix} #{Colorizer.colorize(name, :normal, :bold)}"
       end
 
       def rerun_line(padding)
         unformatted = "Rerun: #{rerun_command}"
-        "#{padding}#{Colorin.blue_a700(unformatted)}"
+        "#{padding}#{Colorizer.colorize(unformatted, :tests)}"
+      end
+
+      def formatted_label
+        state.formatted_label
+      end
+
+      def time_with_unit_and_padding_right
+        time_in_s = time
+        case time_in_s
+        when 0...1
+          sprintf('%.0fms ', time_in_s * 1000)
+        when 1...10
+          sprintf('%.2fs ', time_in_s)
+        when 10...100
+          sprintf('%.1fs ', time_in_s)
+        when 100...10000
+          sprintf('%.0fs ', time_in_s)
+        else
+          sprintf('%.0fs', time_in_s)
+        end
+      end
+
+      def formatted_time
+        Colorizer.colorize(time_with_unit_and_padding_right.rjust(6), :time)
+      end
+
+      def formatted_label_and_time
+        "#{formatted_label} #{formatted_time}"
       end
 
-      def state?(some_state)
-        state.class == some_state.class
+      def formatted_message
+        state.formatted_message(self)
+      end
+
+      def file_path
+        @file_path ||= source_location[0]
+      end
+
+      def source_line_number
+        @source_line_number ||= source_location[1]
       end
 
-      def line_for_slowness_podium
-        "#{formatted_time} #{Colorin.white(context)} > #{name}"
+      # credit where credit is due: minitest-line
+      def source_location
+        if minitest_at_least_5_11?
+          minitest_result.source_location
+        else
+          minitest_result.method(minitest_result.name).source_location rescue ['unknown', -1]
+        end
       end
 
       private
 
-      attr_reader :minitest_result, :state
+      attr_reader :minitest_result
 
-      def formatted_label
-        "    #{state.formatted_label}"
+      def adjusted_class_name
+        class_name
       end
 
-      def formatted_message
-        " #{state.formatted_message(self)}"
+      def class_name
+        if minitest_at_least_5_11?
+          minitest_result.klass
+        else
+          minitest_result.class.name
+        end
       end
 
-      def formatted_time
-        time_in_s = time
-        time_with_unit =
-          case time_in_s
-          when 0...1
-            sprintf('%.0fms ', time_in_s * 1000)
-          when 1...10
-            sprintf('%.2fs ', time_in_s)
-          when 10...100
-            sprintf('%.1fs ', time_in_s)
-          when 100...10000
-            sprintf('%.0fs ', time_in_s)
-          else
-            sprintf('%.0fs', time_in_s)
-          end
-        Colorin.grey_700(time_with_unit.rjust(6))
+      def minitest_at_least_5_11?
+        @minitest_at_least_5_11 ||= Gem.loaded_specs['minitest'].version >= Gem::Version.new('5.11')
+      end
+
+      def class_separator
+        CLASS_SEPARATOR
+      end
+
+      def name_prefix
+        NAME_PREFIX
       end
 
       def rerun_command
-        relative_location = state.test_location(self).split(':').first
-        "rake TEST=#{relative_location} TESTOPTS=\"--name=#{name_for_rerun_command}\""
+        return unless (relative_location = state.test_location(self))
+
+        relative_location = relative_location.split(':').first
+
+        stem = Minitest::Bender.configuration.rerun_command_stem
+
+        if stem.include?('rake')
+          "#{stem} TEST=#{relative_location} TESTOPTS=\"--name=#{name_for_rerun_command}\""
+        else
+          "#{stem} #{relative_location} --name=#{name_for_rerun_command}"
+        end
       end
     end
   end