knapsack_pro 5.1.2 → 5.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fdfe0b2398cdf5c6f7721d4984569bc0adce8438406e97b005868c9f32478b24
-  data.tar.gz: '09411e2a2b2cddb9d5dd1e6cdb544c1b7b2898058155d135850836c7409b8551'
+  metadata.gz: ebf6661c02960aefdda92e8fd84ce692e22e59e9fb4d593f677577cda62acb57
+  data.tar.gz: c56e9069badb8ed6090e1b8afb6f2c7d5aebb52a8b7e484484399586b3b7f82c
 SHA512:
-  metadata.gz: 822b2c1ef56ebaf68fbe24800511cdcd3cab21ef0aa8a74fb38a9cf85f682587cf41d59b7c347fa8609458ed35886215f535587016022c2952a5867441fe8f90
-  data.tar.gz: c2a94c2a81befbc09f0b7d77b656335275c028bc1ea40c1c06ad07f60e1623a3b0a4079eba83525234f6f7e460d3f8b067cd274fa4e0adfd1835d903df8f25d2
+  metadata.gz: df2abe40a8b601acd3db8eedce7fc985273e4ff0100346300080ef0de7c8a3df96ecb98f7ab038f121f365d852da0ff877c579a57a652c1c5fb8318a72bc53a7
+  data.tar.gz: 032132b675a088531b3dd112f3258823ec4fd7b9beae8f982dbb3abc85508b4d4ccbd56112ab0e8f1faaf9b0af74edc98fd6896fb01d25ba77f20bef14f61273

data/.github/dependabot.yml

@@ -0,0 +1,11 @@
+# To get started with Dependabot version updates, you'll need to specify which
+# package ecosystems to update and where the package manifests are located.
+# Please see the documentation for all configuration options:
+# https://docs.github.com/github/administering-a-repository/configuration-options-for-dependency-updates
+
+version: 2
+updates:
+  - package-ecosystem: "bundler"
+    directory: "/"
+    schedule:
+      interval: "weekly"

data/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Changelog
 
+### 5.2.1
+
+* Shallow fetch the last month of commits only on CI
+* Ensure input to `git shortlog`
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/209
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v5.2.0...v5.2.1
+
+### 5.2.0
+
+* Send authors to the API
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/208
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v5.1.2...v5.2.0
+
 ### 5.1.2
 
 * Fix broken RSpec split by test examples feature when `SPEC_OPTS` is set in Queue Mode. Ignore `SPEC_OPTS` when generating test examples report for slow test files.

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

@@ -16,6 +16,8 @@ module KnapsackPro
                 :node_index => args.fetch(:node_index),
                 :ci_build_id => KnapsackPro::Config::Env.ci_node_build_id,
                 :user_seat => KnapsackPro::Config::Env.masked_user_seat,
+                :build_author => KnapsackPro::RepositoryAdapters::GitAdapter.new.build_author,
+                :commit_authors => KnapsackPro::RepositoryAdapters::GitAdapter.new.commit_authors,
               }
 
               unless request_hash[:cache_read_attempt]

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

@@ -17,6 +17,8 @@ module KnapsackPro
                 :node_index => args.fetch(:node_index),
                 :node_build_id => KnapsackPro::Config::Env.ci_node_build_id,
                 :user_seat => KnapsackPro::Config::Env.masked_user_seat,
+                :build_author => KnapsackPro::RepositoryAdapters::GitAdapter.new.build_author,
+                :commit_authors => KnapsackPro::RepositoryAdapters::GitAdapter.new.commit_authors,
               }
 
               if request_hash[:can_initialize_queue] && !request_hash[:attempt_connect_to_queue]

data/lib/knapsack_pro/config/env.rb

@@ -67,7 +67,7 @@ module KnapsackPro
         def masked_user_seat
           return unless user_seat
 
-          user_seat.gsub(/(?<=\w{2})[a-zA-Z]/, "*")
+          KnapsackPro::MaskString.call(user_seat)
         end
 
         def test_file_pattern
@@ -285,6 +285,11 @@ module KnapsackPro
           ENV['KNAPSACK_PRO_TEST_RUNNER_ADAPTER'] = adapter_class.to_s.split('::').last
         end
 
+        def ci?
+          ENV.fetch('CI', 'false').downcase == 'true' ||
+            detected_ci != KnapsackPro::Config::CI::Base
+        end
+
         private
 
         def required_env(env_name)

data/lib/knapsack_pro/mask_string.rb

