knapsack_pro 3.9.0 → 3.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e420b07793ab2ae891d047a88791a664ef1049cb3011a752e8c38b4c6f6a7ef7
-  data.tar.gz: 378a11f2229b60b8491b705764d71efb241de8b8ab56a5cffd0476157584f0f9
+  metadata.gz: a43c73ef1e2535db96433d5a39659b4a02cf0ecb9baac50f5effef464c0638de
+  data.tar.gz: 49739064aea9230e17d9719bd3ee8bff27dead716bf388530eb4c9e5e079d6e1
 SHA512:
-  metadata.gz: 6fc8003860fd45599a4f7ffe7dd2d0fbe6a472fab237e8ec704da2ca7084bbb84dac1859624f6ec2f40e0555ba266e29568520208c43b74b90dfcebb026a20f9
-  data.tar.gz: a76e2e4d9b598aec22f261c40087c3b19ba67d7d3ad14a84255e135345dd373ac252d2697aa88a982e67f95e1be0ab8dab2cdfce9b4e494843ea87de1bfd5db7
+  metadata.gz: d019adc9a5a78d7b795e380828a5be81eca28702aba936c1fb6759b7294bffe3e90dfcc7ab009c844863cfbe4ead8c9a1bf1159a89dee44305f9d50803d798b7
+  data.tar.gz: c92054f2b1cdc5c842e0130d89ee8f9ebc9a44985721718b9af48e3c9e6db9f3e391fe2a8ead8ad8ebf040f0526a3ce4600014b4bbf62d1f43952c4b5e24025f

data/CHANGELOG.md

@@ -1,5 +1,21 @@
 # Change Log
 
+### 3.11.0
+
+* Send distinguishable user seat info over to the API
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/192
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v3.10.0...v3.11.0
+
+### 3.10.0
+
+* Remove Solano CI and Snap CI support because they do not exist anymore
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/194
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v3.9.0...v3.10.0
+
 ### 3.9.0
 
 * Suppress all RSpec spec file names displayed in stdout at the beginning of running tests in Regular Mode only when the log level is >= `warn`

data/lib/knapsack_pro/client/api/v1/build_distributions.rb

@@ -15,6 +15,7 @@ module KnapsackPro
                 :node_total => args.fetch(:node_total),
                 :node_index => args.fetch(:node_index),
                 :ci_build_id => KnapsackPro::Config::Env.ci_node_build_id,
+                :user_seat => KnapsackPro::Config::Env.user_seat_hash,
               }
 
               unless request_hash[:cache_read_attempt]

data/lib/knapsack_pro/client/api/v1/queues.rb

@@ -16,6 +16,7 @@ module KnapsackPro
                 :node_total => args.fetch(:node_total),
                 :node_index => args.fetch(:node_index),
                 :node_build_id => KnapsackPro::Config::Env.ci_node_build_id,
+                :user_seat => KnapsackPro::Config::Env.user_seat_hash,
               }
 
               if request_hash[:can_initialize_queue] && !request_hash[:attempt_connect_to_queue]

data/lib/knapsack_pro/config/ci/base.rb

@@ -22,6 +22,9 @@ module KnapsackPro
 
         def project_dir
         end
+
+        def user_seat
+        end
       end
     end
   end

data/lib/knapsack_pro/config/ci/buildkite.rb

@@ -29,6 +29,10 @@ module KnapsackPro
         def project_dir
           ENV['BUILDKITE_BUILD_CHECKOUT_PATH']
         end
+
+        def user_seat
+          ENV['BUILDKITE_BUILD_AUTHOR'] || ENV['BUILDKITE_BUILD_CREATOR']
+        end
       end
     end
   end

data/lib/knapsack_pro/config/ci/circle.rb

@@ -25,6 +25,10 @@ module KnapsackPro
         def project_dir
           ENV['CIRCLE_WORKING_DIRECTORY']
         end
