mutant 0.8.24 → 0.9.0

data/spec/unit/mutant/reporter/cli/printer/config_spec.rb

@@ -6,26 +6,12 @@ RSpec.describe Mutant::Reporter::CLI::Printer::Config do
   let(:reportable) { config }
 
   describe '.call' do
-    context 'on default config' do
-      it_reports(<<~REPORT)
-        Mutant configuration:
-        Matcher:         #<Mutant::Matcher::Config empty>
-        Integration:     Mutant::Integration::Null
-        Jobs:            1
-        Includes:        []
-        Requires:        []
-      REPORT
-    end
-
-    context 'with non default coverage expectation' do
-      it_reports(<<~REPORT)
-        Mutant configuration:
-        Matcher:         #<Mutant::Matcher::Config empty>
-        Integration:     Mutant::Integration::Null
-        Jobs:            1
-        Includes:        []
-        Requires:        []
-      REPORT
-    end
+    it_reports(<<~'REPORT')
+      Matcher:         #<Mutant::Matcher::Config empty>
+      Integration:     null
+      Jobs:            1
+      Includes:        []
+      Requires:        []
+    REPORT
   end
 end

data/spec/unit/mutant/reporter/cli/printer/env_progress_spec.rb

@@ -10,13 +10,16 @@ RSpec.describe Mutant::Reporter::CLI::Printer::EnvProgress do
       with(:subject_a_result) { { mutation_results: [] } }
 
       it_reports <<~'STR'
-        Mutant configuration:
+        Mutant environment:
         Matcher:         #<Mutant::Matcher::Config empty>
-        Integration:     Mutant::Integration::Null
+        Integration:     null
         Jobs:            1
         Includes:        []
         Requires:        []
         Subjects:        1
+        Total-Tests:     1
+        Selected-Tests:  1
+        Tests/Subject:   1.00 avg
         Mutations:       2
         Results:         0
         Kills:           0
@@ -31,13 +34,16 @@ RSpec.describe Mutant::Reporter::CLI::Printer::EnvProgress do
 
     context 'on full coverage' do
       it_reports <<~'STR'
-        Mutant configuration:
+        Mutant environment:
         Matcher:         #<Mutant::Matcher::Config empty>
-        Integration:     Mutant::Integration::Null
+        Integration:     null
         Jobs:            1
         Includes:        []
         Requires:        []
         Subjects:        1
+        Total-Tests:     1
+        Selected-Tests:  1
+        Tests/Subject:   1.00 avg
         Mutations:       2
         Results:         2
         Kills:           2
@@ -54,13 +60,16 @@ RSpec.describe Mutant::Reporter::CLI::Printer::EnvProgress do
       with(:mutation_a_test_result) { { passed: true } }
 
       it_reports <<~'STR'
-        Mutant configuration:
+        Mutant environment:
         Matcher:         #<Mutant::Matcher::Config empty>
-        Integration:     Mutant::Integration::Null
+        Integration:     null
         Jobs:            1
         Includes:        []
         Requires:        []
         Subjects:        1
+        Total-Tests:     1
+        Selected-Tests:  1
+        Tests/Subject:   1.00 avg
         Mutations:       2
         Results:         2
         Kills:           1

data/spec/unit/mutant/reporter/cli/printer/env_result_spec.rb

@@ -16,13 +16,21 @@ RSpec.describe Mutant::Reporter::CLI::Printer::EnvResult do
       -true
       +false
       -----------------------
-      Mutant configuration:
+      - 1 @ runtime: 1.0
+        - test-a
+      Test Output:
+      mutation a test result output
+      -----------------------
+      Mutant environment:
       Matcher:         #<Mutant::Matcher::Config empty>
-      Integration:     Mutant::Integration::Null
+      Integration:     null
       Jobs:            1
       Includes:        []
       Requires:        []
       Subjects:        1
+      Total-Tests:     1
+      Selected-Tests:  1
+      Tests/Subject:   1.00 avg
       Mutations:       2
       Results:         2
       Kills:           1

data/spec/unit/mutant/reporter/cli/printer/isolation_result_spec.rb

@@ -77,13 +77,17 @@ RSpec.describe Mutant::Reporter::CLI::Printer::IsolationResult do
           instance_double(
             Process::Status,
             'unsuccessful status'
-          )
+          ),
+          'log message'
         )
       end
 
       it_reports <<~'STR'
-        Killfork exited nonzero. Its result (if any) was ignored:
+        Killfork exited nonzero. Its result (if any) was ignored.
+        Process status:
         #<InstanceDouble(Process::Status) "unsuccessful status">
