rspeed 0.6.0 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5b8a143543e9f44b4c5641d60642968b7cc4a7ec85953b17f6ef62743e66196e
-  data.tar.gz: f8fc781825804fe67084c08b05164ca8a31fc87e458952ba0ca86e567576e669
+  metadata.gz: 121d5558331de345c2147e1b1ee8b6679c09ac8cb83b33af4ad5673d3e534806
+  data.tar.gz: b739938586f9ce82c94c9fe74f79583153ad268e5443c54602f32650482595dd
 SHA512:
-  metadata.gz: 7b50821b549363010ab566dd5f654a5b2b042e76fd696054b795126b1343ec3a8d73ed1c3a7a08cb63f1eeba38dbbae8922877bb8ec8649bb4601577b75737b0
-  data.tar.gz: 0f74bea31bf0c550ce6f89b8f22e8c0fedb59acfebe6cddf1dc4fc4499b74e7e73505b4a24327948f1330d9ff46b16f70de0855b4178a452149b33d2b5396051
+  metadata.gz: 4109a7ca7ccc74e43c09eafd0aabf5fba559f9dbc5350a8439cbfc47cd400c03de3d862405059e60adb6ee11b83592985a89b877982c3094bcecbf3f69c363df
+  data.tar.gz: 6f706fe116da661fe858fc85979e5ee718ec4fe4da19cb13cd06ee6bbabad4688e6182f8504db93215a549fa9efe18991314faf97d8ca8e0f55125771b2de772

data/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 - None;
 
+## v0.7.0
+
+#### News
+
+- Prints pipe info into a table on the console;
+- The previous result is keeped alive for further checks;
+- Adds final report show actual, added and removed;
+
 ## v0.6.0
 
 #### Break Change

data/lib/rspeed.rb

@@ -7,6 +7,7 @@ module RSpeed
   require 'rspeed/logger'
   require 'rspeed/observer'
   require 'rspeed/redis'
+  require 'rspeed/reporter'
   require 'rspeed/runner'
   require 'rspeed/splitter'
   require 'rspeed/variable'

data/lib/rspeed/database.rb

@@ -4,8 +4,16 @@ module RSpeed
   module Database
     module_function
 
+    def previous_result
+      list(RSpeed::Variable.previous_result)
+    end
+
     def result
-      RSpeed::Redis.list(RSpeed::Variable.result).map { |item| JSON.parse(item, symbolize_names: true) }
+      list(RSpeed::Variable.result)
+    end
+
+    def list(key)
+      RSpeed::Redis.list(key).map { |item| JSON.parse(item, symbolize_names: true) }
     end
   end
 end

data/lib/rspeed/differ.rb

@@ -13,43 +13,59 @@ module RSpeed
             data << "#{file}:#{index + 1}" if /^it/.match?(item.gsub(/\s+/, ''))
           end
         end
-
-        RSpeed::Logger.log(self, __method__, data)
       end
     end
 
     def actual_data(files:, result:)
-      result
-        .select { |item| files.include?(item[:file]) }
-        .tap { |data| RSpeed::Logger.log(self, __method__, data) }
+      result.select { |item| files.include?(item[:file]) }
     end
 
     def added_data(files:, result:)
-      (files - result.map { |item| item[:file] })
-        .map { |file| { file: file, time: 0 } }
-        .tap { |data| RSpeed::Logger.log(self, __method__, data) }
+      (files - result.map { |item| item[:file] }).map { |file| { file: file, time: nil } }
     end
 
-    def diff
-      files  = actual_files
-      result = RSpeed::Database.result.uniq { |item| item[:file] }
+    def diff(from: actual_files, to: RSpeed::Database.result)
+      to      = to.uniq { |item| item[:file] }
+      added   = added_data(files: from, result: to)
+      actual  = actual_data(files: from, result: to) + added_data(files: from, result: to)
+      removed = removed_data(files: from, result: to)
+
+      {
+        actual_files:  actual,
+        actual_time:   sum_time(data: actual),
+        added_files:   added,
+        added_time:    sum_time(data: added),
+        removed_files: removed,
+        removed_time:  sum_time(data: removed),
+      }
+    end
 
-      removed_data(files: files, result: result) # called just for log purpose
+    def final_diff(from: RSpeed::Database.previous_result, to: RSpeed::Database.result)
+      from           = from.uniq { |item| item[:file] }
+      to             = to.uniq { |item| item[:file] }
+      previous_files = from.map { |item| item[:file] }
+      actual_files   = to.map { |item| item[:file] }
+      added          = to.reject { |item| previous_files.include?(item[:file]) }
+      removed        = from.reject { |item| actual_files.include?(item[:file]) }
 