+
+        def user_seat
+          ENV['CIRCLE_USERNAME'] || ENV['CIRCLE_PR_USERNAME']
+        end
       end
     end
   end

data/lib/knapsack_pro/config/ci/github_actions.rb

@@ -29,6 +29,10 @@ module KnapsackPro
         def project_dir
           ENV['GITHUB_WORKSPACE']
         end
+
+        def user_seat
+          ENV['GITHUB_ACTOR']
+        end
       end
     end
   end

data/lib/knapsack_pro/config/ci/gitlab_ci.rb

@@ -16,7 +16,7 @@ module KnapsackPro
 
         def node_build_id
           ENV['CI_PIPELINE_ID'] || # Gitlab Release 9.0+
-          ENV['CI_BUILD_ID']  # Gitlab Release 8.x
+          ENV['CI_BUILD_ID'] # Gitlab Release 8.x
         end
 
         def commit_hash
@@ -32,6 +32,11 @@ module KnapsackPro
         def project_dir
           ENV['CI_PROJECT_DIR']
         end
+
+        def user_seat
+          ENV['GITLAB_USER_NAME'] || # Gitlab Release 10.0
+          ENV['GITLAB_USER_EMAIL'] # Gitlab Release 8.12
+        end
       end
     end
   end

data/lib/knapsack_pro/config/env.rb

@@ -58,6 +58,16 @@ module KnapsackPro
             ci_env_for(:project_dir)
         end
 
+        def user_seat
+          ENV['KNAPSACK_PRO_USER_SEAT'] ||
+            ci_env_for(:user_seat)
+        end
+
+        def user_seat_hash
+          return unless user_seat
+          Digest::SHA2.hexdigest(user_seat)
+        end
+
         def test_file_pattern
           ENV['KNAPSACK_PRO_TEST_FILE_PATTERN']
         end

data/lib/knapsack_pro/version.rb

@@ -1,3 +1,3 @@
 module KnapsackPro
-  VERSION = '3.9.0'
+  VERSION = '3.11.0'
 end

data/lib/knapsack_pro.rb

@@ -21,8 +21,6 @@ require_relative 'knapsack_pro/config/ci/semaphore'
 require_relative 'knapsack_pro/config/ci/semaphore2'
 require_relative 'knapsack_pro/config/ci/buildkite'
 require_relative 'knapsack_pro/config/ci/travis'
-require_relative 'knapsack_pro/config/ci/snap_ci'
-require_relative 'knapsack_pro/config/ci/solano_ci'
 require_relative 'knapsack_pro/config/ci/codeship'
 require_relative 'knapsack_pro/config/ci/github_actions'
 require_relative 'knapsack_pro/config/ci/heroku'

data/spec/knapsack_pro/client/api/v1/build_distributions_spec.rb

@@ -6,6 +6,7 @@ describe KnapsackPro::Client::API::V1::BuildDistributions do
     let(:node_total) { double }
     let(:node_index) { double }
     let(:ci_build_id) { double }
+    let(:user_seat_hash) { double }
     let(:test_files) { double }
 
     subject do
@@ -22,6 +23,7 @@ describe KnapsackPro::Client::API::V1::BuildDistributions do
     before do
       expect(KnapsackPro::Config::Env).to receive(:fixed_test_suite_split).and_return(fixed_test_suite_split)
       expect(KnapsackPro::Config::Env).to receive(:ci_node_build_id).and_return(ci_build_id)
+      expect(KnapsackPro::Config::Env).to receive(:user_seat_hash).and_return(user_seat_hash)
     end
 
     context 'when cache_read_attempt=true' do
@@ -40,6 +42,7 @@ describe KnapsackPro::Client::API::V1::BuildDistributions do
             node_total: node_total,
             node_index: node_index,
             ci_build_id: ci_build_id,
+            user_seat: user_seat_hash,
           }
         }).and_return(action)
         expect(subject).to eq action
