knapsack_pro 0.17.0 → 0.18.0

data/spec/knapsack_pro/report_spec.rb

@@ -2,11 +2,80 @@ describe KnapsackPro::Report do
   describe '.save' do
     subject { described_class.save }
 
+    it do
+      test_files = double
+      tracker = instance_double(KnapsackPro::Tracker, to_a: test_files)
+      expect(KnapsackPro).to receive(:tracker).and_return(tracker)
+      expect(described_class).to receive(:create_build_subset).with(test_files)
+
+      subject
+    end
+  end
+
+  describe '.save_subset_queue_to_file' do
+    let(:fake_path) { SecureRandom.uuid }
+
+    subject { described_class.save_subset_queue_to_file }
+
     before do
+      test_files = [{path: fake_path}]
       tracker = instance_double(KnapsackPro::Tracker, to_a: test_files)
       expect(KnapsackPro).to receive(:tracker).and_return(tracker)
+
+      subset_queue_id = 'fake-subset-queue-id'
+      expect(KnapsackPro::Config::Env).to receive(:subset_queue_id).and_return(subset_queue_id)
+
+      queue_id = 'fake-queue-id'
+      expect(KnapsackPro::Config::Env).to receive(:queue_id).twice.and_return(queue_id)
     end
 
+    it do
+      subject
+
+      expect(
+        JSON.parse(
+          File.read('tmp/knapsack_pro/queue/fake-queue-id/fake-subset-queue-id.json')
+        )
+      ).to eq([
+        { 'path' => fake_path }
+      ])
+    end
+  end
+
+  describe '.save_node_queue_to_api' do
+    let(:json_test_file_a_path) { double }
+    let(:json_test_file_a) { [{ 'path' => 'a_spec.rb' }] }
+
+    let(:json_test_file_b_path) { double }
+    let(:json_test_file_b) { [{ 'path' => 'b_spec.rb' }] }
+
+    subject { described_class.save_node_queue_to_api }
+
+    before do
+      queue_id = 'fake-queue-id'
+      expect(KnapsackPro::Config::Env).to receive(:queue_id).and_return(queue_id)
+
+      expect(Dir).to receive(:glob).with('tmp/knapsack_pro/queue/fake-queue-id/*.json').and_return([
+        json_test_file_a_path,
+        json_test_file_b_path
+      ])
+
+      expect(File).to receive(:read).with(json_test_file_a_path).and_return(json_test_file_a.to_json)
+      expect(File).to receive(:read).with(json_test_file_b_path).and_return(json_test_file_b.to_json)
+    end
+
+    it do
+      expect(described_class).to receive(:create_build_subset).with(
+        json_test_file_a + json_test_file_b
+      )
+
+      subject
+    end
+  end
+
+  describe '.create_build_subset' do
+    subject { described_class.create_build_subset(test_files) }
+
     context "when test files doesn't exist" do
       let(:test_files) { [] }
 

data/spec/knapsack_pro/runners/queue/base_runner_spec.rb

@@ -0,0 +1,67 @@
+describe KnapsackPro::Runners::Queue::BaseRunner do
+  describe '.run' do
+    it do
+      expect {
+        described_class.run(nil)
+      }.to raise_error NotImplementedError
+    end
+  end
+
+  describe '.run_tests' do
+    it do
+      expect {
+        described_class.run_tests(nil, nil, nil, nil)
+      }.to raise_error NotImplementedError
+    end
+  end
+
+  describe 'instance methods' do
+    let(:adapter_class) { double }
+    let(:runner) do
+      described_class.new(adapter_class)
+    end
+    let(:allocator) { instance_double(KnapsackPro::QueueAllocator) }
+    let(:allocator_builder) { instance_double(KnapsackPro::QueueAllocatorBuilder) }
+
+    before do
+      expect(KnapsackPro::QueueAllocatorBuilder).to receive(:new).with(adapter_class).and_return(allocator_builder)
+      expect(allocator_builder).to receive(:allocator).and_return(allocator)
+    end
+
+    describe '#test_file_paths' do
+      subject { runner.test_file_paths(args) }
+
+      context 'when can_initialize_queue flag has value' do
+        let(:can_initialize_queue) { double }
+        let(:args) { { can_initialize_queue: can_initialize_queue } }
+        let(:test_file_paths) { double }
+
+        before do
+          expect(allocator).to receive(:test_file_paths).and_return(test_file_paths)
+        end
+
+        it { should eq test_file_paths }
+      end
+
+      context 'when can_initialize_queue flag has no value' do
+        let(:args) { {} }
+
+        it do
+          expect { subject }.to raise_error(KeyError)
+        end
+      end
+    end
+
+    describe '#test_dir' do
+      let(:test_dir) { double }
+
+      subject { runner.test_dir }
+
+      before do
+        expect(allocator_builder).to receive(:test_dir).and_return(test_dir)
+      end
+
+      it { should eq test_dir }
+    end
+  end
+end