+        Log messages (combined stderr and stdout):
+        log message
       STR
     end
 
@@ -97,7 +101,8 @@ RSpec.describe Mutant::Reporter::CLI::Printer::IsolationResult do
           instance_double(
             Process::Status,
             'unsuccessful status'
-          )
+          ),
+          'log message'
         )
       end
 
@@ -116,8 +121,11 @@ RSpec.describe Mutant::Reporter::CLI::Printer::IsolationResult do
         Possible solutions are:
         * Reduce concurrency
         * Reduce locks
-        Killfork exited nonzero. Its result (if any) was ignored:
+        Killfork exited nonzero. Its result (if any) was ignored.
+        Process status:
         #<InstanceDouble(Process::Status) "unsuccessful status">
+        Log messages (combined stderr and stdout):
+        log message
       STR
     end
   end

data/spec/unit/mutant/reporter/cli/printer/mutation_result_spec.rb

@@ -12,7 +12,7 @@ RSpec.describe Mutant::Reporter::CLI::Printer::MutationResult do
       end
 
       let(:mutation_a_isolation_result) do
-        Mutant::Isolation::Fork::ChildError.new(status)
+        Mutant::Isolation::Fork::ChildError.new(status, 'log message')
       end
 
       it_reports(<<~'REPORT')
@@ -20,8 +20,12 @@ RSpec.describe Mutant::Reporter::CLI::Printer::MutationResult do
         @@ -1,2 +1,2 @@
         -true
         +false
-        Killfork exited nonzero. Its result (if any) was ignored:
+        -----------------------
+        Killfork exited nonzero. Its result (if any) was ignored.
+        Process status:
         #<InstanceDouble(Process::Status) (anonymous)>
+        Log messages (combined stderr and stdout):
+        log message
         -----------------------
       REPORT
     end
@@ -42,6 +46,11 @@ RSpec.describe Mutant::Reporter::CLI::Printer::MutationResult do
                 [Mutant::Color::NONE,  "@@ -1,2 +1,2 @@\n"],
                 [Mutant::Color::RED,   "-true\n"],
                 [Mutant::Color::GREEN, "+false\n"],
+                [Mutant::Color::NONE,  "-----------------------\n"],
+                [Mutant::Color::NONE,  "- 1 @ runtime: 1.0\n"],
+                [Mutant::Color::NONE,  "  - test-a\n"],
+                [Mutant::Color::NONE,  "Test Output:\n"],
+                [Mutant::Color::NONE,  "mutation a test result output\n"],
                 [Mutant::Color::NONE,  "-----------------------\n"]
               ].map { |color, text| color.format(text) }.join
             )
@@ -54,6 +63,11 @@ RSpec.describe Mutant::Reporter::CLI::Printer::MutationResult do
               -true
               +false
               -----------------------
+              - 1 @ runtime: 1.0
+                - test-a
+              Test Output:
+              mutation a test result output
+              -----------------------
             STR
           end
         end
@@ -80,6 +94,11 @@ RSpec.describe Mutant::Reporter::CLI::Printer::MutationResult do
             Mutated AST:
             s(:zsuper)
             -----------------------
+            - 1 @ runtime: 1.0
+              - test-a
+            Test Output:
+            mutation a test result output
+            -----------------------
           REPORT
         end
       end
@@ -101,6 +120,11 @@ RSpec.describe Mutant::Reporter::CLI::Printer::MutationResult do
           Unparsed Source:
           true
           -----------------------
+          - 1 @ runtime: 1.0
+            - test-a
+          Test Output:
+          mutation a test result output
+          -----------------------
         REPORT
       end
 
@@ -117,6 +141,11 @@ RSpec.describe Mutant::Reporter::CLI::Printer::MutationResult do
           No code was inserted. And the test did NOT PASS.
           This is typically a problem of your specs not passing unmutated.
           -----------------------
+          - 1 @ runtime: 1.0
+            - test-a
+          Test Output:
+          mutation a test result output
+          -----------------------
         REPORT
       end
     end

data/spec/unit/mutant/reporter/cli/printer/status_progressive_spec.rb

@@ -10,7 +10,7 @@ RSpec.describe Mutant::Reporter::CLI::Printer::StatusProgressive do
       with(:env_result) { { subject_results: [] } }
 
       it_reports <<~REPORT
-        (00/02) 100% - killtime: 0.00s runtime: 4.00s overhead: 4.00s
+        progress: 00/02 alive: 0 runtime: 4.00s killtime: 0.00s mutations/s: 0.00
       REPORT
     end
 