@@ -62,6 +65,7 @@ describe KnapsackPro::Client::API::V1::BuildDistributions do
             node_total: node_total,
             node_index: node_index,
             ci_build_id: ci_build_id,
+            user_seat: user_seat_hash,
             test_files: test_files
           }
         }).and_return(action)

data/spec/knapsack_pro/client/api/v1/queues_spec.rb

@@ -7,6 +7,7 @@ describe KnapsackPro::Client::API::V1::Queues do
     let(:node_index) { double }
     let(:test_files) { double }
     let(:node_build_id) { double }
+    let(:user_seat_hash) { double }
 
     subject do
       described_class.queue(
@@ -23,6 +24,7 @@ describe KnapsackPro::Client::API::V1::Queues do
     before do
       expect(KnapsackPro::Config::Env).to receive(:fixed_queue_split).and_return(fixed_queue_split)
       expect(KnapsackPro::Config::Env).to receive(:ci_node_build_id).and_return(node_build_id)
+      expect(KnapsackPro::Config::Env).to receive(:user_seat_hash).and_return(user_seat_hash)
     end
 
     context 'when can_initialize_queue=true and attempt_connect_to_queue=true' do
@@ -43,6 +45,7 @@ describe KnapsackPro::Client::API::V1::Queues do
             node_total: node_total,
             node_index: node_index,
             node_build_id: node_build_id,
+            user_seat: user_seat_hash,
           }
         }).and_return(action)
         expect(subject).to eq action
@@ -67,6 +70,7 @@ describe KnapsackPro::Client::API::V1::Queues do
             node_total: node_total,
             node_index: node_index,
             node_build_id: node_build_id,
+            user_seat: user_seat_hash,
             test_files: test_files
           }
         }).and_return(action)
@@ -92,6 +96,7 @@ describe KnapsackPro::Client::API::V1::Queues do
             node_total: node_total,
             node_index: node_index,
             node_build_id: node_build_id,
+            user_seat: user_seat_hash,
           }
         }).and_return(action)
         expect(subject).to eq action

data/spec/knapsack_pro/config/ci/base_spec.rb

@@ -6,4 +6,5 @@ describe KnapsackPro::Config::CI::Base do
   its(:commit_hash) { should be nil }
   its(:branch) { should be nil }
   its(:project_dir) { should be nil }
+  its(:user_seat) { should be nil }
 end

data/spec/knapsack_pro/config/ci/buildkite_spec.rb

@@ -76,8 +76,8 @@ describe KnapsackPro::Config::CI::Buildkite do
     subject { described_class.new.branch }
 
     context 'when environment exists' do
-      let(:env) { { 'BUILDKITE_BRANCH' => 'master' } }
-      it { should eql 'master' }
+      let(:env) { { 'BUILDKITE_BRANCH' => 'main' } }
+      it { should eql 'main' }
     end
 
     context "when environment doesn't exist" do
@@ -97,4 +97,39 @@ describe KnapsackPro::Config::CI::Buildkite do
       it { should be nil }
     end
   end
+
+  describe '#user_seat' do
+    subject { described_class.new.user_seat }
+
+    context 'when the BUILDKITE_BUILD_AUTHOR env var exists' do
+      let(:env) do
+        { 'BUILDKITE_BUILD_AUTHOR' => 'Jane Doe',
+          'BUILDKITE_BUILD_CREATOR' => nil }
+      end
+
+      it { should eql 'Jane Doe' }
+    end
+
+    context 'when the BUILDKITE_BUILD_CREATOR env var exists' do
+      let(:env) do
+        { 'BUILDKITE_BUILD_AUTHOR' => nil,
+          'BUILDKITE_BUILD_CREATOR' => 'John Doe' }
+      end
+
+      it { should eql 'John Doe' }
+    end
+
+    context 'when both BUILDKITE_BUILD_AUTHOR and BUILDKITE_BUILD_CREATOR env vars exist' do
+      let(:env) do
+        { 'BUILDKITE_BUILD_AUTHOR' => 'Jane Doe',
+          'BUILDKITE_BUILD_CREATOR' => 'John Doe' }
+      end
+
+      it { should eql 'Jane Doe' }
+    end
+
+    context "when neither env var exists" do
+      it { should be nil }
+    end
+  end
 end

