rspeed 0.5.2 → 0.6.0

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/set_spec.rb

@@ -1,9 +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(described_class.client.get('key')).to eq('value')
+    expect(redis.get('key')).to eq('value')
   end
 end

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

@@ -2,7 +2,7 @@
 
 RSpec.describe RSpeed::Redis, '.specs_finished?' do
   before do
-    described_class.set('rspeed_pipe_1', '1.0')
+    described_class.set('rspeed:pipe_01', '1.0')
 
     allow(RSpeed::Env).to receive(:pipes).and_return(2)
   end
@@ -12,7 +12,7 @@ RSpec.describe RSpeed::Redis, '.specs_finished?' do
   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') }
+    before { described_class.set('rspeed:pipe_02', '2.0') }
 
     it { expect(described_class.specs_finished?).to be(true) }
   end

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

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

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

@@ -2,20 +2,16 @@
 
 RSpec.describe RSpeed::Runner, '#run' do
   let!(:shell) { double('shell') }
-  let!(:splitter) { instance_double('RSpeed::Splitter') }
 
-  before do
-    allow(shell).to receive(:call)
-    allow(RSpeed::Splitter).to receive(:new).and_return(splitter)
-  end
+  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(splitter).to receive(:first_pipe?).and_return(true)
-        allow(splitter).to receive(:pipe_files).and_return('spec_1.rb spec_2.rb')
+        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
@@ -27,8 +23,8 @@ RSpec.describe RSpeed::Runner, '#run' do
 
     context 'when is not the first pipe' do
       before do
-        allow(splitter).to receive(:first_pipe?).and_return(false)
-        allow(splitter).to receive(:pipe_files).and_return('spec_1.rb spec_2.rb')
+        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
@@ -41,12 +37,12 @@ RSpec.describe RSpeed::Runner, '#run' do
 
   context 'when has no result' do
     before do
-      allow(splitter).to receive(:first_pipe?).and_return(false)
-      allow(splitter).to receive(:pipe_files).and_return('spec_1.rb spec_2.rb')
+      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(splitter).to receive(:first_pipe?).and_return(true) }
+      before { allow(RSpeed::Splitter).to receive(:first_pipe?).and_return(true) }
 
       it 'runs the pipe specs' do
         described_class.run(shell)
@@ -57,7 +53,7 @@ RSpec.describe RSpeed::Runner, '#run' do
 
     context 'when is not the first pipe' do
       before do
-        allow(splitter).to receive(:first_pipe?).and_return(false)
+        allow(RSpeed::Splitter).to receive(:first_pipe?).and_return(false)
         allow(RSpeed::Observer).to receive(:after_suite)
       end
 

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