-      (actual_data(files: files, result: result) + added_data(files: files, result: result))
-        .tap { |data| RSpeed::Logger.log(self, __method__, data) }
+      {
+        actual_files:  to,
+        actual_time:   sum_time(data: to),
+        added_files:   added,
+        added_time:    sum_time(data: added),
+        removed_files: removed,
+        removed_time:  sum_time(data: removed),
+      }
     end
 
     def removed_data(files:, result:)
-      result
-        .reject { |item| files.include?(item[:file]) }
-        .tap { |data| RSpeed::Logger.log(self, __method__, data) }
+      result.reject { |item| files.include?(item[:file]) }
     end
 
-    def removed_time(data:)
-      data
-        .sum { |item| item[:time].to_f }
-        .tap { |result| RSpeed::Logger.log(self, __method__, result) }
+    def sum_time(data:)
+      return '?' if data.all? { |item| item[:time].nil? }
+
+      data.sum { |item| item[:time].to_f }
     end
   end
 end

data/lib/rspeed/observer.rb

@@ -19,8 +19,9 @@ module RSpeed
 
       return unless RSpeed::Redis.specs_finished?
 
+      RSpeed::Redis.version_the_result if RSpeed::Redis.result?
       RSpeed::Splitter.consolidate
-
+      RSpeed::Reporter.call
       RSpeed::Redis.clean
 
       RSpeed::Logger.log(self, __method__, 'RSpeed finished.')
@@ -31,10 +32,6 @@ module RSpeed
     end
 
     def before_suite
-      clean_profile
-    end
-
-    def clean_profile
       RSpeed::Logger.log(self, __method__, 'Cleanning current flag and profile.')
 
       RSpeed::Redis.destroy(pattern: RSpeed::Variable.pipe)

data/lib/rspeed/redis.rb

@@ -68,5 +68,11 @@ module RSpeed
         RSpeed::Logger.log(self, __method__, "Specs #{boo ? 'initialized.' : 'not initialized yet.'}")
       end
     end
+
+    def version_the_result
+      RSpeed::Logger.log(self, __method__, 'Versioning the result.')
+
+      client.rename(RSpeed::Variable.result, RSpeed::Variable.previous_result)
+    end
   end
 end

data/lib/rspeed/reporter.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+module RSpeed
+  module Reporter
+    module_function
+
+    require 'terminal-table'
+
+    def call
+      diff = RSpeed::Differ.final_diff
+
+      print_table(
+        headings: %w[Global Value],
+
+        rows: [
+          ['Actual Time', diff[:actual_time]],
+          ['Removed Time', diff[:removed_time]],
+          ['Added Time', diff[:added_time]],
+        ]
+      )
+    end
+
+    def print_files(items)
+      total_specs = items.size
+      headings    = ["#{total_specs} specs", "Pipe #{RSpeed::Env.pipe}/#{RSpeed::Env.pipes}", 'Last Time']
+
+      rows = items.map.with_index do |item, index|
+        [format('%02d', index + 1), item[:file], item[:time].to_s]
+      end
+
+      print_table(headings: headings, rows: rows)
+    end
+
+    def print_table(headings:, rows:)
+      puts(Terminal::Table.new(headings: headings, rows: rows, title: 'RSpeed'))
+    end
+  end
+end

data/lib/rspeed/runner.rb

@@ -9,7 +9,7 @@ module RSpeed
         return shell.call(['bundle exec rspec', RSpeed::Splitter.pipe_files].compact.join(' '))
       end
 
-      RSpeed::Logger.log(self, __method__, "Skipped! Only Pipe 1 can warm.")
+      RSpeed::Logger.log(self, __method__, 'Skipped! Only Pipe 1 can warm.')
 
       RSpeed::Observer.after_suite
     end

data/lib/rspeed/splitter.rb

@@ -31,9 +31,11 @@ module RSpeed
     def pipe_files
       return unless RSpeed::Redis.result?
 
-      splitted = split(data: RSpeed::Differ.diff)
+      splitted = split(data: RSpeed::Differ.diff[:actual_files])
 
-      splitted[RSpeed::Variable.key(RSpeed::Env.pipe)][:files].map { |item| item[:file] }.join(' ')
+      splitted[RSpeed::Variable.key(RSpeed::Env.pipe)][:files].tap do |items|
+        RSpeed::Reporter.print_files(items)
+      end.map { |item| item[:file] }.join(' ')
     end
 
     def split(data:)
@@ -44,7 +46,7 @@ module RSpeed
         json[RSpeed::Variable.key(index + 1)] = { total: 0, files: [], number: index + 1 }
       end
 
-      sorted_data = data.sort_by { |item| item[:time] }.reverse
+      sorted_data = data.sort_by { |item| item[:time] || 0.0 }.reverse
 
       sorted_data.each do |record|
         selected_pipe_data = json.min_by { |pipe| pipe[1][:total] }

data/lib/rspeed/variable.rb