data/spec/knapsack_pro/runners/queue/rspec_runner_spec.rb

@@ -0,0 +1,118 @@
+describe KnapsackPro::Runners::Queue::RSpecRunner do
+  describe '.run' do
+    let(:test_suite_token_rspec) { 'fake-token' }
+    let(:queue_id) { 'fake-queue-id' }
+    let(:runner) { double }
+
+    subject { described_class.run(args) }
+
+    before do
+      expect(KnapsackPro::Config::Env).to receive(:test_suite_token_rspec).and_return(test_suite_token_rspec)
+      expect(KnapsackPro::Config::EnvGenerator).to receive(:set_queue_id).and_return(queue_id)
+
+      expect(ENV).to receive(:[]=).with('KNAPSACK_PRO_TEST_SUITE_TOKEN', test_suite_token_rspec)
+      expect(ENV).to receive(:[]=).with('KNAPSACK_PRO_QUEUE_RECORDING_ENABLED', 'true')
+      expect(ENV).to receive(:[]=).with('KNAPSACK_PRO_QUEUE_ID', queue_id)
+
+      expect(described_class).to receive(:new).with(KnapsackPro::Adapters::RSpecAdapter).and_return(runner)
+    end
+
+    context 'when args provided' do
+      let(:args) { '--example-arg example-value' }
+
+      it do
+        result = double
+        expect(described_class).to receive(:run_tests).with(runner, true, ['--example-arg', 'example-value'], 0).and_return(result)
+
+        expect(subject).to eq result
+      end
+    end
+
+    context 'when args not provided' do
+      let(:args) { nil }
+
+      it do
+        result = double
+        expect(described_class).to receive(:run_tests).with(runner, true, [], 0).and_return(result)
+
+        expect(subject).to eq result
+      end
+    end
+  end
+
+  describe '.run_tests' do
+    let(:test_dir) { 'fake-test-dir' }
+    let(:runner) do
+      instance_double(described_class, test_dir: test_dir)
+    end
+    let(:can_initialize_queue) { double(:can_initialize_queue) }
+    let(:args) { ['--example-arg', 'example-value'] }
+    let(:exitstatus) { double }
+
+    subject { described_class.run_tests(runner, can_initialize_queue, args, exitstatus) }
+
+    before do
+      expect(runner).to receive(:test_file_paths).with(can_initialize_queue: can_initialize_queue).and_return(test_file_paths)
+    end
+
+    context 'when test files exist' do
+      let(:test_file_paths) { ['a_spec.rb', 'b_spec.rb'] }
+
+      before do
+        subset_queue_id = 'fake-subset-queue-id'
+        expect(KnapsackPro::Config::EnvGenerator).to receive(:set_subset_queue_id).and_return(subset_queue_id)
+
+        expect(ENV).to receive(:[]=).with('KNAPSACK_PRO_SUBSET_QUEUE_ID', subset_queue_id)
+
+        options = double
+        expect(RSpec::Core::ConfigurationOptions).to receive(:new).with([
+          '--example-arg', 'example-value',
+          '--default-path', test_dir,
+          'a_spec.rb', 'b_spec.rb',
+        ]).and_return(options)
+
+        rspec_core_runner = double
+        expect(RSpec::Core::Runner).to receive(:new).with(options).and_return(rspec_core_runner)
+        expect(rspec_core_runner).to receive(:run).with($stderr, $stdout).and_return(exit_code)
+
+        expect(RSpec).to receive_message_chain(:world, :example_groups, :clear)
+
+        # second call of run_tests because of recursion
+        expect(runner).to receive(:test_file_paths).with(can_initialize_queue: false).and_return([])
+      end
+
+      context 'when exit code is zero' do
+        let(:exit_code) { 0 }
+
+        it do
+          expect(KnapsackPro::Report).to receive(:save_node_queue_to_api)
+          expect(described_class).to receive(:exit).with(exitstatus)
+
+          subject
+        end
+      end
+
+      context 'when exit code is not zero' do
+        let(:exit_code) { double }
+
+        it do
+          expect(KnapsackPro::Report).to receive(:save_node_queue_to_api)
+          expect(described_class).to receive(:exit).with(exit_code)
+
+          subject
+        end
+      end
+    end
+
+    context "when test files don't exist" do
+      let(:test_file_paths) { [] }
+
+      it do
+        expect(KnapsackPro::Report).to receive(:save_node_queue_to_api)
+        expect(described_class).to receive(:exit).with(exitstatus)
+
+        subject
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: knapsack_pro
 version: !ruby/object:Gem::Version
