knapsack_pro 8.3.3 → 9.0.0

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: knapsack_pro
 version: !ruby/object:Gem::Version
-  version: 8.3.3
+  version: 9.0.0
 platform: ruby
 authors:
 - ArturT
@@ -215,7 +215,6 @@ files:
 - README.md
 - Rakefile
 - bin/knapsack_pro
-- bin/test
 - knapsack_pro.gemspec
 - lib/knapsack_pro.rb
 - lib/knapsack_pro/adapters/base_adapter.rb
@@ -258,7 +257,6 @@ files:
 - 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
@@ -359,9 +357,8 @@ files:
 - 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/formatters/time_tracker_spec.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_spec.rb
@@ -403,6 +400,7 @@ files:
 - spec_fake/models/admin_spec.rb
 - spec_fake/models/user_spec.rb
 - spec_fake/spec_helper.rb
+- spec_time_tracker/spec_helper.rb
 - test_fake/a_test.rb
 - test_fake/b_test.rb
 homepage: https://knapsackpro.com
@@ -421,14 +419,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.7.0
+      version: 3.0.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.7
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Knapsack Pro splits tests across parallel CI nodes and ensures each parallel
   job finish work at a similar time.
@@ -481,9 +479,8 @@ test_files:
 - 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/formatters/time_tracker_spec.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_spec.rb

data/bin/test

@@ -1,15 +0,0 @@
-#!/bin/bash
-
-bundle exec ruby spec/knapsack_pro/formatters/time_tracker_specs.rb
-FORMATTERS_EXIT_CODE=$?
-
-bundle exec rspec spec
-RSPEC_EXIT_CODE=$?
-
-if [ "$FORMATTERS_EXIT_CODE" -ne "0" ]; then
-  exit $FORMATTERS_EXIT_CODE
-fi
-
-if [ "$RSPEC_EXIT_CODE" -ne "0" ]; then
-  exit $RSPEC_EXIT_CODE
-fi

data/lib/knapsack_pro/logger_wrapper.rb

@@ -1,22 +0,0 @@
-# frozen_string_literal: true
-
-module KnapsackPro
-  class LoggerWrapper
-    def initialize(logger)
-      @logger = logger
-    end
-
-    private
-
-    attr_reader :logger
-
-    LOG_LEVEL_METHODS = KnapsackPro::Config::Env::LOG_LEVELS.keys.map(&:to_sym)
-
-    def method_missing(method, *args, &block)
-      if LOG_LEVEL_METHODS.include?(method)
-        args[0] = "[knapsack_pro] #{args[0]}"
-      end
-      logger.send(method, *args, &block)
-    end
-  end
-end

data/spec/knapsack_pro/formatters/time_tracker_specs.rb

