knapsack_pro 9.0.0 → 9.1.0

data/spec/knapsack_pro/hooks/queue_spec.rb

@@ -1,199 +0,0 @@
-describe KnapsackPro::Hooks::Queue do
-  describe '.call_before_queue' do
-    subject { described_class.call_before_queue }
-
-    before do
-      described_class.reset_before_queue
-    end
-
-    context 'when no callback is set' do
-      it { should be_nil }
-    end
-
-    context 'when multiple callbacks are set' do
-      let(:queue_id) { double }
-
-      it 'calls each block' do
-        expect(KnapsackPro::Config::Env).to receive(:queue_id).twice.and_return(queue_id)
-
-        expected_called_blocks = []
-
-        described_class.before_queue do |q_id|
-          expected_called_blocks << [:block_1_called, q_id]
-        end
-        described_class.before_queue do |q_id|
-          expected_called_blocks << [:block_2_called, q_id]
-        end
-
-        subject
-
-        expect(expected_called_blocks).to eq([
-          [:block_1_called, queue_id],
-          [:block_2_called, queue_id],
-        ])
-      end
-    end
-  end
-
-  describe '.call_before_subset_queue' do
-    subject { described_class.call_before_subset_queue }
-
-    before do
-      described_class.reset_before_subset_queue
-    end
-
-    context 'when no callback is set' do
-      it { should be_nil }
-    end
-
-    context 'when multiple callbacks are set' do
-      let(:queue_id) { double }
-      let(:subset_queue_id) { double }
-
-      it 'calls each block' do
-        expect(KnapsackPro::Config::Env).to receive(:queue_id).twice.and_return(queue_id)
-        expect(KnapsackPro::Config::Env).to receive(:subset_queue_id).twice.and_return(subset_queue_id)
-
-        expected_called_blocks = []
-
-        described_class.before_subset_queue do |q_id, subset_q_id|
-          expected_called_blocks << [:block_1_called, q_id, subset_q_id]
-        end
-        described_class.before_subset_queue do |q_id, subset_q_id|
-          expected_called_blocks << [:block_2_called, q_id, subset_q_id]
-        end
-
-        subject
-
-        expect(expected_called_blocks).to eq([
-          [:block_1_called, queue_id, subset_queue_id],
-          [:block_2_called, queue_id, subset_queue_id],
-        ])
-      end
-    end
-
-    context 'when a callback is set AND the queue is passed' do
-      let(:queue_id) { double }
-      let(:subset_queue_id) { double }
-      let(:queue) { instance_double(KnapsackPro::Queue) }
-
-      subject { described_class.call_before_subset_queue(queue) }
-
-      it 'calls each block' do
-        expect(KnapsackPro::Config::Env).to receive(:queue_id).and_return(queue_id)
-        expect(KnapsackPro::Config::Env).to receive(:subset_queue_id).and_return(subset_queue_id)
-
-        expected_called_blocks = []
-
-        described_class.before_subset_queue do |q_id, subset_q_id, queue|
-          expected_called_blocks << [:block_1_called, q_id, subset_q_id, queue]
-        end
-
-        subject
-
-        expect(expected_called_blocks).to eq([
-          [:block_1_called, queue_id, subset_queue_id, queue],
-        ])
-      end
-    end
-  end
-
-  describe '.call_after_subset_queue' do
-    subject { described_class.call_after_subset_queue }
-
-    before do
-      described_class.reset_after_subset_queue
-    end
-
-    context 'when no callback is set' do
-      it { should be_nil }
-    end
-
-    context 'when multiple callbacks are set' do
-      let(:queue_id) { double }
-      let(:subset_queue_id) { double }
-
-      it 'calls each block' do
-        expect(KnapsackPro::Config::Env).to receive(:queue_id).at_least(:once).and_return(queue_id)
-        expect(KnapsackPro::Config::Env).to receive(:subset_queue_id).at_least(:once).and_return(subset_queue_id)
-
-        expected_called_blocks = []
-
-        described_class.after_subset_queue do |q_id, subset_q_id|
-          expected_called_blocks << [:block_1_called, q_id, subset_q_id]
-        end
-        described_class.after_subset_queue do |q_id, subset_q_id|
-          expected_called_blocks << [:block_2_called, q_id, subset_q_id]
-        end
-
-        subject
-
-        expect(expected_called_blocks).to eq([
-          [:block_1_called, queue_id, subset_queue_id],
-          [:block_2_called, queue_id, subset_queue_id],
-        ])
-      end
-    end
-
-    context 'when a callback is set AND the queue is passed' do
-      let(:queue_id) { double }
-      let(:subset_queue_id) { double }
-      let(:queue) { instance_double(KnapsackPro::Queue) }
-
-      subject { described_class.call_after_subset_queue(queue) }
-
-      it 'calls each block' do
-        expect(KnapsackPro::Config::Env).to receive(:queue_id).and_return(queue_id)
-        expect(KnapsackPro::Config::Env).to receive(:subset_queue_id).and_return(subset_queue_id)
-
-        expected_called_blocks = []
-
-        described_class.after_subset_queue do |q_id, subset_q_id, queue|
-          expected_called_blocks << [:block_1_called, q_id, subset_q_id, queue]
-        end
-
-        subject
-
-        expect(expected_called_blocks).to eq([
-          [:block_1_called, queue_id, subset_queue_id, queue],
-        ])
-      end
-    end
-  end
-
-  describe '.call_after_queue' do
-    subject { described_class.call_after_queue }
-
-    before do
-      described_class.reset_after_queue
-    end
-
-    context 'when no callback is set' do
-      it { should be_nil }
-    end
-
-    context 'when multiple callbacks are set' do
-      let(:queue_id) { double }
-
-      it 'calls each block' do
-        expect(KnapsackPro::Config::Env).to receive(:queue_id).twice.and_return(queue_id)
-
-        expected_called_blocks = []
-
-        described_class.after_queue do |q_id|
-          expected_called_blocks << [:block_1_called, q_id]
-        end
-        described_class.after_queue do |q_id|
-          expected_called_blocks << [:block_2_called, q_id]
-        end
-
-        subject
-
-        expect(expected_called_blocks).to eq([
-          [:block_1_called, queue_id],
-          [:block_2_called, queue_id],
-        ])
-      end
-    end
-  end
-end

