knapsack_pro 9.0.0 → 9.1.0

data/spec/knapsack_pro/runners/test_unit_runner_spec.rb

@@ -1,62 +0,0 @@
-describe KnapsackPro::Runners::TestUnitRunner do
-  subject { described_class.new(KnapsackPro::Adapters::TestUnitAdapter) }
-
-  it { should be_kind_of KnapsackPro::Runners::BaseRunner }
-
-  describe '.run' do
-    let(:args) { '--verbose --order=random' }
-
-    subject { described_class.run(args) }
-
-    before do
-      stub_const("ENV", { 'KNAPSACK_PRO_TEST_SUITE_TOKEN_TEST_UNIT' => 'test-unit-token' })
-    end
-
-    context 'when test files were returned by Knapsack Pro API' do
-      let(:test_file_paths) { ['test-unit_fake/a_test.rb', 'test-unit_fake/b_test.rb'] }
-
-      it 'runs tests' do
-        expect(KnapsackPro::Adapters::TestUnitAdapter).to receive(:verify_bind_method_called)
-
-        tracker = instance_double(KnapsackPro::Tracker)
-        expect(KnapsackPro).to receive(:tracker).and_return(tracker)
-        expect(tracker).to receive(:set_prerun_tests).with(test_file_paths)
-
-        runner = instance_double(described_class,
-                                 test_dir: 'test-unit_fake',
-                                 test_file_paths: test_file_paths,
-                                 test_files_to_execute_exist?: true)
-        expect(described_class).to receive(:new)
-        .with(KnapsackPro::Adapters::TestUnitAdapter).and_return(runner)
-
-        auto_runner_exit_code = 0
-        expect(described_class).to receive(:test_unit_autorunner_run) do |flag, test_dir, cli_args|
-          expect(flag).to be true
-          expect(test_dir).to eq 'test-unit_fake'
-          expect(cli_args.size).to eq 4
-          expect(cli_args[0]).to eq '--verbose'
-          expect(cli_args[1]).to eq '--order=random'
-          expect(cli_args[2]).to end_with 'test-unit_fake/a_test.rb'
-          expect(cli_args[3]).to end_with 'test-unit_fake/b_test.rb'
-        end.and_return(auto_runner_exit_code)
-        expect(described_class).to receive(:exit).with(auto_runner_exit_code)
-
-        subject
-
-        expect(ENV['KNAPSACK_PRO_TEST_SUITE_TOKEN']).to eq 'test-unit-token'
-        expect(ENV['KNAPSACK_PRO_REGULAR_MODE_ENABLED']).to eq 'true'
-      end
-    end
-
-    context 'when test files were not returned by Knapsack Pro API' do
-      it "doesn't run tests" do
-        runner = instance_double(described_class,
-                                 test_files_to_execute_exist?: false)
-        expect(described_class).to receive(:new)
-        .with(KnapsackPro::Adapters::TestUnitAdapter).and_return(runner)
-
-        subject
-      end
-    end
-  end
-end

data/spec/knapsack_pro/slow_test_file_determiner_spec.rb