@@ -23,6 +23,10 @@ module RSpeed
       append_app_name('rspeed:pipe', plus: format('%02d', RSpeed::Env.pipe))
     end
 
+    def previous_result
+      append_app_name('rspeed', plus: 'previous')
+    end
+
     def profile
       append_app_name('rspeed:profile', plus: format('%02d', RSpeed::Env.pipe))
     end

data/lib/rspeed/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RSpeed
-  VERSION = '0.6.0'
+  VERSION = '0.7.0'
 end

data/spec/models/rspeed/database/list_spec.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Database, '#list' do
+  before do
+    redis_object.rpush(RSpeed::Variable.result, { file: '1_spec.rb', time: 1.0 }.to_json)
+    redis_object.rpush(RSpeed::Variable.result, { file: '2_spec.rb', time: 2.0 }.to_json)
+  end
+
+  it 'list and parse the given key' do
+    expect(described_class.list(RSpeed::Variable.result)).to eq [
+      { file: '1_spec.rb', time: 1.0 },
+      { file: '2_spec.rb', time: 2.0 },
+    ]
+  end
+end

data/spec/models/rspeed/database/previous_result_spec.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Database, '#previous_result' do
+  let!(:redis) { redis_object }
+
+  before do
+    redis.rpush(RSpeed::Variable.previous_result, { file: '1_spec.rb', time: 1.0 }.to_json)
+    redis.rpush(RSpeed::Variable.previous_result, { file: '2_spec.rb', time: 2.0 }.to_json)
+  end
+
+  it 'returns the previous results data as json' do
+    expect(described_class.previous_result).to eq [
+      { file: '1_spec.rb', time: 1.0 },
+      { file: '2_spec.rb', time: 2.0 },
+    ]
+  end
+end

data/spec/models/rspeed/differ/added_data_spec.rb

@@ -11,9 +11,9 @@ RSpec.describe RSpeed::Differ, '#added_data' do
     ]
   end
 
-  it 'returns the added data files with time as zero since we do not know yet' do
+  it 'returns the added data files' do
     expect(described_class.added_data(files: files, result: result)).to eq [
-      { file: './spec/fixtures/new_spec.rb:4', time: 0 },
+      { file: './spec/fixtures/new_spec.rb:4', time: nil },
     ]
   end
 end

data/spec/models/rspeed/differ/diff_spec.rb

@@ -6,12 +6,12 @@ RSpec.describe RSpeed::Differ, '#diff' do
   let!(:redis) { redis_object }
 
   before do
-    redis.rpush('rspeed', { file: './spec/fixtures/1_spec.rb:4', time: '1.4' }.to_json)
-    redis.rpush('rspeed', { file: './spec/fixtures/1_spec.rb:6', time: '1.6' }.to_json)
-    redis.rpush('rspeed', { file: './spec/fixtures/1_spec.rb:8', time: '1.8' }.to_json)
-    redis.rpush('rspeed', { file: './spec/fixtures/2_spec.rb:4', time: '2.4' }.to_json)
+    redis.rpush('rspeed', { file: './spec/fixtures/1_spec.rb:4', time: 1.4 }.to_json)
+    redis.rpush('rspeed', { file: './spec/fixtures/1_spec.rb:6', time: 1.6 }.to_json)
+    redis.rpush('rspeed', { file: './spec/fixtures/1_spec.rb:8', time: 1.8 }.to_json)
+    redis.rpush('rspeed', { file: './spec/fixtures/2_spec.rb:4', time: 2.4 }.to_json)
     redis.rpush('rspeed', { file: './spec/fixtures/x_spec.rb:1', time: 3 }.to_json)
-    redis.rpush('rspeed', { file: './spec/fixtures/2_spec.rb:666', time: '6' }.to_json)
+    redis.rpush('rspeed', { file: './spec/fixtures/2_spec.rb:666', time: 6 }.to_json)
 
     File.open('spec/fixtures/new_spec.rb', 'a') { |file| file.write('it') }
   end
@@ -20,31 +20,57 @@ RSpec.describe RSpeed::Differ, '#diff' do
 
   it 'removes removed specs and adds new spec and keeps keeped specs based on rspeed key values' do
     EnvMock.mock(rspeed_spec_path: './spec/fixtures/*_spec.rb') do