data/spec/knapsack_pro/presenter_spec.rb

@@ -1,57 +0,0 @@
-describe KnapsackPro::Presenter do
-  describe '.global_time' do
-    let(:tracker) { instance_double(KnapsackPro::Tracker, global_time: 60*62+3) }
-
-    subject { described_class.global_time }
-
-    before do
-      expect(KnapsackPro).to receive(:tracker).and_return(tracker)
-    end
-
-    it { should eql "Global test execution duration: 01h 02m 03s" }
-  end
-
-  describe '.pretty_seconds' do
-    subject { described_class.pretty_seconds(seconds) }
-
-    context 'when less then one second' do
-      let(:seconds) { 0.987 }
-      it { should eql '0.987s' }
-    end
-
-    context 'when one second' do
-      let(:seconds) { 1 }
-      it { should eql '01s' }
-    end
-
-    context 'when only seconds' do
-      let(:seconds) { 5 }
-      it { should eql '05s' }
-    end
-
-    context 'when only minutes' do
-      let(:seconds) { 120 }
-      it { should eql '02m' }
-    end
-
-    context 'when only hours' do
-      let(:seconds) { 60*60*3 }
-      it { should eql '03h' }
-    end
-
-    context 'when minutes and seconds' do
-      let(:seconds) { 180+9 }
-      it { should eql '03m 09s' }
-    end
-
-    context 'when all' do
-      let(:seconds) { 60*60*4+120+7 }
-      it { should eql '04h 02m 07s' }
-    end
-
-    context 'when negative seconds' do
-      let(:seconds) { -67 }
-      it { should eql '-01m 07s' }
-    end
-  end
-end

data/spec/knapsack_pro/pure/queue/rspec_pure_spec.rb

