flatware-rspec 1.0.0 → 2.0.0.rc3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 5222039b04efa4cdd03b8a1fd99755ce7b5a796d
-  data.tar.gz: 1c4ff5266e4f837ebed84de24b33cc2a86e98138
+SHA256:
+  metadata.gz: 8c7ed50452251f91a122a1870cf0034e737bd9e245a2effca63876d352b6dfaa
+  data.tar.gz: 204dc1af97debe14947500af0bf54da88f5b5d6a8e95eda320b94ef81315be7f
 SHA512:
-  metadata.gz: 373b77233a023fb74f712c4ece31b9690b9cfa495577fb8b681d227a58a7ed312af32e0185694c852231ca68ecbf92c9cda750a14fc8d154faffdf3caac44508
-  data.tar.gz: a485e4f6d3da78f2802a143a416c0ea79211a0c5e23ad6c7b9ea68afbe916c2e4073ed35fa50b566b51f06e2ea3b2ef7e21b9d3b7d68590e6afec4c9293568d4
+  metadata.gz: 19970bba746199d0514531e83ceacdfd48a89ef3f0531ff6a9d42c5b89927e107fd429c11594f9a3e784c03014ee12911be9217188150609582e78091c4e578d
+  data.tar.gz: 8b1284ae49367edfa7ba0e5e06f95fe7df5523f9091340f0f5e08fba3e9a6332dcb6a342d075990ed78ed5fe723ae25b4604336a5cb60e16fc95918d8807cd45

data/README.md

@@ -1,45 +1,10 @@
-# Flatware [![Build Status][travis-badge]][travis] [![Code Climate][code-climate-badge]][code-climate]
+# Flatware [![Code Climate][code-climate-badge]][code-climate]
 
-[travis-badge]: https://travis-ci.org/briandunn/flatware.svg?branch=master
-[travis]: http://travis-ci.org/briandunn/flatware
 [code-climate-badge]: https://codeclimate.com/github/briandunn/flatware.png
 [code-climate]: https://codeclimate.com/github/briandunn/flatware
 
 Flatware parallelizes your test suite to significantly reduce test time.
 
-## Requirements
-
-* ZeroMQ > 4.0
-
-## Installation
-
-### ZeroMQ
-
-#### Linux Ubuntu
-
-```sh
-sudo apt-get install -qq libzmq3-dev
-```
-
-(Never you mind the 3. This package contains ZMQ version 4.)
-
-#### Mac OSX
-
-
-If you're on macOS 10.12, use the custom hashrocket ZMQ homebrew formula.
-
-```sh
-brew tap hashrocket/formulas
-brew install hashrocket/formulas/zeromq
-```
-
-The stock homebrew version will likely work on older versions of macOS.
-
-
-```sh
-brew install zeromq
-```
-
 ### Flatware
 
 Add the runners you need to your Gemfile:
@@ -73,6 +38,18 @@ To run your entire suite with the default rspec options add the `flatware-rspec`
 $ flatware rspec
 ```
 
+The rspec runner can balance worker loads, making your suite even faster.
+
+It forms balaced groups of spec files according to their last run times, if you've set `example_status_persistence_file_path` [in your RSpec config](https://relishapp.com/rspec/rspec-core/v/3-8/docs/command-line/only-failures).
+
+For this to work the configuration option must be loaded before any specs are run. The `.rspec` file is one way to achive this:
+
+    --require spec_helper
+
+But beware, if you're using ActiveRecord in your suite you'll need to avoid doing things that cause it to establish a database connection in `spec_helper.rb`. If ActiveRecord connects before flatware forks off workers, each will die messily. All of this will just work if you're following [the recomended pattern of splitting your helpers into `spec_helper` and `rails_helper`](https://github.com/rspec/rspec-rails/blob/v3.8.2/lib/generators/rspec/install/templates/spec/rails_helper.rb). Another option is to use [the configurable hooks](
+#faster-startup-with-activerecord
+).
+
 ### Options
 
 If you'd like to limit the number of forked workers, you can pass the 'w' flag:
@@ -123,9 +100,31 @@ Now you are ready to rock:
 $ flatware rspec && flatware cucumber
 ```
 
-## Planned Features
+### Faster Startup With ActiveRecord
 
