knapsack_pro 9.0.0 → 9.1.0

data/spec/knapsack_pro/test_file_finder_spec.rb

@@ -1,176 +0,0 @@
-describe KnapsackPro::TestFileFinder do
-  describe '.call' do
-    let(:test_file_pattern) { double }
-    let(:test_files) { double }
-
-    subject { described_class.call(test_file_pattern) }
-
-    before do
-      test_file_finder = instance_double(described_class, call: test_files)
-      expect(described_class).to receive(:new).with(test_file_pattern, true).and_return(test_file_finder)
-    end
-
-    it { should eq test_files }
-  end
-
-  describe '.slow_test_files_by_pattern' do
-    let(:adapter_class) { double }
-
-    subject { described_class.slow_test_files_by_pattern(adapter_class) }
-
-    before do
-      expect(KnapsackPro::Config::Env).to receive(:slow_test_file_pattern).at_least(1).and_return(slow_test_file_pattern)
-    end
-
-    context 'when slow_test_file_pattern is present' do
-      let(:slow_test_file_pattern) { double }
-      let(:test_file_entities) do
-        [
-          { 'path' => 'a_spec.rb' },
-          { 'path' => 'b_spec.rb' },
-          { 'path' => 'c_spec.rb' },
-        ]
-      end
-      let(:slow_test_file_entities) do
-        [
-          { 'path' => 'b_spec.rb' },
-          { 'path' => 'c_spec.rb' },
-          { 'path' => 'd_spec.rb' },
-        ]
-      end
-
-      it do
-        test_file_pattern = double
-        expect(KnapsackPro::TestFilePattern).to receive(:call).with(adapter_class).and_return(test_file_pattern)
-        expect(described_class).to receive(:call).with(test_file_pattern).and_return(test_file_entities)
-
-        expect(described_class).to receive(:call).with(slow_test_file_pattern, test_file_list_enabled: false).and_return(slow_test_file_entities)
-
-        expect(subject).to eq([
-          { 'path' => 'b_spec.rb' },
-          { 'path' => 'c_spec.rb' },
-        ])
-      end
-    end
-
-    context 'when slow_test_file_pattern is not present' do
-      let(:slow_test_file_pattern) { nil }
-
-      it do
-        expect { subject }.to raise_error RuntimeError, 'KNAPSACK_PRO_SLOW_TEST_FILE_PATTERN is not defined'
-      end
-    end
-  end
-
-  describe '.select_test_files_that_can_be_run' do
-    let(:adapter_class) { double }
-    let(:test_file_entities_to_run) do
-      [
-        { 'path' => 'a_spec.rb' },
-        { 'path' => 'b_spec.rb' },
-        { 'path' => 'not_existing_on_disk_spec.rb' },
-      ]
-    end
-    # test files existing on disk
-    let(:test_file_entities) do
-      [
-        { 'path' => 'a_spec.rb' },
-        { 'path' => 'b_spec.rb' },
-        { 'path' => 'c_spec.rb' },
-      ]
-    end
-
-    subject { described_class.select_test_files_that_can_be_run(adapter_class, test_file_entities_to_run) }
-
-    it do
-      test_file_pattern = double
-      expect(KnapsackPro::TestFilePattern).to receive(:call).with(adapter_class).and_return(test_file_pattern)
-      expect(described_class).to receive(:call).with(test_file_pattern).and_return(test_file_entities)
-
-      expect(subject).to eq([
-        { 'path' => 'a_spec.rb' },
-        { 'path' => 'b_spec.rb' },
-      ])
-    end
-  end
-
-  describe '#call' do
-    let(:test_file_list_enabled) { true }
-    let(:test_file_pattern) { 'spec_fake/**{,/*/**}/*_spec.rb' }
-
-    subject { described_class.new(test_file_pattern, test_file_list_enabled).call }
-
-    context 'when KNAPSACK_PRO_TEST_FILE_EXCLUDE_PATTERN is not defined' do
-      it do
-        should eq([
-          { 'path' => 'spec_fake/controllers/users_controller_spec.rb' },
-          { 'path' => 'spec_fake/models/admin_spec.rb' },
-          { 'path' => 'spec_fake/models/user_spec.rb' },
-        ])
-      end
-    end
-
-    context 'when KNAPSACK_PRO_TEST_FILE_EXCLUDE_PATTERN is defined' do
-      let(:test_file_exclude_pattern) { 'spec_fake/controllers/*_spec.rb' }
-
-      before do
-        stub_const("ENV", { 'KNAPSACK_PRO_TEST_FILE_EXCLUDE_PATTERN' => test_file_exclude_pattern })
-      end
-
-      it do
-        should eq([
-          { 'path' => 'spec_fake/models/admin_spec.rb' },
-          { 'path' => 'spec_fake/models/user_spec.rb' },
-        ])
-      end
-    end
-
-    context 'when KNAPSACK_PRO_TEST_FILE_LIST is defined' do
-      # added spaces next to comma to check space is removed later
-      let(:test_file_list) { 'spec/bar_spec.rb,spec/foo_spec.rb, spec/time_helpers_spec.rb:10 , spec/time_helpers_spec.rb:38' }
-
-      before do
-        stub_const("ENV", { 'KNAPSACK_PRO_TEST_FILE_LIST' => test_file_list })
-      end
-
-      it do
-        expect(subject).to eq([
-          { 'path' => 'spec/bar_spec.rb' },
-          { 'path' => 'spec/foo_spec.rb' },
-          { 'path' => 'spec/time_helpers_spec.rb:10' },
-          { 'path' => 'spec/time_helpers_spec.rb:38' },
-        ])
-      end
-
-      context 'when test_file_list_enabled=false' do
-        let(:test_file_list_enabled) { false }
-
-        it do
-          should eq([
-            { 'path' => 'spec_fake/controllers/users_controller_spec.rb' },
-            { 'path' => 'spec_fake/models/admin_spec.rb' },
-            { 'path' => 'spec_fake/models/user_spec.rb' },
-          ])
-        end
-      end
-    end
-
-    context 'when KNAPSACK_PRO_TEST_FILE_LIST_SOURCE_FILE is defined' do
-      let(:test_file_list_source_file) { 'spec/fixtures/test_file_list_source_file.txt' }
-
-      before do
-        stub_const("ENV", { 'KNAPSACK_PRO_TEST_FILE_LIST_SOURCE_FILE' => test_file_list_source_file })
-      end
-
-      it do
-        expect(subject).to eq([
-          { 'path' => 'spec/test1_spec.rb' },
-          { 'path' => 'spec/test2_spec.rb[1]' },
-          { 'path' => 'spec/test3_spec.rb[1:2:3:4]' },
-          { 'path' => 'spec/test4_spec.rb:4' },
-          { 'path' => 'spec/test4_spec.rb:5' },
-        ])
-      end
-    end
-  end
-end

