knapsack_pro 6.0.4 → 7.0.1

data/spec/knapsack_pro/runners/queue/cucumber_runner_spec.rb

@@ -24,7 +24,7 @@ describe KnapsackPro::Runners::Queue::CucumberRunner do
       expect(described_class).to receive(:new).with(KnapsackPro::Adapters::CucumberAdapter).and_return(runner)
     end
 
-    context 'when args provided' do
+    context 'when args are provided' do
       let(:args) { '--retry 5 --no-strict-flaky' }
 
       it do
@@ -39,7 +39,7 @@ describe KnapsackPro::Runners::Queue::CucumberRunner do
           can_initialize_queue: true,
           args: args,
           exitstatus: 0,
-          all_test_file_paths: [],
+          node_test_file_paths: [],
         }
         expect(described_class).to receive(:handle_signal!)
         expect(described_class).to receive(:run_tests).with(accumulator).and_return(expected_accumulator)
@@ -50,7 +50,7 @@ describe KnapsackPro::Runners::Queue::CucumberRunner do
       end
     end
 
-    context 'when args not provided' do
+    context 'when args are not provided' do
       let(:args) { nil }
 
       it do
@@ -65,7 +65,7 @@ describe KnapsackPro::Runners::Queue::CucumberRunner do
           can_initialize_queue: true,
           args: nil,
           exitstatus: 0,
-          all_test_file_paths: [],
+          node_test_file_paths: [],
         }
         expect(described_class).to receive(:handle_signal!)
         expect(described_class).to receive(:run_tests).with(accumulator).and_return(expected_accumulator)
@@ -85,21 +85,21 @@ describe KnapsackPro::Runners::Queue::CucumberRunner do
     let(:can_initialize_queue) { double(:can_initialize_queue) }
     let(:args) { '--retry 5 --no-strict-flaky' }
     let(:exitstatus) { 0 }
-    let(:all_test_file_paths) { [] }
+    let(:node_test_file_paths) { [] }
     let(:accumulator) do
       {
         runner: runner,
         can_initialize_queue: can_initialize_queue,
         args: args,
         exitstatus: exitstatus,
-        all_test_file_paths: all_test_file_paths,
+        node_test_file_paths: node_test_file_paths,
       }
     end
 
     subject { described_class.run_tests(accumulator) }
 
     before do
-      expect(runner).to receive(:test_file_paths).with(can_initialize_queue: can_initialize_queue, executed_test_files: all_test_file_paths).and_return(test_file_paths)
+      expect(runner).to receive(:test_file_paths).with(can_initialize_queue: can_initialize_queue, executed_test_files: node_test_file_paths).and_return(test_file_paths)
     end
 
     context 'when test files exist' do
@@ -129,7 +129,7 @@ describe KnapsackPro::Runners::Queue::CucumberRunner do
         allow(child_status).to receive(:exitstatus).and_return(exitstatus)
       end
 
-      context 'when system process finished its work (exited)' do
+      context 'when system process finished (exited)' do
         let(:process_exited) { true }
 
         context 'when tests are passing' do
@@ -142,7 +142,7 @@ describe KnapsackPro::Runners::Queue::CucumberRunner do
               can_initialize_queue: false,
               args: args,
               exitstatus: exitstatus,
-              all_test_file_paths: test_file_paths,
+              node_test_file_paths: test_file_paths,
             })
           end
         end
@@ -157,13 +157,13 @@ describe KnapsackPro::Runners::Queue::CucumberRunner do
               can_initialize_queue: false,
               args: args,
               exitstatus: 1, # tests failed
-              all_test_file_paths: test_file_paths,
+              node_test_file_paths: test_file_paths,
             })
           end
         end
       end
 
-      context "when system process didn't finish its work (hasn't exited)" do
+      context 'when system process did not finish (it has not exited)' do
         let(:process_exited) { false }
 
         it 'raises an error' do
@@ -172,11 +172,11 @@ describe KnapsackPro::Runners::Queue::CucumberRunner do
       end
     end
 
-    context "when test files don't exist" do
+    context 'when test files do not exist' do
       let(:test_file_paths) { [] }
 