-* Use heuristics to run your slowest tests first
+Flatware has a couple lifecycle callbacks that you can use to avoid booting your app
+over again on every core. One way to take advantage of this via a `spec/flatware_helper.rb` file like so:
+
+```ruby
+Flatware.configure do |conf|
+  conf.before_fork do
+    require 'rails_helper'
+
+    ActiveRecord::Base.connection.disconnect!
+  end
+
+  conf.after_fork do |test_env_number|
+    config = ActiveRecord::Base.connection_config
+
+    ActiveRecord::Base.establish_connection(
+      config.merge(
+        database: config.fetch(:database) + test_env_number.to_s
+      )
+    )
+  end
+end
+```
+Now when I run `bundle exec flatware rspec -r ./spec/flatware_helper` My app only boots once, rather than once per core.
 
 ## Design Goals
 
@@ -156,20 +155,10 @@ directory. CD there and `flatware` will be in your path so you can tinker away.
 
 ## How it works
 
-Flatware relies on a message passing system to enable concurrency.
-The main process declares a worker for each cpu in the computer. Each
-worker forks from the main process and is then assigned a portion of the
-test suite.  As the worker runs the test suite it sends progress
-messages to the main process.  These messages are collected and when
-the last worker is finished the main process provides a report on the
-collected progress messages.
+Flatware relies on a message passing system to enable concurrency. The main process forks a worker for each cpu in the computer. These workers are each given a chunk of the tests to run. The workers report back to the main process about their progress. The main process prints those progress messages. When the last worker is finished the main process prints the results.
 
 ## Resources
 
-To learn more about the messaging system that Flatware uses, take a look at the
-[excellent ZeroMQ guide][z].
-
-[z]: http://zguide.zeromq.org/page:all
 [a]: https://github.com/cucumber/aruba
 
 ## Contributing to Flatware

data/lib/flatware/rspec.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+require 'rspec/core'
+require 'rspec/expectations'
+require 'flatware/rspec/cli'
+
+module Flatware
+  module RSpec
+    require 'flatware/rspec/formatter'
+    require 'flatware/rspec/job_builder'
+
+    module_function
+
+    def extract_jobs_from_args(args, workers:)
+      JobBuilder.new(args, workers: workers).jobs
+    end
+
+    def runner
+      ::RSpec::Core::Runner.tap do |runner|
+        def runner.trap_interrupt() end
+      end
+    end
+
+    def run(job, _options = [])
+      ::RSpec.configuration.deprecation_stream = StringIO.new
+      ::RSpec.configuration.output_stream = StringIO.new
+      ::RSpec.configuration.add_formatter(Flatware::RSpec::Formatter)
+
+      runner.run(Array(job), $stderr, $stdout)
+      ::RSpec.reset # prevents duplicate runs
+    end
+  end
+end

data/lib/flatware/rspec/checkpoint.rb

@@ -1,28 +1,74 @@
-require 'flatware/rspec/examples_notification'
+# frozen_string_literal: true
+
+require 'forwardable'
+require 'flatware/rspec/marshalable'
 
 module Flatware
   module RSpec
+    ##
+    # Marshalable container for the run details from a worker.
+    # Can be added to other checkpoints to create a final run summary.
     class Checkpoint
-      attr_reader :summary, :failures_notification
+      extend Forwardable
+
+      def_delegator :summary, :failures?
+      def_delegators :failures_notification, :fully_formatted_failed_examples, :failure_notifications
+      def_delegators :pending_notification, :fully_formatted_pending_examples, :pending_examples
+
+      EVENTS = %i[
+        deprecation
+        dump_failures
+        dump_pending
+        dump_profile
+        dump_summary
+      ].freeze
+
+      attr_reader :events
+
+      def initialize(events = {})
+        @events = { deprecation: [] }.merge(events)
+      end
+
+      def self.listen_for(event, &block)
+        define_method(event) do |notification|
+          instance_exec(Marshalable.for_event(event).from_rspec(notification), &block)
+        end
+      end
 
-      def initialize(summary, failures_notification)
-        @summary, @failures_notification = summary, ExamplesNotification.new(failures_notification.failure_notifications)
+      (EVENTS - %i[deprecation]).each do |event|
+        listen_for(event) do |notification|
+          events[event] = notification
+        end
+      end
+
+      listen_for(:deprecation) do |deprecation|
+        events[:deprecation] << deprecation
       end
 
       def +(other)