@@ -1,119 +0,0 @@
-describe KnapsackPro::SlowTestFileDeterminer do
-  describe '.call' do
-    let(:node_total) { 4 }
-
-    before do
-      expect(KnapsackPro::Config::Env).to receive(:ci_node_total).and_return(node_total)
-    end
-
-    subject { described_class.call(test_files) }
-
-    context 'when test files have recorded execution time' do
-      let(:test_files) do
-        [
-          { 'path' => 'a_spec.rb', 'time_execution' => 1.0 },
-          { 'path' => 'b_spec.rb', 'time_execution' => 3.4 },
-          { 'path' => 'c_spec.rb', 'time_execution' => 3.5 },
-          { 'path' => 'd_spec.rb', 'time_execution' => 12.1 },
-        ]
-      end
-
-      it 'detects slow tests above 3.5s threshold (20.0 / 4 nodes * 70% threshold = 3.5)' do
-        expect(subject).to eq([
-          { 'path' => 'c_spec.rb', 'time_execution' => 3.5 },
-          { 'path' => 'd_spec.rb', 'time_execution' => 12.1 },
-        ])
-      end
-    end
-
-    context 'when test files have recorded execution time AND slow test files threshold is set' do
-      let(:test_files) do
-        [
-          { 'path' => 'a_spec.rb', 'time_execution' => 1.0 },
-          { 'path' => 'b_spec.rb', 'time_execution' => 3.4 },
-          { 'path' => 'c_spec.rb', 'time_execution' => 3.5 },
-          { 'path' => 'd_spec.rb', 'time_execution' => 12.1 },
-        ]
-      end
-
-      before do
-        stub_const("ENV", { 'KNAPSACK_PRO_SLOW_TEST_FILE_THRESHOLD' => '2' })
-      end
-
-      it 'detects slow tests above 2.0s threshold' do
-        expect(subject).to eq([
-          { 'path' => 'b_spec.rb', 'time_execution' => 3.4 },
-          { 'path' => 'c_spec.rb', 'time_execution' => 3.5 },
-          { 'path' => 'd_spec.rb', 'time_execution' => 12.1 },
-        ])
-      end
-    end
-
-    context 'when test files have no recorded execution time' do
-      let(:test_files) do
-        [
-          { 'path' => 'a_spec.rb', 'time_execution' => 0.0 },
-          { 'path' => 'b_spec.rb', 'time_execution' => 0.0 },
-        ]
-      end
-
-      it do
-        expect(subject).to eq([])
-      end
-    end
-
-    context 'when there are no test files' do
-      let(:test_files) { [] }
-
-      it do
-        expect(subject).to eq([])
-      end
-    end
-  end
-
-  describe '.save_to_json_report', :clear_tmp do
-    let(:json_report_path) { '.knapsack_pro/slow_test_file_determiner/slow_test_files_node_0.json' }
-    let(:test_files) do
-      [
-        { 'path' => 'a_spec.rb', 'time_execution' => 1.0 },
-        # unique path to ensure we saved on disk a completely new report
-        { 'path' => "#{SecureRandom.hex}_spec.rb", 'time_execution' => 3.4 },
-      ]
-    end
-
-    subject { described_class.save_to_json_report(test_files) }
-
-    it do
-      subject
-      expect(File.read(json_report_path)).to eq(test_files.to_json)
-    end
-  end
-
-  describe '.read_from_json_report', :clear_tmp do
-    let(:test_files) do
-      [
-        { 'path' => 'a_spec.rb', 'time_execution' => 1.0 },
-        # unique path to ensure we saved on disk a completely new report
-        { 'path' => "#{SecureRandom.hex}_spec.rb", 'time_execution' => 3.4 },
-      ]
-    end
-
-    subject { described_class.read_from_json_report }
-
-    context 'when json report exists' do
-      before do
-        described_class.save_to_json_report(test_files)
-      end
-
-      it do
-        expect(subject).to eq test_files
-      end
-    end
-
-    context 'when json report does not exist' do
-      it do
-        expect { subject }.to raise_error(RuntimeError, 'The report with slow test files has not been generated yet. If you have enabled split by test cases https://knapsackpro.com/perma/ruby/split-by-test-examples and you see this error it means that your tests accidentally cleaned up the .knapsack_pro directory. Please do not remove this directory during tests runtime!')
-      end
-    end
-  end
-end

data/spec/knapsack_pro/slow_test_file_finder_spec.rb