-  version: 0.17.0
+  version: 0.18.0
 platform: ruby
 authors:
 - ArturT
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-10-14 00:00:00.000000000 Z
+date: 2017-01-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -225,10 +225,12 @@ files:
 - lib/knapsack_pro/adapters/spinach_adapter.rb
 - lib/knapsack_pro/allocator.rb
 - lib/knapsack_pro/allocator_builder.rb
+- lib/knapsack_pro/base_allocator_builder.rb
 - lib/knapsack_pro/client/api/action.rb
 - lib/knapsack_pro/client/api/v1/base.rb
 - lib/knapsack_pro/client/api/v1/build_distributions.rb
 - lib/knapsack_pro/client/api/v1/build_subsets.rb
+- lib/knapsack_pro/client/api/v1/queues.rb
 - lib/knapsack_pro/client/connection.rb
 - lib/knapsack_pro/config/ci/base.rb
 - lib/knapsack_pro/config/ci/buildkite.rb
@@ -237,11 +239,14 @@ files:
 - lib/knapsack_pro/config/ci/snap_ci.rb
 - lib/knapsack_pro/config/ci/travis.rb
 - lib/knapsack_pro/config/env.rb
+- lib/knapsack_pro/config/env_generator.rb
 - lib/knapsack_pro/crypto/decryptor.rb
 - lib/knapsack_pro/crypto/digestor.rb
 - lib/knapsack_pro/crypto/encryptor.rb
 - lib/knapsack_pro/logger_wrapper.rb
 - lib/knapsack_pro/presenter.rb
+- lib/knapsack_pro/queue_allocator.rb
+- lib/knapsack_pro/queue_allocator_builder.rb
 - lib/knapsack_pro/report.rb
 - lib/knapsack_pro/repository_adapter_initiator.rb
 - lib/knapsack_pro/repository_adapters/base_adapter.rb
@@ -250,6 +255,8 @@ files:
 - lib/knapsack_pro/runners/base_runner.rb
 - lib/knapsack_pro/runners/cucumber_runner.rb
 - lib/knapsack_pro/runners/minitest_runner.rb
+- lib/knapsack_pro/runners/queue/base_runner.rb
+- lib/knapsack_pro/runners/queue/rspec_runner.rb
 - lib/knapsack_pro/runners/rspec_runner.rb
 - lib/knapsack_pro/runners/spinach_runner.rb
 - lib/knapsack_pro/task_loader.rb
@@ -265,6 +272,7 @@ files:
 - lib/tasks/encrypted_test_file_names.rake
 - lib/tasks/install.rake
 - lib/tasks/minitest.rake
+- lib/tasks/queue/rspec.rake
 - lib/tasks/rspec.rake
 - lib/tasks/salt.rake
 - lib/tasks/spinach.rake
@@ -281,10 +289,12 @@ files:
 - spec/knapsack_pro/adapters/spinach_adapter_spec.rb
 - spec/knapsack_pro/allocator_builder_spec.rb
 - spec/knapsack_pro/allocator_spec.rb
+- spec/knapsack_pro/base_allocator_builder_spec.rb
 - spec/knapsack_pro/client/api/action_spec.rb
 - spec/knapsack_pro/client/api/v1/base_spec.rb
 - spec/knapsack_pro/client/api/v1/build_distributions_spec.rb
 - spec/knapsack_pro/client/api/v1/build_subsets_spec.rb
+- spec/knapsack_pro/client/api/v1/queues_spec.rb
 - spec/knapsack_pro/client/connection_spec.rb
 - spec/knapsack_pro/config/ci/base_spec.rb
 - spec/knapsack_pro/config/ci/buildkite_spec.rb
@@ -292,12 +302,15 @@ files:
 - spec/knapsack_pro/config/ci/semaphore_spec.rb
 - spec/knapsack_pro/config/ci/snap_ci_spec.rb
 - spec/knapsack_pro/config/ci/travis_spec.rb
