rspeed 0.5.2 → 0.6.0

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

@@ -5,15 +5,15 @@ require 'support/env_mock'
 RSpec.describe RSpeed::Variable, '.result' do
   it { expect(described_class.result).to eq('rspeed') }
 
-  context 'when env name is given' do
-    it 'includes the name' do
-      EnvMock.mock(rspeed_name: 'name') do
-        expect(described_class.result).to eq('rspeed_name')
+  context 'when env app is given' do
+    it 'includes the app' do
+      EnvMock.mock(rspeed_app: 'app') do
+        expect(described_class.result).to eq('rspeed_app')
       end
     end
   end
 
-  context 'when env name is not given' do
+  context 'when env app is not given' do
     it { expect(described_class.result).to eq('rspeed') }
   end
 end

data/spec/spec_helper.rb

@@ -5,7 +5,7 @@ def delete_file(file_path)
 end
 
 def truncate_profiles
-  RSpeed::Redis.destroy(RSpeed::Variable::PROFILE_PATTERN)
+  RSpeed::Redis.destroy(pattern: RSpeed::Variable::PROFILE_PATTERN)
 end
 
 def populate_profiles
@@ -19,7 +19,7 @@ def populate_profiles
     { file: './spec/0_2_spec.rb', time: 0.2 }.to_json,
   ]
 
-  RSpeed::Splitter.new.append(items: data, key: RSpeed::Variable.profile)
+  RSpeed::Splitter.append(items: data, key: RSpeed::Variable.profile)
 end
 
 def redis_object

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rspeed
 version: !ruby/object:Gem::Version
-  version: 0.5.2
+  version: 0.6.0
 platform: ruby
 authors:
 - Washington Botelho
@@ -165,6 +165,8 @@ files:
 - lib/generators/rspeed/install_generator.rb
 - lib/generators/rspeed/templates/lib/tasks/rspeed.rb
 - lib/rspeed.rb
+- lib/rspeed/database.rb
+- lib/rspeed/differ.rb
 - lib/rspeed/env.rb
 - lib/rspeed/extension.rb
 - lib/rspeed/logger.rb
@@ -178,15 +180,21 @@ files:
 - spec/fixtures/1_spec.rb
 - spec/fixtures/2_spec.rb
 - spec/fixtures/empty.rb
-- spec/fixtures/new_spec.rb.csv
+- 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/removed_data_spec.rb
+- spec/models/rspeed/differ/removed_time_spec.rb
+- spec/models/rspeed/env/app_spec.rb
 - spec/models/rspeed/env/db_spec.rb
 - spec/models/rspeed/env/host_spec.rb
-- spec/models/rspeed/env/name_spec.rb
 - spec/models/rspeed/env/pipe_spec.rb
 - spec/models/rspeed/env/pipes_spec.rb
 - spec/models/rspeed/env/port_spec.rb
-- spec/models/rspeed/env/result_key_spec.rb
 - spec/models/rspeed/env/rspeed_spec.rb
+- spec/models/rspeed/env/spec_path_spec.rb
 - spec/models/rspeed/observer/after_spec.rb
 - spec/models/rspeed/observer/after_suite_spec.rb
 - spec/models/rspeed/observer/before_spec.rb
@@ -196,28 +204,25 @@ files:
 - spec/models/rspeed/redis/destroy_spec.rb
 - spec/models/rspeed/redis/get_spec.rb
 - spec/models/rspeed/redis/keys_spec.rb
+- spec/models/rspeed/redis/list_spec.rb
+- spec/models/rspeed/redis/profiles_content_spec.rb
 - spec/models/rspeed/redis/result_spec.rb
 - 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/runner/run_spec.rb
-- spec/models/rspeed/splitter/actual_examples_spec.rb
-- spec/models/rspeed/splitter/append_question_spec.rb
 - spec/models/rspeed/splitter/append_spec.rb
