knapsack_pro 7.14.0 → 8.0.0

data/spec/integration/runners/queue/rspec_runner_spec.rb

@@ -1959,23 +1959,20 @@ describe "#{KnapsackPro::Runners::Queue::RSpecRunner} - Integration tests", :cle
     end
   end
 
-  context 'when the RSpec split by examples is enabled' do
+  context 'when the RSpec split by test examples is enabled' do
     before do
-      ENV['KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES'] = 'true'
-
-      # remember to stub Queue API batches to include test examples (example: a_spec.rb[1:1])
-      # for the following slow test files
+      # Remember to stub the Queue API batches to include test examples (example: a_spec.rb[1:1])
+      # for the following slow test files.
       ENV['KNAPSACK_PRO_SLOW_TEST_FILE_PATTERN'] = "#{SPEC_DIRECTORY}/a_spec.rb"
 
       ENV['KNAPSACK_PRO_CI_NODE_TOTAL'] = '2'
     end
     after do
-      ENV.delete('KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES')
       ENV.delete('KNAPSACK_PRO_SLOW_TEST_FILE_PATTERN')
       ENV.delete('KNAPSACK_PRO_CI_NODE_TOTAL')
     end
 
-    it 'splits slow test files by examples AND ensures the test examples are executed only once' do
+    it 'splits slow test files by test examples AND ensures the test examples are executed only once' do
       rspec_options = '--format d'
 
       spec_a = Spec.new('a_spec.rb', <<~SPEC)
@@ -2058,23 +2055,20 @@ describe "#{KnapsackPro::Runners::Queue::RSpecRunner} - Integration tests", :cle
     end
   end
 
-  context 'when the RSpec split by examples is enabled AND --tag is set' do
+  context 'when the RSpec split by test examples is enabled AND --tag is set' do
     before do
-      ENV['KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES'] = 'true'
-
-      # remember to stub Queue API batches to include test examples (example: a_spec.rb[1:1])
-      # for the following slow test files
+      # Remember to stub the Queue API batches to include test examples (example: a_spec.rb[1:2])
+      # for the following slow test files.
       ENV['KNAPSACK_PRO_SLOW_TEST_FILE_PATTERN'] = "#{SPEC_DIRECTORY}/a_spec.rb"
 
       ENV['KNAPSACK_PRO_CI_NODE_TOTAL'] = '2'
     end
     after do
-      ENV.delete('KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES')
       ENV.delete('KNAPSACK_PRO_SLOW_TEST_FILE_PATTERN')
       ENV.delete('KNAPSACK_PRO_CI_NODE_TOTAL')
     end
 
-    it 'sets 1 as exit code AND raises an error (a test example path as a_spec.rb[1:1] would always be executed even when it does not have the tag that is set via the --tag option. We cannot run tests because it could lead to running unintentional tests)' do
+    it 'splits slow test files by test examples AND ensures the test examples are executed only once' do
       rspec_options = '--format d --tag my_tag'
 
       spec_a = Spec.new('a_spec.rb', <<~SPEC)
@@ -2082,7 +2076,7 @@ describe "#{KnapsackPro::Runners::Queue::RSpecRunner} - Integration tests", :cle
           it 'A1 test example' do
             expect(1).to eq 1
           end
-          it 'A2 test example' do
+          it 'A2 test example', :my_tag do
             expect(1).to eq 1
           end
         end
@@ -2101,7 +2095,7 @@ describe "#{KnapsackPro::Runners::Queue::RSpecRunner} - Integration tests", :cle
 
       spec_c = Spec.new('c_spec.rb', <<~SPEC)
         describe 'C_describe' do
-          it 'C1 test example' do
+          it 'C1 test example', :my_tag do
             expect(1).to eq 1
           end
           it 'C2 test example' do
