selective-ruby-core 0.2.2-x86_64-darwin → 0.2.4-x86_64-darwin

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2b31916fb7c624fac39cd7b3a2ccf05e076c189e96685e1bba20469f2fa5e4fd
-  data.tar.gz: 909ac8b3216b515a5a2cbeb2481e6bffe035f1482b7cb40bd425cc730a0c3c94
+  metadata.gz: bf4ede6530f65732361cb3edca6e3bf1fe1b75702d42460710248e6afd77d042
+  data.tar.gz: 8d09d05ec38cc9aea45834835bd98fe6d94f7387d06cfa337d632ba97c68b69d
 SHA512:
-  metadata.gz: 9e209155405afc706b36bae7fd1ffbc7bc526096b675ba65416e2129ee3e61b855fb92878badde69630effb01d35425411a9a14329e33123e3c36f2e22b234b7
-  data.tar.gz: 55a7aafc15597659a88afd9ab4a51db7b7713a7eb47e9290b68c2d4257a13d554c7381d1bdf63c36e07b3d8727d1a47b2b90b1c0e9ea456826453f3ad0358a73
+  metadata.gz: 27a5595b47add478a1c99c71f7574a66b6fbe0cf8deafb503bedd48f61302397469658b7f0d9cc71c147668980a50169c48c2e4a3c912afb0a6ed8dab3088429
+  data.tar.gz: 5f1a0a6a1644f4193eb2d29eaf820278a7a74b96a0f1c8a49e2b3e7623485379b7f1f207285c145120a97f9dabb69308c4a1bbf64f3b4836102420f016b531a3

data/lib/bin/build_env.sh

@@ -31,22 +31,26 @@ elif [ -n "$SEMAPHORE" ]; then
   pr_title=$SEMAPHORE_GIT_PR_NAME
 fi
 