-      context 'when all_test_file_paths exist' do
-        let(:all_test_file_paths) { ['features/a.feature'] }
+      context 'when node_test_file_paths exists' do
+        let(:node_test_file_paths) { ['features/a.feature'] }
 
         it 'returns exit code 0' do
           expect(KnapsackPro::Adapters::CucumberAdapter).to receive(:verify_bind_method_called)
@@ -191,8 +191,8 @@ describe KnapsackPro::Runners::Queue::CucumberRunner do
         end
       end
 
-      context "when all_test_file_paths don't exist" do
-        let(:all_test_file_paths) { [] }
+      context 'when node_test_file_paths do not exist' do
+        let(:node_test_file_paths) { [] }
 
         it 'returns exit code 0' do
           expect(KnapsackPro::Hooks::Queue).to receive(:call_after_queue)

data/spec/knapsack_pro/runners/queue/minitest_runner_spec.rb

@@ -26,7 +26,7 @@ describe KnapsackPro::Runners::Queue::MinitestRunner do
       expect($LOAD_PATH).to receive(:unshift).with(test_dir)
     end
 
-    context 'when args provided' do
+    context 'when args are provided' do
       let(:args) { '--verbose --pride' }
 
       it do
@@ -41,7 +41,7 @@ describe KnapsackPro::Runners::Queue::MinitestRunner do
           can_initialize_queue: true,
           args: ['--verbose', '--pride'],
           exitstatus: 0,
-          all_test_file_paths: [],
+          node_test_file_paths: [],
         }
         expect(described_class).to receive(:handle_signal!)
         expect(described_class).to receive(:run_tests).with(accumulator).and_return(expected_accumulator)
@@ -52,7 +52,7 @@ describe KnapsackPro::Runners::Queue::MinitestRunner do
       end
     end
 
-    context 'when args not provided' do
+    context 'when args are not provided' do
       let(:args) { nil }
 
       it do
@@ -67,7 +67,7 @@ describe KnapsackPro::Runners::Queue::MinitestRunner do
           can_initialize_queue: true,
           args: [],
           exitstatus: 0,
-          all_test_file_paths: [],
+          node_test_file_paths: [],
         }
         expect(described_class).to receive(:handle_signal!)
         expect(described_class).to receive(:run_tests).with(accumulator).and_return(expected_accumulator)
@@ -84,21 +84,21 @@ describe KnapsackPro::Runners::Queue::MinitestRunner do
     let(:can_initialize_queue) { double(:can_initialize_queue) }
     let(:args) { ['--verbose', '--pride'] }
     let(:exitstatus) { 0 }
-    let(:all_test_file_paths) { [] }
+    let(:node_test_file_paths) { [] }
     let(:accumulator) do
       {
         runner: runner,
         can_initialize_queue: can_initialize_queue,
         args: args,
         exitstatus: exitstatus,
-        all_test_file_paths: all_test_file_paths,
+        node_test_file_paths: node_test_file_paths,
       }
     end
 
     subject { described_class.run_tests(accumulator) }
 
     before do
-      expect(runner).to receive(:test_file_paths).with(can_initialize_queue: can_initialize_queue, executed_test_files: all_test_file_paths).and_return(test_file_paths)
+      expect(runner).to receive(:test_file_paths).with(can_initialize_queue: can_initialize_queue, executed_test_files: node_test_file_paths).and_return(test_file_paths)
     end
 
     context 'when test files exist' do
@@ -145,7 +145,7 @@ describe KnapsackPro::Runners::Queue::MinitestRunner do
             can_initialize_queue: false,
             args: args,
             exitstatus: exitstatus,
-            all_test_file_paths: test_file_paths,
+            node_test_file_paths: test_file_paths,
           })
         end
       end
@@ -160,17 +160,17 @@ describe KnapsackPro::Runners::Queue::MinitestRunner do
             can_initialize_queue: false,
             args: args,
             exitstatus: 1, # tests failed
-            all_test_file_paths: test_file_paths,
+            node_test_file_paths: test_file_paths,
           })
         end
       end
     end
 