@@ -2114,43 +2108,39 @@ describe "#{KnapsackPro::Runners::Queue::RSpecRunner} - Integration tests", :cle
         [spec_a, spec_b, spec_c]
       ])
       stub_test_cases_for_slow_test_files([
-        "#{spec_a.path}[1:1]",
-        "#{spec_a.path}[1:2]",
+        "#{spec_a.path}[1:2]", # only this test example is tagged
       ])
       stub_spec_batches([
-        ["#{spec_a.path}[1:1]", spec_b.path],
+        [spec_b.path],
         ["#{spec_a.path}[1:2]", spec_c.path],
       ])
 
       actual = subject
 
-      expect(actual.stdout).to include('ERROR -- : [knapsack_pro] It is not allowed to use the RSpec tag option together with the RSpec split by test examples feature. Please see: https://knapsackpro.com/perma/ruby/rspec-split-by-test-examples-tag')
-
       expect(actual.stdout).to_not include('A1 test example')
-      expect(actual.stdout).to_not include('A2 test example')
-      expect(actual.stdout).to_not include('B1 test example')
-      expect(actual.stdout).to_not include('B2 test example')
-      expect(actual.stdout).to_not include('C1 test example')
+      expect(actual.stdout).to include('A2 test example')
+      expect(actual.stdout).to include('B1 test example')
+      expect(actual.stdout).to include('B2 test example')
+      expect(actual.stdout).to include('C1 test example')
       expect(actual.stdout).to_not include('C2 test example')
 
-      expect(actual.exit_code).to eq 1
+      expect(actual.stdout).to include('4 examples, 0 failures')
+
+      expect(actual.exit_code).to eq 0
     end
   end
 
-  context 'when the RSpec split by examples is enabled AND JSON formatter is used' do
+  context 'when the RSpec split by test examples is enabled AND JSON formatter is used' do
     let(:json_file) { "#{SPEC_DIRECTORY}/rspec.json" }
 
     before do
-      ENV['KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES'] = 'true'
-
-      # remember to stub Queue API batches to include test examples (example: a_spec.rb[1:1])
-      # for the following slow test files
+      # Remember to stub the Queue API batches to include test examples (example: a_spec.rb[1:1])
+      # for the following slow test files.
       ENV['KNAPSACK_PRO_SLOW_TEST_FILE_PATTERN'] = "#{SPEC_DIRECTORY}/a_spec.rb"
 
       ENV['KNAPSACK_PRO_CI_NODE_TOTAL'] = '2'
     end
     after do
-      ENV.delete('KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES')
       ENV.delete('KNAPSACK_PRO_SLOW_TEST_FILE_PATTERN')
       ENV.delete('KNAPSACK_PRO_CI_NODE_TOTAL')
     end
@@ -2239,20 +2229,17 @@ describe "#{KnapsackPro::Runners::Queue::RSpecRunner} - Integration tests", :cle
     end
   end
 
-  context 'when the RSpec split by examples is enabled AND JUnit XML formatter is used' do
+  context 'when the RSpec split by test examples is enabled AND JUnit XML formatter is used' do
     let(:xml_file) { "#{SPEC_DIRECTORY}/rspec.xml" }
 
     before do
-      ENV['KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES'] = 'true'
-
-      # remember to stub Queue API batches to include test examples (example: a_spec.rb[1:1])
-      # for the following slow test files
+      # Remember to stub the Queue API batches to include test examples (example: a_spec.rb[1:1])
+      # for the following slow test files.
       ENV['KNAPSACK_PRO_SLOW_TEST_FILE_PATTERN'] = "#{SPEC_DIRECTORY}/a_spec.rb"
 
       ENV['KNAPSACK_PRO_CI_NODE_TOTAL'] = '2'
     end
     after do
-      ENV.delete('KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES')
       ENV.delete('KNAPSACK_PRO_SLOW_TEST_FILE_PATTERN')
       ENV.delete('KNAPSACK_PRO_CI_NODE_TOTAL')
     end
@@ -2338,21 +2325,18 @@ describe "#{KnapsackPro::Runners::Queue::RSpecRunner} - Integration tests", :cle
     end
   end
 
-  context 'when the RSpec split by examples is enabled AND simplecov is used' do
+  context 'when the RSpec split by test examples is enabled AND simplecov is used' do
     let(:coverage_dir) { "#{KNAPSACK_PRO_TMP_DIR}/coverage" }
     let(:coverage_file) { "#{coverage_dir}/index.html" }
 
     before do
