rspeed 0.3.0 → 0.6.0

data/lib/rspeed/variable.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module RSpeed
+  module Variable
+    module_function
+
+    PIPES_PATTERN   = 'rspeed:pipe_*'
+    PROFILE_PATTERN = 'rspeed:profile_*'
+
+    def append_app_name(value, plus: nil)
+      [value, RSpeed::Env.app, plus].compact.join('_')
+    end
+
+    def key(number)
+      append_app_name('rspeed', plus: number).to_sym
+    end
+
+    def result
+      append_app_name('rspeed')
+    end
+
+    def pipe
+      append_app_name('rspeed:pipe', plus: format('%02d', RSpeed::Env.pipe))
+    end
+
+    def profile
+      append_app_name('rspeed:profile', plus: format('%02d', RSpeed::Env.pipe))
+    end
+  end
+end

data/lib/rspeed/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RSpeed
-  VERSION = '0.3.0'
+  VERSION = '0.6.0'
 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/models/rspeed/database/result_spec.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Database, '#result' do
+  let!(:redis) { redis_object }
+
+  before do
+    redis.rpush('rspeed', { file: '1_spec.rb', time: 1.0 }.to_json)
+    redis.rpush('rspeed', { file: '2_spec.rb', time: 2.0 }.to_json)
+  end
+
+  it 'returns the results data as json' do
+    expect(described_class.result).to eq [
+      { file: '1_spec.rb', time: 1.0 },
+      { file: '2_spec.rb', time: 2.0 },
+    ]
+  end
+end

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

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Differ, '#actual_data' do
+  let!(:files) { ['./spec/fixtures/1_spec.rb:4', './spec/fixtures/1_spec.rb:6', './spec/fixtures/new_spec.rb:1'] }
+
+  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 consolidated data still existent as spec' do
+    expect(described_class.actual_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/actual_files_spec.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Differ, '#actual_files' do
+  it 'returns all examples' do
+    expect(described_class.actual_files(spec_path: './spec/fixtures/**/*_spec.rb')).to eq [
+      './spec/fixtures/1_spec.rb:4',
+      './spec/fixtures/1_spec.rb:6',
+      './spec/fixtures/1_spec.rb:8',
+      './spec/fixtures/2_spec.rb:4',
+    ]
+  end
+
+  it 'does not raise when no file match' do
+    expect(described_class.actual_files(spec_path: './spec/fixtures/**/*_missing.rb')).to eq []
+  end
+
+  it 'does not raise when file is empty' do
+    expect(described_class.actual_files(spec_path: './spec/fixtures/**/empty.rb')).to eq []
+  end
+end

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

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Differ, '#added_data' do
+  let!(:files) { ['./spec/fixtures/1_spec.rb:4', './spec/fixtures/1_spec.rb:6', './spec/fixtures/new_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 the added data files with time as zero since we do not know yet' do
+    expect(described_class.added_data(files: files, result: result)).to eq [
+      { file: './spec/fixtures/new_spec.rb:4', time: 0 },
+    ]
+  end
+end

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

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+require 'support/env_mock'
+
+RSpec.describe RSpeed::Differ, '#diff' do
+  let!(:redis) { redis_object }
+
+  before do
+    redis.rpush('rspeed', { file: './spec/fixtures/1_spec.rb:4', time: '1.4' }.to_json)
+    redis.rpush('rspeed', { file: './spec/fixtures/1_spec.rb:6', time: '1.6' }.to_json)
+    redis.rpush('rspeed', { file: './spec/fixtures/1_spec.rb:8', time: '1.8' }.to_json)
+    redis.rpush('rspeed', { file: './spec/fixtures/2_spec.rb:4', time: '2.4' }.to_json)
+    redis.rpush('rspeed', { file: './spec/fixtures/x_spec.rb:1', time: 3 }.to_json)
+    redis.rpush('rspeed', { file: './spec/fixtures/2_spec.rb:666', time: '6' }.to_json)
+
+    File.open('spec/fixtures/new_spec.rb', 'a') { |file| file.write('it') }
+  end
+
+  after { delete_file('spec/fixtures/new_spec.rb') }
+
+  it 'removes removed specs and adds new spec and keeps keeped specs based on rspeed key values' do
+    EnvMock.mock(rspeed_spec_path: './spec/fixtures/*_spec.rb') do
+      expect(described_class.diff).to eq [
+        { file: './spec/fixtures/1_spec.rb:4', time: '1.4' },
+        { file: './spec/fixtures/1_spec.rb:6', time: '1.6' },
+        { file: './spec/fixtures/1_spec.rb:8', time: '1.8' },
+        { file: './spec/fixtures/2_spec.rb:4', time: '2.4' },
+        { file: './spec/fixtures/new_spec.rb:1', time: 0 },
+      ]
+    end
+  end
+
+  context 'when has duplicated example' do
+    before do
+      redis.rpush('rspeed', { file: './spec/fixtures/1_spec.rb:4', time: '1.4' }.to_json)
+      redis.rpush('rspeed', { file: './spec/fixtures/1_spec.rb:4', time: '1.4' }.to_json)
+    end
+
+    it 'is removed' do
+      EnvMock.mock(rspeed_spec_path: './spec/fixtures/*_spec.rb') do
+        expect(described_class.diff).to eq [
+          { file: './spec/fixtures/1_spec.rb:4', time: '1.4' },
+          { file: './spec/fixtures/1_spec.rb:6', time: '1.6' },
+          { file: './spec/fixtures/1_spec.rb:8', time: '1.8' },
+          { file: './spec/fixtures/2_spec.rb:4', time: '2.4' },
+          { file: './spec/fixtures/new_spec.rb:1', time: 0 },
+        ]
+      end
+    end
+  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/removed_time_spec.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+RSpec.describe RSpeed::Differ, '#removed_time' do
+  let!(:data) do
+    [
+      { file: './spec/fixtures/1_spec.rb:4', time: 1.4 },
+      { file: './spec/fixtures/1_spec.rb:6', time: 1.6 },
+    ]
+  end
+
+  it 'returns the sum of removed examples' do
+    expect(described_class.removed_time(data: data)).to be(3.0)
+  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 { 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