@@ -19,7 +19,7 @@ RSpec.describe Mutant::Reporter::CLI::Printer::StatusProgressive do
         with(:status) { { active_jobs: [].to_set } }
 
         it_reports(<<~REPORT)
-          (02/02) 100% - killtime: 2.00s runtime: 4.00s overhead: 2.00s
+          progress: 02/02 alive: 0 runtime: 4.00s killtime: 2.00s mutations/s: 0.50
         REPORT
       end
 
@@ -30,13 +30,13 @@ RSpec.describe Mutant::Reporter::CLI::Printer::StatusProgressive do
           with(:mutation_a_test_result) { { passed: true } }
 
           it_reports(<<~REPORT)
-            (01/02)  50% - killtime: 2.00s runtime: 4.00s overhead: 2.00s
+            progress: 02/02 alive: 1 runtime: 4.00s killtime: 2.00s mutations/s: 0.50
           REPORT
         end
 
         context 'on success' do
           it_reports(<<~REPORT)
-            (02/02) 100% - killtime: 2.00s runtime: 4.00s overhead: 2.00s
+            progress: 02/02 alive: 0 runtime: 4.00s killtime: 2.00s mutations/s: 0.50
           REPORT
         end
       end

data/spec/unit/mutant/reporter/cli/printer/subject_result_spec.rb

@@ -24,6 +24,11 @@ RSpec.describe Mutant::Reporter::CLI::Printer::SubjectResult do
         -true
         +false
         -----------------------
+        - 1 @ runtime: 1.0
+          - test-a
+        Test Output:
+        mutation a test result output
+        -----------------------
       STR
     end
 

data/spec/unit/mutant/reporter/cli_spec.rb

@@ -4,30 +4,12 @@ RSpec.describe Mutant::Reporter::CLI do
   setup_shared_context
 
   let(:object) { described_class.new(output, format) }
+  let(:tty?)   { false                               }
 
-  let(:tput) do
-    instance_double(
-      Mutant::Reporter::CLI::Tput,
-      restore: '[tput-restore]',
-      prepare: '[tput-prepare]'
-    )
+  let(:format) do
+    described_class::Format::Progressive.new(tty?)
   end
 
-  let(:framed_format) do
-    described_class::Format::Framed.new(
-      tty:  false,
-      tput: tput
-    )
-  end
-
-  let(:tty?) { false }
-
-  let(:progressive_format) do
-    described_class::Format::Progressive.new(tty: tty?)
-  end
-
-  let(:format) { framed_format }
-
   def contents
     output.rewind
     output.read
@@ -47,56 +29,23 @@ RSpec.describe Mutant::Reporter::CLI do
   describe '.build' do
     subject { described_class.build(output) }
 
-    let(:progressive_format) do
-      described_class::Format::Progressive.new(tty: tty?)
-    end
-
-    let(:framed_format) do
-      described_class::Format::Framed.new(
-        tty:  true,
-        tput: tput
-      )
-    end
+    context 'when output is a tty' do
+      let(:tty?)   { true                            }
+      let(:output) { instance_double(IO, tty?: true) }
 
-    before do
-      expect(ENV).to receive(:key?).with('CI').and_return(ci?)
+      it { should eql(described_class.new(output, format)) }
     end
 
-    let(:output) { instance_double(IO, tty?: tty?) }
-    let(:tty?)   { true                            }
-    let(:ci?)    { false                           }
+    context 'when output is not a tty' do
+      context 'and does not respond to #tty?' do
+        let(:output) { nil }
 
-    context 'when not on CI and on a tty' do
-      before do
-        expect(described_class::Tput).to receive(:detect).and_return(tput)
+        it { should eql(described_class.new(output, format)) }
       end
 
-      context 'and tput is available' do
-        it { should eql(described_class.new(output, framed_format)) }
+      context 'and does respond to #tty?' do
+        it { should eql(described_class.new(output, format)) }
       end
-
-      context 'and tput is not available' do
-        let(:tput) { nil }
-
-        it { should eql(described_class.new(output, progressive_format)) }
-      end
-    end
-
-    context 'when on CI' do
-      let(:ci?) { true }
-      it { should eql(described_class.new(output, progressive_format)) }
-    end
-
-    context 'when output is not a tty?' do
-      let(:tty?) { false }
-      it { should eql(described_class.new(output, progressive_format)) }
-    end
-
-    context 'when output does not respond to #tty?' do
-      let(:output) { instance_double(IO) }
-      let(:tty?)   { false               }
-
-      it { should eql(described_class.new(output, progressive_format)) }
     end
   end
 