-      ENV['KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES'] = 'true'
-
-      # remember to stub Queue API batches to include test examples (example: a_spec.rb[1:1])
-      # for the following slow test files
+      # Remember to stub the Queue API batches to include test examples (example: a_spec.rb[1:1])
+      # for the following slow test files.
       ENV['KNAPSACK_PRO_SLOW_TEST_FILE_PATTERN'] = "#{SPEC_DIRECTORY}/a_spec.rb"
 
       ENV['KNAPSACK_PRO_CI_NODE_TOTAL'] = '2'
     end
     after do
-      ENV.delete('KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES')
       ENV.delete('KNAPSACK_PRO_SLOW_TEST_FILE_PATTERN')
       ENV.delete('KNAPSACK_PRO_CI_NODE_TOTAL')
     end
@@ -2427,6 +2411,104 @@ describe "#{KnapsackPro::Runners::Queue::RSpecRunner} - Integration tests", :cle
     end
   end
 
+  context 'when the RSpec split by test examples is enabled AND test files are split by test examples AND slow test files are not detected (for example, the user could have passed test examples like a_spec.rb[1:1] directly using KNAPSACK_PRO_TEST_FILE_LIST_SOURCE_FILE or KNAPSACK_PRO_TEST_FILE_LIST)' do
+    before do
+      ENV['KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES'] = 'true'
+      ENV['KNAPSACK_PRO_SLOW_TEST_FILE_PATTERN'] = ""
+      ENV['KNAPSACK_PRO_CI_NODE_TOTAL'] = '2'
+    end
+    after do
+      ENV.delete('KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES')
+      ENV.delete('KNAPSACK_PRO_SLOW_TEST_FILE_PATTERN')
+      ENV.delete('KNAPSACK_PRO_CI_NODE_TOTAL')
+    end
+
+    it 'detects test execution times correctly for individual test examples even though they are not considered slow test files' do
+      ENV['TEST__LOG_EXECUTION_TIMES'] = 'true'
+      rspec_options = '--format d'
+
+      spec_a = Spec.new('a_spec.rb', <<~SPEC)
+        describe 'A_describe' do
+          it 'A1 test example' do
+            expect(1).to eq 1
+          end
+          it 'A2 test example' do
+            expect(1).to eq 1
+          end
+        end
+      SPEC
+
+      spec_b = Spec.new('b_spec.rb', <<~SPEC)
+        describe 'B_describe' do
+          it 'B1 test example' do
+            expect(1).to eq 1
+          end
+          it 'B2 test example' do
+            expect(1).to eq 1
+          end
+        end
+      SPEC
+
+      spec_c = Spec.new('c_spec.rb', <<~SPEC)
+        describe 'C_describe' do
+          it 'C1 test example' do
+            expect(1).to eq 1
+          end
+          it 'C2 test example' do
+            expect(1).to eq 1
+          end
+        end
+      SPEC
+
+      generate_specs(spec_helper_with_knapsack, rspec_options, [
+        [spec_a, spec_b, spec_c]
+      ])
+      stub_test_cases_for_slow_test_files([
+        "#{spec_a.path}[1:1]",
+        "#{spec_a.path}[1:2]",
+      ])
+      stub_spec_batches([
+        ["#{spec_a.path}[1:1]", spec_b.path],
+        ["#{spec_a.path}[1:2]", spec_c.path],
+      ])
+
+      actual = subject
+
+      expect(actual.stdout).to include('DEBUG -- : [knapsack_pro] Detected 0 slow test files: []')
+
+      expect(actual.stdout).to include(
+        <<~OUTPUT
+        A_describe
+          A1 test example
+
+        B_describe
+          B1 test example
+          B2 test example
+        OUTPUT
+      )
+
+      expect(actual.stdout).to include(
+        <<~OUTPUT
+        A_describe
+          A2 test example
+
+        C_describe
+          C1 test example
+          C2 test example
+        OUTPUT
+      )
+
+      expect(actual.stdout.scan(/A1 test example/).size).to eq 1
+      expect(actual.stdout.scan(/A2 test example/).size).to eq 1
+
+      expect(actual.stdout).to include('6 examples, 0 failures')
+
+      expect(actual.stdout).to include('[INTEGRATION TEST] test_files: 4, test files have execution time: true')
+
+      expect(actual.exit_code).to eq 0
+    end
+  end
+
   context 'when the example_status_persistence_file_path option is used and multiple batches of tests are fetched from the Queue API and some tests are pending and failing' do
     let(:examples_file_path) { "#{SPEC_DIRECTORY}/examples.txt" }
 