-      expect(described_class.diff).to eq [
-        { file: './spec/fixtures/1_spec.rb:4', time: '1.4' },
-        { file: './spec/fixtures/1_spec.rb:6', time: '1.6' },
-        { file: './spec/fixtures/1_spec.rb:8', time: '1.8' },
-        { file: './spec/fixtures/2_spec.rb:4', time: '2.4' },
-        { file: './spec/fixtures/new_spec.rb:1', time: 0 },
-      ]
+      expect(described_class.diff).to eq(
+        actual_files: [
+          { file: './spec/fixtures/1_spec.rb:4', time: 1.4 },
+          { file: './spec/fixtures/1_spec.rb:6', time: 1.6 },
+          { file: './spec/fixtures/1_spec.rb:8', time: 1.8 },
+          { file: './spec/fixtures/2_spec.rb:4', time: 2.4 },
+          { file: './spec/fixtures/new_spec.rb:1', time: nil },
+        ],
+
+        actual_time: 7.2,
+        added_files: [{ file: './spec/fixtures/new_spec.rb:1', time: nil }],
+        added_time:  '?',
+
+        removed_files: [
+          { file: './spec/fixtures/x_spec.rb:1', time: 3 },
+          { file: './spec/fixtures/2_spec.rb:666', time: 6 },
+        ],
+
+        removed_time: 9.0
+      )
     end
   end
 
   context 'when has duplicated example' do
     before do
-      redis.rpush('rspeed', { file: './spec/fixtures/1_spec.rb:4', time: '1.4' }.to_json)
-      redis.rpush('rspeed', { file: './spec/fixtures/1_spec.rb:4', time: '1.4' }.to_json)
+      redis.rpush('rspeed', { file: './spec/fixtures/1_spec.rb:4', time: 1.4 }.to_json)
+      redis.rpush('rspeed', { file: './spec/fixtures/1_spec.rb:4', time: 1.4 }.to_json)
     end
 
     it 'is removed' do
       EnvMock.mock(rspeed_spec_path: './spec/fixtures/*_spec.rb') do
-        expect(described_class.diff).to eq [
-          { file: './spec/fixtures/1_spec.rb:4', time: '1.4' },
-          { file: './spec/fixtures/1_spec.rb:6', time: '1.6' },
-          { file: './spec/fixtures/1_spec.rb:8', time: '1.8' },
-          { file: './spec/fixtures/2_spec.rb:4', time: '2.4' },
-          { file: './spec/fixtures/new_spec.rb:1', time: 0 },
-        ]
+        expect(described_class.diff).to eq(
+          actual_files: [
+            { file: './spec/fixtures/1_spec.rb:4', time: 1.4 },
+            { file: './spec/fixtures/1_spec.rb:6', time: 1.6 },
+            { file: './spec/fixtures/1_spec.rb:8', time: 1.8 },
+            { file: './spec/fixtures/2_spec.rb:4', time: 2.4 },
+            { file: './spec/fixtures/new_spec.rb:1', time: nil },
+          ],
+
+          actual_time: 7.2,
+          added_files: [{ file: './spec/fixtures/new_spec.rb:1', time: nil }],
+          added_time:  '?',
+
+          removed_files: [
+            { file: './spec/fixtures/x_spec.rb:1', time: 3 },
+            { file: './spec/fixtures/2_spec.rb:666', time: 6 },
+          ],
+
+          removed_time: 9.0
+        )
       end
     end
   end

data/spec/models/rspeed/differ/final_diff_spec.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+require 'support/env_mock'
+
+RSpec.describe RSpeed::Differ, '#final_diff' do
+  let!(:redis) { redis_object }
+
+  before do
+    redis.rpush('rspeed_previous', { file: './spec/fixtures/1_spec.rb:1', time: 1.1 }.to_json)
+    redis.rpush('rspeed_previous', { file: './spec/fixtures/2_spec.rb:2', time: 2.2 }.to_json)
+    redis.rpush('rspeed_previous', { file: './spec/fixtures/3_spec.rb:3', time: 3.3 }.to_json)
+    redis.rpush('rspeed_previous', { file: './spec/fixtures/4_spec.rb:4', time: 4.4 }.to_json)
+
+    redis.rpush('rspeed', { file: './spec/fixtures/1_spec.rb:1', time: 1.1 }.to_json)
+    redis.rpush('rspeed', { file: './spec/fixtures/2_spec.rb:2', time: 2.2 }.to_json)
+    redis.rpush('rspeed', { file: './spec/fixtures/5_spec.rb:5', time: 5.5 }.to_json)
+    redis.rpush('rspeed', { file: './spec/fixtures/6_spec.rb:6', time: 6.6 }.to_json)
+  end
+
+  it 'returns the diff' do
+    expect(described_class.final_diff).to eq(
+      actual_files: [
+        { file: './spec/fixtures/1_spec.rb:1', time: 1.1 },
+        { file: './spec/fixtures/2_spec.rb:2', time: 2.2 },
+        { file: './spec/fixtures/5_spec.rb:5', time: 5.5 },
+        { file: './spec/fixtures/6_spec.rb:6', time: 6.6 },
+      ],
+
+      actual_time: 15.4,
+
+      added_files: [
+        { file: './spec/fixtures/5_spec.rb:5', time: 5.5 },
+        { file: './spec/fixtures/6_spec.rb:6', time: 6.6 },
+      ],
+
+      added_time: 12.1,
+
+      removed_files: [
+        { file: './spec/fixtures/3_spec.rb:3', time: 3.3 },
+        { file: './spec/fixtures/4_spec.rb:4', time: 4.4 },
+      ],
+
+      removed_time: 7.7
+    )
+  end
+end