@@ -1,248 +0,0 @@
-require(KnapsackPro.root + '/lib/knapsack_pro/formatters/time_tracker')
-require(KnapsackPro.root + '/lib/knapsack_pro/extensions/rspec_extension')
-
-describe KnapsackPro::Pure::Queue::RSpecPure do
-  let(:rspec_pure) { described_class.new }
-
-  describe '#add_knapsack_pro_formatters_to' do
-    subject { rspec_pure.add_knapsack_pro_formatters_to(spec_opts) }
-
-    context 'when no spec_opts' do
-      let(:spec_opts) { nil }
-
-      it 'returns no spec_opts' do
-        expect(subject).to be nil
-      end
-    end
-
-    context 'when spec_opts have Knapsack Pro formatters' do
-      let(:spec_opts) { '--color --format d --format KnapsackPro::Formatters::TimeTracker' }
-
-      it 'returns spec_opts' do
-        expect(subject).to eq spec_opts
-      end
-    end
-
-    context 'when spec_opts have no Knapsack Pro formatters' do
-      let(:spec_opts) { '--color --format d' }
-
-      it 'returns spec_opts with added Knapsack Pro formatters' do
-        expect(subject).to eq '--color --format d --format KnapsackPro::Formatters::TimeTracker'
-      end
-    end
-  end
-
-  describe '#error_exit_code' do
-    subject { rspec_pure.error_exit_code(rspec_error_exit_code) }
-
-    context 'when RSpec has no defined error exit code' do
-      let(:rspec_error_exit_code) { nil }
-
-      it 'returns 1 as the default exit code' do
-        expect(subject).to eq 1
-      end
-    end
-
-    context 'when RSpec has a defined error exit code' do
-      let(:rspec_error_exit_code) { 2 }
-
-      it 'returns the custom exit code' do
-        expect(subject).to eq rspec_error_exit_code
-      end
-    end
-  end
-
-  describe '#args_with_seed_option_added_when_viable' do
-    let(:order_option) { KnapsackPro::Adapters::RSpecAdapter.order_option(args) }
-
-    subject { rspec_pure.args_with_seed_option_added_when_viable(order_option, seed, args) }
-
-    context 'when the order option is not random' do
-      let(:args) { ['--order', 'defined'] }
-      let(:seed) { KnapsackPro::Extensions::RSpecExtension::Seed.new(nil, false) }
-
-      it 'does not add the seed option to args' do
-        expect(subject).to eq ['--order', 'defined']
-      end
-    end
-
-    ['random', 'rand'].each do |random_option_value|
-      context "when the order option is `#{random_option_value}`" do
-        let(:args) { ['--order', random_option_value] }
-
-        context 'when the seed is not used' do
-          let(:seed) { KnapsackPro::Extensions::RSpecExtension::Seed.new('123', false) }
-
-          it 'does not add the seed option to args' do
-            expect(subject).to eq ['--order', random_option_value]
-          end
-        end
-
-        context 'when the seed is used' do
-          let(:seed) { KnapsackPro::Extensions::RSpecExtension::Seed.new('123', true) }
-
-          it 'adds the seed option to args' do
-            expect(subject).to eq ['--order', random_option_value, '--seed', '123']
-          end
-        end
-      end
-    end
-
-    context 'when the order option is `rand:123`' do
-      let(:args) { ['--order', 'rand:123'] }
-      let(:seed) { KnapsackPro::Extensions::RSpecExtension::Seed.new('123', true) }
-
-      it 'does not add the seed option to args' do
-        expect(subject).to eq ['--order', 'rand:123']
-      end
-    end
-
-    context 'when the order option is not set in args AND seed is used' do
-      let(:args) { ['--format', 'documentation'] }
-      let(:seed) { KnapsackPro::Extensions::RSpecExtension::Seed.new('123', true) }
-
-      it 'adds the seed option to args' do
-        expect(subject).to eq ['--format', 'documentation', '--seed', '123']
-      end
-    end
-
-    context 'when the order option is not set in args AND seed is not used' do
-      let(:args) { ['--format', 'documentation'] }
-      let(:seed) { KnapsackPro::Extensions::RSpecExtension::Seed.new(value: '123', used?: false) }
-
-      it 'does not add the seed option to args' do
-        expect(subject).to eq ['--format', 'documentation']
-      end
-    end
-  end
-
-  describe '#prepare_cli_args' do
-    subject { rspec_pure.prepare_cli_args(args, has_format_option, has_require_rails_helper_option, rails_helper_exists, test_dir) }
-
-    context 'when no args' do
-      let(:args) { nil }
-      let(:has_format_option) { false }
-      let(:has_require_rails_helper_option) { false }
-      let(:test_dir) { 'spec' }
-
-      context 'when rails_helper does not exist' do
-        let(:rails_helper_exists) { false }
-
-        it 'adds the default progress formatter, the default path and the time tracker formatter, does not add require rails_helper' do
-          expect(subject).to eq [
-            '--format', 'progress',
-            '--default-path', 'spec',
-            '--format', 'KnapsackPro::Formatters::TimeTracker',
-          ]
-        end
-      end
-
-      context 'when rails_helper exists' do
-        let(:rails_helper_exists) { true }
-
-        it 'adds the default progress formatter, require rails_helper, the default path and the time tracker formatter' do
-          expect(subject).to eq [
-            '--format', 'progress',
-            '--require', 'rails_helper',
-            '--default-path', 'spec',
-            '--format', 'KnapsackPro::Formatters::TimeTracker',
-          ]
-        end
-      end
-    end
-
-    context 'when args are present and a custom test directory is set' do
-      let(:args) { '--color --profile --require rails_helper' }
-      let(:has_format_option) { false }
-      let(:has_require_rails_helper_option) { true }
-      let(:rails_helper_exists) { true }
-      let(:test_dir) { 'custom_spec_dir' }
-
-      it do
-        expect(subject).to eq [
-          '--color',
-          '--profile',
-          '--require', 'rails_helper',
-          '--format', 'progress',
-          '--default-path', 'custom_spec_dir',
-          '--format', 'KnapsackPro::Formatters::TimeTracker',
-        ]
-      end
-    end
-
-    context 'when args are present and has format option' do
-      let(:args) { '--color --profile --format d --require rails_helper' }
-      let(:has_format_option) { true }
-      let(:has_require_rails_helper_option) { true }
-      let(:rails_helper_exists) { true }
-      let(:test_dir) { 'spec' }
-
-      it 'uses the format option from args instead of the default formatter' do
-        expect(subject).to eq [
-          '--color',
-          '--profile',
-          '--format', 'd',
-          '--require', 'rails_helper',
-          '--default-path', 'spec',
-          '--format', 'KnapsackPro::Formatters::TimeTracker',
-        ]
-      end
-    end
-  end
-
-  describe '#rspec_command' do
-    let(:args) { ['--format', 'documentation'] }
-    let(:test_file_paths) { ['a_spec.rb', 'b_spec.rb'] }
-
-    subject { rspec_pure.rspec_command(args, test_file_paths, scope) }
-
-    context 'when there are no test file paths' do
-      let(:scope) { :queue_finished }
-      let(:test_file_paths) { [] }
-
-      it 'returns no messages' do
-        expect(subject).to eq []
-      end
-    end
-
-    context 'when a subset of queue (a batch of tests fetched from the Queue API)' do
-      let(:scope) { :batch_finished }
-
-      it 'returns messages with the RSpec command' do
-        expect(subject).to eq([
-          'To retry the last batch of tests fetched from the Queue API, please run the following command on your machine:',
-          'bundle exec rspec --format documentation "a_spec.rb" "b_spec.rb"',
-        ])
-      end
-    end
-
-    context 'when all tests fetched from the Queue API' do
-      let(:scope) { :queue_finished }
-
-      it 'returns messages with the RSpec command' do
-        expect(subject).to eq([
-          'To retry all the tests assigned to this CI node, please run the following command on your machine:',
-          'bundle exec rspec --format documentation "a_spec.rb" "b_spec.rb"',
-        ])
-      end
-    end
-
-    describe '#exit_summary' do
-      subject { rspec_pure.exit_summary(unexecuted_test_files) }
-
-      context 'when there are no unexecuted test files' do
-        let(:unexecuted_test_files) { [] }
-
-        it { expect(subject).to be_nil }
-      end
-
-      context 'when there are unexecuted test files' do
-        let(:unexecuted_test_files) { ['b_spec.rb', 'c_spec.rb'] }
-
-        it 'returns a warning message' do
-          expect(subject).to eq 'Unexecuted tests on this CI node (including pending tests): b_spec.rb c_spec.rb'
-        end
-      end
-    end
-  end
-end

