rspeed 0.3.0 → 0.6.0

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

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Observer, '.after_suite' do
+  context 'when all specs is not finished' do
+    let!(:redis) { redis_object }
+
+    before { allow(RSpeed::Redis).to receive(:specs_finished?).and_return(false) }
+
+    it 'sets true on pipe key to indicates that its finished' do
+      described_class.after_suite
+
+      expect(redis.get('rspeed:pipe_01')).to eq('true')
+    end
+  end
+
+  context 'when all specs finished' do
+    before do
+      allow(RSpeed::Redis).to receive(:specs_finished?).and_return(true)
+      allow(RSpeed::Splitter).to receive(:consolidate)
+      allow(RSpeed::Redis).to receive(:clean)
+    end
+
+    it 'consolidates profiles' do
+      described_class.after_suite
+
+      expect(RSpeed::Splitter).to have_received(:consolidate)
+    end
+
+    it 'destroyes pipe finished flag keys' do
+      described_class.after_suite
+
+      expect(RSpeed::Redis).to have_received(:clean)
+    end
+  end
+end

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

@@ -1,7 +1,5 @@
 # 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) }

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

@@ -1,15 +1,27 @@
 # frozen_string_literal: true
 
-require 'rails_helper'
-
 RSpec.describe RSpeed::Observer, '.before_suite' do
-  before { clean_csv_file }
+  let!(:redis) { redis_object }
+
+  before { truncate_profiles }
+
+  after { truncate_profiles }
+
+  it 'cleans only the current pipe profile' do
+    redis.rpush('rspeed:profile_01', { file: '1_spec.rb', time: 1 }.to_json)
+    redis.rpush('rspeed:profile_02', { file: '2_spec.rb', time: 2 }.to_json)
+
+    described_class.before_suite
+
+    expect(redis.keys).to eq ['rspeed:profile_02']
+  end
 
-  it 'cleans the csv file' do
-    File.open('rspeed.csv', 'a') { |file| file.write('content') }
+  it 'cleans the pipe flag' do
+    redis.set('rspeed:pipe_01', true)
+    redis.set('rspeed:pipe_02', true)
 
     described_class.before_suite
 
-    expect(File.open('rspeed.csv').read).to eq ''
+    expect(redis.keys).to eq ['rspeed:pipe_02']
   end
 end

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

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '.clean' do
+  it 'destroys all keys that keeps the pipe finished info' do
+    redis_object.set('rspeed:pipe_01', true)
+    redis_object.set('rspeed:pipe_02', true)
+
+    described_class.clean
+
+    expect(redis_object.keys('*')).to eq([])
+  end
+
+  it 'destroys the pipe profiles' do
+    redis_object.set('rspeed:profile_01', true)
+
+    described_class.clean
+
+    expect(redis_object.keys('*')).to eq([])
+  end
+end

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

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '.client' do
+  it 'returns a redis client' do
+    expect(described_class.client.class).to eq(Redis)
+  end
+end

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

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '#destroy' do
+  before do
+    redis_object.set('rspeed', '{}')
+    redis_object.set('rspeed_1', '{}')
+    redis_object.set('rspeed_2', '{}')
+  end
+
+  it 'destroys via wildcard' do
+    described_class.destroy(pattern: 'rspeed_*')
+
+    expect(redis_object.keys('*')).to eq %w[rspeed]
+  end
+
+  it 'destroys via single name' do
+    described_class.destroy(pattern: 'rspeed')
+
+    expect(redis_object.keys('*')).to eq %w[rspeed_1 rspeed_2]
+  end
+end

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

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '.set' do
+  let!(:redis) { redis_object }
+
+  it 'sets a key on redis' do
+    redis_object.set('key', 'value')
+
+    expect(described_class.get('key')).to eq('value')
+  end
+end

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

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '.keys' do
+  before do
+    redis_object.set('custom_key_1', 'value_1')
+    redis_object.set('custom_key_2', 'value_2')
+    redis_object.set('custom_key_3', 'value_3')
+  end
+
+  it 'shows keys' do
+    expect(described_class.keys(pattern: 'custom_key_*')).to eq %w[custom_key_1 custom_key_2 custom_key_3]
+  end
+end

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

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '#list' do
+  let!(:redis) { redis_object }
+
+  before do
+    redis.rpush('key', { file: '1_spec.rb', time: 1.0 }.to_json)
+    redis.rpush('key', { file: '2_spec.rb', time: 2.0 }.to_json)
+    redis.rpush('key', { file: '3_spec.rb', time: 3.0 }.to_json)
+  end
+
+  it 'list all key entries' do
+    expect(described_class.list('key')).to eq [
+      '{"file":"1_spec.rb","time":1.0}',
+      '{"file":"2_spec.rb","time":2.0}',
+      '{"file":"3_spec.rb","time":3.0}',
+    ]
+  end
+end

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

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '#profiles_content' do
+  let!(:redis) { redis_object }
+
+  before do
+    redis.rpush('rspeed:profile_01', { file: '1_spec.rb', time: 1.0 }.to_json)
+    redis.rpush('rspeed:profile_02', { file: '2_spec.rb', time: 2.0 }.to_json)
+    redis.rpush('rspeed:profile_03', { file: '3_spec.rb', time: 3.0 }.to_json)
+  end
+
+  it 'returns the content of all profiles keys' do
+    expect(described_class.profiles_content).to eq [
+      '{"file":"1_spec.rb","time":1.0}',
+      '{"file":"2_spec.rb","time":2.0}',
+      '{"file":"3_spec.rb","time":3.0}',
+    ]
+  end
+end

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

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '#result?' do
+  context 'when has no key rspeed on redis' do
+    it { expect(described_class.result?).to be(false) }
+  end
+
+  context 'when has key rspeed on redis' do
+    before { described_class.set('rspeed', { files: [[1, '1_spec.rb']], number: 0, total: 1 }.to_json) }
+
+    it { expect(described_class.result?).to be(true) }
+  end
+end

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

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '.set' do
+  let!(:redis) { redis_object }
+
+  it 'sets a key on redis' do
+    described_class.set('key', 'value')
+
+    expect(redis.get('key')).to eq('value')
+  end
+end

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

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '.specs_finished?' do
+  before do
+    described_class.set('rspeed:pipe_01', '1.0')
+
+    allow(RSpeed::Env).to receive(:pipes).and_return(2)
+  end
+
+  context 'when the quantity of pipe result is not the same as the quantity of pipes' do
+    it { expect(described_class.specs_finished?).to be(false) }
+  end
+
+  context 'when the quantity of pipe result is the same as the quantity of pipes' do
+    before { described_class.set('rspeed:pipe_02', '2.0') }
+
+    it { expect(described_class.specs_finished?).to be(true) }
+  end
+end

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

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '.specs_initiated?' do
+  context 'when has no pipe flag key' do
+    it { expect(described_class.specs_finished?).to be(false) }
+  end
+
+  context 'when has at least one pipe flag key' do
+    before { described_class.set('rspeed:pipe_01', '1.0') }
+
+    it { expect(described_class.specs_finished?).to be(true) }
+  end
+end