data/spec/models/rspeed/differ/sum_time_spec.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Differ, '#sum_time' do
+  context 'when has at least one time with value' do
+    let!(:data) { [{ time: 1.4 }, { time: 1.6 }, { time: 2.4 }, { time: nil }] }
+
+    it 'sums the time key' do
+      expect(described_class.sum_time(data: data)).to be(5.4)
+    end
+  end
+
+  context 'when all time are nil' do
+    let!(:data) { [{ time: nil }, { time: nil }] }
+
+    it 'returns a question mark' do
+      expect(described_class.sum_time(data: data)).to eq('?')
+    end
+  end
+end

data/spec/models/rspeed/observer/after_suite_spec.rb

@@ -16,7 +16,9 @@ RSpec.describe RSpeed::Observer, '.after_suite' do
   context 'when all specs finished' do
     before do
       allow(RSpeed::Redis).to receive(:specs_finished?).and_return(true)
+      allow(RSpeed::Redis).to receive(:version_the_result)
       allow(RSpeed::Splitter).to receive(:consolidate)
+      allow(RSpeed::Reporter).to receive(:call)
       allow(RSpeed::Redis).to receive(:clean)
     end
 
@@ -26,6 +28,12 @@ RSpec.describe RSpeed::Observer, '.after_suite' do
       expect(RSpeed::Splitter).to have_received(:consolidate)
     end
 
+    it 'reports' do
+      described_class.after_suite
+
+      expect(RSpeed::Reporter).to have_received(:call)
+    end
+
     it 'destroyes pipe finished flag keys' do
       described_class.after_suite
 

data/spec/models/rspeed/redis/specs_initiated_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-RSpec.describe RSpeed::Redis, '.specs_initiated?' do
+RSpec.describe RSpeed::Redis, '.specs_finished?' do
   context 'when has no pipe flag key' do
     it { expect(described_class.specs_finished?).to be(false) }
   end

data/spec/models/rspeed/redis/version_the_result_spec.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '.version_the_result' do
+  before { redis_object.rpush(RSpeed::Variable.result, true) }
+
+  it 'renames the result to a previous result key' do
+    described_class.version_the_result
+
+    expect(redis_object.keys).to eq([RSpeed::Variable.previous_result])
+  end
+
+  context 'when already have a previous result' do
+    before { redis_object.rpush(RSpeed::Variable.previous_result, true) }
+
+    it 'overrides' do
+      described_class.version_the_result
+
+      expect(redis_object.keys).to eq([RSpeed::Variable.previous_result])
+    end
+  end
+end

data/spec/models/rspeed/reporter/call_spec.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Reporter, '#call' do
+  let!(:previous_result) do
+    [
+      { file: './spec/fixtures/1_spec.rb:1', time: 1.1 },
+      { file: './spec/fixtures/2_spec.rb:2', time: 2.2 },
+      { file: './spec/fixtures/3_spec.rb:3', time: 3.3 },
+      { file: './spec/fixtures/4_spec.rb:4', time: 4.4 },
+    ]
+  end
+
+  let!(:result) do
+    [
+      { file: './spec/fixtures/1_spec.rb:1', time: 1.1 },
+      { file: './spec/fixtures/2_spec.rb:2', time: 2.2 },
+      { file: './spec/fixtures/5_spec.rb:5', time: 5.5 },
+      { file: './spec/fixtures/6_spec.rb:6', time: 6.6 },
+    ]
+  end
+
+  before do
+    allow(RSpeed::Database).to receive(:previous_result).and_return(previous_result)
+    allow(RSpeed::Database).to receive(:result).and_return(result)
+  end
+
+  it 'prints a report' do
+    output = <<~HEREDOC
+      +--------------+-------+
+      |        RSpeed        |
+      +--------------+-------+
+      | Global       | Value |
+      +--------------+-------+
+      | Actual Time  | 15.4  |
+      | Removed Time | 7.7   |
+      | Added Time   | 12.1  |
+      +--------------+-------+
+    HEREDOC
+
+    expect { described_class.call }.to output(output).to_stdout
+  end
+end