-    context "when test files don't exist" do
+    context 'when test files do not exist' do
       let(:test_file_paths) { [] }
 
-      context 'when all_test_file_paths exist' do
-        let(:all_test_file_paths) { ['a_test.rb'] }
+      context 'when node_test_file_paths exists' do
+        let(:node_test_file_paths) { ['a_test.rb'] }
 
         it 'returns exit code 0' do
           expect(KnapsackPro::Adapters::MinitestAdapter).to receive(:verify_bind_method_called)
@@ -185,8 +185,8 @@ describe KnapsackPro::Runners::Queue::MinitestRunner do
         end
       end
 
-      context "when all_test_file_paths don't exist" do
-        let(:all_test_file_paths) { [] }
+      context 'when node_test_file_paths do not exist' do
+        let(:node_test_file_paths) { [] }
 
         it 'returns exit code 0' do
           expect(KnapsackPro::Hooks::Queue).to receive(:call_after_queue)

data/spec/knapsack_pro_spec.rb

@@ -24,7 +24,7 @@ describe KnapsackPro do
           })
 
           expect(Logger).to receive(:new).with('log/knapsack_pro_node_1.log').and_return(logger)
-          expect(logger).to receive(:level=).with(Logger::DEBUG)
+          expect(logger).to receive(:level=).with(Logger::INFO)
           expect(KnapsackPro::LoggerWrapper).to receive(:new).with(logger).and_return(logger_wrapper)
         end
 
@@ -38,7 +38,7 @@ describe KnapsackPro do
           })
 
           expect(Logger).to receive(:new).with('log/knapsack_pro_node_0.log').and_return(logger)
-          expect(logger).to receive(:level=).with(Logger::DEBUG)
+          expect(logger).to receive(:level=).with(Logger::INFO)
           expect(KnapsackPro::LoggerWrapper).to receive(:new).with(logger).and_return(logger_wrapper)
         end
 
@@ -51,7 +51,7 @@ describe KnapsackPro do
 
       before do
         expect(Logger).to receive(:new).with(STDOUT).and_return(logger)
-        expect(logger).to receive(:level=).with(Logger::DEBUG)
+        expect(logger).to receive(:level=).with(Logger::INFO)
         expect(KnapsackPro::LoggerWrapper).to receive(:new).with(logger).and_return(logger_wrapper)
       end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: knapsack_pro
 version: !ruby/object:Gem::Version
-  version: 6.0.4
+  version: 7.0.1
 platform: ruby
 authors:
 - ArturT
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-01-09 00:00:00.000000000 Z
+date: 2024-03-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -178,10 +178,10 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.9.4
-description: Run tests in parallel across CI server nodes based on tests execution
-  time. Split tests in a dynamic way to ensure parallel jobs are done at a similar
-  time. Thanks to that your CI build time is as fast as possible. It works with many
-  CI providers.
+description: Knapsack Pro wraps your current test runner(s) and works with your existing
+  CI infrastructure to parallelize tests optimally. It dynamically splits your tests
+  based on up-to-date test execution data. It's designed from the ground up for CI
+  and supports all of them.
 email:
 - support@knapsackpro.com
 executables:
@@ -193,6 +193,7 @@ files:
 - ".github/assets/install-button.png"
 - ".github/assets/knapsack-diamonds.png"
 - ".github/dependabot.yml"
+- ".github/pull_request_template.md"
 - ".gitignore"
 - ".rspec"
 - CHANGELOG.md
@@ -240,15 +241,15 @@ files:
 - lib/knapsack_pro/crypto/decryptor.rb
 - lib/knapsack_pro/crypto/digestor.rb
 - lib/knapsack_pro/crypto/encryptor.rb
+- lib/knapsack_pro/extensions/rspec_extension.rb
 - lib/knapsack_pro/formatters/rspec_json_formatter.rb