data/spec/models/rspeed/runner/run_spec.rb

@@ -0,0 +1,73 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Runner, '#run' do
+  let!(:shell) { double('shell') }
+
+  before { allow(shell).to receive(:call) }
+
+  context 'when has result' do
+    before { allow(RSpeed::Redis).to receive(:result?).and_return(true) }
+
+    context 'when is the first pipe' do
+      before do
+        allow(RSpeed::Splitter).to receive(:first_pipe?).and_return(true)
+        allow(RSpeed::Splitter).to receive(:pipe_files).and_return('spec_1.rb spec_2.rb')
+      end
+
+      it 'runs the pipe specs' do
+        described_class.run(shell)
+
+        expect(shell).to have_received(:call).with('bundle exec rspec spec_1.rb spec_2.rb')
+      end
+    end
+
+    context 'when is not the first pipe' do
+      before do
+        allow(RSpeed::Splitter).to receive(:first_pipe?).and_return(false)
+        allow(RSpeed::Splitter).to receive(:pipe_files).and_return('spec_1.rb spec_2.rb')
+      end
+
+      it 'runs the pipe specs' do
+        described_class.run(shell)
+
+        expect(shell).to have_received(:call).with('bundle exec rspec spec_1.rb spec_2.rb')
+      end
+    end
+  end
+
+  context 'when has no result' do
+    before do
+      allow(RSpeed::Splitter).to receive(:first_pipe?).and_return(false)
+      allow(RSpeed::Splitter).to receive(:pipe_files).and_return('spec_1.rb spec_2.rb')
+    end
+
+    context 'when is the first pipe' do
+      before { allow(RSpeed::Splitter).to receive(:first_pipe?).and_return(true) }
+
+      it 'runs the pipe specs' do
+        described_class.run(shell)
+
+        expect(shell).to have_received(:call).with('bundle exec rspec spec_1.rb spec_2.rb')
+      end
+    end
+
+    context 'when is not the first pipe' do
+      before do
+        allow(RSpeed::Splitter).to receive(:first_pipe?).and_return(false)
+        allow(RSpeed::Observer).to receive(:after_suite)
+      end
+
+      it 'does not run the pipe specs' do
+        described_class.run(shell)
+
+        expect(shell).not_to have_received(:call)
+      end
+
+      it 'executes the after suite callback' do
+        described_class.run(shell)
+
+        expect(RSpeed::Observer).to have_received(:after_suite)
+      end
+    end
+  end
+end

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