@@ -1,545 +0,0 @@
-# Named _specs.rb on purpose because it hangs if run as part of `bundle exec rspec`.
-# Use `bundle exec ruby spec/knapsack_pro/formatters/time_tracker_specs.rb` instead.
-
-require 'rspec/core'
-require 'knapsack_pro'
-require 'stringio'
-require 'tempfile'
-require_relative '../../../lib/knapsack_pro/formatters/time_tracker'
-
-class TestTimeTracker
-  def test_single_example
-    KnapsackPro::Formatters::TimeTracker.define_method(:rspec_split_by_test_example?) do |_file|
-      false
-    end
-
-    spec = <<~SPEC
-      describe "KnapsackPro::Formatters::TimeTracker" do
-        it do
-          sleep 0.1
-          expect(1).to eq 1
-        end
-      end
-    SPEC
-
-    run_specs(spec) do |spec_paths, times|
-      raise unless times.size == 1
-      raise unless times[0]["path"] == spec_paths.first
-      raise unless times[0]["time_execution"] > 0.10
-      raise unless times[0]["time_execution"] < 0.15
-    end
-  end
-
-  def test_two_files
-    KnapsackPro::Formatters::TimeTracker.define_method(:rspec_split_by_test_example?) do |_file|
-      false
-    end
-
-    spec_1 = <<~SPEC
-      describe "KnapsackPro::Formatters::TimeTracker 1" do
-        it do
-          sleep 0.1
-          expect(1).to eq 1
-        end
-      end
-    SPEC
-
-    spec_2 = <<~SPEC
-      describe "KnapsackPro::Formatters::TimeTracker 2" do
-        it do
-          sleep 0.2
-          expect(1).to eq 1
-        end
-      end
-    SPEC
-
-    run_specs([spec_1, spec_2]) do |spec_paths, times|
-      raise unless times.size == 2
-      raise unless times.first["path"] == spec_paths.first
-      raise unless times.first["time_execution"] > 0.10
-      raise unless times.first["time_execution"] < 0.15
-      raise unless times.last["path"] == spec_paths.last
-      raise unless times.last["time_execution"] > 0.20
-      raise unless times.last["time_execution"] < 0.25
-    end
-  end
-
-  def test_failing_example
-    KnapsackPro::Formatters::TimeTracker.define_method(:rspec_split_by_test_example?) do |_file|
-      false
-    end
-
-    spec = <<~SPEC
-      describe "KnapsackPro::Formatters::TimeTracker" do
-        it do
-          sleep 0.1
-          expect(1).to eq 2
-        end
-      end
-    SPEC
-
-    run_specs(spec) do |spec_paths, times|
-      raise unless times.size == 1
-      raise unless times[0]["path"] == spec_paths.first
-      raise unless times[0]["time_execution"] > 0.10
-      raise unless times[0]["time_execution"] < 0.15
-    end
-  end
-
-  def test_pending_example
-    KnapsackPro::Formatters::TimeTracker.define_method(:rspec_split_by_test_example?) do |_file|
-      false
-    end
-
-    spec = <<~SPEC
-      describe "KnapsackPro::Formatters::TimeTracker" do
-        xit do
-          sleep 0.1
-          expect(1).to eq 2
-        end
-      end
-    SPEC
-
-    run_specs(spec) do |spec_paths, times|
-      raise unless times.size == 1
-      raise unless times[0]["path"] == spec_paths.first
-      raise unless times[0]["time_execution"] == 0.0
-    end
-  end
-
-  def test_multiple_top_level_groups
-    KnapsackPro::Formatters::TimeTracker.define_method(:rspec_split_by_test_example?) do |_file|
-      false
-    end
-
-    spec = <<~SPEC
-      describe "KnapsackPro::Formatters::TimeTracker 1" do
-        it do
-          sleep 0.1
-          expect(1).to eq 1
-        end
-      end
-
-      describe "KnapsackPro::Formatters::TimeTracker 2" do
-        it do
-          sleep 0.2
-          expect(1).to eq 1
-        end
-      end
-    SPEC
-
-    run_specs(spec) do |spec_paths, times|
-      raise unless times.size == 1
-      raise unless times[0]["path"] == spec_paths.first
-      raise unless times[0]["time_execution"] > 0.30
-      raise unless times[0]["time_execution"] < 0.35
-    end
-  end
-
-  def test_rspec_split_by_test_example
-    KnapsackPro::Formatters::TimeTracker.define_method(:rspec_split_by_test_example?) do |_file|
-      true
-    end
-
-    spec = <<~SPEC
-      describe "KnapsackPro::Formatters::TimeTracker 1" do
-        it do
-          expect(1).to eq 1
-        end
-
-        it do
-          sleep 0.1
-          expect(1).to eq 1
-        end
-      end
-
-      describe "KnapsackPro::Formatters::TimeTracker 2" do
-        it do
-          sleep 0.2
-          expect(1).to eq 1
-        end
-
-        it do
-          sleep 0.3
-          expect(1).to eq 1
-        end
-      end
-    SPEC
-
-    run_specs(spec) do |spec_paths, times|
-      raise unless times.size == 4
-      spec_path = spec_paths.first
-      raise unless times.find { |time| time["path"] == "#{spec_path}[1:1]" }["time_execution"] < 0.05
-      raise unless times.find { |time| time["path"] == "#{spec_path}[1:2]" }["time_execution"] > 0.10
-      raise unless times.find { |time| time["path"] == "#{spec_path}[1:2]" }["time_execution"] < 0.15
-      raise unless times.find { |time| time["path"] == "#{spec_path}[2:1]" }["time_execution"] > 0.20
-      raise unless times.find { |time| time["path"] == "#{spec_path}[2:1]" }["time_execution"] < 0.25
-      raise unless times.find { |time| time["path"] == "#{spec_path}[2:2]" }["time_execution"] > 0.30
-      raise unless times.find { |time| time["path"] == "#{spec_path}[2:2]" }["time_execution"] < 0.35
-    end
-  end
-
-  def test_hooks
-    KnapsackPro::Formatters::TimeTracker.define_method(:rspec_split_by_test_example?) do |_file|
-      false
-    end
-
-    spec = <<~SPEC
-      describe "KnapsackPro::Formatters::TimeTracker" do
-        before(:all) do
-          sleep 0.1
-        end
-
-        before(:each) do
-          sleep 0.1
-        end
-
-        after(:each) do
-          sleep 0.1
-        end
-
-        it do
-          expect(1).to eq 1
-        end
-
-        it do
-          expect(1).to eq 1
-        end
-
-        after(:all) do
-          sleep 0.1
-        end
-      end
-    SPEC
-
-    run_specs(spec) do |spec_paths, times|
-      raise unless times.size == 1
-      raise unless times[0]["path"] == spec_paths.first
-      raise unless times[0]["time_execution"] > 0.60
-      raise unless times[0]["time_execution"] < 0.65
-    end
-  end
-
-  def test_nested_hooks
-    KnapsackPro::Formatters::TimeTracker.define_method(:rspec_split_by_test_example?) do |_file|
-      false
-    end
-
-    spec = <<~SPEC
-      describe "KnapsackPro::Formatters::TimeTracker" do
-        before(:all) do
-          sleep 0.1
-        end
-
-        after(:all) do
-          sleep 0.1
-        end
-
-        it do
-          expect(1).to eq 1
-        end
-
-        describe do
-          before(:all) do
-            sleep 0.1
-          end
-
-          after(:all) do
-            sleep 0.1
-          end
-
-          it do
-            expect(1).to eq 1
-          end
-        end
-
-        describe do
-          before(:all) do
-            sleep 0.1
-          end
-
-          after(:all) do
-            sleep 0.1
-          end
-
-          it do
-            expect(1).to eq 1
-          end
-        end
-      end
-    SPEC
-
-    run_specs(spec) do |spec_paths, times|
-      raise unless times.size == 1
-      raise unless times[0]["path"] == spec_paths.first
-      raise unless times[0]["time_execution"] > 0.60
-      raise unless times[0]["time_execution"] < 0.65
-    end
-  end
-
-  def test_hooks_with_rspec_split_by_test_example
-    KnapsackPro::Formatters::TimeTracker.define_method(:rspec_split_by_test_example?) do |_file|
-      true
-    end
-
-    spec = <<~SPEC
-      describe "KnapsackPro::Formatters::TimeTracker" do
-        before(:all) do
-          sleep 0.1
-        end
-
-        before(:each) do
-          sleep 0.1
-        end
-
-        after(:each) do
-          sleep 0.1
-        end
-
-        it do
-          expect(1).to eq 1
-        end
-
-        it do
-          expect(1).to eq 1
-        end
-
-        after(:all) do
-          sleep 0.1
-        end
-      end
-    SPEC
-
-    run_specs(spec) do |spec_paths, times|
-      raise unless times.size == 2
-      spec_path = spec_paths.first
-      raise unless times.find { |time| time["path"] == "#{spec_path}[1:1]" }["time_execution"] > 0.40
-      raise unless times.find { |time| time["path"] == "#{spec_path}[1:1]" }["time_execution"] < 0.45
-      raise unless times.find { |time| time["path"] == "#{spec_path}[1:2]" }["time_execution"] > 0.40
-      raise unless times.find { |time| time["path"] == "#{spec_path}[1:2]" }["time_execution"] < 0.45
-    end
-  end
-
-  def test_nested_hooks_with_rspec_split_by_test_example
-    KnapsackPro::Formatters::TimeTracker.define_method(:rspec_split_by_test_example?) do |_file|
-      true
-    end
-
-    spec = <<~SPEC
-      describe "KnapsackPro::Formatters::TimeTracker" do
-        before(:all) do
-          sleep 0.1
-        end
-
-        after(:all) do
-          sleep 0.1
-        end
-
-        it do
-          expect(1).to eq 1
-        end
-
-        describe do
-          before(:all) do
-            sleep 0.1
-          end
-
-          after(:all) do
-            sleep 0.1
-          end
-
-          it do
-            expect(1).to eq 1
-          end
-        end
-
-        describe do
-          before(:all) do
-            sleep 0.1
-          end
-
-          after(:all) do
-            sleep 0.1
-          end
-
-          it do
-            expect(1).to eq 1
-          end
-        end
-      end
-    SPEC
-
-    run_specs(spec) do |spec_paths, times|
-      raise unless times.size == 3
-      spec_path = spec_paths.first
-      raise unless times.find { |time| time["path"] == "#{spec_path}[1:1]" }["time_execution"] > 0.20
-      raise unless times.find { |time| time["path"] == "#{spec_path}[1:1]" }["time_execution"] < 0.25
-      raise unless times.find { |time| time["path"] == "#{spec_path}[1:2:1]" }["time_execution"] > 0.40
-      raise unless times.find { |time| time["path"] == "#{spec_path}[1:2:1]" }["time_execution"] < 0.45
-      raise unless times.find { |time| time["path"] == "#{spec_path}[1:3:1]" }["time_execution"] > 0.40
-      raise unless times.find { |time| time["path"] == "#{spec_path}[1:3:1]" }["time_execution"] < 0.45
-    end
-  end
-
-
-  def test_unknown_path
-    KnapsackPro::Formatters::TimeTracker.class_eval do
-      alias_method :original_file_path_for, :file_path_for
-
-      define_method(:file_path_for) do |_example|
-        ""
-      end
-    end
-
-    spec = <<~SPEC
-      describe "KnapsackPro::Formatters::TimeTracker" do
-        it do
-          expect(1).to eq 1
-        end
-      end
-    SPEC
-
-    run_specs(spec) do |spec_paths, times|
-      raise unless times.size == 1
-      raise unless times[0]["path"] == spec_paths.first
-      raise unless times[0]["time_execution"] == 0.0
-    end
-
-  ensure
-    KnapsackPro::Formatters::TimeTracker.class_eval do
-      undef :file_path_for
-      alias_method :file_path_for, :original_file_path_for
-    end
-  end
-
-  def test_empty_group
-    KnapsackPro::Formatters::TimeTracker.define_method(:rspec_split_by_test_example?) do |_file|
-      false
-    end
-
-    spec = <<~SPEC
-      describe "KnapsackPro::Formatters::TimeTracker" do
-      end
-    SPEC
-
-    run_specs(spec) do |spec_paths, times|
-      raise unless times.size == 1
-      raise unless times[0]["path"] == spec_paths.first
-      raise unless times[0]["time_execution"] == 0.0
-    end
-  end
-
-  def test_duration
-    KnapsackPro::Formatters::TimeTracker.define_method(:rspec_split_by_test_example?) do |_file|
-      false
-    end
-
-    spec = <<~SPEC
-      describe "KnapsackPro::Formatters::TimeTracker" do
-        it do
-          expect(1).to eq 1
-        end
-      end
-    SPEC
-
-    run_specs(spec) do |_, _, time_tracker|
-      raise unless time_tracker.duration > 0.0
-    end
-  end
-
-  def test_unexecuted_test_files
-    KnapsackPro::Formatters::TimeTracker.define_method(:rspec_split_by_test_example?) do |_file|
-      false
-    end
-
-    spec = <<~SPEC
-      describe "KnapsackPro::Formatters::TimeTracker" do
-        xit do
-        end
-      end
-    SPEC
-
-    run_specs(spec) do |spec_paths, _, time_tracker|
-      unexecuted_test_files = ["foo_spec.rb", "bar_spec.rb"]
-      # Need to filter because RSpec keeps accumulating state.
-      time_tracker.scheduled_paths = spec_paths + unexecuted_test_files
-      files = time_tracker
-        .unexecuted_test_files
-        .filter { |file| spec_paths.include?(file) || unexecuted_test_files.include?(file) }
-
-      raise unless files.size == 3
-    end
-  end
-
-  def test_subset
-    KnapsackPro::Formatters::TimeTracker.define_method(:rspec_split_by_test_example?) do |_file|
-      false
-    end
-
-    spec = <<~SPEC
-      describe "KnapsackPro::Formatters::TimeTracker" do
-        it "works" do
-          sleep 0.1
-          expect(1).to eq 1
-        end
-      end
-    SPEC
-
-    run_specs(spec) do |spec_paths, times, time_tracker|
-      # Need to filter because RSpec keeps accumulating state.
-      files = time_tracker
-        .batch
-        .filter { |file| spec_paths.include?(file["path"]) }
-
-      raise unless files.size == 1
-      raise unless files[0]["path"] == spec_paths.first
-      raise unless files[0]["time_execution"] > 0.10
-      raise unless files[0]["time_execution"] < 0.15
-    end
-  end
-
-  private
-
-  def run_specs(specs)
-    files = Array(specs).map.with_index do |spec, i|
-      file = Tempfile.new(["tmp_time_tracker_#{i}", "_spec.rb"], "./spec/knapsack_pro/formatters/")
-      file.write(spec)
-      file.rewind
-      file
-    end
-
-    paths = files.map(&:path).map { _1.sub("./", "") }
-
-    options = ::RSpec::Core::ConfigurationOptions.new([
-      "--format", KnapsackPro::Formatters::TimeTracker.to_s,
-      *paths,
-    ])
-    runner = ::RSpec::Core::Runner.new(options)
-    runner.run(StringIO.new, StringIO.new)
-
-    time_tracker = runner.configuration.formatters.find { |f| f.class.to_s == KnapsackPro::Formatters::TimeTracker.to_s }
-    # Need to filter because RSpec keeps accumulating state.
-    time_tracker.scheduled_paths = paths
-    times = time_tracker
-      .queue
-      .sort_by { |time| time["path"] }
-      .filter do |time|
-        paths.any? { |path| time["path"].start_with?(path) }
-      end
-    yield(paths, times, time_tracker)
-
-  ensure
-    # Need to reset because RSpec keeps reusing the same instance.
-    time_tracker.instance_variable_set(:@queue, {}) if time_tracker
-    time_tracker.instance_variable_set(:@started, time_tracker.send(:now)) if time_tracker
-  end
-end
-
-TestTimeTracker
-  .instance_methods
-  .filter { |method| method.to_s.start_with?("test_") }
-  .shuffle
-  .each do |method|
-    puts method
-    TestTimeTracker.new.public_send(method)
-  end

data/spec/knapsack_pro/logger_wrapper_spec.rb

@@ -1,21 +0,0 @@
-describe KnapsackPro::LoggerWrapper do
-  let(:io) { StringIO.new }
-  let(:logger) { ::Logger.new(io) }
-  let(:logger_wrapper) { described_class.new(logger) }
-
-  subject { io.string }
-
-  [:debug, :info, :warn, :error, :fatal].each do |log_level|
-    describe "##{log_level}" do
-      before { logger_wrapper.public_send(log_level, 'Test message') }
-
-      it { should include "[knapsack_pro] Test message\n" }
-    end
-  end
-
-  it 'can set logger level via logger wrapper' do
-    logger_wrapper.level = ::Logger::INFO
-
-    expect(logger_wrapper.level).to eq 1
-  end
-end