rspeed 0.3.0 → 0.6.0

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

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Splitter, '#pipe_files' do
+  context 'when has no result' do
+    before { allow(RSpeed::Redis).to receive(:result?).and_return(false) }
+
+    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(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(described_class.pipe_files).to eq('spec_1.rb spec_2.rb')
+    end
+  end
+end

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

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

@@ -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 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
+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

@@ -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/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

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 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 app 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(pattern: 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.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.3.0
+  version: 0.6.0
 platform: ruby
 authors:
 - Washington Botelho
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-08-21 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,70 @@ 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
 - 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/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/pipe_spec.rb
+- spec/models/rspeed/env/pipes_spec.rb
+- spec/models/rspeed/env/port_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
 - 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/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/append_spec.rb
-- spec/models/rspeed/splitter/destroy_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/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/rename_spec.rb
-- spec/models/rspeed/splitter/result_spec.rb
-- spec/models/rspeed/splitter/save_spec.rb
+- spec/models/rspeed/splitter/need_warm_question_spec.rb
+- spec/models/rspeed/splitter/pipe_files_spec.rb
 - spec/models/rspeed/splitter/split_spec.rb
-- spec/rails_helper.rb
+- spec/models/rspeed/variable/append_app_name_spec.rb
+- spec/models/rspeed/variable/key_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
+- spec/support/coverage.rb
+- spec/support/env_mock.rb
+- spec/support/fakeredis.rb
 homepage: https://github.com/wbotelhos/rspeed
 licenses:
 - MIT
@@ -150,25 +248,61 @@ 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/database/result_spec.rb
+- spec/models/rspeed/variable/pipes_pattern_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/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/destroy_spec.rb
-- spec/models/rspeed/splitter/pipe_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/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/rails_helper.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/host_spec.rb
+- spec/models/rspeed/env/db_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/1_spec.rb
+- spec/fixtures/empty.rb
+- spec/fixtures/2_spec.rb