data/spec/knapsack_pro/adapters/base_adapter_spec.rb

@@ -135,21 +135,21 @@ describe KnapsackPro::Adapters::BaseAdapter do
 
   describe '#bind' do
     let(:temp_directory_path) { '.knapsack_pro' }
-    let(:recording_enabled?) { false }
-    let(:queue_recording_enabled?) { false }
+    let(:regular_mode?) { false }
+    let(:queue_mode?) { false }
 
     before do
       expect(KnapsackPro::Config::TempFiles).to receive(:ensure_temp_directory_exists!)
       expect(File).to receive(:write).with('.knapsack_pro/KnapsackPro-Adapters-BaseAdapter-bind_method_called_for_node_0.txt', nil)
 
-      expect(KnapsackPro::Config::Env).to receive(:recording_enabled?).and_return(recording_enabled?)
-      expect(KnapsackPro::Config::Env).to receive(:queue_recording_enabled?).and_return(queue_recording_enabled?)
+      expect(KnapsackPro::Config::Env).to receive(:regular_mode?).and_return(regular_mode?)
+      expect(KnapsackPro::Config::Env).to receive(:queue_mode?).and_return(queue_mode?)
     end
 
     after { subject.bind }
 
-    context 'when recording enabled' do
-      let(:recording_enabled?) { true }
+    context 'when regular mode enabled' do
+      let(:regular_mode?) { true }
 
       before do
         allow(subject).to receive(:bind_time_tracker)
@@ -165,8 +165,8 @@ describe KnapsackPro::Adapters::BaseAdapter do
       it { expect(subject).to receive(:bind_save_report) }
     end
 
-    context 'when queue recording enabled' do
-      let(:queue_recording_enabled?) { true }
+    context 'when queue mode enabled' do
+      let(:queue_mode?) { true }
 
       it 'calls queue hooks in proper order before binding time tracker' do
         logger = instance_double(Logger)
@@ -179,7 +179,7 @@ describe KnapsackPro::Adapters::BaseAdapter do
       end
     end
 
-    context 'when recording disabled' do
+    context 'when regular mode and queue mode disabled' do
       it { expect(subject).not_to receive(:bind_save_report) }
       it { expect(subject).not_to receive(:bind_before_queue_hook) }
       it { expect(subject).not_to receive(:bind_after_queue_hook) }

data/spec/knapsack_pro/adapters/rspec_adapter_spec.rb

@@ -89,76 +89,6 @@ describe KnapsackPro::Adapters::RSpecAdapter do
     end
   end
 