data/spec/models/rspeed/reporter/print_files_spec.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Reporter, '#print_files' do
+  let!(:items) do
+    [
+      { file: './spec/fixtures/1_spec.rb:1', time: 1.1 },
+      { file: './spec/fixtures/2_spec.rb:2', time: 2.2 },
+      { file: './spec/fixtures/3_spec.rb:3', time: 3.3 },
+    ]
+  end
+
+  it 'prints the attributes' do
+    output = <<~HEREDOC
+      +---------+-----------------------------+-----------+
+      |                      RSpeed                       |
+      +---------+-----------------------------+-----------+
+      | 3 specs | Pipe 1/1                    | Last Time |
+      +---------+-----------------------------+-----------+
+      | 01      | ./spec/fixtures/1_spec.rb:1 | 1.1       |
+      | 02      | ./spec/fixtures/2_spec.rb:2 | 2.2       |
+      | 03      | ./spec/fixtures/3_spec.rb:3 | 3.3       |
+      +---------+-----------------------------+-----------+
+    HEREDOC
+
+    expect { described_class.print_files(items) }.to output(output).to_stdout
+  end
+end

data/spec/models/rspeed/reporter/print_table_spec.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Reporter, '#print_table' do
+  let!(:headings) { %w[Global Value] }
+
+  let!(:rows) do
+    [
+      ['Actual Time', 1.0],
+      ['Removed Time', 2.0],
+      ['Added Time', 3.0],
+    ]
+  end
+
+  it 'prints' do
+    output = <<~HEREDOC
+      +--------------+-------+
+      |        RSpeed        |
+      +--------------+-------+
+      | Global       | Value |
+      +--------------+-------+
+      | Actual Time  | 1.0   |
+      | Removed Time | 2.0   |
+      | Added Time   | 3.0   |
+      +--------------+-------+
+    HEREDOC
+
+    expect { described_class.print_table(headings: headings, rows: rows) }.to output(output).to_stdout
+  end
+end

data/spec/models/rspeed/splitter/pipe_files_spec.rb

@@ -10,12 +10,21 @@ RSpec.describe RSpeed::Splitter, '#pipe_files' do
   context 'when has result' do
     before do
       allow(RSpeed::Redis).to receive(:result?).and_return(true)
+      allow(RSpeed::Reporter).to receive(:print_files)
 
-      allow(described_class).to receive(:split).and_return(rspeed_1: { files: [{ file: 'spec_1.rb' }, { file: 'spec_2.rb' }] })
+      allow(described_class).to receive(:split)
+        .and_return(rspeed_1: { files: [{ file: 'spec_1.rb', time: 1.0 }, { file: 'spec_2.rb', time: 2.0 }] })
     end
 
     it 'returns the splitted pipe files' do
       expect(described_class.pipe_files).to eq('spec_1.rb spec_2.rb')
     end
+
+    it 'prints the files' do
+      described_class.pipe_files
+
+      expect(RSpeed::Reporter).to have_received(:print_files)
+        .with([{ file: 'spec_1.rb', time: 1.0 }, { file: 'spec_2.rb', time: 2.0 }])
+    end
   end
 end

data/spec/models/rspeed/splitter/split_spec.rb

@@ -3,49 +3,91 @@
 require 'support/env_mock'
 
 RSpec.describe RSpeed::Splitter, '.split' do
-  let!(:unsorted_data) do
-    [
-      { file: './spec/1_5_spec.rb', time: '1.5' },
-      { file: './spec/1_1_spec.rb', time: '1.1' },
-      { file: './spec/0_7_spec.rb', time: '0.7' },
-      { file: './spec/0_4_spec.rb', time: '0.4' },
-      { file: './spec/0_3_spec.rb', time: '0.3' },
-      { file: './spec/0_2_spec.rb', time: '0.2' },
-      { file: './spec/2_0_spec.rb', time: '2.0' },
-    ]
+  context 'when has no nil time' do
+    let!(:unsorted_data) do
+      [
+        { file: './spec/1_5_spec.rb', time: 1.5 },
+        { file: './spec/1_1_spec.rb', time: 1.1 },
+        { file: './spec/0_7_spec.rb', time: 0.7 },
+        { file: './spec/0_4_spec.rb', time: 0.4 },
+        { file: './spec/0_3_spec.rb', time: 0.3 },
+        { file: './spec/0_2_spec.rb', time: 0.2 },
+        { file: './spec/2_0_spec.rb', time: 2.0 },
+      ]
+    end
+
+    it 'splits the times between the pipes' do
+      EnvMock.mock(rspeed_pipes: 3) do
+        expect(described_class.split(data: unsorted_data)).to eq(
+          rspeed_1: {
+            files: [{ file: './spec/2_0_spec.rb', time: 2.0 }],
+            number: 1,
+            total: 2.0,
+          },
+
+          rspeed_2: {
+            files: [
+              { file: './spec/1_5_spec.rb', time: 1.5 },
+              { file: './spec/0_4_spec.rb', time: 0.4 },
+              { file: './spec/0_2_spec.rb', time: 0.2 },
+            ],
+
+            number: 2,
+            total: 1.5 + 0.4 + 0.2, # 1.5 -> 1.9 -> 2.1
+          },
+
+          rspeed_3: {
+            files: [
+              { file: './spec/1_1_spec.rb', time: 1.1 },
+              { file: './spec/0_7_spec.rb', time: 0.7 },
+              { file: './spec/0_3_spec.rb', time: 0.3 },
+            ],
+
+            number: 3,
+            total: 1.1 + 0.7 + 0.3, # 1.1 -> 1.8 -> 2.1
+          }
+        )
+      end
+    end
   end
 