@@ -1,12 +1,10 @@
 # frozen_string_literal: true
 
 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(
+    described_class.append(
       items: [{ file: '1_spec.rb', time: 1 }.to_json, { file: '2_spec.rb', time: 2 }.to_json],
       key: 'rspeed'
     )

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,17 +1,15 @@
 # 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 }
+    it { expect(described_class.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 }
+    it { expect(described_class.first_pipe?).to eq false }
   end
 end

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

@@ -1,21 +1,19 @@
 # frozen_string_literal: true
 
 RSpec.describe RSpeed::Splitter, '#need_warm?' do
-  subject(:splitter) { described_class.new }
-
   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(splitter).to receive(:first_pipe?).and_return(false) }
+      before { allow(described_class).to receive(:first_pipe?).and_return(false) }
 
-      it { expect(splitter.need_warm?).to be(false) }
+      it { expect(described_class.need_warm?).to be(false) }
     end
 
     context 'when is the first pipe' do
-      before { allow(splitter).to receive(:first_pipe?).and_return(true) }
+      before { allow(described_class).to receive(:first_pipe?).and_return(true) }
 
-      it { expect(splitter.need_warm?).to be(true) }
+      it { expect(described_class.need_warm?).to be(true) }
     end
   end
 
@@ -23,15 +21,15 @@ RSpec.describe RSpeed::Splitter, '#need_warm?' do
     before { allow(RSpeed::Redis).to receive(:result?).and_return(true) }
 
     context 'when is not the first pipe' do
-      before { allow(splitter).to receive(:first_pipe?).and_return(false) }
+      before { allow(described_class).to receive(:first_pipe?).and_return(false) }
 
-      it { expect(splitter.need_warm?).to be(false) }
+      it { expect(described_class.need_warm?).to be(false) }
     end
 
     context 'when is the first pipe' do
-      before { allow(splitter).to receive(:first_pipe?).and_return(true) }
+      before { allow(described_class).to receive(:first_pipe?).and_return(true) }
 
-      it { expect(splitter.need_warm?).to be(false) }
+      it { expect(described_class.need_warm?).to be(false) }
     end
   end
 end

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

@@ -1,26 +1,21 @@
 # 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) }
+    it { expect(described_class.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' }] })
+      allow(described_class).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'
+      expect(described_class.pipe_files).to eq('spec_1.rb spec_2.rb')
     end
   end
 end

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

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-RSpec.describe RSpeed::Splitter, '.split' do
-  subject(:splitter) { described_class.new }
+require 'support/env_mock'
 
+RSpec.describe RSpeed::Splitter, '.split' do
   let!(:unsorted_data) do
     [
       { file: './spec/1_5_spec.rb', time: '1.5' },
@@ -15,47 +15,9 @@ RSpec.describe RSpeed::Splitter, '.split' do
     ]
   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(
+  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,

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

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+require 'support/env_mock'
+
+RSpec.describe RSpeed::Variable, '.append_app_name' do
+  context 'when env app is given' do
+    it 'appends the rspeed app name on the given value' do
+      EnvMock.mock(rspeed_app: 'app') do
+        expect(described_class.append_app_name('value')).to eq('value_app')
+      end
+    end
+
+    context 'when a plus is given' do
+      it 'appends the rspeed app name on the given value with the plus value' do
+        EnvMock.mock(rspeed_app: 'app', plus: 'plus') do
+          expect(described_class.append_app_name('value', plus: 'plus')).to eq('value_app_plus')
+        end
+      end
+    end
+  end
+
+  context 'when env app is not given' do
+    it 'uses just the given string' do
+      expect(described_class.append_app_name('value')).to eq('value')
+    end
+
+    context 'when a plus is given' do
+      it 'appends the plus value' do
+        expect(described_class.append_app_name('value', plus: 'plus')).to eq('value_plus')
+      end
+    end
+  end
+end

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

@@ -5,10 +5,10 @@ require 'support/env_mock'
 RSpec.describe RSpeed::Variable, '.key' do
   it { expect(described_class.key(1)).to be(:rspeed_1) }
 
-  context 'when env name is given' do
-    it 'includes the name' do
-      EnvMock.mock(rspeed_name: 'name') do
-        expect(described_class.key(1)).to be(:rspeed_name_1)
+  context 'when env app is given' do
+    it 'includes the app' do
+      EnvMock.mock(rspeed_app: 'app') do
+        expect(described_class.key(1)).to be(:rspeed_app_1)
       end
     end
   end

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

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+require 'support/env_mock'
+
+RSpec.describe RSpeed::Variable, '.pipe' do
+  it { expect(described_class.pipe).to eq('rspeed:pipe_01') }
+
+  context 'when env name is given' do
+    it 'includes the name' do
+      EnvMock.mock(rspeed_app: 'app') do
+        expect(described_class.pipe).to eq('rspeed:pipe_app_01')
+      end
+    end
+  end
+
+  context 'when pipe is bigger than 9' do
+    it 'does not prefix zero on the number' do
+      EnvMock.mock(rspeed_pipe: 10) do
+        expect(described_class.pipe).to eq('rspeed:pipe_10')
+      end
+    end
+  end
+end

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

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 RSpec.describe RSpeed::Variable, 'PIPES_PATTERN' do
-  it { expect(described_class::PIPES_PATTERN).to eq('rspeed_pipe_*') }
+  it { expect(described_class::PIPES_PATTERN).to eq('rspeed:pipe_*') }
 end

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

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 RSpec.describe RSpeed::Variable, 'PROFILE_PATTERN' do
-  it { expect(described_class::PROFILE_PATTERN).to eq('rspeed_profile_*') }
+  it { expect(described_class::PROFILE_PATTERN).to eq('rspeed:profile_*') }
 end

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

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+require 'support/env_mock'
+
+RSpec.describe RSpeed::Variable, '.profile' do
+  it { expect(described_class.profile).to eq('rspeed:profile_01') }
+
+  context 'when env name is given' do
+    it 'includes the name' do
+      EnvMock.mock(rspeed_app: 'name') do
+        expect(described_class.profile).to eq('rspeed:profile_name_01')
+      end
+    end
+  end
+
+  context 'when pipe is bigger than 9' do
+    it 'does not prefix zero on the number' do
+      EnvMock.mock(rspeed_pipe: 10) do
+        expect(described_class.profile).to eq('rspeed:profile_10')
+      end
+    end
+  end
+end