rspeed 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fa530fc36a8a38f31b2f241cb0c322bd268f0928cb5929f89d4b939e54b740a3
-  data.tar.gz: 4228e7997bd6d4b51da7aa66ef10ddc25474a184cd3c42d42afd6077db777f8d
+  metadata.gz: 27a64c0a7ba3f1c113bbdf8f060be44dad913a684d234663b342a537937ac7f2
+  data.tar.gz: 47d3a190302b361c6bb76279094a7c04982492e59d099d6acdbe92fcbd26d648
 SHA512:
-  metadata.gz: ce7fa5fe6f56cdf9b3f9a02cc040a8180d21e3c5ad1c08bcc792df3ee336c300c6fd5d8438cef908815a896dc961a5743910a63686653c6fdc91707ec24b4a6b
-  data.tar.gz: ae309883ef9f2f63792644583c1b9277e8c290e84172b59e7eecb0a961015d48b9411a798a6514d6dd3855a3553ce4d31e420634514f37025f4023a4c181e133
+  metadata.gz: ac35df999459a32e06989e84d0e703f52a61b896e3a68ef7993c6eaf70952dd05826dffdf36f01fe5843b3916e57b24112708cbcadd19b83eb018c990f2bda4c
+  data.tar.gz: '09feaa28dc6a306f8ba039bcfbf02b500450b6f117160a726f326daae76c9584524825ffd693a4f3a84a61cb181fef7c00216557ce04ccaea736f42d3e9370d8'

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## v0.2.0
+
+- Splits specs by examples over files;
+
 ## v0.1.0
 
 - First release;

data/lib/rspeed.rb

@@ -6,5 +6,6 @@ module RSpeed
 end
 
 require 'rspeed/extension'
+require 'rspeed/observer'
 require 'rspeed/runner'
 require 'rspeed/splitter'

data/lib/rspeed/extension.rb

@@ -4,40 +4,8 @@ if ENV['RSPEED'] == 'true'
   require 'rspec/rails'
 
   RSpec.configure do |config|
-    config.before :suite do
-      File.open('rspeed.csv', 'w') { |file| file.truncate 0 }
-    end
-
-    config.before do |example|
-      example.update_inherited_metadata start_at: example.clock.now
-    end
-
-    config.after do |example|
-      file_path       = example.metadata[:file_path]
-      time_difference = example.clock.now - example.metadata[:start_at]
-
-      File.open('rspeed.csv', 'a') do |file|
-        file.write "#{time_difference},#{file_path}\n"
-      end
-    end
-
-    config.after :suite do |_example|
-      result = {}
-
-      CSV.read('rspeed.csv').each do |line|
-        result[line[1]] ||= 0
-        result[line[1]] += line[0].to_d
-      end
-
-      result = result.sort_by { |line| line[1] }.reverse
-
-      File.open('rspeed.csv', 'w') { |file| file.truncate 0 }
-
-      File.open('rspeed.csv', 'a') do |file|
-        result.each do |file_spec, time|
-          file.write "#{time},#{file_spec}\n"
-        end
-      end
-    end
+    config.before(:suite) { RSpeed::Observer.before_suite }
+    config.before { |example| RSpeed::Observer.before(example) }
+    config.after { |example| RSpeed::Observer.after(example) }
   end
 end

data/lib/rspeed/observer.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+module RSpeed
+  module Observer
+    module_function
+
+    def after(example)
+      file_path   = example.metadata[:file_path]
+      line_number = example.metadata[:line_number]
+      spent_time  = example.clock.now - example.metadata[:start_at]
+
+      File.open('rspeed.csv', 'a') do |file|
+        file.write "#{spent_time},#{file_path}:#{line_number}\n"
+      end
+    end
+
+    def before(example)
+      example.update_inherited_metadata(start_at: example.clock.now)
+    end
+
+    def before_suite
+      truncate_csv_file
+    end
+
+    def truncate_csv_file
+      File.open('rspeed.csv', 'w') { |file| file.truncate(0) }
+    end
+  end
+end

data/lib/rspeed/splitter.rb

@@ -80,7 +80,9 @@ module RSpeed
         json["rspeed_#{index + 1}".to_sym] = { total: 0, files: [], number: index + 1 }
       end
 
-      data.each do |record|
+      sorted_data = data.sort_by { |item| item[:time] }.reverse
+
+      sorted_data.each do |record|
         selected_pipe_data = json.min_by { |pipe| pipe[1][:total] }
         selected_pipe      = json["rspeed_#{selected_pipe_data[1][:number]}".to_sym]
         time               = record[:time].to_f