-  it 'splits the times between the pipes' do
-    EnvMock.mock(rspeed_pipes: 3) do
-      expect(described_class.split(data: unsorted_data)).to eq(
-        rspeed_1: {
-          files: [{ file: './spec/2_0_spec.rb', time: 2.0 }],
-          number: 1,
-          total: 2.0,
-        },
-
-        rspeed_2: {
-          files: [
-            { file: './spec/1_5_spec.rb', time: 1.5 },
-            { file: './spec/0_4_spec.rb', time: 0.4 },
-            { file: './spec/0_2_spec.rb', time: 0.2 },
-          ],
-
-          number: 2,
-          total: 1.5 + 0.4 + 0.2, # 1.5 -> 1.9 -> 2.1
-        },
-
-        rspeed_3: {
-          files: [
-            { file: './spec/1_1_spec.rb', time: 1.1 },
-            { file: './spec/0_7_spec.rb', time: 0.7 },
-            { file: './spec/0_3_spec.rb', time: 0.3 },
-          ],
-
-          number: 3,
-          total: 1.1 + 0.7 + 0.3, # 1.1 -> 1.8 -> 2.1
-        }
-      )
+  context 'when has nil time' do
+    let!(:unsorted_data) do
+      [
+        { file: './spec/1_5_spec.rb', time: 1.5 },
+        { file: './spec/1_1_spec.rb', time: 1.1 },
+        { file: './spec/2_0_spec.rb', time: nil },
+      ]
+    end
+
+    it 'uses zero as default' do
+      EnvMock.mock(rspeed_pipes: 3) do
+        expect(described_class.split(data: unsorted_data)).to eq(
+          rspeed_1: {
+            files: [{ file: './spec/1_5_spec.rb', time: 1.5 }],
+            number: 1,
+            total: 1.5,
+          },
+
+          rspeed_2: {
+            files: [
+              { file: './spec/1_1_spec.rb', time: 1.1 },
+            ],
+
+            number: 2,
+            total: 1.1
+          },
+
+          rspeed_3: {
+            files: [
+              { file: './spec/2_0_spec.rb', time: 0.0 },
+            ],
+
+            number: 3,
+            total: 0.0,
+          }
+        )
+      end
     end
   end
 end

data/spec/models/rspeed/variable/previous_result_spec.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+require 'support/env_mock'
+
+RSpec.describe RSpeed::Variable, '.previous_result' do
+  it { expect(described_class.previous_result).to eq('rspeed_previous') }
+
+  context 'when env app is given' do
+    it 'includes the app' do
+      EnvMock.mock(rspeed_app: 'app') do
+        expect(described_class.previous_result).to eq('rspeed_app_previous')
+      end
+    end
+  end
+
+  context 'when env app is not given' do
+    it { expect(described_class.previous_result).to eq('rspeed_previous') }
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rspeed
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.7.0
 platform: ruby
 authors:
 - Washington Botelho
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-02-07 00:00:00.000000000 Z
+date: 2021-02-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis
@@ -38,6 +38,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: terminal-table
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: codecov
   requirement: !ruby/object:Gem::Requirement
@@ -172,6 +186,7 @@ files:
 - lib/rspeed/logger.rb
 - lib/rspeed/observer.rb
 - lib/rspeed/redis.rb
+- lib/rspeed/reporter.rb
 - lib/rspeed/runner.rb
 - lib/rspeed/splitter.rb
 - lib/rspeed/variable.rb
@@ -180,13 +195,16 @@ files:
 - spec/fixtures/1_spec.rb
 - spec/fixtures/2_spec.rb
 - spec/fixtures/empty.rb
+- spec/models/rspeed/database/list_spec.rb
+- spec/models/rspeed/database/previous_result_spec.rb
 - spec/models/rspeed/database/result_spec.rb
 - spec/models/rspeed/differ/actual_data_spec.rb
 - spec/models/rspeed/differ/actual_files_spec.rb
 - spec/models/rspeed/differ/added_data_spec.rb
 - spec/models/rspeed/differ/diff_spec.rb
+- spec/models/rspeed/differ/final_diff_spec.rb
 - spec/models/rspeed/differ/removed_data_spec.rb
