make all tests 1s so buildkite doesn't think they're skipped

Author: IanButterworth

test/buildkitetestjson.jl

@@ -72,7 +72,6 @@ function result_dict(testset::Test.DefaultTestSet, prefix::String="")
             "julia_version" => string(VERSION),
             "testset" => testset.description,
         ),
-        # note we drop some of this from common_data before merging into individual results
         "history" => if !isnothing(testset.time_end)
             Dict{String,Any}(
                 "start_at" => testset.time_start,
@@ -177,8 +176,10 @@ end
 function collect_results!(results::Vector{Dict{String,Any}}, testset::Test.DefaultTestSet, prefix::String="")
     common_data = result_dict(testset, prefix)
     # testset duration is not relevant for individual test results
-    common_data["history"]["duration"] = 0.0 # required field
-    delete!(common_data["history"], "end_at")
+    # but buildkite considers zero duration tests to have been skipped, so set all tests to 1.0s
+    # TODO: get buildkite to support zero duration tests
+    common_data["history"]["duration"] = 1.0 # required field
+    common_data["history"]["end_at"] = common_data["history"]["start_at"] + 1
     result_offset = length(results) + 1
     result_counts = Dict{Tuple{String,String,UInt64},Int}()
     get_rid(rdata) = (rdata["location"], rdata["result"], haskey(rdata, "failure_expanded") ? hash(rdata["failure_expanded"]) : UInt64(0))