@@ -0,0 +1,7 @@
+module KnapsackPro
+  class MaskString
+    def self.call(string)
+      string.gsub(/(?<=\w{2})[a-zA-Z]/, "*")
+    end
+  end
+end

data/lib/knapsack_pro/repository_adapters/git_adapter.rb

@@ -14,8 +14,44 @@ module KnapsackPro
         str_branches.split("\n")
       end
 
+      def commit_authors
+        authors = git_commit_authors
+          .split("\n")
+          .map { |line| line.strip }
+          .map { |line| line.split("\t") }
+          .map do |commits, author|
+            { commits: commits.to_i, author: KnapsackPro::MaskString.call(author) }
+          end
+
+        raise if authors.empty?
+
+        authors
+      rescue Exception
+        []
+      end
+
+      def build_author
+        author = KnapsackPro::MaskString.call(git_build_author.strip)
+        raise if author.empty?
+        author
+      rescue Exception
+        "no git <no.git@example.com>"
+      end
+
       private
 
+      def git_commit_authors
+        if KnapsackPro::Config::Env.ci?
+          `git fetch --shallow-since "one month ago" --quiet`
+        end
+
+        `git log --since "one month ago" | git shortlog --summary --email`
+      end
+
+      def git_build_author
+        `git log --format="%aN <%aE>" -1`
+      end
+
       def working_dir
         dir = KnapsackPro::Config::Env.project_dir
         File.expand_path(dir)

data/lib/knapsack_pro/version.rb

@@ -1,3 +1,3 @@
 module KnapsackPro
-  VERSION = '5.1.2'
+  VERSION = '5.2.1'
 end

data/lib/knapsack_pro.rb

@@ -55,6 +55,7 @@ require_relative 'knapsack_pro/adapters/test_unit_adapter'
 require_relative 'knapsack_pro/adapters/spinach_adapter'
 require_relative 'knapsack_pro/allocator'
 require_relative 'knapsack_pro/queue_allocator'
+require_relative 'knapsack_pro/mask_string'
 require_relative 'knapsack_pro/test_case_mergers/base_merger'
 require_relative 'knapsack_pro/test_case_mergers/rspec_merger'
 require_relative 'knapsack_pro/build_distribution_fetcher'

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

@@ -8,6 +8,7 @@ describe KnapsackPro::Client::API::V1::BuildDistributions do
     let(:ci_build_id) { double }
     let(:masked_user_seat) { double }
     let(:test_files) { double }