data/spec/knapsack_pro/config/ci/circle_spec.rb

@@ -63,8 +63,8 @@ describe KnapsackPro::Config::CI::Circle do
     subject { described_class.new.branch }
 
     context 'when environment exists' do
-      let(:env) { { 'CIRCLE_BRANCH' => 'master' } }
-      it { should eql 'master' }
+      let(:env) { { 'CIRCLE_BRANCH' => 'main' } }
+      it { should eql 'main' }
     end
 
     context "when environment doesn't exist" do
@@ -84,4 +84,39 @@ describe KnapsackPro::Config::CI::Circle do
       it { should be nil }
     end
   end
+
+  describe '#user_seat' do
+    subject { described_class.new.user_seat }
+
+    context 'when the CIRCLE_USERNAME env var exists' do
+      let(:env) do
+        { 'CIRCLE_USERNAME' => 'Jane Doe',
+          'CIRCLE_PR_USERNAME' => nil }
+      end
+
+      it { should eql 'Jane Doe' }
+    end
+
+    context 'when the CIRCLE_PR_USERNAME env var exists' do
+      let(:env) do
+        { 'CIRCLE_USERNAME' => nil,
+          'CIRCLE_PR_USERNAME' => 'John Doe' }
+      end
+
+      it { should eql 'John Doe' }
+    end
+
+    context 'when both CIRCLE_USERNAME and CIRCLE_PR_USERNAME env vars exist' do
+      let(:env) do
+        { 'CIRCLE_USERNAME' => 'Jane Doe',
+          'CIRCLE_PR_USERNAME' => 'John Doe' }
+      end
+
+      it { should eql 'Jane Doe' }
+    end
+
+    context "when neither env var exists" do
+      it { should be nil }
+    end
+  end
 end

data/spec/knapsack_pro/config/ci/github_actions_spec.rb

@@ -52,12 +52,12 @@ describe KnapsackPro::Config::CI::GithubActions do
       context 'when GITHUB_REF has value' do
         let(:env) do
           {
-            'GITHUB_REF' => 'master',
+            'GITHUB_REF' => 'main',
             'GITHUB_SHA' => '2e13512fc230d6f9ebf4923352718e4d',
           }
         end
 
-        it { should eql 'master' }
+        it { should eql 'main' }
       end
 
       context 'when GITHUB_REF is not set' do
@@ -88,4 +88,18 @@ describe KnapsackPro::Config::CI::GithubActions do
       it { should be nil }
     end
   end
+
+  describe '#user_seat' do
+    subject { described_class.new.user_seat }
+
+    context 'when the GITHUB_ACTOR environment variable exists' do
+      let(:env) { { 'GITHUB_ACTOR' => 'jane_doe' } }
+
+      it { should eql 'jane_doe' }
+    end
+
+    context "when the GITHUB_ACTOR environment variable doesn't exist" do
+      it { should be nil }
+    end
+  end
 end

data/spec/knapsack_pro/config/ci/gitlab_ci_spec.rb

@@ -73,13 +73,13 @@ describe KnapsackPro::Config::CI::GitlabCI do
     subject { described_class.new.branch }
 
     context 'when Gitlab Release 9.0+ and environment exists' do
-      let(:env) { { 'CI_COMMIT_REF_NAME' => 'master' } }
-      it { should eql 'master' }
+      let(:env) { { 'CI_COMMIT_REF_NAME' => 'main' } }
+      it { should eql 'main' }
     end
 
     context 'when Gitlab Release 8.x and environment exists' do
