knapsack_pro 5.4.1 → 5.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e62965afc4edc8d9b00b2a2706499f2013b7e17df08d1864ba8bd6e25aae29ab
-  data.tar.gz: 2b5c2d0a48780c1f71fa8d99dcae03d23d1001cabcd9a7dbe6a243b32e152e75
+  metadata.gz: fa2ae33f8d962bbd8d5f1639e050f2a43e9ad28bf655d637d43ca2c23bc1d7d3
+  data.tar.gz: 70e747f428235b65bba04ed2f748221170a6971e8982e0ab0fbbdb60177d9a94
 SHA512:
-  metadata.gz: 4a4d493dc1b46234b78f231567d7bfc7c1a6cf356dc0e02306f7a7c3261afd600b72be07c389c051ed96e09e491abc95f8d703c1432a22d1ed97d875b8d1fdf6
-  data.tar.gz: d406b157ef48520309ac3fb043738515105612f89389753a153ed074398f9b7f5bf5f6766f3c57a3407c25eb0982d81b5bc8109f8f8a4ac09911847a2c512c16
+  metadata.gz: ed58c0b63af2f3301e7233c142da4165a9c2c9a4f31987d465a803d3b4fdab590137d2ef71bbf6b80b9abc95f513cea605091b5caa7fcd0598620790977cc29b
+  data.tar.gz: 607af355f302481ca595f577428ea49565e711a790c284d7fb595f0f15106070a99e85acf4ebd5cf0e073e672ee37f9d851eb2b464c372dca550bf5c878c8050

data/.circleci/config.yml

@@ -30,10 +30,14 @@ jobs:
           command: |
             bundle install --jobs=4 --retry=3 --path vendor/bundle
 
+      - run: gem install rubocop
+
       - save_cache:
           paths:
             - ./vendor/bundle
           key: v2-dependencies-bundler-{{ checksum "knapsack_pro.gemspec" }}
 
-      # run tests!
+      # enforce `frozen_string_literal: true` in the gem files
+      - run: rubocop -A --only Style/FrozenStringLiteralComment,Layout/EmptyLineAfterMagicComment lib/
+
       - run: bundle exec rspec spec

data/CHANGELOG.md

@@ -1,5 +1,25 @@
 # Changelog
 