+function escape() {
+  echo -n "$1" | sed 's/"/\\"/g'
+}
+
 # Output the JSON
 cat <<EOF
   {
-    "api_key": "$SELECTIVE_API_KEY",
-    "host": "${SELECTIVE_HOST:-wss://app.selective.ci}",
-    "platform": "${SELECTIVE_PLATFORM:-$platform}",
-    "branch": "${SELECTIVE_BRANCH:-$branch}",
-    "pr_title": "${SELECTIVE_PR_TITLE:-$pr_title}",
-    "target_branch": "${SELECTIVE_TARGET_BRANCH:-$target_branch}",
-    "actor": "${SELECTIVE_ACTOR:-$actor}",
-    "sha": "${SELECTIVE_SHA:-$sha}",
-    "run_id": "${SELECTIVE_RUN_ID:-$run_id}",
-    "run_attempt": "${SELECTIVE_RUN_ATTEMPT:-$run_attempt}",
-    "runner_id": "${SELECTIVE_RUNNER_ID:-$runner_id}",
-    "commit_message": "$(git log --format=%s -n 1 $sha)",
-    "committer_name": "$(git show -s --format='%an' -n 1 $sha)",
-    "committer_email": "$(git show -s --format='%ae' -n 1 $sha)"
+    "api_key": "$(escape "${SELECTIVE_API_KEY}")",
+    "host": "$(escape "${SELECTIVE_HOST:-wss://app.selective.ci}")",
+    "platform": "$(escape "${SELECTIVE_PLATFORM:-$platform}")",
+    "branch": "$(escape "${SELECTIVE_BRANCH:-$branch}")",
+    "pr_title": "$(escape "${SELECTIVE_PR_TITLE:-$pr_title}")",
+    "target_branch": "$(escape "${SELECTIVE_TARGET_BRANCH:-$target_branch}")",
+    "actor": "$(escape "${SELECTIVE_ACTOR:-$actor}")",
+    "sha": "$(escape "${SELECTIVE_SHA:-$sha}")",
+    "run_id": "$(escape "${SELECTIVE_RUN_ID:-$run_id}")",
+    "run_attempt": "$(escape "${SELECTIVE_RUN_ATTEMPT:-$run_attempt}")",
+    "runner_id": "$(escape "${SELECTIVE_RUNNER_ID:-$runner_id}")",
+    "commit_message": "$(escape "$(git log --format=%s -n 1 $sha)")",
+    "committer_name": "$(escape "$(git show -s --format='%an' -n 1 $sha)")",
+    "committer_email": "$(escape "$(git show -s --format='%ae' -n 1 $sha)")"
   }
 EOF

data/lib/selective/ruby/core/controller.rb

@@ -8,6 +8,7 @@ module Selective
       class Controller
         include Helper
         @@selective_suppress_reporting = false
+        @@report_at_finish = {}
 
         REQUIRED_CONFIGURATION = {
           "host" => "SELECTIVE_HOST",
@@ -33,7 +34,7 @@ module Selective
           handle_termination_signals(transport_pid)
           wait_for_connectivity
           run_main_loop
-        rescue NamedPipe::PipeClosedError
+        rescue ConnectionLostError
           retry!
         rescue => e
           with_error_handling { raise e }
@@ -57,12 +58,16 @@ module Selective
           @@selective_suppress_reporting
         end
 
+        def self.report_at_finish
+          @@report_at_finish
+        end
+
         private
 
         attr_reader :runner, :pipe, :transport_pid, :retries, :logger, :runner_id, :diff
 
         def get_runner_id
-          runner_id = build_env.delete("runner_id")
+          runner_id = build_env["runner_id"]
           return generate_runner_id if runner_id.nil? || runner_id.empty?
 
           runner_id
@@ -90,10 +95,10 @@ module Selective
         def retry!
           @retries += 1
 
-          with_error_handling { raise "Too many retries" } if retries > 4
+          with_error_handling { raise "Too many retries" } if retries > 10
 
-          puts("Retrying in #{retries} seconds...")
-          sleep(retries)
+          puts("Retrying in #{retries} seconds...") if debug?
+          sleep(retries > 4 ? 4 : retries)
           kill_transport
 
           pipe.reset!
@@ -109,29 +114,30 @@ module Selective
         end
 
         def transport_url(reconnect: false)
-          @transport_url ||= begin
-            api_key = build_env.delete("api_key")
-            host = build_env.delete("host")
-            run_id = build_env.delete("run_id")
-            run_attempt = build_env.delete("run_attempt")
+          base_transport_url_params[:reconnect] = true if reconnect
+          query_string = URI.encode_www_form(base_transport_url_params)
+          "#{build_env["host"]}/transport/websocket?#{query_string}"
+        end
+
+        def base_transport_url_params
+          @base_transport_url_params ||= begin
+            api_key = build_env["api_key"]
+            run_id = build_env["run_id"]
+            run_attempt = build_env["run_attempt"]
 
-            params = {
+            metadata = build_env.reject { |k,v| %w(host runner_id api_key run_id run_attempt).include?(k) }
+
+            {
+              "api_key" => api_key,
               "run_id" => run_id,
               "run_attempt" => run_attempt,
-              "api_key" => api_key,
               "runner_id" => runner_id,
               "language" => "ruby",
               "core_version" => Selective::Ruby::Core::VERSION,
               "framework" => runner.framework,
               "framework_version" => runner.framework_version,
               "framework_wrapper_version" => runner.wrapper_version,
-            }.merge(metadata: build_env.to_json)
-
-            prams[:reconnect] = true if reconnect
-
-            query_string = URI.encode_www_form(params)
-
-            "#{host}/transport/websocket?#{query_string}"
+            }.merge(metadata: metadata.to_json)
           end
         end
 
@@ -224,7 +230,7 @@ module Selective
 
               sleep(1)
             end
-          rescue NamedPipe::PipeClosedError, IOError
+          rescue ConnectionLostError, IOError
             # If the pipe is close, move straight to killing
             # it forcefully.
           end
@@ -270,8 +276,11 @@ module Selective
           runner.run_test_cases(data[:test_case_ids])
         end
 
+        # Todo: Rename this command to match the method name
+        # on the runner wrapper. We should do something similar
+        # to normalize handle_print_notice and handle_print_message
         def handle_remove_failed_test_case_result(data)
-          runner.remove_failed_test_case_result(data[:test_case_id])
+          runner.remove_test_case_result(data[:test_case_id])
         end
 
         def handle_print_message(data)
@@ -281,7 +290,13 @@ module Selective
         def handle_close(data)
           exit_status = data[:exit_status]
           self.class.restore_reporting!
-          with_error_handling { runner.finish } unless exit_status.is_a?(Integer)
+
+          with_error_handling do
+            Selective::Ruby::Core::Controller.report_at_finish[:connection_retries] = @retries
+            write({type: "report_at_finish", data: Selective::Ruby::Core::Controller.report_at_finish})
+
+            runner.finish unless exit_status.is_a?(Integer)
+          end
 
           kill_transport
           pipe.delete_pipes

data/lib/selective/ruby/core/file_correlator.rb

@@ -9,7 +9,7 @@ module Selective
         FILE_CORRELATION_COLLECTOR_PATH = File.join(ROOT_GEM_PATH, "lib", "bin", "file_correlation_collector.sh")
 
         def initialize(diff, num_commits, target_branch)
-          @diff = diff
+          @diff = diff.reject {|f| f =~ /^spec\// }
           @num_commits = num_commits
           @target_branch = target_branch
         end

data/lib/selective/ruby/core/named_pipe.rb

@@ -2,8 +2,6 @@ module Selective
   module Ruby
     module Core
       class NamedPipe
-        class PipeClosedError < StandardError; end
-
         attr_reader :read_pipe_path, :write_pipe_path
 
         def initialize(read_pipe_path, write_pipe_path, skip_reset: false)
@@ -44,7 +42,7 @@ module Selective
             write_pipe.write("\n")
             write_pipe.flush
           rescue Errno::EPIPE
-            raise PipeClosedError
+            raise ConnectionLostError
           end
         end
 
@@ -54,7 +52,7 @@ module Selective
             message = read_pipe.gets.chomp
           rescue NoMethodError => e
             if e.name == :chomp
-              raise PipeClosedError
+              raise ConnectionLostError
             else
               raise e
             end

data/lib/selective/ruby/core/version.rb

@@ -3,7 +3,7 @@
 module Selective
   module Ruby
     module Core
-      VERSION = "0.2.2"
+      VERSION = "0.2.4"
     end
   end
 end

data/lib/selective-ruby-core.rb

@@ -14,6 +14,7 @@ module Selective
   module Ruby
     module Core
       class Error < StandardError; end
+      class ConnectionLostError < StandardError; end
 
       ROOT_GEM_PATH = Gem.loaded_specs["selective-ruby-core"].full_gem_path
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: selective-ruby-core
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.2.4
 platform: x86_64-darwin
 authors:
 - Benjamin Wood
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2024-01-26 00:00:00.000000000 Z
+date: 2024-02-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: zeitwerk
@@ -71,7 +71,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.10
+rubygems_version: 3.5.3
 signing_key: 
 specification_version: 4
 summary: Selective Ruby Client Core