data/spec/knapsack_pro/test_file_pattern_spec.rb

@@ -1,49 +0,0 @@
-describe KnapsackPro::TestFilePattern do
-  describe '.call' do
-    let(:adapter_class) { KnapsackPro::Adapters::BaseAdapter }
-
-    subject { described_class.call(adapter_class) }
-
-    before do
-      expect(KnapsackPro::Config::Env).to receive(:test_file_pattern).and_return(env_test_file_pattern)
-    end
-
-    context 'when ENV defined' do
-      let(:env_test_file_pattern) { 'spec/**{,/*/**}/*_spec.rb' }
-
-      it { should eq env_test_file_pattern }
-    end
-
-    context 'when ENV not defined' do
-      let(:env_test_file_pattern) { nil }
-
-      it { should eq 'test/**{,/*/**}/*_test.rb' }
-    end
-  end
-
-  describe '#test_dir' do
-    let(:adapter_class) { KnapsackPro::Adapters::BaseAdapter }
-
-    subject { described_class.test_dir(adapter_class) }
-
-    before do
-      expect(described_class).to receive(:call).with(adapter_class).and_return(test_file_pattern)
-    end
-
-    context 'when default test file pattern' do
-      let(:test_file_pattern) { 'spec/**{,/*/**}/*_spec.rb' }
-
-      it 'extracts test directory from the pattern' do
-        expect(subject).to eq 'spec'
-      end
-    end
-
-    context 'when test file pattern has multiple patterns' do
-      let(:test_file_pattern) { '{spec/*_spec.rb,spec2/controllers/**/*_spec.rb}' }
-
-      it 'extracts test directory from the first pattern' do
-        expect(subject).to eq 'spec'
-      end
-    end
-  end
-end