@@ -1,42 +0,0 @@
-describe KnapsackPro::SlowTestFileFinder do
-  describe '.call' do
-    let(:adapter_class) { double }
-
-    subject { described_class.call(adapter_class) }
-
-    before do
-      expect(KnapsackPro::Config::Env).to receive(:test_files_encrypted?).and_return(test_files_encrypted?)
-    end
-
-    context 'when test files are not encrypted' do
-      let(:test_files_encrypted?) { false }
-
-      it do
-        test_files_from_api = double
-        build_distribution_entity = instance_double(KnapsackPro::BuildDistributionFetcher::BuildDistributionEntity, test_files: test_files_from_api)
-        expect(KnapsackPro::BuildDistributionFetcher).to receive(:call).and_return(build_distribution_entity)
-
-        merged_test_files_from_api = double
-        expect(KnapsackPro::TestCaseMergers::BaseMerger).to receive(:call).with(adapter_class, test_files_from_api).and_return(merged_test_files_from_api)
-
-        test_files_existing_on_disk = double
-        expect(KnapsackPro::TestFileFinder).to receive(:select_test_files_that_can_be_run).with(adapter_class, merged_test_files_from_api).and_return(test_files_existing_on_disk)
-
-        slow_test_files = double
-        expect(KnapsackPro::SlowTestFileDeterminer).to receive(:call).with(test_files_existing_on_disk).and_return(slow_test_files)
-
-        expect(KnapsackPro::SlowTestFileDeterminer).to receive(:save_to_json_report).with(slow_test_files)
-
-        expect(subject).to eq slow_test_files
-      end
-    end
-
-    context 'when test files are encrypted' do
-      let(:test_files_encrypted?) { true }
-
-      it do
-        expect { subject }.to raise_error RuntimeError, 'Split by test cases is not possible when you have enabled test file names encryption ( https://knapsackpro.com/perma/ruby/encryption ). You need to disable encryption with KNAPSACK_PRO_TEST_FILES_ENCRYPTED=false in order to use split by test cases https://knapsackpro.com/perma/ruby/split-by-test-examples'
-      end
-    end
-  end
-end

data/spec/knapsack_pro/task_loader_spec.rb

@@ -1,14 +0,0 @@
-describe KnapsackPro::TaskLoader do
-  describe '#load_tasks' do
-    let(:rspec_rake_task_path) { "#{KnapsackPro.root}/lib/tasks/rspec.rake" }
-    let(:cucumber_rake_task_path) { "#{KnapsackPro.root}/lib/tasks/cucumber.rake" }
-    let(:minitest_rake_task_path) { "#{KnapsackPro.root}/lib/tasks/minitest.rake" }
-
-    before { allow(subject).to receive(:import) }
-    after { subject.load_tasks }
-
-    it { expect(subject).to receive(:import).with(rspec_rake_task_path) }
-    it { expect(subject).to receive(:import).with(cucumber_rake_task_path) }
-    it { expect(subject).to receive(:import).with(minitest_rake_task_path) }
-  end
-end

data/spec/knapsack_pro/test_case_detectors/rspec_test_example_detector_spec.rb