-  describe '.ensure_no_tag_option_when_rspec_split_by_test_examples_enabled!' do
-    let(:cli_args) { double }
-
-    subject { described_class.ensure_no_tag_option_when_rspec_split_by_test_examples_enabled!(cli_args) }
-
-    before do
-      expect(KnapsackPro::Config::Env).to receive(:rspec_split_by_test_examples?).and_return(rspec_split_by_test_examples_enabled)
-    end
-
-    context 'when RSpec split by test examples enabled' do
-      let(:rspec_split_by_test_examples_enabled) { true }
-
-      before do
-        expect(described_class).to receive(:has_tag_option?).with(cli_args).and_return(has_tag_option)
-      end
-
-      context 'when RSpec tag option is provided' do
-        let(:has_tag_option) { true }
-
-        it do
-          expect { subject }.to raise_error(/It is not allowed to use the RSpec tag option together with the RSpec split by test examples feature/)
-        end
-      end
-
-      context 'when RSpec tag option is not provided' do
-        let(:has_tag_option) { false }
-
-        it 'does nothing' do
-          expect(subject).to be_nil
-        end
-      end
-    end
-
-    context 'when RSpec split by test examples disabled' do
-      let(:rspec_split_by_test_examples_enabled) { false }
-
-      it 'does nothing' do
-        expect(subject).to be_nil
-      end
-    end
-  end
-
-  describe '.has_tag_option?' do
-    subject { described_class.has_tag_option?(cli_args) }
-
-    context 'when tag option is provided as -t' do
-      let(:cli_args) { ['-t', 'mytag'] }
-
-      it { expect(subject).to be true }
-    end
-
-    context 'when tag option is provided as --tag' do
-      let(:cli_args) { ['--tag', 'mytag'] }
-
-      it { expect(subject).to be true }
-    end
-
-    context 'when tag option is provided without delimiter' do
-      let(:cli_args) { ['-tmytag'] }
-
-      it { expect(subject).to be true }
-    end
-
-    context 'when tag option is not provided' do
-      let(:cli_args) { ['--fake', 'value'] }
-
-      it { expect(subject).to be false }
-    end
-  end
-
   describe '.has_format_option?' do
     subject { described_class.has_format_option?(cli_args) }
 
@@ -215,6 +145,28 @@ describe KnapsackPro::Adapters::RSpecAdapter do
     end
   end
 
+  describe '.id_path?' do
+    subject { described_class.id_path?(path) }
+
+    context 'when the path resembles the RSpec path with id' do
+      let(:path) { 'spec/features/a_spec.rb[1:1:7:1]' }
+
+      it { is_expected.to be true }
+    end
+
+    context 'when the path resembles the RSpec path with multiple ids' do
+      let(:path) { 'spec/features/a_spec.rb[1:1:7:1, 1:2]' }
+
+      it { is_expected.to be true }
+    end
+
+    context "when the path doesn't resemble the RSpec path with id" do
+      let(:path) { 'spec/features/a_spec.rb' }
+
+      it { is_expected.to be false }
+    end
+  end
+
   describe '.rails_helper_exists?' do
     subject { described_class.rails_helper_exists?(test_dir) }
 
@@ -293,6 +245,18 @@ describe KnapsackPro::Adapters::RSpecAdapter do
     end
   end
 
+  describe '.remove_formatters' do
+    subject { described_class.remove_formatters(cli_args) }
+
+    context "when CLI args include formatters" do
+      let(:cli_args) { ['-t', 'awesome_tag', '-f', 'documentation', '-o', '/tmp/documentation.txt', '--tag', 'mytag', '--format', 'json', '--out', '/tmp/file.json', '--no-color'] }
+
+      it 'removes formatters and the related output file options' do
+        expect(subject).to eq ['-t', 'awesome_tag', '--tag', 'mytag', '--no-color']
+      end
+    end
+  end
+
   describe '.file_path_for' do
     let(:current_example) { ::RSpec.describe.example }
 
@@ -370,6 +334,26 @@ describe KnapsackPro::Adapters::RSpecAdapter do
     end
   end
 
+  describe 'private .scheduled_paths' do
+    subject { described_class.send(:scheduled_paths) }
+
+    context 'when the RSpec configuration has files or directories to run' do
+      it 'returns list of test files passed to RSpec (if this fails then the internal RSpec API changed and we must start supporting a new RSpec version as well)' do
+        expect(subject).not_to be_empty
+      end
+    end
+
+    context "when the RSpec configuration has no files or directories to run (when the internal RSpec API changed and we don't support the new RSpec version yet)" do
+      before do
+        expect(described_class).to receive(:rspec_configuration).and_return(double(:object_with_no_instance_variables))
+      end
+
+      it 'fallbacks to an empty array to not blow up Knapsack Pro' do
+        expect(subject).to eq([])
+      end
+    end
+  end
+
   describe 'bind methods' do
     let(:config) { double }
 