data/spec/knapsack_pro/test_file_presenter_spec.rb

@@ -1,22 +0,0 @@
-describe KnapsackPro::TestFilePresenter do
-  describe '.stringify_paths' do
-    let(:test_file_paths) { ['a_spec.rb', 'b_spec.rb'] }
-
-    subject { described_class.stringify_paths(test_file_paths) }
-
-    it { should eq '"a_spec.rb" "b_spec.rb"' }
-  end
-
-  describe '.paths' do
-    let(:test_files) do
-      [
-        { 'path' => 'a_spec.rb' },
-        { 'path' => 'b_spec.rb' },
-      ]
-    end
-
-    subject { described_class.paths(test_files) }
-
-    it { should eq ['a_spec.rb', 'b_spec.rb'] }
-  end
-end

data/spec/knapsack_pro/test_files_with_test_cases_composer_spec.rb

@@ -1,41 +0,0 @@
-describe KnapsackPro::TestFilesWithTestCasesComposer do
-  let(:test_files) do
-    [
-      { 'path' => 'spec/a_spec.rb' },
-      { 'path' => 'spec/b_spec.rb' },
-      { 'path' => 'spec/c_spec.rb' },
-      { 'path' => 'spec/slow_1_spec.rb' },
-      { 'path' => 'spec/slow_2_spec.rb' },
-    ]
-  end
-  let(:slow_test_files) do
-    [
-      { 'path' => 'spec/slow_1_spec.rb', 'time_execution' => 1.0 },
-      { 'path' => 'spec/slow_2_spec.rb', 'time_execution' => 2.0 },
-    ]
-  end
-  let(:test_file_cases) do
-    [
-      { 'path' => 'spec/slow_1_spec.rb[1:1]' },
-      { 'path' => 'spec/slow_1_spec.rb[1:2]' },
-      { 'path' => 'spec/slow_2_spec.rb[1:1:1]' },
-      { 'path' => 'spec/slow_2_spec.rb[1:1:2]' },
-      { 'path' => 'spec/slow_2_spec.rb[1:1:3]' },
-    ]
-  end
-
-  subject { described_class.call(test_files, slow_test_files, test_file_cases) }
-
-  it 'returns test files that are not slow and test file cases for slow test files' do
-    expect(subject).to eq([
-      { 'path' => 'spec/a_spec.rb' },
-      { 'path' => 'spec/b_spec.rb' },
-      { 'path' => 'spec/c_spec.rb' },
-      { 'path' => 'spec/slow_1_spec.rb[1:1]' },
-      { 'path' => 'spec/slow_1_spec.rb[1:2]' },
-      { 'path' => 'spec/slow_2_spec.rb[1:1:1]' },
-      { 'path' => 'spec/slow_2_spec.rb[1:1:2]' },
-      { 'path' => 'spec/slow_2_spec.rb[1:1:3]' },
-    ])
-  end
-end

data/spec/knapsack_pro/test_flat_distributor_spec.rb