-        self.class.new summary + other.summary, failures_notification + other.failures_notification
+        self.class.new(events.merge(other.events) { |_, event, other_event| event + other_event })
+      end
+
+      def summary
+        events.fetch(:dump_summary)
       end
 
-      def failures?
-        summary.failure_count > 0
+      def deprecations
+        events.fetch(:deprecation)
       end
 
-      def failure_notifications
-        failures_notification.failure_notifications
+      def profile
+        events[:dump_profile]
+      end
+
+      private
+
+      def failures_notification
+        events.fetch(:dump_failures)
       end
 
-      def fully_formatted_failed_examples(*args)
-        failures_notification.fully_formatted_failed_examples(*args)
+      def pending_notification
+        events.fetch(:dump_pending)
       end
     end
   end

data/lib/flatware/rspec/cli.rb

@@ -1,16 +1,30 @@
+# frozen_string_literal: true
+
 require 'flatware/cli'
+require 'flatware/rspec'
+require 'flatware/rspec/formatters/console'
 
 module Flatware
+  # rspec thor command
   class CLI
     worker_option
-    method_option 'dispatch-endpoint', type: :string, default: 'ipc://dispatch'
-    method_option 'sink-endpoint', type: :string, default: 'ipc://task'
-    desc "rspec [FLATWARE_OPTS]", "parallelizes rspec"
+    method_option(
+      'sink-endpoint',
+      type: :string,
+      default: 'drbunix:flatware-sink'
+    )
+    desc 'rspec [FLATWARE_OPTS]', 'parallelizes rspec'
     def rspec(*rspec_args)
       jobs = RSpec.extract_jobs_from_args rspec_args, workers: workers
+
+      formatter = Flatware::RSpec::Formatters::Console.new(
+        ::RSpec.configuration.output_stream,
+        deprecation_stream: ::RSpec.configuration.deprecation_stream
+      )
+
       Flatware.verbose = options[:log]
-      Worker.spawn count: workers, runner: RSpec, dispatch: options['dispatch-endpoint'], sink: options['sink-endpoint']
-      start_sink jobs: jobs, workers: workers, formatter: Flatware::RSpec::Formatters::Console.new($stdout, $stderr)
+      Worker.spawn count: workers, runner: RSpec, sink: options['sink-endpoint']
+      start_sink(jobs: jobs, workers: workers, formatter: formatter)
     end
   end
 end

data/lib/flatware/rspec/formatter.rb

@@ -1,50 +1,57 @@
 require 'flatware/rspec/checkpoint'
-require 'flatware/rspec/summary'
 require 'rspec/core/formatters/console_codes'
+require 'forwardable'
 
 module Flatware
   module RSpec
     ProgressMessage = Struct.new(:progress)
 
     class Formatter
-      attr_reader :summary, :output
+      extend Forwardable
+
+      def_delegators :checkpoint, *Checkpoint::EVENTS
+
+      attr_reader :output
 
       def initialize(stdout)
         @output = stdout
       end
 
-      def example_passed(example)
+      def example_passed(_example)
         send_progress :passed
       end
 
-      def example_failed(example)
+      def example_failed(_example)
         send_progress :failed
       end
 
-      def example_pending(example)
+      def example_pending(_example)
         send_progress :pending
       end
 
-      def dump_summary(summary)
-        @summary = Summary.from_notification(summary)
-      end
-
-      def dump_failures(failure_notification)
-        @failure_notification = failure_notification
-      end
-
       def close(*)
-        Sink::client.checkpoint Checkpoint.new(summary, @failure_notification)
-        @failure_notification = nil
+        Sink.client.checkpoint checkpoint
+        @checkpoint = nil
       end
 
       private
 
       def send_progress(status)
-        Sink::client.progress ProgressMessage.new status
+        Sink.client.progress ProgressMessage.new status
       end
-    end
 
-    ::RSpec::Core::Formatters.register Formatter, :example_passed, :example_failed, :example_pending, :dump_summary, :dump_failures, :close
+      def checkpoint
+        @checkpoint ||= Checkpoint.new
+      end
+
+      ::RSpec::Core::Formatters.register(
+        self,
+        *Checkpoint::EVENTS,
+        :example_passed,
+        :example_failed,
+        :example_pending,
+        :close
+      )
+    end
   end
 end

data/lib/flatware/rspec/formatters/console.rb