-- spec/models/rspeed/splitter/diff_spec.rb
+- spec/models/rspeed/splitter/consolidate_spec.rb
 - spec/models/rspeed/splitter/first_pipe_spec.rb
-- spec/models/rspeed/splitter/get_spec.rb
 - spec/models/rspeed/splitter/need_warm_question_spec.rb
 - spec/models/rspeed/splitter/pipe_files_spec.rb
-- spec/models/rspeed/splitter/rename_spec.rb
 - spec/models/rspeed/splitter/split_spec.rb
-- spec/models/rspeed/variable/append_name_spec.rb
-- spec/models/rspeed/variable/default_partner_spec.rb
+- spec/models/rspeed/variable/append_app_name_spec.rb
 - spec/models/rspeed/variable/key_spec.rb
-- spec/models/rspeed/variable/pipe_name_spec.rb
-- spec/models/rspeed/variable/pipe_profile_spec.rb
+- spec/models/rspeed/variable/pipe_spec.rb
 - spec/models/rspeed/variable/pipes_pattern_spec.rb
 - spec/models/rspeed/variable/profile_pattern_spec.rb
+- spec/models/rspeed/variable/profile_spec.rb
 - spec/models/rspeed/variable/result_spec.rb
 - spec/spec_helper.rb
 - spec/support/common.rb
@@ -249,52 +254,55 @@ specification_version: 4
 summary: Split and speed up your RSpec tests.
 test_files:
 - spec/spec_helper.rb
+- spec/models/rspeed/database/result_spec.rb
 - spec/models/rspeed/variable/pipes_pattern_spec.rb
-- spec/models/rspeed/variable/pipe_profile_spec.rb
-- spec/models/rspeed/variable/default_partner_spec.rb
-- spec/models/rspeed/variable/append_name_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/pipe_name_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
 - spec/models/rspeed/redis/destroy_spec.rb
 - spec/models/rspeed/redis/get_spec.rb
 - 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/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/runner/run_spec.rb
 - spec/models/rspeed/splitter/append_spec.rb
-- spec/models/rspeed/splitter/actual_examples_spec.rb
-- spec/models/rspeed/splitter/get_spec.rb
-- spec/models/rspeed/splitter/rename_spec.rb
 - spec/models/rspeed/splitter/need_warm_question_spec.rb
 - spec/models/rspeed/splitter/pipe_files_spec.rb
 - spec/models/rspeed/splitter/first_pipe_spec.rb
-- spec/models/rspeed/splitter/diff_spec.rb
 - spec/models/rspeed/splitter/split_spec.rb
-- spec/models/rspeed/splitter/append_question_spec.rb
+- spec/models/rspeed/splitter/consolidate_spec.rb
 - spec/models/rspeed/observer/before_suite_spec.rb
 - 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/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/added_data_spec.rb
 - spec/models/rspeed/env/pipe_spec.rb
+- spec/models/rspeed/env/spec_path_spec.rb
 - spec/models/rspeed/env/rspeed_spec.rb
 - spec/models/rspeed/env/port_spec.rb
-- spec/models/rspeed/env/name_spec.rb
 - spec/models/rspeed/env/host_spec.rb
 - spec/models/rspeed/env/db_spec.rb
-- spec/models/rspeed/env/result_key_spec.rb
+- spec/models/rspeed/env/app_spec.rb
 - spec/models/rspeed/env/pipes_spec.rb
 - spec/common_helper.rb
 - spec/support/fakeredis.rb
 - spec/support/env_mock.rb
 - spec/support/common.rb
 - spec/support/coverage.rb
-- spec/fixtures/new_spec.rb.csv
 - spec/fixtures/1_spec.rb
 - spec/fixtures/empty.rb
 - spec/fixtures/2_spec.rb

data/spec/fixtures/new_spec.rb.csv

@@ -1 +0,0 @@
-10,./spec/fixtures/new_spec.rb.csv:1010,./spec/fixtures/new_spec.rb.csv:10

data/spec/models/rspeed/env/result_key_spec.rb