@@ -1,60 +0,0 @@
-describe KnapsackPro::TestFlatDistributor do
-  let(:test_flat_distributor) { described_class.new(test_files, node_total) }
-  let(:node_total) { 2 }
-  let(:test_files) do
-    [
-      { 'path' => 'spec/dir1/a_spec.rb' },
-      { 'path' => 'spec/dir2/a_spec.rb' },
-      { 'path' => 'spec/dir3/a_spec.rb' },
-      { 'path' => 'spec/feature/e_spec.rb' },
-      { 'path' => 'spec/models/b_spec.rb' },
-      { 'path' => 'spec/feature/c_spec.rb' },
-      { 'path' => 'spec/models/d_spec.rb' },
-      { 'path' => 'spec/feature/a_spec.rb' },
-      { 'path' => 'spec/models/f_spec.rb' },
-      { 'path' => 'spec/models/g_spec.rb' },
-      { 'path' => 'spec/dir4/h_spec.rb' },
-      { 'path' => 'spec/models/i_spec.rb' },
-    ]
-  end
-
-  describe '#nodes' do
-    subject { test_flat_distributor.nodes }
-
-    it do
-      should eq({
-        0 => [
-          { 'path' => 'spec/feature/a_spec.rb' },
-          { 'path' => 'spec/feature/e_spec.rb' },
-          { 'path' => 'spec/models/d_spec.rb' },
-          { 'path' => 'spec/models/g_spec.rb' },
-          { 'path' => 'spec/dir1/a_spec.rb' },
-          { 'path' => 'spec/dir3/a_spec.rb' },
-        ],
-        1 => [
-          { 'path' => 'spec/feature/c_spec.rb' },
-          { 'path' => 'spec/models/b_spec.rb' },
-          { 'path' => 'spec/models/f_spec.rb' },
-          { 'path' => 'spec/models/i_spec.rb' },
-          { 'path' => 'spec/dir2/a_spec.rb' },
-          { 'path' => 'spec/dir4/h_spec.rb' },
-        ],
-      })
-    end
-  end
-
-  describe '#test_files_for_node' do
-    subject { test_flat_distributor.test_files_for_node(1) }
-
-    it do
-      should eq([
-        { 'path' => 'spec/feature/c_spec.rb' },
-        { 'path' => 'spec/models/b_spec.rb' },
-        { 'path' => 'spec/models/f_spec.rb' },
-        { 'path' => 'spec/models/i_spec.rb' },
-        { 'path' => 'spec/dir2/a_spec.rb' },
-        { 'path' => 'spec/dir4/h_spec.rb' },
-      ])
-    end
-  end
-end

data/spec/knapsack_pro/tracker_spec.rb