-      let(:env) { { 'CI_BUILD_REF_NAME' => 'master' } }
-      it { should eql 'master' }
+      let(:env) { { 'CI_BUILD_REF_NAME' => 'main' } }
+      it { should eql 'main' }
     end
 
     context "when environment doesn't exist" do
@@ -99,4 +99,39 @@ describe KnapsackPro::Config::CI::GitlabCI do
       it { should be nil }
     end
   end
+
+  describe '#user_seat' do
+    subject { described_class.new.user_seat }
+
+    context 'when the GITLAB_USER_NAME env var exists' do
+      let(:env) do
+        { 'GITLAB_USER_NAME' => 'Jane Doe',
+          'GITLAB_USER_EMAIL' => nil }
+      end
+
+      it { should eql 'Jane Doe' }
+    end
+
+    context 'when the GITLAB_USER_EMAIL env var exists' do
+      let(:env) do
+        { 'GITLAB_USER_NAME' => nil,
+          'GITLAB_USER_EMAIL' => 'janed@example.com' }
+      end
+
+      it { should eql 'janed@example.com' }
+    end
+
+    context 'when both GITLAB_USER_NAME and GITLAB_USER_EMAIL env vars exist' do
+      let(:env) do
+        { 'GITLAB_USER_NAME' => 'Jane Doe',
+          'GITLAB_USER_EMAIL' => 'janed@example.com' }
+      end
+
+      it { should eql 'Jane Doe' }
+    end
+
+    context "when neither env var exists" do
+      it { should be nil }
+    end
+  end
 end

data/spec/knapsack_pro/config/env_spec.rb

@@ -182,6 +182,51 @@ describe KnapsackPro::Config::Env do
     end
   end
 
+  describe '.user_seat' do
+    subject { described_class.user_seat }
+
+    context 'when ENV exists' do
+      context 'when KNAPSACK_PRO_USER_SEAT has value' do
+        before { stub_const("ENV", { 'KNAPSACK_PRO_USER_SEAT' => 'John Doe' }) }
+        it { should eq 'John Doe' }
+      end
+
+      context 'when CI environment has value' do
+        before do
+          expect(described_class).to receive(:ci_env_for).with(:user_seat).and_return('Jane Doe')
+        end
+
+        it { should eq 'Jane Doe' }
+      end
+    end
+
+    context "when ENV doesn't exist" do
+      it { should be nil }
+    end
+  end
+
+  describe '.user_seat_hash' do
+    subject { described_class.user_seat_hash }
+
+    before do
+      expect(described_class).to receive(:user_seat).at_least(1).and_return(user_seat)
+    end
+
+    context 'when the user seat has a value' do
+      let(:user_seat) { 'John Doe' }
+
+      it 'returns a SHA256 hash for the user seat' do
+        expect(subject).to eq '6cea57c2fb6cbc2a40411135005760f241fffc3e5e67ab99882726431037f908'
+      end
+    end
+
+    context 'when the user seat has no value' do
+      let(:user_seat) { nil }
+
+      it { expect(subject).to be_nil }
+    end
+  end
+
   describe '.test_file_pattern' do
     subject { described_class.test_file_pattern }
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: knapsack_pro
 version: !ruby/object:Gem::Version
-  version: 3.9.0
+  version: 3.11.0
 platform: ruby
 authors:
 - ArturT
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-03-22 00:00:00.000000000 Z
+date: 2023-05-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -230,8 +230,6 @@ files:
 - lib/knapsack_pro/config/ci/heroku.rb
 - lib/knapsack_pro/config/ci/semaphore.rb
 - lib/knapsack_pro/config/ci/semaphore2.rb
-- lib/knapsack_pro/config/ci/snap_ci.rb
-- lib/knapsack_pro/config/ci/solano_ci.rb
 - lib/knapsack_pro/config/ci/travis.rb
 - lib/knapsack_pro/config/env.rb
 - lib/knapsack_pro/config/env_generator.rb