@@ -1,217 +0,0 @@
-describe KnapsackPro::TestCaseDetectors::RSpecTestExampleDetector do
-  let(:report_dir) { '.knapsack_pro/test_case_detectors/rspec' }
-  let(:report_path) { '.knapsack_pro/test_case_detectors/rspec/rspec_dry_run_json_report_node_0.json' }
-  let(:rspec_test_example_detector) { described_class.new }
-
-  describe '#generate_json_report' do
-    subject { rspec_test_example_detector.generate_json_report(rspec_args) }
-
-    shared_examples 'generate_json_report runs RSpec::Core::Runner' do
-      before do
-        expect(KnapsackPro::Config::TempFiles).to receive(:ensure_temp_directory_exists!)
-
-        expect(FileUtils).to receive(:mkdir_p).with(report_dir)
-
-        allow(File).to receive(:exist?)
-        expect(File).to receive(:exist?).at_least(:once).with(report_path).and_return(true)
-        expect(File).to receive(:delete).with(report_path)
-
-        expect(rspec_test_example_detector).to receive(:slow_test_files).and_return(test_file_entities)
-      end
-
-      context 'when there are no slow test files' do
-        let(:test_file_entities) { [] }
-
-        before do
-          expect(File).to receive(:write).with(report_path, { examples: [] }.to_json)
-        end
-
-        it do
-          expect(subject).to be_nil
-        end
-      end
-
-      context 'when slow test files exist' do
-        let(:test_file_entities) do
-          [
-            { 'path' => 'spec/a_spec.rb' },
-            { 'path' => 'spec/b_spec.rb' },
-          ]
-        end
-
-        before do
-          test_dir = 'spec'
-          expect(KnapsackPro::Config::Env).to receive(:test_dir).and_return(nil)
-          expect(KnapsackPro::TestFilePattern).to receive(:test_dir).with(KnapsackPro::Adapters::RSpecAdapter).and_return(test_dir)
-
-          options = double
-          expect(RSpec::Core::ConfigurationOptions).to receive(:new).with(expected_args + [
-            '--format', expected_format,
-            '--dry-run',
-            '--out', report_path,
-            '--default-path', test_dir,
-            'spec/a_spec.rb', 'spec/b_spec.rb',
-          ]).and_return(options)
-
-          rspec_core_runner = double
-          expect(RSpec::Core::Runner).to receive(:new).with(options).and_return(rspec_core_runner)
-          expect(rspec_core_runner).to receive(:run).with($stderr, $stdout).and_return(exit_code)
-        end
-
-        context 'when exit code from RSpec::Core::Runner is 0' do
-          let(:exit_code) { 0 }
-
-          it do
-            expect(subject).to be_nil
-          end
-        end
-
-        context 'when exit code from RSpec::Core::Runner is 1' do
-          let(:exit_code) { 1 }
-
-          before do
-            json_file = %{
-            {"version":"3.11.0","messages":["An error occurred while loading ./spec/a_spec.rb"],"examples":[],"summary":{"duration":3.6e-05,"example_count":0,"failure_count":0,"pending_count":0,"errors_outside_of_examples_count":1},"summary_line":"0 examples, 0 failures, 1 error occurred outside of examples"}
-            }.strip
-            expect(File).to receive(:read).with(report_path).and_return(json_file)
-          end
-
-          it do
-            expect { subject }.to raise_error(SystemExit) { |error| expect(error.status).to eq exit_code }
-          end
-        end
-      end
-    end
-
-    context 'when RSpec >= 3.6.0' do
-      let(:rspec_args) { '' }
-      let(:expected_args) { [] }
-      let(:expected_format) { 'json' }
-
-      it_behaves_like 'generate_json_report runs RSpec::Core::Runner'
-    end
-
-    context 'when RSpec < 3.6.0' do
-      let(:rspec_args) { '' }
-      let(:expected_args) { [] }
-      let(:expected_format) { 'KnapsackPro::Formatters::RSpecJsonFormatter' }
-
-      before do
-        stub_const('RSpec::Core::Version::STRING', '3.5.0')
-      end
-
-      it_behaves_like 'generate_json_report runs RSpec::Core::Runner'
-    end
-
-    context 'when RSpec CLI args are present including format option' do
-      let(:rspec_args) { '-t mytag --format documentation --out /tmp/documentation.txt --tag ~@skip --example-matches regexp --example string' }
-      let(:expected_args) { ['-t', 'mytag', '--tag', '~@skip', '--example-matches', 'regexp', '--example', 'string'] }
-      let(:expected_format) { 'json' }
-
-      describe 'removes formatters from RSpec CLI args' do
-        it_behaves_like 'generate_json_report runs RSpec::Core::Runner'
-      end
-    end
-
-    context 'when RSpec CLI args are not set' do
-      let(:rspec_args) { nil }
-      let(:expected_args) { [] }
-      let(:expected_format) { 'json' }
-
-      it do
-        expect { subject }.to raise_error("The internal KNAPSACK_PRO_RSPEC_OPTIONS environment variable is unset. Ensure it is not overridden accidentally. Otherwise, please report this as a bug: https://knapsackpro.com/perma/ruby/support")
-      end
-    end
-
-    context 'with --force-color' do
-      let(:rspec_args) { '--force-color' }
-      let(:expected_args) { ['--force-color'] }
-      let(:expected_format) { 'json' }
-
-      it_behaves_like 'generate_json_report runs RSpec::Core::Runner'
-    end
-
-    context 'with --no-color and --force-color' do
-      let(:rspec_args) { '--no-color --force-color' }
-
-      after { KnapsackPro.reset_logger! }
-
-      it do
-        subject_class = Class.new(KnapsackPro::TestCaseDetectors::RSpecTestExampleDetector) do
-          define_method(:slow_test_files) do
-            [{ 'path' => 'spec/a_spec.rb' }]
-          end
-        end
-
-        expect do
-          KnapsackPro.logger = ::Logger.new($stdout)
-          subject_class.new.generate_json_report(rspec_args)
-        end
-          .to output(/Please only use one of `--force-color` and `--no-color`/).to_stderr
-          .and output(%r{ERROR -- knapsack_pro: Failed to generate the slow test files report: bundle exec rspec --no-color --force-color --format json --dry-run --out .knapsack_pro/test_case_detectors/rspec/rspec_dry_run_json_report_node_0.json --default-path spec spec/a_spec.rb}).to_stdout
-          .and raise_error(SystemExit) { |error| expect(error.status).to eq 1 }
-      end
-    end
-  end
-
-  describe '#test_file_example_paths' do
-    subject { described_class.new.test_file_example_paths }
-
-    context 'when JSON report exists' do
-      it do
-        expect(File).to receive(:exist?).with(report_path).and_return(true)
-
-        json_file = {
-          'examples' => [
-            { id: './spec/a_spec.rb[1:1]' },
-            { id: './spec/a_spec.rb[1:2]' },
-          ]
-        }.to_json
-        expect(File).to receive(:read).with(report_path).and_return(json_file)
-
-        expect(subject).to eq([
-          { 'path' => 'spec/a_spec.rb[1:1]' },
-          { 'path' => 'spec/a_spec.rb[1:2]' },
-        ])
-      end
-    end
-
-    context 'when JSON report does not exist' do
-      it do
-        expect(File).to receive(:exist?).with(report_path).and_return(false)
-
-        expect { subject }.to raise_error(RuntimeError, "No report found at .knapsack_pro/test_case_detectors/rspec/rspec_dry_run_json_report_node_0.json")
-      end
-    end
-  end
-
-  describe '#slow_test_files' do
-    subject { described_class.new.slow_test_files }
-
-    before do
-      expect(KnapsackPro::Config::Env).to receive(:slow_test_file_pattern).and_return(slow_test_file_pattern)
-    end
-
-    context 'when slow test file pattern is present' do
-      let(:slow_test_file_pattern) { double }
-
-      it do
-        expected_slow_test_files = double
-        expect(KnapsackPro::TestFileFinder).to receive(:slow_test_files_by_pattern).with(KnapsackPro::Adapters::RSpecAdapter).and_return(expected_slow_test_files)
-
-        expect(subject).to eq expected_slow_test_files
-      end
-    end
-
-    context 'when slow test file pattern is not present' do
-      let(:slow_test_file_pattern) { nil }
-
-      it do
-        expected_slow_test_files = double
-        expect(KnapsackPro::SlowTestFileDeterminer).to receive(:read_from_json_report).and_return(expected_slow_test_files)
-
-        expect(subject).to eq expected_slow_test_files
-      end
-    end
-  end
-end