data/spec/knapsack_pro/config/env_spec.rb

@@ -603,89 +603,43 @@ describe KnapsackPro::Config::Env do
     end
   end
 
-  describe '.recording_enabled' do
-    subject { described_class.recording_enabled }
-
-    context 'when ENV exists' do
-      let(:recording_enabled) { 'true' }
-      before { stub_const("ENV", { 'KNAPSACK_PRO_RECORDING_ENABLED' => recording_enabled }) }
-      it { should eq recording_enabled }
-    end
-
-    context "when ENV doesn't exist" do
-      it { should be_nil }
-    end
-  end
-
   describe '.regular_mode?' do
     subject { described_class.regular_mode? }
 
-    before do
-      expect(described_class).to receive(:recording_enabled?).and_return(recording_enabled)
-    end
-
-    context 'when recording is enabled' do
-      let(:recording_enabled) { true }
-      it { should be true }
-    end
-
-    context 'when recording is not enabled' do
-      let(:recording_enabled) { false }
-      it { should be false }
-    end
-  end
-
-  describe '.recording_enabled?' do
-    subject { described_class.recording_enabled? }
-
-    before do
-      expect(described_class).to receive(:recording_enabled).and_return(recording_enabled)
-    end
-
-    context 'when enabled' do
-      let(:recording_enabled) { 'true' }
-
-      it { should be true }
-    end
-
-    context 'when disabled' do
-      let(:recording_enabled) { nil }
-
-      it { should be false }
+    context 'when regular mode is enabled' do
+      let(:regular_mode_enabled) { 'true' }
+      before { stub_const("ENV", { 'KNAPSACK_PRO_REGULAR_MODE_ENABLED' => regular_mode_enabled }) }
+      it { should eq true }
     end
-  end
 
-  describe '.queue_recording_enabled' do
-    subject { described_class.queue_recording_enabled }
-
-    context 'when ENV exists' do
-      let(:queue_recording_enabled) { 'true' }
-      before { stub_const("ENV", { 'KNAPSACK_PRO_QUEUE_RECORDING_ENABLED' => queue_recording_enabled }) }
-      it { should eq queue_recording_enabled }
+    context 'when regular mode is disabled' do
+      let(:regular_mode_enabled) { 'false' }
+      before { stub_const("ENV", { 'KNAPSACK_PRO_REGULAR_MODE_ENABLED' => regular_mode_enabled }) }
+      it { should eq false }
     end
 
     context "when ENV doesn't exist" do
-      it { should be_nil }
+      it { should false }
     end
   end
 
-  describe '.queue_recording_enabled?' do
-    subject { described_class.queue_recording_enabled? }
+  describe '.queue_mode?' do
+    subject { described_class.queue_mode? }
 
-    before do
-      expect(described_class).to receive(:queue_recording_enabled).and_return(queue_recording_enabled)
+    context 'when queue mode is enabled' do
+      let(:queue_mode_enabled) { 'true' }
+      before { stub_const("ENV", { 'KNAPSACK_PRO_QUEUE_MODE_ENABLED' => queue_mode_enabled }) }
+      it { should eq true }
     end
 
-    context 'when enabled' do
-      let(:queue_recording_enabled) { 'true' }
-
-      it { should be true }
+    context 'when queue mode is disabled' do
+      let(:queue_mode_enabled) { 'false' }
+      before { stub_const("ENV", { 'KNAPSACK_PRO_QUEUE_MODE_ENABLED' => queue_mode_enabled }) }
+      it { should eq false }
     end
 
-    context 'when disabled' do
-      let(:queue_recording_enabled) { nil }
-
-      it { should be false }
+    context "when ENV doesn't exist" do
+      it { should false }
     end
   end
 
@@ -1016,51 +970,43 @@ describe KnapsackPro::Config::Env do
   describe '.rspec_split_by_test_examples?' do
     subject { described_class.rspec_split_by_test_examples? }
 
