rspeed 0.2.0 → 0.5.2

data/lib/rspeed/variable.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+module RSpeed
+  module Variable
+    module_function
+
+    DEFAULT_PATTERN = 'rspeed_*'
+    PIPES_PATTERN   = 'rspeed_pipe_*'
+    PROFILE_PATTERN = 'rspeed_profile_*'
+
+    def append_name(value, suffix = nil)
+      [value, RSpeed::Env.name, suffix].compact.join('_')
+    end
+
+    def key(number)
+      append_name('rspeed', number).to_sym
+    end
+
+    def result
+      append_name('rspeed')
+    end
+
+    def pipe_name
+      append_name('rspeed_pipe', RSpeed::Env.pipe)
+    end
+
+    def profile
+      append_name('rspeed_profile', RSpeed::Env.pipe)
+    end
+  end
+end

data/lib/rspeed/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RSpeed
-  VERSION = '0.2.0'
+  VERSION = '0.5.2'
 end

data/spec/common_helper.rb

@@ -0,0 +1,10 @@
+# frozen_string_literal: true
+
+ENV['RAILS_ENV'] ||= 'test'
+
+require 'support/coverage'
+
+require 'pry-byebug'
+require 'rspeed'
+require 'support/common'
+require 'support/fakeredis'

data/spec/fixtures/1_spec.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+RSpec.describe 'fixture_1' do
+  it { sleep 0.1 }
+
+  it { sleep 0.2 }
+
+  it { sleep 0.3 }
+end

data/spec/fixtures/2_spec.rb

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+RSpec.describe 'fixture_2' do
+  it { sleep 2 }
+end

data/spec/fixtures/empty.rb

@@ -0,0 +1,4 @@
+# frozen_string_literal: true
+
+RSpec.describe 'empty' do
+end

data/spec/fixtures/new_spec.rb.csv

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

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

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+require 'support/env_mock'
+
+RSpec.describe RSpeed::Env, '.db' do
+  context 'when key is not setted on env' do
+    it { expect(described_class.db).to be(nil) }
+  end
+
+  context 'when key is setted on env' do
+    it 'returns env value as integer' do
+      EnvMock.mock(rspeed_db: '10') do
+        expect(described_class.db).to be(10)
+      end
+    end
+  end
+end

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

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+require 'support/env_mock'
+
+RSpec.describe RSpeed::Env, '.host' do
+  context 'when key is not setted on env' do
+    it { expect(described_class.host).to be(nil) }
+  end
+
+  context 'when key is setted on env' do
+    it 'returns env value' do
+      EnvMock.mock(rspeed_host: 'localhost') do
+        expect(described_class.host).to eq('localhost')
+      end
+    end
+  end
+end

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

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+require 'support/env_mock'
+
+RSpec.describe RSpeed::Env, '.name' do
+  context 'when env is given' do
+    it 'returns the env value' do
+      EnvMock.mock(rspeed_name: 'name') do
+        expect(described_class.name).to eq('name')
+      end
+    end
+  end
+
+  context 'when env is not given' do
+    it { expect(described_class.name).to be(nil) }
+  end
+end

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

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+require 'support/env_mock'
+
+RSpec.describe RSpeed::Env, '.pipe' do
+  context 'when pipe env is given' do
+    it 'returns the number of the current pipe as integer' do
+      EnvMock.mock(rspeed_pipe: '2') do
+        expect(described_class.pipe).to be(2)
+      end
+    end
+  end
+
+  context 'when pipe env is not given' do
+    it 'returns 1' do
+      expect(described_class.pipe).to be(1)
+    end
+  end
+end

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

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+require 'support/env_mock'
+
+RSpec.describe RSpeed::Env, '.pipes' do
+  context 'when env is setted' do
+    it 'returns the env value' do
+      EnvMock.mock(rspeed_pipes: '2') do
+        expect(described_class.pipes).to be(2)
+      end
+    end
+  end
+
+  context 'when env is not setted' do
+    it 'returns number 1' do
+      expect(described_class.pipes).to be(1)
+    end
+  end
+end

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

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+require 'support/env_mock'
+
+RSpec.describe RSpeed::Env, '.port' do
+  context 'when key is not setted on env' do
+    it { expect(described_class.port).to be(nil) }
+  end
+
+  context 'when key is setted on env' do
+    it 'returns env value as integer' do
+      EnvMock.mock(rspeed_port: '6379') do
+        expect(described_class.port).to be(6379)
+      end
+    end
+  end
+end

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

@@ -0,0 +1,19 @@
+# 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/env/rspeed_spec.rb

@@ -0,0 +1,43 @@
+# frozen_string_literal: true
+
+require 'support/env_mock'
+
+RSpec.describe RSpeed::Env, '.rspeed' do
+  context 'when env is given' do
+    context 'when true as string' do
+      it 'true' do
+        EnvMock.mock(rspeed: 'true') do
+          expect(described_class.rspeed).to be(true)
+        end
+      end
+    end
+
+    context 'when true' do
+      it 'true' do
+        EnvMock.mock(rspeed: true) do
+          expect(described_class.rspeed).to be(true)
+        end
+      end
+    end
+
+    context 'when false as string' do
+      it 'false' do
+        EnvMock.mock(rspeed: 'false') do
+          expect(described_class.rspeed).to be(false)
+        end
+      end
+    end
+
+    context 'when false' do
+      it 'false' do
+        EnvMock.mock(rspeed: false) do
+          expect(described_class.rspeed).to be(false)
+        end
+      end
+    end
+  end
+
+  context 'when env is not given' do
+    it { expect(described_class.rspeed).to be(false) }
+  end
+end

