knapsack 0.0.3 → 0.1.0

data/spec/knapsack/report_spec.rb

@@ -0,0 +1,73 @@
+describe Knapsack::Report do
+  let(:report) { described_class.send(:new) }
+  let(:report_path) { 'tmp/fake_report.json' }
+  let(:report_json) do
+    %Q[{"a_spec.rb": #{rand(Math::E..Math::PI)}}]
+  end
+
+  describe '#config' do
+    context 'when passed options' do
+      let(:opts) do
+        {
+          report_path: 'new_knapsack_report.json',
+          fake: true
+        }
+      end
+
+      it do
+        expect(report.config(opts)).to eql({
+          report_path: 'new_knapsack_report.json',
+          fake: true
+        })
+      end
+    end
+
+    context "when didn't pass options" do
+      it do
+        expect(report.config).to eql({
+          report_path: 'knapsack_report.json'
+        })
+      end
+    end
+  end
+
+  describe '#save', :clear_tmp do
+    before do
+      expect(report).to receive(:report_json).and_return(report_json)
+      report.config({
+        report_path: report_path
+      })
+      report.save
+    end
+
+    it { expect(File.read(report_path)).to eql report_json }
+  end
+
+  describe '.open' do
+    let(:subject) { report.open }
+
+    before do
+      report.config({
+        report_path: report_path
+      })
+    end
+
+    context 'when report file exists' do
+      before do
+        expect(File).to receive(:read).with(report_path).and_return(report_json)
+      end
+
+      it { should eql(JSON.parse(report_json)) }
+    end
+
+    context "when report file doesn't exist" do
+      let(:report_path) { 'tmp/non_existing_report.json' }
+
+      it do
+        expect {
+          subject
+        }.to raise_error("Knapsack report file doesn't exist. Please generate report first!")
+      end
+    end
+  end
+end

data/spec/knapsack/task_loader_spec.rb

@@ -0,0 +1,10 @@
+describe Knapsack::TaskLoader do
+  describe '#load_tasks' do
+    let(:rake_task_path) { "#{Knapsack.root}/lib/tasks/knapsack.rake" }
+
+    it do
+      expect(subject).to receive(:import).with(rake_task_path)
+      subject.load_tasks
+    end
+  end
+end

data/spec/knapsack/tracker_spec.rb

@@ -1,8 +1,6 @@
-require 'spec_helper'
-
 shared_examples 'default trakcer attributes' do
   it { expect(tracker.global_time).to eql 0 }
-  it { expect(tracker.files).to eql({}) }
+  it { expect(tracker.spec_files_with_time).to eql({}) }
 end
 
 describe Knapsack::Tracker do
@@ -10,23 +8,13 @@ describe Knapsack::Tracker do
 
   it_behaves_like 'default trakcer attributes'
 
-  describe '#generate_report?' do
-    subject { tracker.generate_report? }
-
-    context 'when ENV variable is defined' do
-      before do
-        stub_const("ENV", { 'KNAPSACK_GENERATE_REPORT' => true })
-      end
-      it { should be true }
-    end
-
-    context 'when ENV variable is not defined' do
-      it { should be false }
+  describe '#config' do
+    before do
+      stub_const("ENV", { 'KNAPSACK_GENERATE_REPORT' => generate_report })
     end
-  end
 
-  describe '#config' do
     context 'when passed options' do
+      let(:generate_report) { true }
       let(:opts) do
         {
           enable_time_offset_warning: false,
@@ -37,22 +25,98 @@ describe Knapsack::Tracker do
       it do
         expect(tracker.config(opts)).to eql({
           enable_time_offset_warning: false,
-          time_offset_warning: 30,
+          time_offset_in_seconds: 30,
+          generate_report: true,
           fake: true
         })
       end
     end
 
     context "when didn't pass options" do
+      let(:generate_report) { nil }
+
       it do
         expect(tracker.config).to eql({
           enable_time_offset_warning: true,
-          time_offset_warning: 30,
+          time_offset_in_seconds: 30,
+          generate_report: false
         })
       end
     end
   end
 
+  describe '#spec_path' do
+    subject { tracker.spec_path }
+
+    context 'when spec_path not set' do
+      it do
+        expect { subject }.to raise_error("spec_path needs to be set by Knapsack Adapter's bind method")
+      end
+    end
+
+    context 'when spec_path set' do
+      context 'when spec path has prefix ./' do
+        before { tracker.spec_path = './spec/models/user_spec.rb' }
+        it { should eql 'spec/models/user_spec.rb' }
+      end
+
+      context 'when spec path has not prefix ./' do
+        before { tracker.spec_path = 'spec/models/user_spec.rb' }
+        it { should eql 'spec/models/user_spec.rb' }
+      end
+    end
+  end
+
+  describe '#time_exceeded?' do
+    subject { tracker.time_exceeded? }
+
+    before do
+      expect(tracker).to receive(:global_time).and_return(global_time)
+      expect(tracker).to receive(:max_node_time_execution).and_return(max_node_time_execution)
+    end
+
+    context 'when true' do
+      let(:global_time) { 2 }
+      let(:max_node_time_execution) { 1 }
+      it { should be true }
+    end
+
+    context 'when false' do
+      let(:global_time) { 1 }
+      let(:max_node_time_execution) { 1 }
+      it { should be false }
+    end
+  end
+
+  describe '#max_node_time_execution' do
+    let(:report_distributor) { instance_double(Knapsack::Distributors::ReportDistributor) }
+    let(:node_time_execution) { 3.5 }
+    let(:max_node_time_execution) { node_time_execution + tracker.config[:time_offset_in_seconds] }
+
+    subject { tracker.max_node_time_execution }
+
+    before do
+      expect(tracker).to receive(:report_distributor).and_return(report_distributor)
+      expect(report_distributor).to receive(:node_time_execution).and_return(node_time_execution)
+    end
+
+    it { should eql max_node_time_execution }
+  end
+
+  describe '#exceeded_time' do
+    let(:global_time) { 5 }
+    let(:max_node_time_execution) { 2 }
+
+    subject { tracker.exceeded_time }
+
+    before do
+      expect(tracker).to receive(:global_time).and_return(global_time)
+      expect(tracker).to receive(:max_node_time_execution).and_return(max_node_time_execution)
+    end
+
+    it { should eql 3 }
+  end
+
   describe 'track time execution' do
     let(:now) { Time.now }
     let(:spec_paths) { ['a_spec.rb', 'b_spec.rb'] }
@@ -73,7 +137,7 @@ describe Knapsack::Tracker do
 
     it { expect(tracker.global_time).to eql 3.0 }
     it do
-      expect(tracker.files).to eql({
+      expect(tracker.spec_files_with_time).to eql({
         'a_spec.rb' => 1.0,
         'b_spec.rb' => 2.0,
       })

data/spec/knapsack_spec.rb

@@ -1,5 +1,3 @@
-require 'spec_helper'
-
 describe Knapsack do
   describe '.tracker' do
     subject { described_class.tracker }
@@ -7,4 +5,27 @@ describe Knapsack do
     it { should be_a Knapsack::Tracker }
     it { expect(subject.object_id).to eql described_class.tracker.object_id }
   end
+
+  describe '.report' do
+    subject { described_class.report }
+
+    it { should be_a Knapsack::Report }
+    it { expect(subject.object_id).to eql described_class.report.object_id }
+  end
+
+  describe '.root' do
+    subject { described_class.root }
+
+    it { expect(subject).to match 'knapsack' }
+  end
+
+  describe '.load_tasks' do
+    let(:task_loader) { instance_double(Knapsack::TaskLoader) }
+
+    it do
+      expect(Knapsack::TaskLoader).to receive(:new).and_return(task_loader)
+      expect(task_loader).to receive(:load_tasks)
+      described_class.load_tasks
+    end
+  end
 end

data/spec/spec_helper.rb

@@ -12,4 +12,20 @@ RSpec.configure do |config|
     mocks.syntax = :expect
     mocks.verify_partial_doubles = true
   end
+
+  config.expect_with :rspec do |c|
+    c.syntax = :expect
+  end
+
+  config.before(:each) do
+    if RSpec.current_example.metadata[:clear_tmp]
+      FileUtils.mkdir_p(File.join(Knapsack.root, 'tmp'))
+    end
+  end
+
+  config.after(:each) do
+    if RSpec.current_example.metadata[:clear_tmp]
+      FileUtils.rm_r(File.join(Knapsack.root, 'tmp'))
+    end
+  end
 end

data/spec_examples/fast/1_spec.rb

@@ -1,6 +1,3 @@
-require 'spec_helper'
-
-describe 'Fast 1' do
+describe 'Fast 1', :focus do
   it {}
 end
-

data/spec_examples/fast/2_spec.rb

@@ -1,7 +1,4 @@
-require 'spec_helper'
-
-describe 'Fast 2' do
+describe 'Fast 2', :focus do
   it {}
   it {}
 end
-

data/spec_examples/fast/3_spec.rb

@@ -1,8 +1,5 @@
-require 'spec_helper'
-
-describe 'Fast 3' do
+describe 'Fast 3', :focus do
   it {}
   it {}
   it {}
 end
-

data/spec_examples/fast/4_spec.rb

@@ -1,9 +1,6 @@
-require 'spec_helper'
-
-describe 'Fast 4' do
+describe 'Fast 4', :focus do
   it {}
   it {}
   it {}
   it {}
 end
-

data/spec_examples/fast/5_spec.rb

@@ -1,10 +1,7 @@
-require 'spec_helper'
-
-describe 'Fast 5' do
+describe 'Fast 5', :focus do
   it {}
   it {}
   it {}
   it {}
   it {}
 end
-

data/spec_examples/fast/6_spec.rb

@@ -1,6 +1,4 @@
-require 'spec_helper'
-
-describe 'Fast 6' do
+describe 'Fast 6', :focus do
   it {}
   it {}
   it {}
@@ -8,4 +6,3 @@ describe 'Fast 6' do
   it {}
   it {}
 end
-

data/spec_examples/leftover/1_spec.rb

@@ -0,0 +1,4 @@
+# this file should not be included in knapsack_report.json
+describe 'Leftover Fast 1' do
+  it {}
+end

data/spec_examples/leftover/a_spec.rb

@@ -0,0 +1,8 @@
+# this file should not be included in knapsack_report.json
+describe 'Leftover Slow A' do
+  it { sleep 1 }
+  it { sleep 0.1 }
+  it { sleep 0.5 }
+  it { sleep 3 }
+  it { sleep 1 }
+end

data/spec_examples/slow/a_spec.rb

@@ -1,6 +1,4 @@
-require 'spec_helper'
-
-describe 'Slow A' do
+describe 'Slow A', :focus do
   it { sleep 1 }
   it { sleep 0.1 }
   it { sleep 0.5 }

data/spec_examples/slow/b_spec.rb

@@ -1,7 +1,5 @@
-require 'spec_helper'
-
-describe 'Slow B' do
-  it { sleep 2 }
+describe 'Slow B', :focus do
+  it { sleep 0.3 }
   it { sleep 0.2 }
-  it { sleep 1 }
+  it { sleep 0.4 }
 end

data/spec_examples/slow/c_spec.rb

@@ -1,6 +1,4 @@
-require 'spec_helper'
-
-describe 'Slow C' do
+describe 'Slow C', :focus do
   it { sleep 0.8 }
   it { sleep 0.1 }
   it { sleep 0.1 }

data/spec_examples/spec_helper.rb

@@ -1,9 +1,12 @@
 require 'knapsack'
 Knapsack.tracker.config({
   enable_time_offset_warning: true,
-  time_offset_warning: 5
+  time_offset_in_seconds: 3
 })
-Knapsack::Adapters::Rspec.bind
+Knapsack.report.config({
+  report_path: 'knapsack_report.json'
+})
+Knapsack::Adapters::RspecAdapter.bind
 
 RSpec.configure do |config|
   config.order = :random

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: knapsack
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.1.0
 platform: ruby
 authors:
 - ArturT
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-07-04 00:00:00.000000000 Z
+date: 2014-07-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -98,19 +98,36 @@ files:
 - ".gitignore"
 - ".rspec"
 - ".travis.yml"
+- CHANGELOG.md
 - Gemfile
 - LICENSE.txt
 - README.md
 - Rakefile
+- TODO.md
 - knapsack.gemspec
 - knapsack_report.json
 - lib/knapsack.rb
-- lib/knapsack/adapters/base.rb
-- lib/knapsack/adapters/rspec.rb
+- lib/knapsack/adapters/base_adapter.rb
+- lib/knapsack/adapters/rspec_adapter.rb
+- lib/knapsack/allocator.rb
+- lib/knapsack/distributors/base_distributor.rb
+- lib/knapsack/distributors/leftover_distributor.rb
+- lib/knapsack/distributors/report_distributor.rb
 - lib/knapsack/presenter.rb
 - lib/knapsack/report.rb
+- lib/knapsack/task_loader.rb
 - lib/knapsack/tracker.rb
 - lib/knapsack/version.rb
+- lib/tasks/knapsack.rake
+- spec/knapsack/adapters/base_adapter_spec.rb
+- spec/knapsack/adapters/rspec_adapter_spec.rb
+- spec/knapsack/allocator_spec.rb
+- spec/knapsack/distributors/base_distributor_spec.rb
+- spec/knapsack/distributors/leftover_distributor_spec.rb
+- spec/knapsack/distributors/report_distributor_spec.rb
+- spec/knapsack/presenter_spec.rb
+- spec/knapsack/report_spec.rb
+- spec/knapsack/task_loader_spec.rb
 - spec/knapsack/tracker_spec.rb
 - spec/knapsack_spec.rb
 - spec/spec_helper.rb
@@ -120,12 +137,11 @@ files:
 - spec_examples/fast/4_spec.rb
 - spec_examples/fast/5_spec.rb
 - spec_examples/fast/6_spec.rb
+- spec_examples/leftover/1_spec.rb
+- spec_examples/leftover/a_spec.rb
 - spec_examples/slow/a_spec.rb
 - spec_examples/slow/b_spec.rb
 - spec_examples/slow/c_spec.rb
-- spec_examples/slow/d_spec.rb
-- spec_examples/slow/e_spec.rb
-- spec_examples/slow/f_spec.rb
 - spec_examples/spec_helper.rb
 homepage: https://github.com/ArturT/knapsack
 licenses:
@@ -152,6 +168,15 @@ signing_key:
 specification_version: 4
 summary: Parallel specs across CI server nodes based on each spec file's time execution.
 test_files:
+- spec/knapsack/adapters/base_adapter_spec.rb
+- spec/knapsack/adapters/rspec_adapter_spec.rb
+- spec/knapsack/allocator_spec.rb
+- spec/knapsack/distributors/base_distributor_spec.rb
+- spec/knapsack/distributors/leftover_distributor_spec.rb
+- spec/knapsack/distributors/report_distributor_spec.rb
+- spec/knapsack/presenter_spec.rb
+- spec/knapsack/report_spec.rb
+- spec/knapsack/task_loader_spec.rb
 - spec/knapsack/tracker_spec.rb
 - spec/knapsack_spec.rb
 - spec/spec_helper.rb