selective-ruby-core 0.1.7-aarch64-linux → 0.1.9-aarch64-linux

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cda7bb49825b1322ab9bb0d3bc643744bf79dc70c7a8a611fd4cc39cba7ca494
-  data.tar.gz: 596d3053cb1de84eba9ca0a1488600fc559c647fe963c882cfd62b6b34b60686
+  metadata.gz: d396e26c8bae6e3ace92a0cb027627ee691e818bec433ea1af28dc108b48afc4
+  data.tar.gz: cfcab6ed9eff376aceb940ea4ef7e5ece1353d27aac35254282ab38598cf6779
 SHA512:
-  metadata.gz: 72ed3aba011eafbb171074eff2681ec2003144de42f2cdfdf86299604a88500d6bc510b849664befaab8a4d5b60e60a7168de64c271186bce58ba793c28d451f
-  data.tar.gz: 685d9c77788edc0d1e1b4f2b923982989fdc347fa7710b503b5a3f9beae4238ec3810df37c4491d53ef002d4eaf70502978011ce6b11641a5261f80c546da92b
+  metadata.gz: d1f10139bdd46cd2dc5f05e72754471b01d8e0626b92a4da5a6cbd1c83e180eec6e58899860ecda80a2f4a8d0a499c85ac17193e943d73aef9e701667ae96508
+  data.tar.gz: 66fa3787af219e72ad55b0a2fb625c01d76867cbaa44fac86fd7699c2c273ae0bc60911ea0246fa22af7bcf6755e4ecf550c4f28409382b9bd0f983f7ef47968

data/lib/bin/build_env.sh

@@ -33,6 +33,8 @@ fi
 # Output the JSON
 cat <<EOF
   {
+    "api_key": "$SELECTIVE_API_KEY",
+    "host": "${SELECTIVE_HOST:-wss://app.selective.ci}",
     "platform": "$platform",
     "branch": "$branch",
     "pr_title": "$SELECTIVE_PR_TITLE",

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

@@ -9,6 +9,14 @@ module Selective
         include Helper
         @@selective_suppress_reporting = false
 
+        REQUIRED_CONFIGURATION = {
+          "host" => "SELECTIVE_HOST",
+          "api_key" => "SELECTIVE_API_KEY",
+          "platform" => "SELECTIVE_PLATFORM",
+          "run_id" => "SELECTIVE_RUN_ID",
+          "branch" => "SELECTIVE_BRANCH"
+        }.freeze
+
         def initialize(runner, debug: false, log: false)
           @debug = debug
           @runner = runner
@@ -101,12 +109,8 @@ module Selective
 
         def transport_url(reconnect: false)
           @transport_url ||= begin
-            api_key = ENV.fetch("SELECTIVE_API_KEY")
-            host = ENV.fetch("SELECTIVE_HOST", "wss://app.selective.ci")
-
-            # Validate that host is a valid websocket url(starts with ws:// or wss://)
-            raise "Invalid host: #{host}" unless host.match?(/^wss?:\/\//)
-
+            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")
             run_attempt = SecureRandom.uuid if run_attempt.nil? || run_attempt.empty?
@@ -134,7 +138,20 @@ module Selective
         def build_env
           @build_env ||= begin
             result = `#{File.join(ROOT_GEM_PATH, "lib", "bin", "build_env.sh")}`
-            JSON.parse(result)
+            JSON.parse(result).tap do |env|
+              validate_build_env(env)
+            end
+          end
+        end
+
+        def validate_build_env(env)
+          missing = REQUIRED_CONFIGURATION.each_with_object([]) do |(key, env_var), arry|
+            arry << env_var if env[key].nil? || env[key].empty?
+          end
+          
+          with_error_handling do
+            raise "Missing required environment variables: #{missing.join(", ")}" unless missing.empty?
+            raise "Invalid host: #{env['host']}" unless env['host'].match?(/^wss?:\/\//)
           end
         end
 
@@ -220,9 +237,11 @@ module Selective
         end
 
         def handle_command(data)
-          send("handle_#{data[:command]}", data)
-        rescue NoMethodError
-          raise "Unknown command received: #{data[:command]}" if debug?
+          if respond_to? "handle_#{data[:command]}", true
+            send("handle_#{data[:command]}", data)
+          else
+            raise "Unknown command received: #{data[:command]}" if debug?
+          end
         end
 
         def handle_print_notice(data)
@@ -262,7 +281,7 @@ module Selective
         def handle_close(data)
           exit_status = data[:exit_status]
           self.class.restore_reporting!
-          runner.finish unless exit_status.is_a?(Integer)
+          with_error_handling { runner.finish } unless exit_status.is_a?(Integer)
 
           kill_transport
           pipe.delete_pipes
@@ -291,12 +310,12 @@ module Selective
 
         def get_diff(num_commits)
           target_branch = build_env["target_branch"]
-          return [] if target_branch.nil? || target_branch.empty?
+          return if target_branch.nil? || target_branch.empty?
 
           Open3.capture2e("git fetch origin #{target_branch} --depth=#{num_commits}").then do |output, status|
             unless status.success?
               print_warning "Selective was unable to fetch the target branch. This may result in a sub-optimal test order. If the issue persists, please contact support. The output was:\n\n#{output}"
-              return []
+              return
             end
           end
 
@@ -305,7 +324,7 @@ module Selective
               output.split("\n")
             else
               print_warning "Selective was unable to diff with the target branch. This may result in a sub-optimal test order. If the issue persists, please contact support. The output was:\n\n#{output}"
-              []
+              nil
             end
           end
         end

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

@@ -3,7 +3,7 @@
 module Selective
   module Ruby
     module Core
-      VERSION = "0.1.7"
+      VERSION = "0.1.9"
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: selective-ruby-core
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.1.9
 platform: aarch64-linux
 authors:
 - Benjamin Wood
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2024-01-05 00:00:00.000000000 Z
+date: 2024-01-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: zeitwerk