@@ -1,184 +0,0 @@
-shared_examples 'default trakcer attributes' do
-  it { expect(tracker.global_time).to eql 0 }
-  it { expect(tracker.test_files_with_time).to eql({}) }
-  it { expect(tracker.prerun_tests_loaded).to be false }
-end
-
-describe KnapsackPro::Tracker do
-  let(:adapter) { 'RSpecAdapter' }
-  let(:tracker) { described_class.send(:new) }
-
-  before do
-    allow(KnapsackPro::Config::Env).to receive(:test_runner_adapter).and_return(adapter)
-    allow(KnapsackPro::Config::Env).to receive(:ci_node_index).and_return(0)
-  end
-
-  it_behaves_like 'default trakcer attributes'
-
-  describe '#current_test_path' do
-    subject { tracker.current_test_path }
-
-    context 'when current_test_path not set' do
-      it { should eql nil }
-    end
-
-    context 'when current_test_path set' do
-      context 'when current_test_path has prefix ./' do
-        before { tracker.current_test_path = './spec/models/user_spec.rb' }
-        it { should eql 'spec/models/user_spec.rb' }
-      end
-
-      context 'when current_test_path has no prefix ./' do
-        before { tracker.current_test_path = 'spec/models/user_spec.rb' }
-        it { should eql 'spec/models/user_spec.rb' }
-      end
-    end
-  end
-
-  describe 'track time execution' do
-    let(:test_paths) { ['a_spec.rb', 'b_spec.rb'] }
-    let(:delta) { 0.02 }
-
-    before do
-      tracker.set_prerun_tests(test_paths)
-    end
-
-    shared_examples '#to_a' do
-      subject { tracker.to_a }
-
-      its(:size) { should eq 2 }
-      it { expect(subject[0][:path]).to eq 'a_spec.rb' }
-      it { expect(subject[0][:time_execution]).to be >= 0 }
-      it { expect(subject[1][:path]).to eq 'b_spec.rb' }
-      it { expect(subject[1][:time_execution]).to be >= 0 }
-    end
-
-    context 'without Timecop' do
-      before do
-        test_paths.each_with_index do |test_path, index|
-          tracker.current_test_path = test_path
-          tracker.start_timer
-          sleep index.to_f / 10 + 0.1
-          tracker.stop_timer
-        end
-      end
-
-      it { expect(tracker.global_time).to be_within(delta).of(0.3) }
-      it { expect(tracker.prerun_tests_loaded).to be true }
-      it { expect(tracker.test_files_with_time.keys.size).to eql 2 }
-      it { expect(tracker.test_files_with_time['a_spec.rb'][:time_execution]).to be_within(delta).of(0.1) }
-      it { expect(tracker.test_files_with_time['b_spec.rb'][:time_execution]).to be_within(delta).of(0.2) }
-      it_behaves_like '#to_a'
-    end
-
-    context "with Timecop - Timecop shouldn't have impact on the measured test time" do
-      let(:now) { Time.now }
-
-      before do
-        test_paths.each_with_index do |test_path, index|
-          Timecop.freeze(now) do
-            tracker.current_test_path = test_path
-            tracker.start_timer
-          end
-
-          delay = index + 1
-          Timecop.freeze(now+delay) do
-            tracker.stop_timer
-          end
-        end
-      end
-
-      it { expect(tracker.global_time).to be > 0 }
-      it { expect(tracker.global_time).to be_within(delta).of(0) }
-      it { expect(tracker.prerun_tests_loaded).to be true }
-      it { expect(tracker.test_files_with_time.keys.size).to eql 2 }
-      it { expect(tracker.test_files_with_time['a_spec.rb'][:time_execution]).to be_within(delta).of(0) }
-      it { expect(tracker.test_files_with_time['b_spec.rb'][:time_execution]).to be_within(delta).of(0) }
-      it_behaves_like '#to_a'
-    end
-
-    # https://github.com/KnapsackPro/knapsack_pro-ruby/issues/32
-    context 'when start timer was not called (rspec-retry issue)' do
-      before do
-        test_paths.each_with_index do |test_path, index|
-          tracker.current_test_path = test_path
-          sleep 0.001
-          tracker.stop_timer
-        end
-      end
-
-      it { expect(tracker.global_time).to be > 0 }
-      it { expect(tracker.prerun_tests_loaded).to be true }
-      it { expect(tracker.test_files_with_time.keys.size).to eql 2 }
-      it { expect(tracker.test_files_with_time['a_spec.rb'][:time_execution]).to eq 0 }
-      it '2nd spec (b_spec.rb) should have recorded time execution - because start_time was set during first call of stop_timer for the first spec (a_spec.rb)' do
-        expect(tracker.test_files_with_time['b_spec.rb'][:time_execution]).to be > 0
-      end
-      it_behaves_like '#to_a'
-    end
-
-    context 'when a new tracker instance is created' do
-      let(:non_pending_test_paths) { ['a_spec.rb', 'b_spec.rb'] }
-      let(:test_paths) { non_pending_test_paths + ['pending_spec.rb'] }
-
-      before do
-        # measure tests only for non pending tests
-        non_pending_test_paths.each_with_index do |test_path, index|
-          tracker.current_test_path = test_path
-          tracker.start_timer
-          sleep index.to_f / 10 + 0.1
-          tracker.stop_timer
-        end
-      end
-
-      it '2nd tracker instance loads prerun tests from the disk' do
-        expect(tracker.prerun_tests_loaded).to be true
-        expect(tracker.to_a.size).to eq 3
-        expect(tracker.to_a[0][:path]).to eq 'a_spec.rb'
-        expect(tracker.to_a[0][:time_execution]).to be >= 0
-        expect(tracker.to_a[1][:path]).to eq 'b_spec.rb'
-        expect(tracker.to_a[1][:time_execution]).to be >= 0
-        expect(tracker.to_a[2][:path]).to eq 'pending_spec.rb'
-        expect(tracker.to_a[2][:time_execution]).to eq 0
-
-        tracker2 = described_class.send(:new)
-        expect(tracker2.prerun_tests_loaded).to be false
-        expect(tracker2.to_a.size).to eq 3
-        expect(tracker2.to_a[0][:path]).to eq 'a_spec.rb'
-        expect(tracker2.to_a[0][:time_execution]).to be >= 0
-        expect(tracker2.to_a[1][:path]).to eq 'b_spec.rb'
-        expect(tracker2.to_a[1][:time_execution]).to be >= 0
-        expect(tracker2.to_a[2][:path]).to eq 'pending_spec.rb'
-        expect(tracker2.to_a[2][:time_execution]).to eq 0
-        expect(tracker2.prerun_tests_loaded).to be true
-      end
-    end
-  end
-
-  describe '#reset!' do
-    let(:test_file_path) { 'a_spec.rb' }
-
-    before do
-      tracker.set_prerun_tests([test_file_path])
-    end
-
-    before do
-      expect(tracker.prerun_tests_loaded).to be true
-
-      tracker.current_test_path = test_file_path
-      tracker.start_timer
-      sleep 0.1
-      tracker.stop_timer
-
-      expect(tracker.global_time).not_to eql 0
-
-      tracker.reset!
-    end
-
-    it_behaves_like 'default trakcer attributes'
-
-    it 'resets prerun_tests_loaded to false' do
-      expect(tracker.prerun_tests_loaded).to be false
-    end
-  end
-end