-- lib/knapsack_pro/formatters/rspec_queue_profile_formatter_extension.rb
-- lib/knapsack_pro/formatters/rspec_queue_summary_formatter.rb
 - lib/knapsack_pro/formatters/time_tracker.rb
 - lib/knapsack_pro/formatters/time_tracker_fetcher.rb
 - lib/knapsack_pro/hooks/queue.rb
 - lib/knapsack_pro/logger_wrapper.rb
 - lib/knapsack_pro/mask_string.rb
 - lib/knapsack_pro/presenter.rb
+- lib/knapsack_pro/pure/queue/rspec_pure.rb
 - lib/knapsack_pro/queue_allocator.rb
 - lib/knapsack_pro/queue_allocator_builder.rb
 - lib/knapsack_pro/railtie.rb
@@ -301,6 +302,8 @@ files:
 - spec/fixtures/vcr_cassettes/api/v1/build_subsets/create/success.yml
 - spec/integration/api/build_distributions_subset_spec.rb
 - spec/integration/api/build_subsets_create_spec.rb
+- spec/integration/runners/queue/rspec_runner.rb
+- spec/integration/runners/queue/rspec_runner_spec.rb
 - spec/knapsack_pro/adapters/base_adapter_spec.rb
 - spec/knapsack_pro/adapters/cucumber_adapter_spec.rb
 - spec/knapsack_pro/adapters/minitest_adapter_spec.rb
@@ -337,10 +340,12 @@ files:
 - spec/knapsack_pro/crypto/decryptor_spec.rb
 - spec/knapsack_pro/crypto/digestor_spec.rb
 - spec/knapsack_pro/crypto/encryptor_spec.rb
+- spec/knapsack_pro/formatters/time_tracker_fetcher_spec.rb
 - spec/knapsack_pro/formatters/time_tracker_specs.rb
 - spec/knapsack_pro/hooks/queue_spec.rb
 - spec/knapsack_pro/logger_wrapper_spec.rb
 - spec/knapsack_pro/presenter_spec.rb
+- spec/knapsack_pro/pure/queue/rspec_pure_spec.rb
 - spec/knapsack_pro/queue_allocator_builder_spec.rb
 - spec/knapsack_pro/queue_allocator_spec.rb
 - spec/knapsack_pro/report_spec.rb
@@ -354,7 +359,6 @@ files:
 - spec/knapsack_pro/runners/queue/base_runner_spec.rb
 - spec/knapsack_pro/runners/queue/cucumber_runner_spec.rb
 - spec/knapsack_pro/runners/queue/minitest_runner_spec.rb
-- spec/knapsack_pro/runners/queue/rspec_runner_spec.rb
 - spec/knapsack_pro/runners/rspec_runner_spec.rb
 - spec/knapsack_pro/runners/spinach_runner_spec.rb
 - spec/knapsack_pro/runners/test_unit_runner_spec.rb
@@ -401,14 +405,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.7.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.10
+rubygems_version: 3.5.6
 signing_key:
 specification_version: 4
 summary: Knapsack Pro splits tests across parallel CI nodes and ensures each parallel
@@ -421,6 +425,8 @@ test_files:
 - spec/fixtures/vcr_cassettes/api/v1/build_subsets/create/success.yml
 - spec/integration/api/build_distributions_subset_spec.rb
 - spec/integration/api/build_subsets_create_spec.rb
+- spec/integration/runners/queue/rspec_runner.rb
+- spec/integration/runners/queue/rspec_runner_spec.rb
 - spec/knapsack_pro/adapters/base_adapter_spec.rb
 - spec/knapsack_pro/adapters/cucumber_adapter_spec.rb
 - spec/knapsack_pro/adapters/minitest_adapter_spec.rb
@@ -457,10 +463,12 @@ test_files:
 - spec/knapsack_pro/crypto/decryptor_spec.rb
 - spec/knapsack_pro/crypto/digestor_spec.rb
 - spec/knapsack_pro/crypto/encryptor_spec.rb
+- spec/knapsack_pro/formatters/time_tracker_fetcher_spec.rb
 - spec/knapsack_pro/formatters/time_tracker_specs.rb
 - spec/knapsack_pro/hooks/queue_spec.rb
 - spec/knapsack_pro/logger_wrapper_spec.rb
 - spec/knapsack_pro/presenter_spec.rb