data/spec/knapsack_pro/test_case_mergers/base_merger_spec.rb

@@ -1,27 +0,0 @@
-describe KnapsackPro::TestCaseMergers::BaseMerger do
-  describe '.call' do
-    let(:test_files) { double }
-
-    subject { described_class.call(adapter_class, test_files) }
-
-    context 'when adapter_class is KnapsackPro::Adapters::RSpecAdapter' do
-      let(:adapter_class) { KnapsackPro::Adapters::RSpecAdapter }
-
-      it do
-        result = double
-        rspec_merger = instance_double(KnapsackPro::TestCaseMergers::RSpecMerger, call: result)
-        expect(KnapsackPro::TestCaseMergers::RSpecMerger).to receive(:new).with(test_files).and_return(rspec_merger)
-
-        expect(subject).to eq result
-      end
-    end
-
-    context 'when adapter_class is unknown' do
-      let(:adapter_class) { 'fake-adapter' }
-
-      it do
-        expect { subject }.to raise_error 'Test case merger does not exist for adapter_class: fake-adapter'
-      end
-    end
-  end
-end

data/spec/knapsack_pro/test_case_mergers/rspec_merger_spec.rb

@@ -1,81 +0,0 @@
-describe KnapsackPro::TestCaseMergers::RSpecMerger do
-  describe '#call' do
-    subject { described_class.new(test_files).call }
-
-    context 'when test files are regular file paths (not test example paths)' do
-      let(:test_files) do
-        [
-          { 'path' => 'spec/a_spec.rb', 'time_execution' => 1.1 },
-          { 'path' => 'spec/b_spec.rb', 'time_execution' => 2.2 },
-        ]
-      end
-
-      it 'returns the test files unchanged' do
-        expect(subject).to eq([
-          { 'path' => 'spec/a_spec.rb', 'time_execution' => 1.1 },
-          { 'path' => 'spec/b_spec.rb', 'time_execution' => 2.2 },
-        ])
-      end
-    end
-
-    context 'when test files have test example paths' do
-      let(:test_files) do
-        [
-          { 'path' => 'spec/a_spec.rb', 'time_execution' => 1.1 },
-          # test example paths
-          { 'path' => 'spec/test_case_spec.rb[1:1]', 'time_execution' => 2.2 },
-          { 'path' => 'spec/test_case_spec.rb[1:2]', 'time_execution' => 0.8 },
-        ]
-      end
-
-      it 'merges the test example paths and sums their execution times' do
-        expect(subject).to eq([
-          { 'path' => 'spec/a_spec.rb', 'time_execution' => 1.1 },
-          { 'path' => 'spec/test_case_spec.rb', 'time_execution' => 3.0 },
-        ])
-      end
-    end
-
-    context 'when test files have test example paths and the full test file path exists simultaneously' do
-      context 'when the full test file path has a higher execution time than the sum of test example paths' do
-        let(:test_files) do
-          [
-            { 'path' => 'spec/a_spec.rb', 'time_execution' => 1.1 },
-            # full test file path exists alongside test example paths
-            { 'path' => 'spec/test_case_spec.rb', 'time_execution' => 3.1 },
-            # test example paths
-            { 'path' => 'spec/test_case_spec.rb[1:1]', 'time_execution' => 2.2 },
-            { 'path' => 'spec/test_case_spec.rb[1:2]', 'time_execution' => 0.8 },
-          ]
-        end
-
-        it 'returns the full test file path execution time' do
-          expect(subject).to eq([
-            { 'path' => 'spec/a_spec.rb', 'time_execution' => 1.1 },
-            { 'path' => 'spec/test_case_spec.rb', 'time_execution' => 3.1 },
-          ])
-        end
-      end
-
-      context 'when the full test file path has a lower execution time than the sum of test example paths' do
-        let(:test_files) do
-          [
-            { 'path' => 'spec/a_spec.rb', 'time_execution' => 1.1 },
-            # full test file path exists alongside test example paths
-            { 'path' => 'spec/test_case_spec.rb', 'time_execution' => 2.9 },
-            # test example paths
-            { 'path' => 'spec/test_case_spec.rb[1:1]', 'time_execution' => 2.2 },
-            { 'path' => 'spec/test_case_spec.rb[1:2]', 'time_execution' => 0.8 },
-          ]
-        end
-
-        it "returns the sum of test example paths' execution times" do
-          expect(subject).to eq([
-            { 'path' => 'spec/a_spec.rb', 'time_execution' => 1.1 },
-            { 'path' => 'spec/test_case_spec.rb', 'time_execution' => 3.0 },
-          ])
-        end
-      end
-    end
-  end
-end

data/spec/knapsack_pro/test_file_cleaner_spec.rb

@@ -1,11 +0,0 @@
-describe KnapsackPro::TestFileCleaner do
-  describe '.clean' do
-    let(:test_file_path) { './models/user_spec.rb' }
-
-    subject { described_class.clean(test_file_path) }
-
-    it 'removes ./ from the begining of the test file path' do
-      expect(subject).to eq 'models/user_spec.rb'
-    end
-  end
-end