data/spec/models/rspeed/observer/after_spec.rb

@@ -1,18 +1,18 @@
 # frozen_string_literal: true
 
-require 'rails_helper'
-
 RSpec.describe RSpeed::Observer, '.after' do
   let!(:now) { Time.local(2020, 1, 1, 0, 0, 1) }
   let!(:clock) { class_double(RSpec::Core::Time, now: now) }
   let!(:metadata) { { file_path: 'file_path', line_number: 7, start_at: now - 1 } }
   let!(:example) { instance_double(RSpec::Core::Example, clock: clock, metadata: metadata) }
 
-  before { clean_csv_file }
+  before { truncate_profiles }
 
-  it 'appends the time of example on csv file' do
+  it 'appends the file and time on pipe profile key' do
     described_class.after(example)
 
-    expect(File.open('rspeed.csv').read).to eq "1.0,file_path:7\n"
+    expect(RSpeed::Redis.client.lrange(RSpeed::Variable.profile, 0, -1)).to eq [
+      { file: 'file_path:7', time: 1.0 }.to_json,
+    ]
   end
 end

data/spec/models/rspeed/observer/after_suite_spec.rb

@@ -0,0 +1,57 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Observer, '.after_suite' do
+  let!(:splitter) { instance_double('RSpeed::Splitter') }
+
+  before do
+    allow(RSpeed::Splitter).to receive(:new).and_return(splitter)
+    allow(splitter).to receive(:append)
+  end
+
+  context 'when append? returns false' do
+    before do
+      allow(splitter).to receive(:append?).and_return(false)
+      allow(splitter).to receive(:rename)
+    end
+
+    it 'does not append the time result' do
+      described_class.after_suite
+
+      expect(splitter).not_to have_received(:append)
+    end
+  end
+
+  context 'when all specs is not finished' do
+    before do
+      allow(splitter).to receive(:append?)
+      allow(RSpeed::Redis).to receive(:specs_finished?).and_return(false)
+    end
+
+    it 'sets true on pipe key to indicates that its finished' do
+      described_class.after_suite
+
+      expect(RSpeed::Redis.get('rspeed_pipe_1')).to eq('true')
+    end
+  end
+
+  context 'when all specs finished' do
+    before do
+      allow(splitter).to receive(:append?)
+      allow(RSpeed::Redis).to receive(:specs_finished?).and_return(true)
+      allow(splitter).to receive(:rename)
+      allow(RSpeed::Redis).to receive(:clean)
+    end
+
+    it 'consolidates profiles' do
+      described_class.after_suite
+
+      expect(splitter).to have_received(:rename)
+    end
+
+    it 'destroyes pipe finished flag keys' do
+      described_class.after_suite
+
+      expect(RSpeed::Redis).to have_received(:clean)
+    end
+  end
+end

data/spec/models/rspeed/observer/before_spec.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require 'rails_helper'
-
 RSpec.describe RSpeed::Observer, '.before' do
   let!(:now) { Time.local(2020) }
   let!(:clock) { class_double(RSpec::Core::Time, now: now) }

data/spec/models/rspeed/observer/before_suite_spec.rb

@@ -1,15 +1,26 @@
 # frozen_string_literal: true
 
-require 'rails_helper'
-
 RSpec.describe RSpeed::Observer, '.before_suite' do
-  before { clean_csv_file }
+  let!(:redis) { redis_object }
+
+  before { truncate_profiles }
+  after { truncate_profiles }
+
+  it 'cleans only the current pipe profile' do
+    redis.lpush('rspeed_profile_1', { file: '1_spec.rb', time: 1 }.to_json)
+    redis.lpush('rspeed_profile_2', { file: '2_spec.rb', time: 2 }.to_json)
+
+    described_class.before_suite
+
+    expect(redis.keys).to eq ['rspeed_profile_2']
+  end
 
-  it 'cleans the csv file' do
-    File.open('rspeed.csv', 'a') { |file| file.write('content') }
+  it 'cleans the pipe flag' do
+    redis.set('rspeed_pipe_1', true)
+    redis.set('rspeed_pipe_2', true)
 
     described_class.before_suite
 
-    expect(File.open('rspeed.csv').read).to eq ''
+    expect(redis.keys).to eq ['rspeed_pipe_2']
   end
 end

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

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '.clean' do
+  it 'destroys all keys that keeps the pipe finished info' do
+    described_class.set('rspeed_pipe_1', true)
+    described_class.set('rspeed_pipe_2', true)
+
+    described_class.clean
+
+    expect(described_class.keys('*')).to eq([])
+  end
+
+  it 'destroys the pipe profiles' do
+    described_class.set('rspeed_profile_1', true)
+
+    described_class.clean
+
+    expect(described_class.keys('*')).to eq([])
+  end
+end

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

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '.client' do
+  it 'returns a redis client' do
+    expect(described_class.client.class).to eq(Redis)
+  end
+end

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

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