+- spec/knapsack_pro/pure/queue/rspec_pure_spec.rb
 - spec/knapsack_pro/queue_allocator_builder_spec.rb
 - spec/knapsack_pro/queue_allocator_spec.rb
 - spec/knapsack_pro/report_spec.rb
@@ -474,7 +482,6 @@ test_files:
 - spec/knapsack_pro/runners/queue/base_runner_spec.rb
 - spec/knapsack_pro/runners/queue/cucumber_runner_spec.rb
 - spec/knapsack_pro/runners/queue/minitest_runner_spec.rb
-- spec/knapsack_pro/runners/queue/rspec_runner_spec.rb
 - spec/knapsack_pro/runners/rspec_runner_spec.rb
 - spec/knapsack_pro/runners/spinach_runner_spec.rb
 - spec/knapsack_pro/runners/test_unit_runner_spec.rb

data/lib/knapsack_pro/formatters/rspec_queue_profile_formatter_extension.rb

@@ -1,58 +0,0 @@
-# frozen_string_literal: true
-
-RSpec::Support.require_rspec_core('formatters/profile_formatter')
-
-module KnapsackPro
-  module Formatters
-    module RSpecQueueProfileFormatterExtension
-      def self.print_summary
-        return unless KnapsackPro::Config::Env.modify_default_rspec_formatters?
-        ::RSpec::Core::Formatters::ProfileFormatter.print_profile_summary
-      end
-
-      def initialize(output)
-        @output = output
-        self.class.registered_output = output
-      end
-
-      def dump_profile(profile)
-        self.class.most_recent_profile = profile
-      end
-    end
-  end
-end
-
-if KnapsackPro::Config::Env.modify_default_rspec_formatters?
-  class RSpec::Core::Formatters::ProfileFormatter
-    prepend KnapsackPro::Formatters::RSpecQueueProfileFormatterExtension
-
-    def self.registered_output=(output)
-      @registered_output = {
-        ENV['KNAPSACK_PRO_QUEUE_ID'] => output
-      }
-    end
-
-    def self.registered_output
-      @registered_output ||= {}
-      @registered_output[ENV['KNAPSACK_PRO_QUEUE_ID']]
-    end
-
-    def self.most_recent_profile=(profile)
-      @most_recent_profile = {
-        ENV['KNAPSACK_PRO_QUEUE_ID'] => profile
-      }
-    end
-
-    def self.most_recent_profile
-      @most_recent_profile ||= {}
-      @most_recent_profile[ENV['KNAPSACK_PRO_QUEUE_ID']] || []
-    end
-
-    def self.print_profile_summary
-      return unless registered_output
-      profile_formatter = new(registered_output)
-      profile_formatter.send(:dump_profile_slowest_examples,  most_recent_profile)
-      profile_formatter.send(:dump_profile_slowest_example_groups, most_recent_profile)
-    end
-  end
-end

data/lib/knapsack_pro/formatters/rspec_queue_summary_formatter.rb