@@ -1,33 +1,84 @@
-module Flatware::RSpec::Formatters
-  class Console
-    attr_reader :formatter
-
-    def initialize(out, err)
-      ::RSpec::configuration.tty = true
-      ::RSpec::configuration.color = true
-      @formatter = ::RSpec::Core::Formatters::ProgressFormatter.new(out)
-    end
+require 'rspec/core'
 
-    def progress(result)
-      formatter.send(message_for(result),nil)
-    end
+module Flatware
+  module RSpec
+    module Formatters
+      class Console
+        attr_reader :progress_formatter, :out, :deprecation_stream
 
-    def summarize(checkpoints)
-      result = checkpoints.reduce :+
-      if result
-        formatter.dump_failures result
-        formatter.dump_summary result.summary
-      end
-    end
+        def initialize(out, deprecation_stream: StringIO.new)
+          @out = out
+          @deprecation_stream = deprecation_stream
+          ::RSpec.configuration.backtrace_exclusion_patterns += [%r{/lib/flatware/worker}, %r{/lib/flatware/rspec}]
+          @progress_formatter = ::RSpec::Core::Formatters::ProgressFormatter.new(out)
+        end
+
+        def progress(result)
+          progress_formatter.public_send(message_for(result), nil)
+        end
+
+        def summarize(checkpoints)
+          return if checkpoints.empty?
+
+          result = checkpoints.reduce :+
+
+          progress_formatter.dump_pending(result) if result.pending_examples.any?
+          progress_formatter.dump_failures(result)
+          dump_deprecations(result.deprecations)
+          dump_profile(result.profile) if result.profile
+          progress_formatter.dump_summary(result.summary)
+        end
+
+        def summarize_remaining(remaining)
+          progress_formatter.output.puts(colorizer.wrap(<<~MESSAGE, :detail))
 
-    private
+            The following specs weren't run:
 
-    def message_for(result)
-      {
-        passed:  :example_passed,
-        failed:  :example_failed,
-        pending: :example_pending
-      }.fetch result.progress
+            #{spec_list(remaining)}
+
+          MESSAGE
+        end
+
+        private
+
+        def dump_deprecations(deprecations)
+          formatter = ::RSpec::Core::Formatters::DeprecationFormatter.new(
+            deprecation_stream,
+            out
+          )
+
+          deprecations.each(&formatter.method(:deprecation))
+          formatter.deprecation_summary(nil)
+        end
+
+        def dump_profile(profile)
+          ::RSpec::Core::Formatters::ProfileFormatter.new(out).dump_profile(profile)
+        end
+
+        def spec_list(remaining)
+          remaining
+            .flat_map(&:id).sort.each_with_index
+            .map do |example, index|
+            format(
+              '%<index>4d) %<example>s',
+              index: index.next,
+              example: example
+            )
+          end.join("\n")
+        end
+
+        def colorizer
+          ::RSpec::Core::Formatters::ConsoleCodes
+        end
+
+        def message_for(result)
+          {
+            passed: :example_passed,
+            failed: :example_failed,
+            pending: :example_pending
+          }.fetch result.progress
+        end
+      end
     end
   end
 end

data/lib/flatware/rspec/job_builder.rb