@@ -327,8 +325,6 @@ files:
 - spec/knapsack_pro/config/ci/heroku_spec.rb
 - spec/knapsack_pro/config/ci/semaphore2_spec.rb
 - spec/knapsack_pro/config/ci/semaphore_spec.rb
-- spec/knapsack_pro/config/ci/snap_ci_spec.rb
-- spec/knapsack_pro/config/ci/solano_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
@@ -408,7 +404,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.6
+rubygems_version: 3.4.10
 signing_key:
 specification_version: 4
 summary: Knapsack Pro splits tests across parallel CI nodes and ensures each parallel
@@ -449,8 +445,6 @@ test_files:
 - spec/knapsack_pro/config/ci/heroku_spec.rb
 - spec/knapsack_pro/config/ci/semaphore2_spec.rb
 - spec/knapsack_pro/config/ci/semaphore_spec.rb
-- spec/knapsack_pro/config/ci/snap_ci_spec.rb
-- spec/knapsack_pro/config/ci/solano_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

data/lib/knapsack_pro/config/ci/snap_ci.rb

@@ -1,35 +0,0 @@
-module KnapsackPro
-  module Config
-    module CI
-      class SnapCI < Base
-        def node_total
-          ENV['SNAP_WORKER_TOTAL']
-        end
-
-        def node_index
-          index = ENV['SNAP_WORKER_INDEX']
-          index.to_i - 1 if index
-        end
-
-        def node_build_id
-          ENV['SNAP_PIPELINE_COUNTER']
-        end
-
-        def commit_hash
-          ENV['SNAP_COMMIT']
-        end
-
-        # https://docs.snap-ci.com/environment-variables/
-        # SNAP_BRANCH - the name of the git branch (not present on pull requests)
-        # SNAP_UPSTREAM_BRANCH - the upstream branch for which the pull request was opened
-        def branch
-          ENV['SNAP_BRANCH'] || ENV['SNAP_UPSTREAM_BRANCH']
-        end
-
-        def project_dir
-          ENV['SNAP_WORKING_DIR']
-        end
-      end
-    end
-  end
-end

data/lib/knapsack_pro/config/ci/solano_ci.rb

@@ -1,32 +0,0 @@
-# http://docs.solanolabs.com/Setup/tddium-set-environment-variables/
-module KnapsackPro
-  module Config
-    module CI
-      class SolanoCI < Base
-        def node_total
-          # not provided
-        end
-
-        def node_index
-          # not provided
-        end
-
-        def node_build_id
-          ENV['TDDIUM_SESSION_ID']
-        end
-
-        def commit_hash
-          ENV['TDDIUM_CURRENT_COMMIT']
-        end
-
-        def branch
-          ENV['TDDIUM_CURRENT_BRANCH']
-        end
-
-        def project_dir
-          ENV['TDDIUM_REPO_ROOT']
-        end
-      end
-    end
-  end
-end

data/spec/knapsack_pro/config/ci/snap_ci_spec.rb