@@ -1,145 +0,0 @@
-# frozen_string_literal: true
-
-require_relative './time_tracker_fetcher'
-
-RSpec::Support.require_rspec_core('formatters/base_formatter')
-RSpec::Support.require_rspec_core('formatters/base_text_formatter')
-
-module KnapsackPro
-  module Formatters
-    module RSpecHideFailuresAndPendingExtension
-      def dump_failures(notification); end
-      def dump_pending(notification); end
-      def dump_summary(summary); end
-    end
-
-    class RSpecQueueSummaryFormatter < ::RSpec::Core::Formatters::BaseFormatter
-      ::RSpec::Core::Formatters.register self, :dump_summary, :dump_failures, :dump_pending
-
-      def self.registered_output=(output)
-        @registered_output = {
-          ENV['KNAPSACK_PRO_QUEUE_ID'] => output
-        }
-      end
-
-      def self.registered_output
-        @registered_output[ENV['KNAPSACK_PRO_QUEUE_ID']]
-      end
-
-      def self.most_recent_failures_summary=(fully_formatted_failed_examples)
-        @most_recent_failures_summary = {
-          ENV['KNAPSACK_PRO_QUEUE_ID'] => fully_formatted_failed_examples
-        }
-      end
-
-      def self.most_recent_failures_summary
-        @most_recent_failures_summary ||= {}
-        @most_recent_failures_summary[ENV['KNAPSACK_PRO_QUEUE_ID']] || []
-      end
-
-      def self.most_recent_pending=(fully_formatted_pending_examples)
-        @most_recent_pending = {
-          ENV['KNAPSACK_PRO_QUEUE_ID'] => fully_formatted_pending_examples
-        }
-      end
-
-      def self.most_recent_pending
-        @most_recent_pending ||= {}
-        @most_recent_pending[ENV['KNAPSACK_PRO_QUEUE_ID']] || []
-      end
-
-      def self.most_recent_summary=(fully_formatted)
-        @most_recent_summary = {
-          ENV['KNAPSACK_PRO_QUEUE_ID'] => fully_formatted
-        }
-      end
-
-      def self.most_recent_summary
-        @most_recent_summary ||= {}
-        @most_recent_summary[ENV['KNAPSACK_PRO_QUEUE_ID']] || []
-      end
-
-      def self.print_summary
-        registered_output.puts('Knapsack Pro Queue finished!')
-        registered_output.puts('')
-
-        unless most_recent_pending.empty?
-          registered_output.puts('All pending tests on this CI node:')
-          registered_output.puts(most_recent_pending)
-          registered_output.puts('')
-        end
-
-        unless most_recent_failures_summary.empty?
-          registered_output.puts('All failed tests on this CI node:')
-          registered_output.puts(most_recent_failures_summary)
-          registered_output.puts('')
-        end
-
-        registered_output.puts(most_recent_summary)
-      end
-
-      def self.print_exit_summary(all_test_file_paths)
-        registered_output.puts('Knapsack Pro Queue exited/aborted!')
-        registered_output.puts('')
-
-        time_tracker = KnapsackPro::Formatters::TimeTrackerFetcher.call
-        unexecuted_test_files = time_tracker&.unexecuted_test_files(all_test_file_paths) || []
-        unless unexecuted_test_files.empty?
-          registered_output.puts('Unexecuted tests on this CI node:')
-          registered_output.puts(unexecuted_test_files)
-          registered_output.puts('')
-        end
-
-        unless most_recent_pending.empty?
-          registered_output.puts('All pending tests on this CI node:')
-          registered_output.puts(most_recent_pending)
-          registered_output.puts('')
-        end
-
-        unless most_recent_failures_summary.empty?
-          registered_output.puts('All failed tests on this CI node:')
-          registered_output.puts(most_recent_failures_summary)
-          registered_output.puts('')
-        end
-
-        registered_output.puts(most_recent_summary)
-      end
-
-      def initialize(output)
-        super
-        self.class.registered_output = output
-      end
-
-      def dump_failures(notification)
-        return if notification.failure_notifications.empty?
-        self.class.most_recent_failures_summary = notification.fully_formatted_failed_examples
-      end
-
-      def dump_pending(notification)
-        return if notification.pending_examples.empty?
-        self.class.most_recent_pending = notification.fully_formatted_pending_examples
-      end
-
-      def dump_summary(summary)
-        colorizer = ::RSpec::Core::Formatters::ConsoleCodes
-        duration = KnapsackPro::Formatters::TimeTrackerFetcher.call.duration
-        formatted_duration = ::RSpec::Core::Formatters::Helpers.format_duration(duration)
-
-        formatted = "\nFinished in #{formatted_duration}\n" \
-          "#{summary.colorized_totals_line(colorizer)}\n"
-
-        unless summary.failed_examples.empty?
-          formatted += (summary.colorized_rerun_commands(colorizer) + "\n")
-        end
-
-        self.class.most_recent_summary = formatted
-      end
-    end
-  end
-end
-
-if KnapsackPro::Config::Env.modify_default_rspec_formatters?
-  class RSpec::Core::Formatters::BaseTextFormatter
-    prepend KnapsackPro::Formatters::RSpecHideFailuresAndPendingExtension
-  end
-end