data/spec/knapsack_pro/utils_spec.rb

@@ -1,55 +0,0 @@
-describe KnapsackPro::Utils do
-  describe '.unsymbolize' do
-    let(:test_files) do
-      [
-        { path: 'a_spec.rb', time_execution: 0.1 },
-        { path: 'b_spec.rb', time_execution: 0.2 },
-      ]
-    end
-
-    subject { described_class.unsymbolize(test_files) }
-
-    it do
-      should eq([
-        { 'path' => 'a_spec.rb', 'time_execution' => 0.1 },
-        { 'path' => 'b_spec.rb', 'time_execution' => 0.2 },
-      ])
-    end
-  end
-
-  describe '.time_now' do
-    subject { described_class.time_now }
-
-    context 'when Timecop does not mock the time' do
-      it do
-        now = Process.clock_gettime(Process::CLOCK_MONOTONIC)
-        expect(subject).to be_within(0.01).of(now)
-      end
-    end
-
-    context 'when Timecop does mock the process clock' do
-      around(:example) do |ex|
-        unless Gem::Version.new(Timecop::VERSION) >= Gem::Version.new('0.9.9')
-          raise 'Timecop >= 0.9.9 is required to run this test. Please run: bundle update'
-        end
-
-        if Gem::Version.new(Timecop::VERSION) >= Gem::Version.new('0.9.10')
-          Timecop.mock_process_clock = true
-          ex.run
-          Timecop.mock_process_clock = false
-        else
-          ex.run
-        end
-      end
-
-      it do
-        now = Process.clock_gettime(Process::CLOCK_MONOTONIC)
-
-        time = Time.local(2020, 1, 31)
-        Timecop.travel(time) do
-          expect(subject).to be_within(0.01).of(now)
-        end
-      end
-    end
-  end
-end