@@ -0,0 +1,120 @@
+# frozen_string_literal: true
+
+require 'forwardable'
+
+module Flatware
+  module RSpec
+    # groups spec files into one job per worker.
+    # reads from persisted example statuses, if available,
+    # and attempts to ballence the jobs accordingly.
+    class JobBuilder
+      extend Forwardable
+      attr_reader :args, :workers, :configuration
+
+      def_delegators(
+        :configuration,
+        :files_to_run,
+        :example_status_persistence_file_path
+      )
+
+      def initialize(args, workers:)
+        @args = args
+        @workers = workers
+
+        @configuration = ::RSpec.configuration
+        configuration.define_singleton_method(:command) { 'rspec' }
+
+        ::RSpec::Core::ConfigurationOptions.new(args).configure(@configuration)
+      end
+
+      def jobs
+        timed_files, untimed_files = timed_and_untimed_files(
+          sum_seconds(load_persisted_example_statuses)
+        )
+
+        balance_jobs(
+          bucket_count: [files_to_run.size, workers].min,
+          timed_files: timed_files,
+          untimed_files: untimed_files
+        )
+      end
+
+      private
+
+      def balance_jobs(bucket_count:, timed_files:, untimed_files:)
+        balance_by(bucket_count, timed_files, &:last)
+          .map { |bucket| bucket.map(&:first) }
+          .zip(
+            round_robin(bucket_count, untimed_files)
+          ).map(&:flatten)
+          .map { |files| Job.new(files, args) }
+      end
+
+      def timed_and_untimed_files(seconds_per_file)
+        files_to_run
+          .map(&method(:normalize_path))
+          .reduce([[], []]) do |(timed, untimed), file|
+          if (time = seconds_per_file[file])
+            [timed + [[file, time]], untimed]
+          else
+            [timed, untimed + [file]]
+          end
+        end
+      end
+
+      def normalize_path(path)
+        ::RSpec::Core::Metadata.relative_path(File.expand_path(path))
+      end
+
+      def load_persisted_example_statuses
+        ::RSpec::Core::ExampleStatusPersister.load_from(
+          example_status_persistence_file_path || ''
+        )
+      end
+
+      def sum_seconds(statuses)
+        statuses.select(&passing)
+                .map { |example| parse_example(**example) }
+                .reduce({}) do |times, example|
+          times.merge(
+            example.fetch(:file_name) => example.fetch(:seconds)
+          ) do |_, old = 0, new|
+            old + new
+          end
+        end
+      end
+
+      def passing
+        ->(example) { example.fetch(:status) =~ /pass/i }
+      end
+
+      def parse_example(example_id:, run_time:, **)
+        seconds = run_time.match(/\d+(\.\d+)?/).to_s.to_f
+        file_name = ::RSpec::Core::Example.parse_id(example_id).first
+        { seconds: seconds, file_name: file_name }
+      end
+
+      def round_robin(count, items)
+        Array.new(count) { [] }.tap do |groups|
+          items.each_with_index do |entry, i|
+            groups[i % count] << entry
+          end
+        end
+      end
+
+      def balance_by(count, items, &block)
+        # find the group with the smallest sum and add it there
+        Array.new(count) { [] }.tap do |groups|
+          items
+            .sort_by(&block)
+            .reverse
+            .each do |entry|
+            groups.min_by do |group|
+              group.map(&block).reduce(:+) || 0
+            end.push(entry)
+          end
+        end
+      end
+    end
+  end
+end

data/lib/flatware/rspec/marshalable.rb

@@ -0,0 +1,22 @@
+module Flatware
+  module RSpec
+    module Marshalable
+      require 'flatware/rspec/marshalable/deprecation_notification'
+      require 'flatware/rspec/marshalable/examples_notification'
+      require 'flatware/rspec/marshalable/profile_notification'
+      require 'flatware/rspec/marshalable/summary_notification'
+
+      module_function
+
+      def for_event(event)
+        {
+          dump_pending: ExamplesNotification,
+          dump_failures: ExamplesNotification,
+          dump_profile: ProfileNotification,
+          dump_summary: SummaryNotification,
+          deprecation: DeprecationNotification
+        }.fetch(event)
+      end
+    end
+  end
+end

data/lib/flatware/rspec/marshalable/deprecation_notification.rb

@@ -0,0 +1,11 @@
+module Flatware
+  module RSpec
+    module Marshalable
+      class DeprecationNotification < ::RSpec::Core::Notifications::SeedNotification
+        def self.from_rspec(notification)
+          notification
+        end
+      end
+    end
+  end
+end

data/lib/flatware/rspec/marshalable/example.rb

@@ -0,0 +1,44 @@
+module Flatware
+  module RSpec
+    module Marshalable
+      require 'flatware/rspec/marshalable/execution_result'
+      require 'flatware/rspec/marshalable/shared_group_inclusion_backtrace'
+
+      ##
+      # a subset of the rspec example interface that can traverse drb
+      Example = Struct.new(
+        *%i[
+          execution_result
+          full_description
+          location
+          location_rerun_argument
+        ]
+      ) do
+        def initialize(rspec_example)
+          super(marshalable_execution_result(rspec_example.execution_result), *members[1..].map do |attribute|
+            rspec_example.public_send(attribute)
+          end)
+
+          @metadata = marshalable_metadata(rspec_example.metadata)
+        end
+
+        attr_reader :metadata
+
+        private
+
+        def marshalable_execution_result(execution_result)
+          ExecutionResult.from_rspec(execution_result)
+        end
+
+        def marshalable_metadata(rspec_metadata)
+          rspec_metadata.slice(:extra_failure_lines).tap do |metadata|
+            if (backtraces = rspec_metadata[:shared_group_inclusion_backtrace])
+              metadata[:shared_group_inclusion_backtrace] =
+                backtraces.map(&SharedGroupInclusionBacktrace.method(:from_rspec))
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/flatware/rspec/marshalable/example_group.rb