@@ -1,104 +0,0 @@
-describe KnapsackPro::Config::CI::SnapCI do
-  let(:env) { {} }
-
-  before do
-    stub_const('ENV', env)
-  end
-
-  it { should be_kind_of KnapsackPro::Config::CI::Base }
-
-  describe '#node_total' do
-    subject { described_class.new.node_total }
-
-    context 'when environment exists' do
-      let(:env) { { 'SNAP_WORKER_TOTAL' => 4 } }
-      it { should eql 4 }
-    end
-
-    context "when environment doesn't exist" do
-      it { should be nil }
-    end
-  end
-
-  describe '#node_index' do
-    subject { described_class.new.node_index }
-
-    context 'when environment exists' do
-      let(:env) { { 'SNAP_WORKER_INDEX' => 4 } }
-      it { should eql 3 }
-    end
-
-    context "when environment doesn't exist" do
-      it { should be nil }
-    end
-  end
-
-  describe '#node_build_id' do
-    subject { described_class.new.node_build_id }
-
-    context 'when environment exists' do
-      let(:env) { { 'SNAP_PIPELINE_COUNTER' => 123 } }
-      it { should eql 123 }
-    end
-
-    context "when environment doesn't exist" do
-      it { should be nil }
-    end
-  end
-
-  describe '#commit_hash' do
-    subject { described_class.new.commit_hash }
-
-    context 'when environment exists' do
-      let(:env) { { 'SNAP_COMMIT' => '3fa64859337f6e56409d49f865d13fd7' } }
-      it { should eql '3fa64859337f6e56409d49f865d13fd7' }
-    end
-
-    context "when environment doesn't exist" do
-      it { should be nil }
-    end
-  end
-
-  describe '#branch' do
-    subject { described_class.new.branch }
-
-    context 'when environment exists' do
-      context 'when branch present' do
-        let(:env) { { 'SNAP_BRANCH' => 'master' } }
-        it { should eql 'master' }
-      end
-
-      context 'when pull request' do
-        let(:env) { { 'SNAP_UPSTREAM_BRANCH' => 'feature-branch' } }
-        it { should eql 'feature-branch' }
-      end
-
-      context 'branch has higher priority' do
-        let(:env) do
-          {
-            'SNAP_BRANCH' => 'master',
-            'SNAP_UPSTREAM_BRANCH' => 'feature-branch'
-          }
-        end
-        it { should eql 'master' }
-      end
-    end
-
-    context "when environment doesn't exist" do
-      it { should be nil }
-    end
-  end
-
-  describe '#project_dir' do
-    subject { described_class.new.project_dir }
-
-    context 'when environment exists' do
-      let(:env) { { 'SNAP_WORKING_DIR' => '/var/snap-ci/repo' } }
-      it { should eql '/var/snap-ci/repo' }
-    end
-
-    context "when environment doesn't exist" do
-      it { should be nil }
-    end
-  end
-end

data/spec/knapsack_pro/config/ci/solano_ci_spec.rb

@@ -1,73 +0,0 @@
-describe KnapsackPro::Config::CI::SolanoCI do
-  let(:env) { {} }
-
-  before do
-    stub_const('ENV', env)
-  end
-
-  it { should be_kind_of KnapsackPro::Config::CI::Base }
-
-  describe '#node_total' do
-    subject { described_class.new.node_total }
-
-    it { should be nil }
-  end
-
-  describe '#node_index' do
-    subject { described_class.new.node_index }
-
-    it { should be nil }
-  end
-
-  describe '#node_build_id' do
-    subject { described_class.new.node_build_id }
-
-    context 'when environment exists' do
-      let(:env) { { 'TDDIUM_SESSION_ID' => 123 } }
-      it { should eql 123 }
-    end
-
-    context "when environment doesn't exist" do
-      it { should be nil }
-    end
-  end
-
-  describe '#commit_hash' do
-    subject { described_class.new.commit_hash }
-
-    context 'when environment exists' do
-      let(:env) { { 'TDDIUM_CURRENT_COMMIT' => '2e13512fc230d6f9ebf4923352718e4d' } }
-      it { should eql '2e13512fc230d6f9ebf4923352718e4d' }
-    end
-
-    context "when environment doesn't exist" do
-      it { should be nil }
-    end
-  end
-
-  describe '#branch' do
-    subject { described_class.new.branch }
-
-    context 'when environment exists' do
-      let(:env) { { 'TDDIUM_CURRENT_BRANCH' => 'master' } }
-      it { should eql 'master' }
-    end
-
-    context "when environment doesn't exist" do
-      it { should be nil }
-    end
-  end
-
-  describe '#project_dir' do
-    subject { described_class.new.project_dir }
-
-    context 'when environment exists' do
-      let(:env) { { 'TDDIUM_REPO_ROOT' => '/example/repo/root' } }
-      it { should eql '/example/repo/root' }
-    end
-
-    context "when environment doesn't exist" do
-      it { should be nil }
-    end
-  end
-end