@@ -111,41 +60,41 @@ RSpec.describe Mutant::Reporter::CLI do
   describe '#delay' do
     subject { object.delay }
 
-    it { should eql(0.05) }
+    it { should eql(1.0) }
   end
 
   describe '#start' do
     subject { object.start(env) }
 
-    context 'on progressive format' do
-      let(:format) { progressive_format }
-
-      it_reports(<<~REPORT)
-        Mutant configuration:
-        Matcher:         #<Mutant::Matcher::Config empty>
-        Integration:     Mutant::Integration::Null
-        Jobs:            1
-        Includes:        []
-        Requires:        []
-      REPORT
-    end
-
-    context 'on framed format' do
-      it_reports '[tput-prepare]'
-    end
+    it_reports(<<~REPORT)
+      Mutant environment:
+      Matcher:         #<Mutant::Matcher::Config empty>
+      Integration:     null
+      Jobs:            1
+      Includes:        []
+      Requires:        []
+      Subjects:        1
+      Total-Tests:     1
+      Selected-Tests:  1
+      Tests/Subject:   1.00 avg
+      Mutations:       2
+    REPORT
   end
 
   describe '#report' do
     subject { object.report(env_result) }
 
     it_reports(<<~REPORT)
-      Mutant configuration:
+      Mutant environment:
       Matcher:         #<Mutant::Matcher::Config empty>
-      Integration:     Mutant::Integration::Null
+      Integration:     null
       Jobs:            1
       Includes:        []
       Requires:        []
       Subjects:        1
+      Total-Tests:     1
+      Selected-Tests:  1
+      Tests/Subject:   1.00 avg
       Mutations:       2
       Results:         2
       Kills:           2
@@ -161,53 +110,27 @@ RSpec.describe Mutant::Reporter::CLI do
   describe '#progress' do
     subject { object.progress(status) }
 
-    context 'on framed format' do
-      let(:format) { framed_format }
-
-      it_reports(<<~REPORT)
-        [tput-restore]Mutant configuration:
-        Matcher:         #<Mutant::Matcher::Config empty>
-        Integration:     Mutant::Integration::Null
-        Jobs:            1
-        Includes:        []
-        Requires:        []
-        Subjects:        1
-        Mutations:       2
-        Results:         2
-        Kills:           2
-        Alive:           0
-        Runtime:         4.00s
-        Killtime:        2.00s
-        Overhead:        100.00%
-        Mutations/s:     0.50
-        Coverage:        100.00%
-        Active subjects: 0
-      REPORT
-    end
+    context 'with empty scheduler' do
+      with(:env_result) { { subject_results: [] } }
 
-    context 'on progressive format' do
-      let(:format) { progressive_format }
+      let(:tty?) { true }
 
-      context 'with empty scheduler' do
-        with(:env_result) { { subject_results: [] } }
+      # rubocop:disable Metrics/LineLength
+      it_reports Mutant::Color::GREEN.format('progress: 00/02 alive: 0 runtime: 4.00s killtime: 0.00s mutations/s: 0.00') + "\n"
+      # rubocop:enable Metrics/LineLength
+    end
 
-        let(:tty?) { true }
+    context 'with last mutation present' do
+      with(:env_result) { { subject_results: [subject_a_result] } }
 
-        it_reports Mutant::Color::GREEN.format('(00/02) 100% - killtime: 0.00s runtime: 4.00s overhead: 4.00s') + "\n"
+      context 'when mutation is successful' do
+        it_reports "progress: 02/02 alive: 0 runtime: 4.00s killtime: 2.00s mutations/s: 0.50\n"
       end
 
-      context 'with last mutation present' do
-        with(:env_result) { { subject_results: [subject_a_result] } }
-
-        context 'when mutation is successful' do
-          it_reports "(02/02) 100% - killtime: 2.00s runtime: 4.00s overhead: 2.00s\n"
-        end
-
-        context 'when mutation is NOT successful' do
-          with(:mutation_a_test_result) { { passed: true } }
+      context 'when mutation is NOT successful' do
+        with(:mutation_a_test_result) { { passed: true } }
 
-          it_reports "(01/02)  50% - killtime: 2.00s runtime: 4.00s overhead: 2.00s\n"
-        end
+        it_reports "progress: 02/02 alive: 1 runtime: 4.00s killtime: 2.00s mutations/s: 0.50\n"
       end
     end
   end