@@ -1,37 +1,17 @@
 # frozen_string_literal: true
 
-require 'rails_helper'
-
 RSpec.describe RSpeed::Splitter, '#append' do
-  subject(:splitter) { described_class.new }
+  let!(:redis) { redis_object }
 
   it 'appends file and time on rspeed key' do
-    splitter.append [[1, '1_spec.rb'], [2, '2_spec.rb']]
+    described_class.append(
+      items: [{ file: '1_spec.rb', time: 1 }.to_json, { file: '2_spec.rb', time: 2 }.to_json],
+      key: 'rspeed'
+    )
 
-    expect(splitter.get('rspeed_tmp')).to eq [
-      '{"file":"2_spec.rb","time":2.0}',
-      '{"file":"1_spec.rb","time":1.0}'
+    expect(redis.lrange('rspeed', 0, -1)).to eq [
+      '{"file":"1_spec.rb","time":1}',
+      '{"file":"2_spec.rb","time":2}',
     ]
   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
-
-      expect(splitter.get('rspeed_tmp')).to eq [
-        '{"file":"./spec/0_2_spec.rb","time":0.2}',
-        '{"file":"./spec/0_3_spec.rb","time":0.3}',
-        '{"file":"./spec/0_4_spec.rb","time":0.4}',
-        '{"file":"./spec/0_7_spec.rb","time":0.7}',
-        '{"file":"./spec/1_1_spec.rb","time":1.1}',
-        '{"file":"./spec/1_5_spec.rb","time":1.5}',
-        '{"file":"./spec/2_0_spec.rb","time":2.0}',
-      ]
-    end
-  end
 end

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

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Splitter, '#consolidate' do
+  let!(:redis) { redis_object }
+
+  before do
+    redis.rpush('rspeed', 'rspeed_content')
+
+    redis.rpush('rspeed:profile_01', { file: '1_spec.rb', time: 1.0 }.to_json)
+    redis.rpush('rspeed:profile_02', { file: '2_spec.rb', time: 2.0 }.to_json)
+    redis.rpush('rspeed:profile_03', { file: '3_spec.rb', time: 3.0 }.to_json)
+  end
+
+  it 'copies profiles to the result key cleanning the previous result' do
+    described_class.consolidate
+
+    expect(redis.lrange('rspeed', 0, -1)).to eq [
+      '{"file":"1_spec.rb","time":1.0}',
+      '{"file":"2_spec.rb","time":2.0}',
+      '{"file":"3_spec.rb","time":3.0}',
+    ]
+  end
+end

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

@@ -1,19 +1,15 @@
 # frozen_string_literal: true
 
-require 'rails_helper'
-
 RSpec.describe RSpeed::Splitter, '.first_pipe?' do
-  subject(:splitter) { described_class.new }
-
   context 'when pipe env is 1' do
-    before { allow(splitter).to receive(:pipe).and_return 1 }
+    before { allow(RSpeed::Env).to receive(:pipe).and_return 1 }
 
-    it { expect(splitter.first_pipe?).to eq true }
+    it { expect(described_class.first_pipe?).to eq true }
   end
 
   context 'when pipe env is not 1' do
-    before { allow(splitter).to receive(:pipe).and_return 2 }
+    before { allow(RSpeed::Env).to receive(:pipe).and_return 2 }
 
-    it { expect(splitter.first_pipe?).to eq false }
+    it { expect(described_class.first_pipe?).to eq false }
   end
 end

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

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Splitter, '#need_warm?' do
+  context 'when has no result' do
+    before { allow(RSpeed::Redis).to receive(:result?).and_return(false) }
+
+    context 'when is not the first pipe' do
+      before { allow(described_class).to receive(:first_pipe?).and_return(false) }
+
+      it { expect(described_class.need_warm?).to be(false) }
+    end
+
+    context 'when is the first pipe' do
+      before { allow(described_class).to receive(:first_pipe?).and_return(true) }
+
+      it { expect(described_class.need_warm?).to be(true) }
+    end
+  end
+
+  context 'when has result' do
+    before { allow(RSpeed::Redis).to receive(:result?).and_return(true) }
+
+    context 'when is not the first pipe' do
+      before { allow(described_class).to receive(:first_pipe?).and_return(false) }
+
+      it { expect(described_class.need_warm?).to be(false) }
+    end
+
+    context 'when is the first pipe' do
+      before { allow(described_class).to receive(:first_pipe?).and_return(true) }
+
+      it { expect(described_class.need_warm?).to be(false) }
+    end
+  end
+end