@@ -0,0 +1,11 @@
+module Flatware
+  module RSpec
+    module Marshalable
+      ExampleGroup = Struct.new(:location) do
+        def initialize(rspec_example_group)
+          super(rspec_example_group.location)
+        end
+      end
+    end
+  end
+end

data/lib/flatware/rspec/marshalable/examples_notification.rb

@@ -0,0 +1,30 @@
+require 'rspec/core'
+require 'flatware/rspec/marshalable/example'
+
+module Flatware
+  module RSpec
+    module Marshalable
+      class ExamplesNotification < ::RSpec::Core::Notifications::ExamplesNotification
+        Reporter = Struct.new(:examples, :failed_examples, :pending_examples) do
+          def self.from_rspec(reporter)
+            new(*members.map { |member| reporter.public_send(member).map(&Example.method(:new)) })
+          end
+
+          def +(other)
+            self.class.new(*zip(other).map { |a, b| a + b })
+          end
+        end
+
+        attr_reader :reporter
+
+        def self.from_rspec(rspec_notification)
+          new Reporter.from_rspec(rspec_notification.instance_variable_get(:@reporter))
+        end
+
+        def +(other)
+          self.class.new reporter + other.reporter
+        end
+      end
+    end
+  end
+end

data/lib/flatware/rspec/marshalable/execution_result.rb

@@ -0,0 +1,18 @@
+module Flatware
+  module RSpec
+    module Marshalable
+      require 'flatware/serialized_exception'
+      class ExecutionResult < ::RSpec::Core::Example::ExecutionResult
+        def self.from_rspec(result)
+          new.tap do |marshalable|
+            marshalable.exception = result.exception && SerializedException.from(result.exception)
+
+            %i[finished_at run_time started_at status].each do |member|
+              marshalable.public_send(:"#{member}=", result.public_send(member))
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/flatware/rspec/marshalable/profile_notification.rb

@@ -0,0 +1,30 @@
+require 'flatware/rspec/marshalable/example'
+require 'flatware/rspec/marshalable/example_group'
+
+module Flatware
+  module RSpec
+    module Marshalable
+      class ProfileNotification < ::RSpec::Core::Notifications::ProfileNotification
+        attr_reader :example_groups
+
+        def +(other)
+          self.class.new(
+            duration + other.duration,
+            examples + other.examples,
+            number_of_examples,
+            example_groups.merge(other.example_groups)
+          )
+        end
+
+        def self.from_rspec(rspec_notification)
+          new(
+            rspec_notification.duration,
+            rspec_notification.examples.map(&Example.method(:new)),
+            rspec_notification.number_of_examples,
+            rspec_notification.instance_variable_get(:@example_groups).transform_keys(&ExampleGroup.method(:new))
+          )
+        end
+      end
+    end
+  end
+end

data/lib/flatware/rspec/marshalable/shared_group_inclusion_backtrace.rb

@@ -0,0 +1,11 @@
+module Flatware
+  module RSpec
+    module Marshalable
+      class SharedGroupInclusionBacktrace < ::RSpec::Core::SharedExampleGroupInclusionStackFrame
+        def self.from_rspec(backtrace)
+          new(backtrace.shared_group_name.to_s, backtrace.inclusion_location)
+        end
+      end
+    end
+  end
+end

data/lib/flatware/rspec/marshalable/summary_notification.rb

@@ -0,0 +1,30 @@
+require 'rspec/core/notifications'
+require 'flatware/rspec/marshalable/example'
+
+module Flatware
+  module RSpec
+    module Marshalable
+      class SummaryNotification < ::RSpec::Core::Notifications::SummaryNotification
+        def +(other)
+          self.class.new(*zip(other).map { |a, b| a + b })
+        end
+
+        def failures?
+          [failure_count, errors_outside_of_examples_count].any?(&:positive?)
+        end
+
+        def self.from_rspec(summary)
+          serialized_examples = [
+            summary.examples,
+            summary.failed_examples,
+            summary.pending_examples
+          ].map do |examples|
+            examples.map(&Example.method(:new))
+          end
+
+          new(summary.duration, *serialized_examples, *summary.to_a[4..])
+        end
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flatware-rspec
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 2.0.0.rc3
 platform: ruby
 authors:
 - Brian Dunn
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2018-08-28 00:00:00.000000000 Z
+date: 2021-03-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: flatware
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 2.0.0.rc3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 2.0.0.rc3
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: '3.6'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: '3.6'
 description: A distributed rspec runner
 email: brian@hashrocket.com
 executables: []