-- spec/models/rspeed/differ/removed_time_spec.rb
+- spec/models/rspeed/differ/sum_time_spec.rb
 - spec/models/rspeed/env/app_spec.rb
 - spec/models/rspeed/env/db_spec.rb
 - spec/models/rspeed/env/host_spec.rb
@@ -210,6 +228,10 @@ files:
 - spec/models/rspeed/redis/set_spec.rb
 - spec/models/rspeed/redis/specs_finished_spec.rb
 - spec/models/rspeed/redis/specs_initiated_spec.rb
+- spec/models/rspeed/redis/version_the_result_spec.rb
+- spec/models/rspeed/reporter/call_spec.rb
+- spec/models/rspeed/reporter/print_files_spec.rb
+- spec/models/rspeed/reporter/print_table_spec.rb
 - spec/models/rspeed/runner/run_spec.rb
 - spec/models/rspeed/splitter/append_spec.rb
 - spec/models/rspeed/splitter/consolidate_spec.rb
@@ -221,6 +243,7 @@ files:
 - spec/models/rspeed/variable/key_spec.rb
 - spec/models/rspeed/variable/pipe_spec.rb
 - spec/models/rspeed/variable/pipes_pattern_spec.rb
+- spec/models/rspeed/variable/previous_result_spec.rb
 - spec/models/rspeed/variable/profile_pattern_spec.rb
 - spec/models/rspeed/variable/profile_spec.rb
 - spec/models/rspeed/variable/result_spec.rb
@@ -254,11 +277,14 @@ specification_version: 4
 summary: Split and speed up your RSpec tests.
 test_files:
 - spec/spec_helper.rb
+- spec/models/rspeed/database/list_spec.rb
 - spec/models/rspeed/database/result_spec.rb
+- spec/models/rspeed/database/previous_result_spec.rb
 - spec/models/rspeed/variable/pipes_pattern_spec.rb
 - spec/models/rspeed/variable/pipe_spec.rb
 - spec/models/rspeed/variable/result_spec.rb
 - spec/models/rspeed/variable/key_spec.rb
+- spec/models/rspeed/variable/previous_result_spec.rb
 - spec/models/rspeed/variable/profile_pattern_spec.rb
 - spec/models/rspeed/variable/profile_spec.rb
 - spec/models/rspeed/variable/append_app_name_spec.rb
@@ -267,12 +293,16 @@ test_files:
 - spec/models/rspeed/redis/clean_spec.rb
 - spec/models/rspeed/redis/client_spec.rb
 - spec/models/rspeed/redis/specs_finished_spec.rb
+- spec/models/rspeed/redis/version_the_result_spec.rb
 - spec/models/rspeed/redis/list_spec.rb
 - spec/models/rspeed/redis/set_spec.rb
 - spec/models/rspeed/redis/specs_initiated_spec.rb
 - spec/models/rspeed/redis/result_spec.rb
 - spec/models/rspeed/redis/keys_spec.rb
 - spec/models/rspeed/redis/profiles_content_spec.rb
+- spec/models/rspeed/reporter/print_files_spec.rb
+- spec/models/rspeed/reporter/call_spec.rb
+- spec/models/rspeed/reporter/print_table_spec.rb
 - spec/models/rspeed/runner/run_spec.rb
 - spec/models/rspeed/splitter/append_spec.rb
 - spec/models/rspeed/splitter/need_warm_question_spec.rb
@@ -284,11 +314,12 @@ test_files:
 - spec/models/rspeed/observer/before_spec.rb
 - spec/models/rspeed/observer/after_suite_spec.rb
 - spec/models/rspeed/observer/after_spec.rb
+- spec/models/rspeed/differ/final_diff_spec.rb
 - spec/models/rspeed/differ/actual_files_spec.rb
 - spec/models/rspeed/differ/removed_data_spec.rb
 - spec/models/rspeed/differ/actual_data_spec.rb
 - spec/models/rspeed/differ/diff_spec.rb
-- spec/models/rspeed/differ/removed_time_spec.rb
+- spec/models/rspeed/differ/sum_time_spec.rb
 - spec/models/rspeed/differ/added_data_spec.rb
 - spec/models/rspeed/env/pipe_spec.rb
 - spec/models/rspeed/env/spec_path_spec.rb

data/spec/models/rspeed/differ/removed_time_spec.rb

@@ -1,14 +0,0 @@
-# frozen_string_literal: true
-
-RSpec.describe RSpeed::Differ, '#removed_time' do
-  let!(:data) do
-    [
-      { file: './spec/fixtures/1_spec.rb:4', time: 1.4 },
-      { file: './spec/fixtures/1_spec.rb:6', time: 1.6 },
-    ]
-  end
-
-  it 'returns the sum of removed examples' do
-    expect(described_class.removed_time(data: data)).to be(3.0)
-  end
-end