+- spec/knapsack_pro/config/env_generator_spec.rb
 - spec/knapsack_pro/config/env_spec.rb
 - spec/knapsack_pro/crypto/decryptor_spec.rb
 - spec/knapsack_pro/crypto/digestor_spec.rb
 - spec/knapsack_pro/crypto/encryptor_spec.rb
 - spec/knapsack_pro/logger_wrapper_spec.rb
 - spec/knapsack_pro/presenter_spec.rb
+- spec/knapsack_pro/queue_allocator_builder_spec.rb
+- spec/knapsack_pro/queue_allocator_spec.rb
 - spec/knapsack_pro/report_spec.rb
 - spec/knapsack_pro/repository_adapter_initiator_spec.rb
 - spec/knapsack_pro/repository_adapters/base_adapter_spec.rb
@@ -306,6 +319,8 @@ files:
 - spec/knapsack_pro/runners/base_runner_spec.rb
 - spec/knapsack_pro/runners/cucumber_runner_spec.rb
 - spec/knapsack_pro/runners/minitest_runner_spec.rb
+- spec/knapsack_pro/runners/queue/base_runner_spec.rb
+- spec/knapsack_pro/runners/queue/rspec_runner_spec.rb
 - spec/knapsack_pro/runners/rspec_runner_spec.rb
 - spec/knapsack_pro/runners/spinach_runner_spec.rb
 - spec/knapsack_pro/task_loader_spec.rb
@@ -330,7 +345,7 @@ files:
 - test_fake/b_test.rb
 homepage: https://github.com/KnapsackPro/knapsack_pro-ruby
 licenses:
-- LGPLv3
+- LGPL-3.0
 metadata: {}
 post_install_message: 
 rdoc_options: []
@@ -348,7 +363,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.5.2
 signing_key: 
 specification_version: 4
 summary: Knapsack Pro splits tests across CI nodes and makes sure that tests will
@@ -367,10 +382,12 @@ test_files:
 - spec/knapsack_pro/adapters/spinach_adapter_spec.rb
 - spec/knapsack_pro/allocator_builder_spec.rb
 - spec/knapsack_pro/allocator_spec.rb
+- spec/knapsack_pro/base_allocator_builder_spec.rb
 - spec/knapsack_pro/client/api/action_spec.rb
 - spec/knapsack_pro/client/api/v1/base_spec.rb
 - spec/knapsack_pro/client/api/v1/build_distributions_spec.rb
 - spec/knapsack_pro/client/api/v1/build_subsets_spec.rb
+- spec/knapsack_pro/client/api/v1/queues_spec.rb
 - spec/knapsack_pro/client/connection_spec.rb
 - spec/knapsack_pro/config/ci/base_spec.rb
 - spec/knapsack_pro/config/ci/buildkite_spec.rb
@@ -378,12 +395,15 @@ test_files:
 - spec/knapsack_pro/config/ci/semaphore_spec.rb
 - spec/knapsack_pro/config/ci/snap_ci_spec.rb
 - spec/knapsack_pro/config/ci/travis_spec.rb
+- spec/knapsack_pro/config/env_generator_spec.rb
 - spec/knapsack_pro/config/env_spec.rb
 - spec/knapsack_pro/crypto/decryptor_spec.rb
 - spec/knapsack_pro/crypto/digestor_spec.rb
 - spec/knapsack_pro/crypto/encryptor_spec.rb
 - spec/knapsack_pro/logger_wrapper_spec.rb
 - spec/knapsack_pro/presenter_spec.rb
+- spec/knapsack_pro/queue_allocator_builder_spec.rb
+- spec/knapsack_pro/queue_allocator_spec.rb
 - spec/knapsack_pro/report_spec.rb
 - spec/knapsack_pro/repository_adapter_initiator_spec.rb
 - spec/knapsack_pro/repository_adapters/base_adapter_spec.rb
@@ -392,6 +412,8 @@ test_files:
 - spec/knapsack_pro/runners/base_runner_spec.rb
 - spec/knapsack_pro/runners/cucumber_runner_spec.rb
 - spec/knapsack_pro/runners/minitest_runner_spec.rb
+- spec/knapsack_pro/runners/queue/base_runner_spec.rb
+- spec/knapsack_pro/runners/queue/rspec_runner_spec.rb
 - spec/knapsack_pro/runners/rspec_runner_spec.rb
 - spec/knapsack_pro/runners/spinach_runner_spec.rb
 - spec/knapsack_pro/task_loader_spec.rb