-    after(:each) do
-      described_class.remove_instance_variable(:@rspec_split_by_test_examples)
-    end
-
-    context 'when KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES=true AND KNAPSACK_PRO_CI_NODE_TOTAL >= 2' do
-      before do
-        stub_const("ENV", { 'KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES' => 'true', 'KNAPSACK_PRO_CI_NODE_TOTAL' => '2' })
-        expect(KnapsackPro).not_to receive(:logger)
-      end
-
-      it { should be true }
+    before do
+      described_class.remove_instance_variable(:@rspec_split_by_test_examples) if described_class.instance_variable_defined?(:@rspec_split_by_test_examples)
     end
-
-    context 'when KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES=false AND KNAPSACK_PRO_CI_NODE_TOTAL >= 2' do
-      before do
-        stub_const("ENV", { 'KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES' => 'false', 'KNAPSACK_PRO_CI_NODE_TOTAL' => '2' })
-        expect(KnapsackPro).not_to receive(:logger)
-      end
-
-      it { should be false }
+    after do
+      described_class.remove_instance_variable(:@rspec_split_by_test_examples)
     end
 
-    context 'when KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES=true AND KNAPSACK_PRO_CI_NODE_TOTAL < 2' do
-      before { stub_const("ENV", { 'KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES' => 'true', 'KNAPSACK_PRO_CI_NODE_TOTAL' => '1' }) }
-
-      it { should be false }
-
-      context 'when called twice' do
-        it 'logs a debug message only once' do
-          logger = instance_double(Logger)
-          expect(KnapsackPro).to receive(:logger).and_return(logger)
-          expect(logger).to receive(:debug).with('Skipping split of test files by test examples because you are running tests on a single CI node (no parallelism)')
+    [
+      ['false', '2', nil, false],
+      ['true',  '2', nil, true],
+      [nil,     '2', nil, true],
+      ['false', '1', nil, false],
+      ['true',  '1', nil, false, :debug, 'Skipping split by test examples because tests are running on a single CI node (no parallelism)'],
+      [nil,     '1', nil, false, :debug, 'Skipping split by test examples because tests are running on a single CI node (no parallelism)'],
+      ['false', '2', 'true', false],
+      ['true',  '2', 'true', true],
+      [nil,     '2', 'true', false, :warn, "Skipping split by test examples because test file names encryption is enabled:\nhttps://knapsackpro.com/perma/ruby/encryption\nhttps://knapsackpro.com/perma/ruby/split-by-test-examples"],
+      ['false', '1', 'true', false],
+      ['true',  '1', 'true', false, :debug, 'Skipping split by test examples because tests are running on a single CI node (no parallelism)'],
+      [nil,     '1', 'true', false, :warn, "Skipping split by test examples because test file names encryption is enabled:\nhttps://knapsackpro.com/perma/ruby/encryption\nhttps://knapsackpro.com/perma/ruby/split-by-test-examples"],
+    ].each do |sbte, node_total, encrypted, expected, log_level, log_message|
+      context "KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES=#{sbte.inspect} AND KNAPSACK_PRO_CI_NODE_TOTAL=#{node_total.inspect} AND KNAPSACK_PRO_TEST_FILES_ENCRYPTED=#{encrypted.inspect}" do
+        before do
+          stub_const("ENV", { 'KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES' => sbte, 'KNAPSACK_PRO_CI_NODE_TOTAL' => node_total, 'KNAPSACK_PRO_TEST_FILES_ENCRYPTED' => encrypted }.compact)
 
-          2.times { described_class.rspec_split_by_test_examples? }
+          if log_level && log_message
+            logger = instance_double(Logger)
+            expect(KnapsackPro).to receive(:logger).and_return(logger)
+            expect(logger).to receive(log_level).once.with(log_message)
+          end
         end
-      end
-    end
 
-    context "when ENV doesn't exist" do
-      before do
-        stub_const("ENV", {})
-        expect(KnapsackPro).not_to receive(:logger)
+        it do
+          expect(described_class.rspec_split_by_test_examples?).to eq(expected)
+          expect(described_class.rspec_split_by_test_examples?).to eq(expected)
+        end
       end
-
-      it { should be false }
     end
   end