rspeed 0.0.1 → 0.5.0

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

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '#destroy' do
+  let!(:redis) { described_class }
+
+  before do
+    redis.set('rspeed', '{}')
+    redis.set('rspeed_1', '{}')
+    redis.set('rspeed_2', '{}')
+  end
+
+  it 'destroys via wildcard' do
+    redis.destroy('rspeed_*')
+
+    expect(redis.keys('*')).to eq %w[rspeed]
+  end
+
+  it 'destroys via single name' do
+    redis.destroy('rspeed')
+
+    expect(redis.keys('*')).to eq %w[rspeed_1 rspeed_2]
+  end
+
+  it 'destroys default partner when no pattern is given' do
+    redis.destroy
+
+    expect(redis.keys('*')).to eq %w[rspeed]
+  end
+end

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

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

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

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '.keys' do
+  subject(:redis) { described_class }
+
+  context 'with default config' do
+    before do
+      described_class.set('rspeed_1', 'value_1')
+      described_class.set('rspeed_2', 'value_2')
+      described_class.set('rspeed_3', 'value_3')
+    end
+
+    it 'shows keys' do
+      expect(redis.keys).to eq %w[rspeed_1 rspeed_2 rspeed_3]
+    end
+  end
+
+  context 'with custom key' do
+    before do
+      described_class.set('custom_key_1', 'value_1')
+      described_class.set('custom_key_2', 'value_2')
+      described_class.set('custom_key_3', 'value_3')
+    end
+
+    it 'shows keys' do
+      expect(redis.keys('custom_key_*')).to eq %w[custom_key_1 custom_key_2 custom_key_3]
+    end
+  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,9 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '.set' do
+  it 'sets a key on redis' do
+    described_class.set('key', 'value')
+
+    expect(described_class.client.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_1', '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_2', '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_1', '1.0') }
+
+    it { expect(described_class.specs_finished?).to be(true) }
+  end
+end

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

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Runner, '#run' do
+  let!(:shell) { double('shell') }
+  let!(:splitter) { instance_double('RSpeed::Splitter') }
+
+  before { allow(RSpeed::Splitter).to receive(:new).and_return(splitter) }
+
+  context 'when is a redundant run' do
+    before { allow(splitter).to receive(:redundant_run?).and_return(true) }
+
+    it 'aborts the run' do
+      expect(described_class.run(shell)).to be(nil)
+    end
+  end
+
+  context 'when is not a redundant run' do
+    before do
+      allow(splitter).to receive(:redundant_run?).and_return(false)
+      allow(shell).to receive(:call)
+      allow(splitter).to receive(:pipe_files).and_return('spec_1.rb spec_2.rb')
+    end
+
+    it 'run 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

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

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Splitter, '#actual_examples' do
+  it 'returns all examples' do
+    splitter = described_class.new(specs_path: 'spec/fixtures/**/*_spec.rb')
+
+    expect(splitter.actual_examples).to eq [
+      'spec/fixtures/1_spec.rb:4',
+      'spec/fixtures/1_spec.rb:6',
+      'spec/fixtures/1_spec.rb:8',
+      'spec/fixtures/2_spec.rb:4',
+    ]
+  end
+
+  it 'does not raise when no file match' do
+    splitter = described_class.new(specs_path: 'spec/fixtures/**/*_missing.rb')
+
+    expect(splitter.actual_examples).to eq []
+  end
+
+  it 'does not raise when file is empty' do
+    splitter = described_class.new(specs_path: 'spec/fixtures/**/empty.rb')
+
+    expect(splitter.actual_examples).to eq []
+  end
+end

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

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Splitter, '#append' do
+  subject(:splitter) { described_class.new }
+
+  it 'appends file and time on rspeed key' do
+    splitter.append [[1, '1_spec.rb'], [2, '2_spec.rb']]
+
+    expect(splitter.get('rspeed_tmp')).to eq [
+      '{"file":"2_spec.rb","time":2.0}',
+      '{"file":"1_spec.rb","time":1.0}',
+    ]
+  end
+
+  context 'when files is not given' do
+    before do
+      truncate_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/diff_spec.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Splitter, '#diff' do
+  subject(:splitter) { described_class.new(specs_path: './spec/fixtures/*_spec.rb') }
+
+  let!(:redis) { redis_object }
+
+  before do
+    redis.lpush 'rspeed', { file: './spec/fixtures/1_spec.rb:4', time: '1.4' }.to_json
+    redis.lpush 'rspeed', { file: './spec/fixtures/1_spec.rb:6', time: '1.6' }.to_json
+    redis.lpush 'rspeed', { file: './spec/fixtures/1_spec.rb:8', time: '1.8' }.to_json
+    redis.lpush 'rspeed', { file: './spec/fixtures/2_spec.rb:4', time: '2.4' }.to_json
+    redis.lpush 'rspeed', { file: './spec/fixtures/2_spec.rb:666', time: '6' }.to_json
+    redis.lpush 'rspeed', { file: './spec/fixtures/x_spec.rb:1', time: 3 }.to_json
+
+    File.open('spec/fixtures/new_spec.rb', 'a') { |file| file.write('it') }
+  end
+
+  after { delete_file('spec/fixtures/new_spec.rb') }
+
+  it 'removes removed specs and adds new spec and keeps keeped specs based on rspeed key values' do
+    expect(splitter.diff).to eq [
+      { file: './spec/fixtures/2_spec.rb:4', time: '2.4' },
+      { file: './spec/fixtures/1_spec.rb:8', time: '1.8' },
+      { file: './spec/fixtures/1_spec.rb:6', time: '1.6' },
+      { file: './spec/fixtures/1_spec.rb:4', time: '1.4' },
+      { file: './spec/fixtures/new_spec.rb:1', time: 0 },
+    ]
+  end
+end

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

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

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

@@ -0,0 +1,74 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Splitter, '#get' do
+  subject(:splitter) { described_class.new }
+
+  let!(:redis) { redis_object }
+
+  context 'when wildcard pattern is given' do
+    before do
+      redis.set 'rspeed_1', { files: [[1, '1_spec.rb'], [2, '2_spec.rb']], number: 0, total: 3 }.to_json
+      redis.set 'rspeed_2', { files: [[3, '3_spec.rb']], number: 1, total: 3 }.to_json
+    end
+
+    it 'returns all values' do
+      expect(splitter.get('rspeed_*')).to eq [
+        {
+          'files' => [[1, '1_spec.rb'], [2, '2_spec.rb']],
+          'number' => 0,
+          'total' => 3,
+        },
+
+        {
+          'files' => [[3, '3_spec.rb']],
+          'number' => 1,
+          'total' => 3,
+        },
+      ]
+    end
+  end
+
+  context 'when normal pattern is given' do
+    before do
+      redis.set 'pattern', { files: [[1, '1_spec.rb'], [2, '2_spec.rb']], number: 0, total: 3 }.to_json
+    end
+
+    it 'returns all values from that key' do
+      expect(splitter.get('pattern')).to eq [
+        {
+          'files' => [[1, '1_spec.rb'], [2, '2_spec.rb']],
+          'number' => 0,
+          'total' => 3,
+        },
+      ]
+    end
+  end
+
+  context 'when pattern is rspeed' do
+    before do
+      redis.lpush 'rspeed', { file: '1_spec.rb', time: 1 }.to_json
+      redis.lpush 'rspeed', { file: '2_spec.rb', time: 2 }.to_json
+    end
+
+    it 'executes the right fetch method' do
+      expect(splitter.get('rspeed')).to eq [
+        '{"file":"2_spec.rb","time":2}',
+        '{"file":"1_spec.rb","time":1}',
+      ]
+    end
+  end
+
+  context 'when pattern is rspeed_tmp' do
+    before do
+      redis.lpush 'rspeed_tmp', { file: '1_spec.rb', time: 1 }.to_json
+      redis.lpush 'rspeed_tmp', { file: '2_spec.rb', time: 2 }.to_json
+    end
+
+    it 'executes the right fetch method' do
+      expect(splitter.get('rspeed_tmp')).to eq [
+        '{"file":"2_spec.rb","time":2}',
+        '{"file":"1_spec.rb","time":1}',
+      ]
+    end
+  end
+end

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

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Splitter, '#pipe_files' do
+  let!(:shell) { double('shell') }
+  let!(:splitter) { described_class.new }
+
+  before { allow(RSpeed::Env).to receive(:pipe).and_return(1) }
+
+  context 'when has no result' do
+    before { allow(RSpeed::Redis).to receive(:result?).and_return(false) }
+
+    it { expect(splitter.pipe_files).to be(nil) }
+  end
+
+  context 'when has result' do
+    before do
+      allow(RSpeed::Redis).to receive(:result?).and_return(true)
+
+      allow(splitter).to receive(:split).and_return(rspeed_1: { files: [{ file: 'spec_1.rb' }, { file: 'spec_2.rb' }] })
+    end
+
+    it 'returns the splitted pipe files' do
+      expect(splitter.pipe_files).to eq 'spec_1.rb spec_2.rb'
+    end
+  end
+end

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

@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Splitter, '.redundant_run?' do
+  subject(:splitter) { described_class.new }
+
+  context 'when is not pipe 1' do
+    before { allow(RSpeed::Env).to receive(:pipe).and_return(2) }
+
+    context 'when result key exists' do
+      let!(:redis) { redis_object }
+
+      before do
+        redis.set('rspeed', '{}')
+
+        allow(RSpeed::Env).to receive(:pipe).and_return(2)
+      end
+
+      it { expect(splitter.redundant_run?).to be(false) }
+    end
+
+    context 'when result key does not exist' do
+      it { expect(splitter.redundant_run?).to be(true) }
+    end
+  end
+
+  context 'when is the first pipe' do
+    before { allow(RSpeed::Env).to receive(:pipe).and_return(1) }
+
+    context 'when result key exists' do
+      let!(:redis) { redis_object }
+
+      before do
+        redis.set('rspeed', '{}')
+
+        allow(RSpeed::Env).to receive(:pipe).and_return(2)
+      end
+
+      it { expect(splitter.redundant_run?).to be(false) }
+    end
+
+    context 'when result key does not exist' do
+      it { expect(splitter.redundant_run?).to be(false) }
+    end
+  end
+end

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

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Splitter, '#rename' do
+  subject(:splitter) { described_class.new }
+
+  let!(:redis) { redis_object }
+
+  before { redis.lpush('rspeed_tmp', { file: '1_spec.rb', time: 1.0 }.to_json) }
+
+  it 'renames the key' do
+    splitter.rename
+
+    expect(redis.lrange('rspeed_tmp', 0, -1)).to eq([])
+    expect(redis.lrange('rspeed', 0, -1)).to     eq(['{"file":"1_spec.rb","time":1.0}'])
+  end
+end

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

@@ -0,0 +1,89 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Splitter, '.split' do
+  subject(:splitter) { described_class.new }
+
+  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
+
+  before { allow(RSpeed::Env).to receive(:pipes).and_return 3 }
+
+  context 'when diff is given' do
+    it 'splits the times between the pipes' do
+      expect(splitter.split(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
+
+  context 'when diff is not given' do
+    before { allow(splitter).to receive(:diff).and_return(unsorted_data) }
+
+    it 'splits the diff result into times between the pipes' do
+      expect(splitter.split).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