+    let(:cache_read_attempt) { [false, true].sample }
 
     subject do
       described_class.subset(
@@ -31,20 +32,9 @@ describe KnapsackPro::Client::API::V1::BuildDistributions do
 
       it 'does not send test_files among other params' do
         action = double
-        expect(KnapsackPro::Client::API::Action).to receive(:new).with({
-          endpoint_path: '/v1/build_distributions/subset',
-          http_method: :post,
-          request_hash: {
-            fixed_test_suite_split: fixed_test_suite_split,
-            cache_read_attempt: cache_read_attempt,
-            commit_hash: commit_hash,
-            branch: branch,
-            node_total: node_total,
-            node_index: node_index,
-            ci_build_id: ci_build_id,
-            user_seat: masked_user_seat,
-          }
-        }).and_return(action)
+        expect(KnapsackPro::Client::API::Action).to receive(:new).with(
+          hash_including(request_hash: hash_excluding(:test_files))
+        ).and_return(action)
         expect(subject).to eq action
       end
     end
@@ -54,24 +44,22 @@ describe KnapsackPro::Client::API::V1::BuildDistributions do
 
       it 'sends test_files among other params' do
         action = double
-        expect(KnapsackPro::Client::API::Action).to receive(:new).with({
-          endpoint_path: '/v1/build_distributions/subset',
-          http_method: :post,
-          request_hash: {
-            fixed_test_suite_split: fixed_test_suite_split,
-            cache_read_attempt: cache_read_attempt,
-            commit_hash: commit_hash,
-            branch: branch,
-            node_total: node_total,
-            node_index: node_index,
-            ci_build_id: ci_build_id,
-            user_seat: masked_user_seat,
-            test_files: test_files
-          }
-        }).and_return(action)
+        expect(KnapsackPro::Client::API::Action).to receive(:new).with(
+          hash_including(request_hash: hash_including(test_files: test_files))
+        ).and_return(action)
         expect(subject).to eq action
       end
     end
+
+    it "sends authors" do
+      action = double
+
+      expect(KnapsackPro::Client::API::Action).to receive(:new).with(
+        hash_including(request_hash: hash_including(:build_author, :commit_authors))
+      ).and_return(action)
+
+      expect(subject).to eq action
+    end
   end
 
   describe '.last' do

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

@@ -8,6 +8,8 @@ describe KnapsackPro::Client::API::V1::Queues do
     let(:test_files) { double }
     let(:node_build_id) { double }
     let(:masked_user_seat) { double }
+    let(:can_initialize_queue) { [false, true].sample }
+    let(:attempt_connect_to_queue) { [false, true].sample }
 
     subject do
       described_class.queue(
@@ -33,21 +35,9 @@ describe KnapsackPro::Client::API::V1::Queues do
 
       it 'does not send test_files among other params' do
         action = double
-        expect(KnapsackPro::Client::API::Action).to receive(:new).with({
-          endpoint_path: '/v1/queues/queue',
-          http_method: :post,
-          request_hash: {
-            fixed_queue_split: fixed_queue_split,
-            can_initialize_queue: can_initialize_queue,
-            attempt_connect_to_queue: attempt_connect_to_queue,
-            commit_hash: commit_hash,
-            branch: branch,
-            node_total: node_total,
-            node_index: node_index,
-            node_build_id: node_build_id,
-            user_seat: masked_user_seat,
-          }
-        }).and_return(action)
+        expect(KnapsackPro::Client::API::Action).to receive(:new).with(
+          hash_including(request_hash: hash_excluding(:test_files))
+        ).and_return(action)
         expect(subject).to eq action
       end
     end
@@ -58,22 +48,9 @@ describe KnapsackPro::Client::API::V1::Queues do
 
       it 'sends test_files among other params' do
         action = double
-        expect(KnapsackPro::Client::API::Action).to receive(:new).with({
-          endpoint_path: '/v1/queues/queue',
-          http_method: :post,
-          request_hash: {
-            fixed_queue_split: fixed_queue_split,
-            can_initialize_queue: can_initialize_queue,
-            attempt_connect_to_queue: attempt_connect_to_queue,
-            commit_hash: commit_hash,
-            branch: branch,
-            node_total: node_total,
-            node_index: node_index,
-            node_build_id: node_build_id,
-            user_seat: masked_user_seat,
-            test_files: test_files
-          }
-        }).and_return(action)
+        expect(KnapsackPro::Client::API::Action).to receive(:new).with(
+          hash_including(request_hash: hash_including(test_files: test_files))
+        ).and_return(action)
         expect(subject).to eq action
       end
     end
@@ -84,23 +61,21 @@ describe KnapsackPro::Client::API::V1::Queues do
 
       it 'does not send test_files among other params' do
         action = double
-        expect(KnapsackPro::Client::API::Action).to receive(:new).with({
-          endpoint_path: '/v1/queues/queue',
-          http_method: :post,
-          request_hash: {
-            fixed_queue_split: fixed_queue_split,
-            can_initialize_queue: can_initialize_queue,
-            attempt_connect_to_queue: attempt_connect_to_queue,
-            commit_hash: commit_hash,
-            branch: branch,
-            node_total: node_total,
-            node_index: node_index,
-            node_build_id: node_build_id,
-            user_seat: masked_user_seat,
-          }
-        }).and_return(action)
+        expect(KnapsackPro::Client::API::Action).to receive(:new).with(
+          hash_including(request_hash: hash_excluding(:test_files))
+        ).and_return(action)
         expect(subject).to eq action
       end
     end
+
+    it "sends authors" do
+      action = double
+
+      expect(KnapsackPro::Client::API::Action).to receive(:new).with(
+        hash_including(request_hash: hash_including(:build_author, :commit_authors))
+      ).and_return(action)
+
+      expect(subject).to eq action
+    end
   end
 end

data/spec/knapsack_pro/config/env_spec.rb

@@ -1041,4 +1041,18 @@ describe KnapsackPro::Config::Env do
       end
     end
   end
+
+  describe '.ci?' do
+    [
+      ['CI from env', { 'CI' => 'True' }, true],
+      ['Travis CI', { 'TRAVIS' => 'true' }, true],
+      ['Unsupported', {}, false],
+    ].each do |ci, env, expected|
+      it "detects #{ci}" do
+        stub_const("ENV", env)
+
+        expect(described_class.ci?).to eq(expected)
+      end
+    end
+  end
 end

data/spec/knapsack_pro/repository_adapters/git_adapter_spec.rb

@@ -31,4 +31,76 @@ describe KnapsackPro::RepositoryAdapters::GitAdapter do
     it { expect(subject.include?('master')).to be true }
     it { expect(subject.include?(circle_branch)).to be true } if ENV['CIRCLECI']
   end
+
+  describe '#build_author' do
+    it "returns the masked build author" do
+      allow_any_instance_of(KnapsackPro::RepositoryAdapters::GitAdapter).to receive(:git_build_author).and_return(
+        "John Doe <john.doe@example.com>" + "\n"
+      )
+
+      subject = KnapsackPro::RepositoryAdapters::GitAdapter.new
+
+      expect(subject.build_author).to eq 'Jo** Do* <jo**.do*@ex*****.co*>'
+    end
+
+    context "when the command raises an exception" do
+      it "returns the no-git author" do
+        allow_any_instance_of(KnapsackPro::RepositoryAdapters::GitAdapter).to receive(:git_build_author).and_raise(Exception)
+
+        subject = KnapsackPro::RepositoryAdapters::GitAdapter.new
+
+        expect(subject.build_author).to eq "no git <no.git@example.com>"
+      end
+    end
+
+    context "when the command returns an empty string" do
+      it "returns the no-git author" do
+        allow_any_instance_of(KnapsackPro::RepositoryAdapters::GitAdapter).to receive(:git_build_author).and_return("")
+
+        subject = KnapsackPro::RepositoryAdapters::GitAdapter.new
+
+        expect(subject.build_author).to eq "no git <no.git@example.com>"
+      end
+    end
+  end
+
+  describe '#commit_authors' do
+    it "returns the masked commit authors" do
+      allow_any_instance_of(KnapsackPro::RepositoryAdapters::GitAdapter).to receive(:git_commit_authors).and_return([
+        "     5\t3v0k4 <riccardo@example.com>\n",
+        "    10\tArtur Nowak <artur@example.com>\n",
+        "     2\tRiccardo <riccardo@example.com>\n",
+        "     3 \tshadre <shadi@example.com>\n",
+      ].join(""))
+
+      subject = KnapsackPro::RepositoryAdapters::GitAdapter.new
+
+      expect(subject.commit_authors).to eq([
+        { commits: 5, author: "3v0*4 <ri******@ex*****.co*>" },
+        { commits: 10, author: "Ar*** No*** <ar***@ex*****.co*>" },
+        { commits: 2, author: "Ri****** <ri******@ex*****.co*>" },
+        { commits: 3, author: "sh**** <sh***@ex*****.co*>" },
+      ])
+    end
+
+    context "when the authors command raises an exception" do
+      it "returns []" do
+        allow_any_instance_of(KnapsackPro::RepositoryAdapters::GitAdapter).to receive(:git_commit_authors).and_raise(Exception)
+
+        subject = KnapsackPro::RepositoryAdapters::GitAdapter.new
+
+        expect(subject.commit_authors).to eq []
+      end
+    end
+
+    context "when the authors command returns an empty string" do
+      it "returns []" do
+        allow_any_instance_of(KnapsackPro::RepositoryAdapters::GitAdapter).to receive(:git_commit_authors).and_return("")
+
+        subject = KnapsackPro::RepositoryAdapters::GitAdapter.new
+
+        expect(subject.commit_authors).to eq []
+      end
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: knapsack_pro
 version: !ruby/object:Gem::Version
-  version: 5.1.2
+  version: 5.2.1
 platform: ruby
 authors:
 - ArturT
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-06-20 00:00:00.000000000 Z
+date: 2023-07-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -192,6 +192,7 @@ files:
 - ".circleci/config.yml"
 - ".github/assets/install-button.png"
 - ".github/assets/knapsack-diamonds.png"
+- ".github/dependabot.yml"
 - ".gitignore"
 - ".rspec"
 - CHANGELOG.md
@@ -244,6 +245,7 @@ files:
 - lib/knapsack_pro/formatters/rspec_queue_summary_formatter.rb
 - lib/knapsack_pro/hooks/queue.rb
 - lib/knapsack_pro/logger_wrapper.rb
+- lib/knapsack_pro/mask_string.rb
 - lib/knapsack_pro/presenter.rb
 - lib/knapsack_pro/queue_allocator.rb
 - lib/knapsack_pro/queue_allocator_builder.rb