rspeed 0.4.0 → 0.5.0

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

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Variable, 'CSV' do
+  it { expect(described_class::CSV).to eq('rspeed.csv') }
+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/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/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/models/rspeed/variable/tmp_spec.rb

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

data/spec/{rails_helper.rb → spec_helper.rb}

@@ -1,25 +1,5 @@
 # frozen_string_literal: true
 
-ENV['RAILS_ENV'] ||= 'test'
-
-require 'fakeredis/rspec'
-require 'json'
-require 'pry-byebug'
-require 'rspec'
-require 'rspeed'
-
-RSpec.configure do |config|
-  config.after do
-    ENV.delete('RSPEED_PIPE')
-    ENV.delete('RSPEED_PIPES')
-  end
-
-  config.disable_monkey_patching!
-
-  config.order = :random
-  config.profile_examples = 10
-end
-
 def delete_file(file_path = 'rspeed.csv')
   File.delete(file_path) if File.exist?(file_path)
 end
@@ -36,12 +16,12 @@ def populate_csv_file
     '0.7,./spec/0_7_spec.rb',
     '0.4,./spec/0_4_spec.rb',
     '0.3,./spec/0_3_spec.rb',
-    '0.2,./spec/0_2_spec.rb'
+    '0.2,./spec/0_2_spec.rb',
   ].join("\n")
 
   File.open('rspeed.csv', 'a') { |file| file.write(data) }
 end
 
 def redis_object
-  @redis_object ||= Redis.new(db: ENV['RSPEED_DB'], host: ENV['RSPEED_HOST'])
+  @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.4.0
+  version: 0.5.0
 platform: ruby
 authors:
 - Washington Botelho
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-08-21 00:00:00.000000000 Z
+date: 2020-12-06 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,32 +165,64 @@ 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/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/env/tmp_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_pipes_flag_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_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/pipe_files_spec.rb
+- spec/models/rspeed/splitter/redundant_run_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/csv_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/pipes_pattern_spec.rb
+- spec/models/rspeed/variable/result_spec.rb
+- spec/models/rspeed/variable/tmp_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
@@ -154,29 +242,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/tmp_spec.rb
+- spec/models/rspeed/variable/default_partner_spec.rb
+- spec/models/rspeed/variable/csv_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/redis/destroy_spec.rb
+- spec/models/rspeed/redis/get_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/clean_pipes_flag_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/redundant_run_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/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/env/pipe_spec.rb
+- spec/models/rspeed/env/rspeed_spec.rb
+- spec/models/rspeed/env/tmp_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
-- spec/rails_helper.rb

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

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