data/lib/rspeed/version.rb

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

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

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe RSpeed::Observer, '.after' do
+  let!(:now) { Time.local(2020, 1, 1, 0, 0, 1) }
+  let!(:clock) { class_double(RSpec::Core::Time, now: now) }
+  let!(:metadata) { { file_path: 'file_path', line_number: 7, start_at: now - 1 } }
+  let!(:example) { instance_double(RSpec::Core::Example, clock: clock, metadata: metadata) }
+
+  before { clean_csv_file }
+
+  it 'appends the time of example on csv file' do
+    described_class.after(example)
+
+    expect(File.open('rspeed.csv').read).to eq "1.0,file_path:7\n"
+  end
+end

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

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe RSpeed::Observer, '.before' do
+  let!(:now) { Time.local(2020) }
+  let!(:clock) { class_double(RSpec::Core::Time, now: now) }
+  let!(:example) { instance_double(RSpec::Core::Example, clock: clock) }
+
+  it 'saves the current time' do
+    allow(example).to receive(:update_inherited_metadata)
+
+    described_class.before(example)
+
+    expect(example).to have_received(:update_inherited_metadata).with(start_at: now)
+  end
+end

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

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe RSpeed::Observer, '.before_suite' do
+  before { clean_csv_file }
+
+  it 'cleans the csv file' do
+    File.open('rspeed.csv', 'a') { |file| file.write('content') }
+
+    described_class.before_suite
+
+    expect(File.open('rspeed.csv').read).to eq ''
+  end
+end

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

@@ -15,6 +15,11 @@ RSpec.describe RSpeed::Splitter, '#append' do
   end
 
   context 'when files is not given' do
+    before do
+      clean_csv_file
+      populate_csv_file
+    end
+
     it 'read csv and append file and time on rspeed key' do
       splitter.append
 

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

@@ -5,22 +5,22 @@ require 'rails_helper'
 RSpec.describe RSpeed::Splitter, '.split' do
   subject(:splitter) { described_class.new }
 
-  let!(:data) do
+  let!(:unsorted_data) do
     [
-      { file: './spec/2_0_spec.rb', time: '2.0' },
       { 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
 
   before { allow(splitter).to receive(:pipes).and_return 3 }
 
   it 'splits the times between the pipes' do
-    expect(splitter.split(data)).to eq(
+    expect(splitter.split(unsorted_data)).to eq(
       rspeed_1: {
         files: [{ file: './spec/2_0_spec.rb', time: 2.0 }],
         number: 1,

data/spec/rails_helper.rb

@@ -9,14 +9,33 @@ require 'rspec'
 require 'rspeed'
 
 RSpec.configure do |config|
+  config.after do
+    ENV.delete('RSPEED_PIPE')
+    ENV.delete('RSPEED_PIPES')
+  end
+
   config.disable_monkey_patching!
 
   config.order = :random
+  config.profile_examples = 10
+end
 
-  config.after do
-    ENV.delete 'RSPEED_PIPE'
-    ENV.delete 'RSPEED_PIPES'
-  end
+def clean_csv_file
+  File.open('rspeed.csv', 'w') { |file| file.truncate(0) }
+end
+
+def populate_csv_file
+  data = [
+    '2.0,./spec/2_0_spec.rb',
+    '1.5,./spec/1_5_spec.rb',
+    '1.1,./spec/1_1_spec.rb',
+    '0.7,./spec/0_7_spec.rb',
+    '0.4,./spec/0_4_spec.rb',
+    '0.3,./spec/0_3_spec.rb',
+    '0.2,./spec/0_2_spec.rb'
+  ].join("\n")
+
+  File.open('rspeed.csv', 'a') { |file| file.write(data) }
 end
 
 def redis_object

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rspeed
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Washington Botelho
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-08-07 00:00:00.000000000 Z
+date: 2020-08-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis
@@ -110,9 +110,13 @@ files:
 - lib/generators/rspeed/templates/lib/tasks/rspeed.rb
 - lib/rspeed.rb
 - lib/rspeed/extension.rb
+- lib/rspeed/observer.rb
 - lib/rspeed/runner.rb
 - lib/rspeed/splitter.rb
 - lib/rspeed/version.rb
+- spec/models/rspeed/observer/after_spec.rb
+- spec/models/rspeed/observer/before_spec.rb
+- spec/models/rspeed/observer/before_suite_spec.rb
 - spec/models/rspeed/splitter/append_spec.rb
 - spec/models/rspeed/splitter/destroy_spec.rb
 - spec/models/rspeed/splitter/diff_spec.rb
@@ -164,4 +168,7 @@ test_files:
 - spec/models/rspeed/splitter/diff_spec.rb
 - spec/models/rspeed/splitter/split_spec.rb
 - spec/models/rspeed/splitter/pipes_spec.rb
+- spec/models/rspeed/observer/before_suite_spec.rb
+- spec/models/rspeed/observer/before_spec.rb
+- spec/models/rspeed/observer/after_spec.rb
 - spec/rails_helper.rb