bitfab 0.63.1 → 0.63.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: be8f745cae80e29bbcb5fc07bfd46cab706d3f6420e1391b39ccd8f297898711
-  data.tar.gz: 383d4f086eb11f5344444bb7506e37d5b5928f679dd4e9daa2b25a5db30780d5
+  metadata.gz: dc609d9aaa9ba7366c855624027df97620baf1309fc38192316305e94e7de4e5
+  data.tar.gz: 2562479d249da81b28b80a8bb176438ac60bdf6c82135f348afb252e09c005a7
 SHA512:
-  metadata.gz: 490e9932572e7e0f6521a3ac855eb1f659c6f00349ebc8c174e35eee062a3dff784c52c1ae77f4ae93be2ad1f6f9fd644ada7391dc4fba46b7089aaf2f8291e1
-  data.tar.gz: de98876f6c6b229c7edc8281adbb5a071b00fcf95d77a2b17620fbee2fa6891a588aed3ae8d773702c23aa239bca92416b00c059ba496e78be3fdd6980b29d16
+  metadata.gz: 4df6fafa2a4ae84186457257339cbdabeb88c43713147541aa4b66b4ccc6888f39822f92ee3e11662f70800892066907c9904f61f03ec8c1c1206ee717f5d3bf
+  data.tar.gz: 9137bc781a7d3f87b8c4f575ddc90a9bf369d1b7c84657c32f038fd12e6bcffb4d33fd95dda28cb6b0f551fc74bb5d3bb627df7a57116ec88a31c8060b459793

data/lib/bitfab/http_client.rb

@@ -262,7 +262,7 @@ module Bitfab
     #   same org and trace function or the server rejects the replay
     def start_replay(trace_function_key, limit, trace_ids: nil, code_change_description: nil,
       code_change_files: nil, experiment_group_id: nil, name: nil, notes: nil, metadata: nil, include_db_branch_lease: false, dataset_ids: nil,
-      grader_ids: nil, db_branch_settings: nil, attempts: 1, only_with_assertions: false, skip_assertion_judging: false, include_original_metadata: false, experimental_selective_replay: nil)
+      grader_ids: nil, db_branch_settings: nil, attempts: 1, only_with_assertions: false, skip_assertion_judging: false, include_original_metadata: false, experimental_selective_replay: nil, dry_run: false)
       payload = {
         "traceFunctionKey" => trace_function_key
       }
@@ -289,6 +289,7 @@ module Bitfab
       payload["attempts"] = attempts if attempts > 1
       payload["onlyWithAssertions"] = true if only_with_assertions
       payload["judgeAssertions"] = true unless skip_assertion_judging
+      payload["dryRun"] = true if dry_run
       payload["includeOriginalMetadata"] = true if include_original_metadata
       payload["experimentalSelectiveReplay"] = experimental_selective_replay if experimental_selective_replay
       payload["dbBranchSettings"] = db_branch_settings unless db_branch_settings.nil?

data/lib/bitfab/replay.rb

@@ -279,9 +279,11 @@ module Bitfab
     #   right after the server creates the experiment and before any item runs,
     #   with a hash { experiment_id:, experiment_url: }. Use it to show the
     #   experiment link even if the run later fails. A raising callback never
-    #   crashes the run.
+    #   crashes the run. Not called for a dry run, which creates no experiment.
     # @return [Hash] with :items, :experiment_id, :experiment_url, and the
-    #   deprecated aliases :test_run_id and :test_run_url carrying the same values
+    #   deprecated aliases :test_run_id and :test_run_url carrying the same values.
+    #   The experiment fields are nil for a dry run, which creates no experiment,
+    #   and progress events then carry a nil :experiment_id too.
     def run(client, receiver, method_name, trace_function_key:, limit: nil, trace_ids: nil, name: nil, notes: nil, metadata: nil,
       concurrency: nil, max_concurrency: ReplayConcurrency::UNSET, code_change_description: CODE_CHANGE_UNSET, code_change_files: CODE_CHANGE_UNSET, experiment_group_id: nil,
       dataset_id: nil, dataset_ids: nil, grader_ids: nil, mock: "marked", attempts: ReplayConcurrency::UNSET, only_with_assertions: false, skip_assertion_judging: false, judge_assertions: nil, dry_run: false,
@@ -393,12 +395,14 @@ module Bitfab
         skip_assertion_judging:,
         include_original_metadata: !adapt_inputs.nil?,
         db_branch_settings: resolved_db_branch_settings,
+        dry_run:,
         **(experimental_selective_replay ? {experimental_selective_replay: SelectiveReplay.wire_options(experimental_selective_replay)} : {})
       )
       experiment_id = replay_data["experimentId"] || replay_data["testRunId"]
       experiment_url = replay_data["experimentUrl"] || replay_data["testRunUrl"]
-      full_experiment_url = "#{client.service_url}#{experiment_url}"
-      if on_experiment_start
+      # Nil for a dry run: current servers create no experiment for one.
+      full_experiment_url = experiment_url && "#{client.service_url}#{experiment_url}"
+      if on_experiment_start && experiment_id
         begin
           on_experiment_start.call({experiment_id:, experiment_url: full_experiment_url})
         rescue => e
@@ -426,10 +430,14 @@ module Bitfab
       end
 
       if dry_run
-        begin
-          http_client.complete_replay(experiment_id)
-        rescue => e
-          warn "Bitfab: could not finalize dry run: #{e.message}"
+        # An older server still creates an experiment for a dry run; finalize it
+        # so it does not sit unfinished in the experiments list.
+        if experiment_id
+          begin
+            http_client.complete_replay(experiment_id)
+          rescue => e
+            warn "Bitfab: could not finalize dry run: #{e.message}"
+          end
         end
         result = {
           items: public_replay_items(result_items), experiment_id:, experiment_url: full_experiment_url,

data/lib/bitfab/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Bitfab
-  VERSION = "0.63.1"
+  VERSION = "0.63.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bitfab
 version: !ruby/object:Gem::Version
-  version: 0.63.1
+  version: 0.63.2
 platform: ruby
 authors:
 - Harvest Team