+### 5.6.0
+
+* Use `frozen_string_literal: true` to reduce memory usage
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/220
+
+* Enforce `frozen_string_literal: true` in the gem files with Rubocop
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/222
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v5.5.0...v5.6.0
+
+### 5.5.0
+
+* Detect user seats for AppVeyor, Codefresh, Codeship
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/221
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v5.4.1...v5.5.0
+
 ### 5.4.1
 
 * Fixes RSpec conflict (see https://github.com/KnapsackPro/knapsack_pro-ruby/issues/217)

data/lib/knapsack_pro/adapters/base_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Adapters
     class BaseAdapter

data/lib/knapsack_pro/adapters/cucumber_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Adapters
     class CucumberAdapter < BaseAdapter

data/lib/knapsack_pro/adapters/minitest_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Adapters
     class MinitestAdapter < BaseAdapter

data/lib/knapsack_pro/adapters/rspec_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Adapters
     class RSpecAdapter < BaseAdapter

data/lib/knapsack_pro/adapters/spinach_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Adapters
     class SpinachAdapter < BaseAdapter

data/lib/knapsack_pro/adapters/test_unit_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Adapters
     class TestUnitAdapter < BaseAdapter

data/lib/knapsack_pro/allocator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class Allocator
     def initialize(args)

data/lib/knapsack_pro/allocator_builder.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class AllocatorBuilder < BaseAllocatorBuilder
     def allocator

data/lib/knapsack_pro/base_allocator_builder.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class BaseAllocatorBuilder
     TEST_RUNNER_MAP = {

data/lib/knapsack_pro/build_distribution_fetcher.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class BuildDistributionFetcher
     class BuildDistributionEntity

data/lib/knapsack_pro/client/api/action.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Client
     module API

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Client
     module API

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Client
     module API

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Client
     module API

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Client
     module API

data/lib/knapsack_pro/client/connection.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Client
     class Connection

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # https://www.appveyor.com/docs/environment-variables/
 module KnapsackPro
   module Config
@@ -27,6 +29,10 @@ module KnapsackPro
           ENV['APPVEYOR_BUILD_FOLDER']
         end
 
+        def user_seat
+          ENV['APPVEYOR_REPO_COMMIT_AUTHOR']
+        end
+
         def detected
           ENV.key?('APPVEYOR') ? self.class : nil
         end

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Config
     module CI

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Config
     module CI

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Config
     module CI

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Config
     module CI
@@ -26,6 +28,10 @@ module KnapsackPro
           ENV['CIRRUS_WORKING_DIR']
         end
 
+        def user_seat
+          # not provided
+        end
+
         def detected
           ENV.key?('CIRRUS_CI') ? self.class : nil
         end

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # https://codefresh.io/docs/docs/codefresh-yaml/variables/#system-provided-variables
 module KnapsackPro
   module Config
@@ -27,6 +29,10 @@ module KnapsackPro
           # not provided
         end
 
+        def user_seat
+          ENV['CF_BUILD_INITIATOR']
+        end
+
         def detected
           ENV.key?('CF_BUILD_ID') ? self.class : nil
         end

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Config
     module CI
@@ -26,6 +28,10 @@ module KnapsackPro
           # not provided
         end
 
+        def user_seat
+          ENV['CI_COMMITTER_NAME']
+        end
+
         def detected
           ENV['CI_NAME'] == 'codeship' ? self.class : nil
         end

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # https://help.github.com/en/actions/configuring-and-managing-workflows/using-environment-variables#default-environment-variables
 module KnapsackPro
   module Config

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # https://docs.gitlab.com/ce/ci/variables/
 module KnapsackPro
   module Config

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Config
     module CI
@@ -26,6 +28,10 @@ module KnapsackPro
           '/app' if node_build_id
         end
 
+        def user_seat
+          # not provided
+        end
+
         def detected
           ENV.key?('HEROKU_TEST_RUN_ID') ? self.class : nil
         end

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

@@ -1,6 +1,10 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Config
     module CI
+      # Semaphore Classic is deprecated
+      # https://semaphoreci.com/blog/semaphore-classic-deprecation
       class Semaphore < Base
         def node_total
           ENV['SEMAPHORE_THREAD_COUNT']

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # https://docs.semaphoreci.com/article/12-environment-variables
 module KnapsackPro
   module Config
@@ -30,6 +32,10 @@ module KnapsackPro
           end
         end
 
+        def user_seat
+          # not provided
+        end
+
         def detected
           # check 2 keys to be sure we are using Semaphore 2.0
           ENV.key?('SEMAPHORE') && ENV.key?('SEMAPHORE_WORKFLOW_ID') ? self.class : nil

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Config
     module CI
@@ -18,6 +20,10 @@ module KnapsackPro
           ENV['TRAVIS_BUILD_DIR']
         end
 
+        def user_seat
+          # not provided
+        end
+
         def detected
           ENV.key?('TRAVIS') ? self.class : nil
         end

data/lib/knapsack_pro/config/env.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Config
     class Env

data/lib/knapsack_pro/config/env_generator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Config
     class EnvGenerator

data/lib/knapsack_pro/config/temp_files.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Config
     class TempFiles
@@ -22,10 +24,12 @@ module KnapsackPro
       end
 
       def self.gitignore_file_content
-        "# This directory is used by knapsack_pro gem for storing temporary files during tests runtime.\n" <<
-        "# Ignore all files, and do not commit this directory into your repository.\n" <<
-        "# Learn more at https://knapsackpro.com\n" <<
-        "*"
+        <<~GITIGNORE
+        # This directory is used by knapsack_pro gem for storing temporary files during tests runtime.
+        # Ignore all files, and do not commit this directory into your repository.
+        # Learn more at https://knapsackpro.com
+        *
+        GITIGNORE
       end
 
       def self.create_gitignore_file!

data/lib/knapsack_pro/crypto/branch_encryptor.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Crypto
     class BranchEncryptor

data/lib/knapsack_pro/crypto/decryptor.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Crypto
     class Decryptor

data/lib/knapsack_pro/crypto/digestor.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Crypto
     class Digestor

data/lib/knapsack_pro/crypto/encryptor.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Crypto
     class Encryptor

data/lib/knapsack_pro/extensions/time.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'time'
 
 class Time

data/lib/knapsack_pro/formatters/rspec_json_formatter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Support.require_rspec_core('formatters/json_formatter')
 
 # based on https://github.com/rspec/rspec-core/blob/master/lib/rspec/core/formatters/json_formatter.rb

data/lib/knapsack_pro/formatters/rspec_queue_profile_formatter_extension.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Support.require_rspec_core('formatters/profile_formatter')
 
 module KnapsackPro

data/lib/knapsack_pro/formatters/rspec_queue_summary_formatter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Support.require_rspec_core('formatters/base_formatter')
 RSpec::Support.require_rspec_core('formatters/base_text_formatter')
 

data/lib/knapsack_pro/hooks/queue.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Hooks
     class Queue

data/lib/knapsack_pro/logger_wrapper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class LoggerWrapper
     def initialize(logger)

data/lib/knapsack_pro/mask_string.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class MaskString
     def self.call(string)

data/lib/knapsack_pro/presenter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class Presenter
     class << self

data/lib/knapsack_pro/queue_allocator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class QueueAllocator
     def initialize(args)

data/lib/knapsack_pro/queue_allocator_builder.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class QueueAllocatorBuilder < BaseAllocatorBuilder
     def allocator

data/lib/knapsack_pro/railtie.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails'
 require 'knapsack_pro'
 

data/lib/knapsack_pro/report.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class Report
     def self.save

data/lib/knapsack_pro/repository_adapter_initiator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class RepositoryAdapterInitiator
     def self.call

data/lib/knapsack_pro/repository_adapters/base_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module RepositoryAdapters
     class BaseAdapter

data/lib/knapsack_pro/repository_adapters/env_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module RepositoryAdapters
     class EnvAdapter < BaseAdapter

data/lib/knapsack_pro/repository_adapters/git_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module RepositoryAdapters
     class GitAdapter < BaseAdapter

data/lib/knapsack_pro/runners/base_runner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Runners
     class BaseRunner

data/lib/knapsack_pro/runners/cucumber_runner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Runners
     class CucumberRunner < BaseRunner

data/lib/knapsack_pro/runners/minitest_runner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Runners
     class MinitestRunner < BaseRunner

data/lib/knapsack_pro/runners/queue/base_runner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Runners
     module Queue

data/lib/knapsack_pro/runners/queue/cucumber_runner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Runners
     module Queue

data/lib/knapsack_pro/runners/queue/minitest_runner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Runners
     module Queue

data/lib/knapsack_pro/runners/queue/rspec_runner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Runners
     module Queue

data/lib/knapsack_pro/runners/rspec_runner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Runners
     class RSpecRunner < BaseRunner

data/lib/knapsack_pro/runners/spinach_runner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Runners
     class SpinachRunner < BaseRunner

data/lib/knapsack_pro/runners/test_unit_runner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module Runners
     class TestUnitRunner < BaseRunner

data/lib/knapsack_pro/slow_test_file_determiner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class SlowTestFileDeterminer
     TIME_THRESHOLD_PER_CI_NODE = 0.7 # 70%

data/lib/knapsack_pro/slow_test_file_finder.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class SlowTestFileFinder
     # Get recorded test files from API.

data/lib/knapsack_pro/task_loader.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rake'
 
 module KnapsackPro

data/lib/knapsack_pro/test_case_detectors/rspec_test_example_detector.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module TestCaseDetectors
     class RSpecTestExampleDetector

data/lib/knapsack_pro/test_case_mergers/base_merger.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module TestCaseMergers
     class BaseMerger

data/lib/knapsack_pro/test_case_mergers/rspec_merger.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   module TestCaseMergers
     class RSpecMerger < BaseMerger

data/lib/knapsack_pro/test_file_cleaner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class TestFileCleaner
     def self.clean(test_file_path)

data/lib/knapsack_pro/test_file_finder.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class TestFileFinder
     def self.call(test_file_pattern, test_file_list_enabled: true)

data/lib/knapsack_pro/test_file_pattern.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class TestFilePattern
     def self.call(adapter_class)

data/lib/knapsack_pro/test_file_presenter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class TestFilePresenter
     def self.stringify_paths(test_file_paths)

data/lib/knapsack_pro/test_files_with_test_cases_composer.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class TestFilesWithTestCasesComposer
     # Args:

data/lib/knapsack_pro/test_flat_distributor.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class TestFlatDistributor
     DIR_TYPES = [

data/lib/knapsack_pro/tracker.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class Tracker
     include Singleton

data/lib/knapsack_pro/utils.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
   class Utils
     def self.unsymbolize(obj)

data/lib/knapsack_pro/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module KnapsackPro
-  VERSION = '5.4.1'
+  VERSION = '5.6.0'
 end

data/lib/knapsack_pro.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'logger'
 require 'singleton'
 require 'net/http'

data/lib/tasks/cucumber.rake

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'knapsack_pro'
 
 namespace :knapsack_pro do

data/lib/tasks/encrypted_branch_names.rake

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'knapsack_pro'
 
 namespace :knapsack_pro do

data/lib/tasks/encrypted_test_file_names.rake

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'knapsack_pro'
 
 namespace :knapsack_pro do

data/lib/tasks/minitest.rake

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'knapsack_pro'
 
 namespace :knapsack_pro do

data/lib/tasks/queue/cucumber.rake

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'knapsack_pro'
 
 namespace :knapsack_pro do

data/lib/tasks/queue/minitest.rake

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'knapsack_pro'
 
 namespace :knapsack_pro do

data/lib/tasks/queue/rspec.rake

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'knapsack_pro'
 
 namespace :knapsack_pro do

data/lib/tasks/rspec.rake

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'knapsack_pro'
 
 namespace :knapsack_pro do

data/lib/tasks/salt.rake

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 namespace :knapsack_pro do
   task :salt, [:size] do |_, args|
     default_size = 32

data/lib/tasks/spinach.rake

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'knapsack_pro'
 
 namespace :knapsack_pro do

data/lib/tasks/test_unit.rake

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'knapsack_pro'
 
 namespace :knapsack_pro do

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

@@ -70,4 +70,18 @@ describe KnapsackPro::Config::CI::AppVeyor do
       it { should be nil }
     end
   end
+
+  describe '#user_seat' do
+    subject { described_class.new.user_seat }
+
+    context 'when the APPVEYOR_REPO_COMMIT_AUTHOR environment variable exists' do
+      let(:env) { { 'APPVEYOR_REPO_COMMIT_AUTHOR' => 'jane_doe' } }
+
+      it { should eql 'jane_doe' }
+    end
+
+    context "when the APPVEYOR_REPO_COMMIT_AUTHOR environment variable doesn't exist" do
+      it { should be nil }
+    end
+  end
 end

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

@@ -63,4 +63,19 @@ describe KnapsackPro::Config::CI::Codefresh do
 
     it { should be nil }
   end
+
+
+  describe '#user_seat' do
+    subject { described_class.new.user_seat }
+
+    context 'when the CF_BUILD_INITIATOR environment variable exists' do
+      let(:env) { { 'CF_BUILD_INITIATOR' => 'jane_doe' } }
+
+      it { should eql 'jane_doe' }
+    end
+
+    context "when the CF_BUILD_INITIATOR environment variable doesn't exist" do
+      it { should be nil }
+    end
+  end
 end

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

@@ -63,4 +63,18 @@ describe KnapsackPro::Config::CI::Codeship do
 
     it { should be nil }
   end
+
+  describe '#user_seat' do
+    subject { described_class.new.user_seat }
+
+    context 'when the CI_COMMITTER_NAME environment variable exists' do
+      let(:env) { { 'CI_COMMITTER_NAME' => 'jane_doe' } }
+
+      it { should eql 'jane_doe' }
+    end
+
+    context "when the CI_COMMITTER_NAME environment variable doesn't exist" do
+      it { should be nil }
+    end
+  end
 end

data/spec/knapsack_pro/config/env_spec.rb

@@ -1082,7 +1082,7 @@ describe KnapsackPro::Config::Env do
       ['Other', { 'CI' => 'true'}],
       [nil, {}],
     ].each do |ci, env|
-      it "detects #{ci} name" do
+      it "detects #{ci || 'missing CI from env or development'}" do
         stub_const("ENV", env)
 
         expect(described_class.ci_provider).to eq(ci)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: knapsack_pro
 version: !ruby/object:Gem::Version
-  version: 5.4.1
+  version: 5.6.0
 platform: ruby
 authors:
 - ArturT
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-08-21 00:00:00.000000000 Z
+date: 2023-08-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake