rspeed 0.4.0 → 0.7.0

data/spec/models/rspeed/differ/final_diff_spec.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+require 'support/env_mock'
+
+RSpec.describe RSpeed::Differ, '#final_diff' do
+  let!(:redis) { redis_object }
+
+  before do
+    redis.rpush('rspeed_previous', { file: './spec/fixtures/1_spec.rb:1', time: 1.1 }.to_json)
+    redis.rpush('rspeed_previous', { file: './spec/fixtures/2_spec.rb:2', time: 2.2 }.to_json)
+    redis.rpush('rspeed_previous', { file: './spec/fixtures/3_spec.rb:3', time: 3.3 }.to_json)
+    redis.rpush('rspeed_previous', { file: './spec/fixtures/4_spec.rb:4', time: 4.4 }.to_json)
+
+    redis.rpush('rspeed', { file: './spec/fixtures/1_spec.rb:1', time: 1.1 }.to_json)
+    redis.rpush('rspeed', { file: './spec/fixtures/2_spec.rb:2', time: 2.2 }.to_json)
+    redis.rpush('rspeed', { file: './spec/fixtures/5_spec.rb:5', time: 5.5 }.to_json)
+    redis.rpush('rspeed', { file: './spec/fixtures/6_spec.rb:6', time: 6.6 }.to_json)
+  end
+
+  it 'returns the diff' do
+    expect(described_class.final_diff).to eq(
+      actual_files: [
+        { file: './spec/fixtures/1_spec.rb:1', time: 1.1 },
+        { file: './spec/fixtures/2_spec.rb:2', time: 2.2 },
+        { file: './spec/fixtures/5_spec.rb:5', time: 5.5 },
+        { file: './spec/fixtures/6_spec.rb:6', time: 6.6 },
+      ],
+
+      actual_time: 15.4,
+
+      added_files: [
+        { file: './spec/fixtures/5_spec.rb:5', time: 5.5 },
+        { file: './spec/fixtures/6_spec.rb:6', time: 6.6 },
+      ],
+
+      added_time: 12.1,
+
+      removed_files: [
+        { file: './spec/fixtures/3_spec.rb:3', time: 3.3 },
+        { file: './spec/fixtures/4_spec.rb:4', time: 4.4 },
+      ],
+
+      removed_time: 7.7
+    )
+  end
+end

data/spec/models/rspeed/differ/removed_data_spec.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Differ, '#removed_data' do
+  let!(:files) { ['./spec/fixtures/2_spec.rb:4'] }
+
+  let!(:result) do
+    [
+      { file: './spec/fixtures/1_spec.rb:4', time: 1.4 },
+      { file: './spec/fixtures/1_spec.rb:6', time: 1.6 },
+      { file: './spec/fixtures/2_spec.rb:4', time: 2.4 },
+    ]
+  end
+
+  it 'returns only removed data' do
+    expect(described_class.removed_data(files: files, result: result)).to eq [
+      { file: './spec/fixtures/1_spec.rb:4', time: 1.4 },
+      { file: './spec/fixtures/1_spec.rb:6', time: 1.6 },
+    ]
+  end
+end

data/spec/models/rspeed/differ/sum_time_spec.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Differ, '#sum_time' do
+  context 'when has at least one time with value' do
+    let!(:data) { [{ time: 1.4 }, { time: 1.6 }, { time: 2.4 }, { time: nil }] }
+
+    it 'sums the time key' do
+      expect(described_class.sum_time(data: data)).to be(5.4)
+    end
+  end
+
+  context 'when all time are nil' do
+    let!(:data) { [{ time: nil }, { time: nil }] }
+
+    it 'returns a question mark' do
+      expect(described_class.sum_time(data: data)).to eq('?')
+    end
+  end
+end

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

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

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/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/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/env/spec_path_spec.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+require 'support/env_mock'
+
+RSpec.describe RSpeed::Env, '.spec_path' do
+  context 'when env is given' do
+    it 'is returned' do
+      EnvMock.mock(rspeed_spec_path: './test/**/*test.rb') do
+        expect(described_class.spec_path).to eq('./test/**/*test.rb')
+      end
+    end
+  end
+
+  context 'when env is not given' do
+    it 'returns the default' do
+      expect(described_class.spec_path).to eq('./spec/**/*spec.rb')
+    end
+  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 { truncate_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,43 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Observer, '.after_suite' do
+  context 'when all specs is not finished' do
+    let!(:redis) { redis_object }
+
+    before { allow(RSpeed::Redis).to receive(:specs_finished?).and_return(false) }
+
+    it 'sets true on pipe key to indicates that its finished' do
+      described_class.after_suite
+
+      expect(redis.get('rspeed:pipe_01')).to eq('true')
+    end
+  end
+
+  context 'when all specs finished' do
+    before do
+      allow(RSpeed::Redis).to receive(:specs_finished?).and_return(true)
+      allow(RSpeed::Redis).to receive(:version_the_result)
+      allow(RSpeed::Splitter).to receive(:consolidate)
+      allow(RSpeed::Reporter).to receive(:call)
+      allow(RSpeed::Redis).to receive(:clean)
+    end
+
+    it 'consolidates profiles' do
+      described_class.after_suite
+
+      expect(RSpeed::Splitter).to have_received(:consolidate)
+    end
+
+    it 'reports' do
+      described_class.after_suite
+
+      expect(RSpeed::Reporter).to have_received(:call)
+    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,27 @@
 # frozen_string_literal: true
 
-require 'rails_helper'
-
 RSpec.describe RSpeed::Observer, '.before_suite' do
-  before { truncate_file }
+  let!(:redis) { redis_object }
+
+  before { truncate_profiles }
+
+  after { truncate_profiles }
+
+  it 'cleans only the current pipe profile' do
+    redis.rpush('rspeed:profile_01', { file: '1_spec.rb', time: 1 }.to_json)
+    redis.rpush('rspeed:profile_02', { file: '2_spec.rb', time: 2 }.to_json)
+
+    described_class.before_suite
+
+    expect(redis.keys).to eq ['rspeed:profile_02']
+  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_01', true)
+    redis.set('rspeed:pipe_02', true)
 
     described_class.before_suite
 
-    expect(File.open('rspeed.csv').read).to eq ''
+    expect(redis.keys).to eq ['rspeed:pipe_02']
   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
+    redis_object.set('rspeed:pipe_01', true)
+    redis_object.set('rspeed:pipe_02', true)
+
+    described_class.clean
+
+    expect(redis_object.keys('*')).to eq([])
+  end
+
+  it 'destroys the pipe profiles' do
+    redis_object.set('rspeed:profile_01', true)
+
+    described_class.clean
+
+    expect(redis_object.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,21 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '#destroy' do
+  before do
+    redis_object.set('rspeed', '{}')
+    redis_object.set('rspeed_1', '{}')
+    redis_object.set('rspeed_2', '{}')
+  end
+
+  it 'destroys via wildcard' do
+    described_class.destroy(pattern: 'rspeed_*')
+
+    expect(redis_object.keys('*')).to eq %w[rspeed]
+  end
+
+  it 'destroys via single name' do
+    described_class.destroy(pattern: 'rspeed')
+
+    expect(redis_object.keys('*')).to eq %w[rspeed_1 rspeed_2]
+  end
+end

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

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Redis, '.set' do
+  let!(:redis) { redis_object }
+
+  it 'sets a key on redis' do
+    redis_object.set('key', 'value')
+
+    expect(described_class.get('key')).to eq('value')
+  end
+end