@@ -49,35 +49,45 @@ files:
 - LICENSE.txt
 - README.md
 - lib/flatware-rspec.rb
+- lib/flatware/rspec.rb
 - lib/flatware/rspec/checkpoint.rb
 - lib/flatware/rspec/cli.rb
-- lib/flatware/rspec/example_notification.rb
-- lib/flatware/rspec/examples_notification.rb
 - lib/flatware/rspec/formatter.rb
 - lib/flatware/rspec/formatters/console.rb
-- lib/flatware/rspec/summary.rb
+- lib/flatware/rspec/job_builder.rb
+- lib/flatware/rspec/marshalable.rb
+- lib/flatware/rspec/marshalable/deprecation_notification.rb
+- lib/flatware/rspec/marshalable/example.rb
+- lib/flatware/rspec/marshalable/example_group.rb
+- lib/flatware/rspec/marshalable/examples_notification.rb
+- lib/flatware/rspec/marshalable/execution_result.rb
+- lib/flatware/rspec/marshalable/profile_notification.rb
+- lib/flatware/rspec/marshalable/shared_group_inclusion_backtrace.rb
+- lib/flatware/rspec/marshalable/summary_notification.rb
 homepage: http://github.com/briandunn/flatware
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - "~>"
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '2.6'
+  - - "<"
     - !ruby/object:Gem::Version
-      version: '2.1'
+      version: '3.1'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.12
-signing_key: 
+rubygems_version: 3.2.3
+signing_key:
 specification_version: 4
 summary: A distributed rspec runner
 test_files: []

data/lib/flatware/rspec/example_notification.rb

@@ -1,21 +0,0 @@
-require 'rspec/core/formatters/console_codes'
-
-module Flatware
-  module RSpec
-    class ExampleNotification
-      attr_reader :formatted
-      def initialize(notification)
-        @formatted = notification.fully_formatted '!', default_colorizer
-      end
-
-      def fully_formatted(i, _=nil)
-        formatted.sub '!', i.to_s
-      end
-
-      private
-      def default_colorizer
-        ::RSpec::Core::Formatters::ConsoleCodes
-      end
-    end
-  end
-end

data/lib/flatware/rspec/examples_notification.rb

@@ -1,24 +0,0 @@
-require 'flatware/rspec/example_notification'
-module Flatware
-  module RSpec
-    class ExamplesNotification
-      attr_reader :failure_notifications
-
-      def initialize(failure_notifications)
-        @failure_notifications = failure_notifications.map(&ExampleNotification.method(:new))
-      end
-
-      def +(other)
-        self.class.new failure_notifications + other.failure_notifications
-      end
-
-      def fully_formatted_failed_examples(*)
-        formatted = "\n\nFailures:\n"
-        failure_notifications.each_with_index do |failure, index|
-          formatted << failure.fully_formatted(index.next)
-        end
-        formatted
-      end
-    end
-  end
-end

data/lib/flatware/rspec/summary.rb

@@ -1,26 +0,0 @@
-require 'rspec/core/notifications'
-module Flatware
-  module RSpec
-    class Example
-      attr_reader :location_rerun_argument, :full_description
-      def initialize(rspec_example)
-        @full_description        = rspec_example.full_description
-        @location_rerun_argument = rspec_example.location_rerun_argument
-      end
-    end
-
-    class Summary < ::RSpec::Core::Notifications::SummaryNotification
-      def +(other)
-        self.class.new(*zip(other).map {|a,b| a + b})
-      end
-
-      def self.from_notification(summary)
-        serialized_examples = [summary.examples, summary.failed_examples, summary.pending_examples].map do |examples|
-          examples.map(&Example.method(:new))
-        end
-
-        new summary.duration, *serialized_examples, *summary.to_a[4..-1]
-      end
-    end
-  end
-end