Allow loading saved benchmark results from version <= 1.3.2 (#351)

* Allow loading saved files without evals_set parameter

* Fix parsing

* Fix scoping in test

* Fix typo

* Fix bug

* Fix typo

* Format

Author: Zentrik

src/serialization.jl

@@ -53,7 +53,11 @@ function recover(x::Vector)
         if ft <: get(SUPPORTED_TYPES, nameof(ft), Union{})
             xsi = recover(fields[fn])
         else
-            xsi = convert(ft, fields[fn])
+            xsi = if fn == "evals_set" && !haskey(fields, fn)
+                false
+            else
+                convert(ft, fields[fn])
+            end
         end
         if T == BenchmarkGroup && xsi isa Dict
             for (k, v) in copy(xsi)

test/SerializationTests.jl

@@ -99,4 +99,18 @@ end
     @test_throws ArgumentError BenchmarkTools.recover([1])
 end
 
+@testset "Backwards Comppatibility with evals_set" begin
+    json_string = "[{\"Julia\":\"1.11.0-DEV.1116\",\"BenchmarkTools\":\"1.4.0\"},[[\"Parameters\",{\"gctrial\":true,\"time_tolerance\":0.05,\"samples\":10000,\"evals\":1,\"gcsample\":false,\"seconds\":5.0,\"overhead\":0.0,\"memory_tolerance\":0.01}]]]"
+    json_io = IOBuffer(json_string)
+
+    @test BenchmarkTools.load(json_io) ==
+        [BenchmarkTools.Parameters(5.0, 10000, 1, false, 0.0, true, false, 0.05, 0.01)]
+
+    json_string = "[{\"Julia\":\"1.11.0-DEV.1116\",\"BenchmarkTools\":\"1.4.0\"},[[\"Parameters\",{\"gctrial\":true,\"time_tolerance\":0.05,\"evals_set\":true,\"samples\":10000,\"evals\":1,\"gcsample\":false,\"seconds\":5.0,\"overhead\":0.0,\"memory_tolerance\":0.01}]]]"
+    json_io = IOBuffer(json_string)
+
+    @test BenchmarkTools.load(json_io) ==
+        [BenchmarkTools.Parameters(5.0, 10000, 1, true, 0.0, true, false, 0.05, 0.01)]
+end
+
 end # module