rspeed 0.2.0 → 0.5.2

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

@@ -1,18 +1,25 @@
 # frozen_string_literal: true
 
-require 'rails_helper'
-
 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 }
+  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 'renames the key' do
+  it 'copies profiles to the result key cleanning the previous result' 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}']
+    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/split_spec.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require 'rails_helper'
-
 RSpec.describe RSpeed::Splitter, '.split' do
   subject(:splitter) { described_class.new }
 
@@ -17,37 +15,75 @@ RSpec.describe RSpeed::Splitter, '.split' do
     ]
   end
 
-  before { allow(splitter).to receive(:pipes).and_return 3 }
-
-  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
-      }
-    )
+  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

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

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

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

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+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)
+      end
+    end
+  end
+end

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

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

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

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

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

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

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

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

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+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')
+      end
+    end
+  end
+
+  context 'when env name is not given' do
+    it { expect(described_class.result).to eq('rspeed') }
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+def delete_file(file_path)
+  File.delete(file_path) if File.exist?(file_path)
+end
+
+def truncate_profiles
+  RSpeed::Redis.destroy(RSpeed::Variable::PROFILE_PATTERN)
+end
+
+def populate_profiles
+  data = [
+    { file: './spec/2_0_spec.rb', time: 2.0 }.to_json,
+    { file: './spec/1_5_spec.rb', time: 1.5 }.to_json,
+    { file: './spec/1_1_spec.rb', time: 1.1 }.to_json,
+    { file: './spec/0_7_spec.rb', time: 0.7 }.to_json,
+    { file: './spec/0_4_spec.rb', time: 0.4 }.to_json,
+    { file: './spec/0_3_spec.rb', time: 0.3 }.to_json,
+    { file: './spec/0_2_spec.rb', time: 0.2 }.to_json,
+  ]
+
+  RSpeed::Splitter.new.append(items: data, key: RSpeed::Variable.profile)
+end
+
+def redis_object
+  @redis_object ||= Redis.new(db: RSpeed::Env.db, host: RSpeed::Env.host)
+end

data/spec/support/common.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+require 'rspec'
+
+RSpec.configure do |config|
+  config.disable_monkey_patching!
+
+  config.mock_with(:rspec) do |mocks|
+    mocks.verify_partial_doubles = true
+  end
+
+  config.order = :random
+end

data/spec/support/coverage.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+if ENV['COVERAGE'] == 'true'
+  require 'simplecov'
+  require 'codecov'
+
+  SimpleCov.formatter = SimpleCov::Formatter::Codecov
+
+  SimpleCov.minimum_coverage(ENV.fetch('MINIMUM_COVERAGE', 80).to_i)
+
+  SimpleCov.start('rails') do
+    add_filter('vendor')
+  end
+end

data/spec/support/env_mock.rb

@@ -0,0 +1,3 @@
+# frozen_string_literal: true
+
+require 'env_mock'

data/spec/support/fakeredis.rb

@@ -0,0 +1,3 @@
+# frozen_string_literal: true
+
+require 'fakeredis/rspec'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rspeed
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.5.2
 platform: ruby
 authors:
 - Washington Botelho
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-08-19 00:00:00.000000000 Z
+date: 2021-02-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis
@@ -24,6 +24,48 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: codecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: env_mock
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: fakeredis
   requirement: !ruby/object:Gem::Requirement
@@ -67,7 +109,21 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: rubocop-performance
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -109,28 +165,65 @@ files:
 - lib/generators/rspeed/install_generator.rb
 - lib/generators/rspeed/templates/lib/tasks/rspeed.rb
 - lib/rspeed.rb
+- lib/rspeed/env.rb
 - lib/rspeed/extension.rb
+- lib/rspeed/logger.rb
 - lib/rspeed/observer.rb
+- lib/rspeed/redis.rb
 - lib/rspeed/runner.rb
 - lib/rspeed/splitter.rb
+- lib/rspeed/variable.rb
 - lib/rspeed/version.rb
+- spec/common_helper.rb
+- spec/fixtures/1_spec.rb
+- spec/fixtures/2_spec.rb
+- spec/fixtures/empty.rb
+- spec/fixtures/new_spec.rb.csv
+- 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/observer/after_spec.rb
+- spec/models/rspeed/observer/after_suite_spec.rb
 - spec/models/rspeed/observer/before_spec.rb
 - spec/models/rspeed/observer/before_suite_spec.rb
+- spec/models/rspeed/redis/clean_spec.rb
+- spec/models/rspeed/redis/client_spec.rb
+- 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/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/destroy_spec.rb
 - spec/models/rspeed/splitter/diff_spec.rb
 - spec/models/rspeed/splitter/first_pipe_spec.rb
 - spec/models/rspeed/splitter/get_spec.rb
-- spec/models/rspeed/splitter/keys_spec.rb
-- spec/models/rspeed/splitter/last_pipe_spec.rb
-- spec/models/rspeed/splitter/pipe_spec.rb
-- spec/models/rspeed/splitter/pipes_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/result_spec.rb
-- spec/models/rspeed/splitter/save_spec.rb
 - spec/models/rspeed/splitter/split_spec.rb
-- spec/rails_helper.rb
+- spec/models/rspeed/variable/append_name_spec.rb
+- spec/models/rspeed/variable/default_partner_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/pipes_pattern_spec.rb
+- spec/models/rspeed/variable/profile_pattern_spec.rb
+- spec/models/rspeed/variable/result_spec.rb
+- spec/spec_helper.rb
+- spec/support/common.rb
+- spec/support/coverage.rb
+- spec/support/env_mock.rb
+- spec/support/fakeredis.rb
 homepage: https://github.com/wbotelhos/rspeed
 licenses:
 - MIT
@@ -150,25 +243,58 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.8
+rubygems_version: 3.1.2
 signing_key:
 specification_version: 4
 summary: Split and speed up your RSpec tests.
 test_files:
+- spec/spec_helper.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/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/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/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/runner/run_spec.rb
 - spec/models/rspeed/splitter/append_spec.rb
-- spec/models/rspeed/splitter/destroy_spec.rb
-- spec/models/rspeed/splitter/pipe_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/last_pipe_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/result_spec.rb
-- spec/models/rspeed/splitter/save_spec.rb
-- spec/models/rspeed/splitter/keys_spec.rb
 - spec/models/rspeed/splitter/diff_spec.rb
 - spec/models/rspeed/splitter/split_spec.rb
-- spec/models/rspeed/splitter/pipes_spec.rb
+- spec/models/rspeed/splitter/append_question_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/rails_helper.rb
+- spec/models/rspeed/env/pipe_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/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