@@ -1,19 +0,0 @@
-# frozen_string_literal: true
-
-require 'support/env_mock'
-
-RSpec.describe RSpeed::Env, '.result_key' do
-  context 'when key is not setted on env' do
-    it 'returns default value' do
-      expect(described_class.result_key).to eq 'rspeed'
-    end
-  end
-
-  context 'when key is setted on env' do
-    it 'returns env value' do
-      EnvMock.mock(respeed_result_key: 'result_customer') do
-        expect(described_class.result_key).to eq 'result_customer'
-      end
-    end
-  end
-end

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

@@ -1,26 +0,0 @@
-# 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_question_spec.rb

@@ -1,37 +0,0 @@
-# frozen_string_literal: true
-
-RSpec.describe RSpeed::Splitter, '#append?' 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) }
-
-      it { expect(splitter.append?).to be(false) }
-    end
-
-    context 'when is the first pipe' do
-      before { allow(splitter).to receive(:first_pipe?).and_return(true) }
-
-      it { expect(splitter.append?).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(splitter).to receive(:first_pipe?).and_return(false) }
-
-      it { expect(splitter.append?).to be(true) }
-    end
-
-    context 'when is the first pipe' do
-      before { allow(splitter).to receive(:first_pipe?).and_return(true) }
-
-      it { expect(splitter.append?).to be(true) }
-    end
-  end
-end

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

@@ -1,30 +0,0 @@
-# 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/get_spec.rb

@@ -1,60 +0,0 @@
-# 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
-end

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

@@ -1,25 +0,0 @@
-# frozen_string_literal: true
-
-RSpec.describe RSpeed::Splitter, '#rename' do
-  subject(:splitter) { described_class.new }
-
-  let!(:redis) { redis_object }
-
-  before do
-    redis.lpush('rspeed', 'rspeed_content')
-
-    redis.lpush('rspeed_profile_1', { file: '1_spec.rb', time: 1.0 }.to_json)
-    redis.lpush('rspeed_profile_2', { file: '2_spec.rb', time: 2.0 }.to_json)
-    redis.lpush('rspeed_profile_3', { file: '3_spec.rb', time: 3.0 }.to_json)
-  end
-
-  it 'copies profiles to the result key cleanning the previous result' do
-    splitter.rename
-
-    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/variable/append_name_spec.rb

@@ -1,19 +0,0 @@
-# frozen_string_literal: true
-
-require 'support/env_mock'
-
-RSpec.describe RSpeed::Variable, '.append_name' do
-  context 'when env name is given' do
-    it 'appends the rspeed name on the given value' do
-      EnvMock.mock(rspeed_name: 'name') do
-        expect(described_class.append_name('value')).to eq('value_name')
-      end
-    end
-  end
-
-  context 'when env name is not given' do
-    it 'ignores the suffix' do
-      expect(described_class.append_name('value')).to eq('value')
-    end
-  end
-end

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

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

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

@@ -1,15 +0,0 @@
-# frozen_string_literal: true
-
-require 'support/env_mock'
-
-RSpec.describe RSpeed::Variable, '.pipe_name' do
-  it { expect(described_class.pipe_name).to eq('rspeed_pipe_1') }
-
-  context 'when env name is given' do
-    it 'includes the name' do
-      EnvMock.mock(rspeed_name: 'name') do
-        expect(described_class.pipe_name).to eq('rspeed_pipe_name_1')
-      end
-    end
-  end
-end

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

@@ -1,15 +0,0 @@
-# frozen_string_literal: true
-
-require 'support/env_mock'
-
-RSpec.describe RSpeed::Variable, '.profile' do
-  it { expect(described_class.profile).to eq('rspeed_profile_1') }
-
-  context 'when env name is given' do
-    it 'includes the name' do
-      EnvMock.mock(rspeed_name: 'name') do
-        expect(described_class.profile).to eq('rspeed_profile_name_1')
-      end
-    end
-  end
-end