data/spec/knapsack_pro/queue_spec.rb

@@ -1,35 +0,0 @@
-describe KnapsackPro::Queue do
-  it 'simulates a Queue Mode build' do
-    queue = described_class.new
-
-    expect(queue.current_batch).to be_nil
-
-    # 1st batch
-    test_files_paths_1 = ['a_spec.rb', 'b_spec.rb']
-    queue.add_batch_for(test_files_paths_1)
-
-    expect(queue.current_batch.test_file_paths).to eq(['a_spec.rb', 'b_spec.rb'])
-    expect(queue.current_batch.status).to eq :not_executed
-    queue.mark_batch_passed
-    expect(queue.current_batch.status).to eq :passed
-
-
-    # 2nd batch
-    test_files_paths_2 = ['c_spec.rb', 'd_spec.rb']
-    queue.add_batch_for(test_files_paths_2)
-
-    expect(queue.current_batch.test_file_paths).to eq(['c_spec.rb', 'd_spec.rb'])
-    expect(queue.current_batch.status).to eq :not_executed
-    queue.mark_batch_failed
-    expect(queue.current_batch.status).to eq :failed
-
-
-    # last batch from the Queue API is always empty
-    test_files_paths_3 = []
-    queue.add_batch_for(test_files_paths_3)
-
-    expect(queue.size).to eq 2
-    expect(queue[0].status).to eq :passed
-    expect(queue[1].status).to eq :failed
-  end
-end