Throw if test_end_expr not a :block expr (#191)

* Throw if `test_end_expr` not a `:block` expression

* Bump version

Project.toml

@@ -1,6 +1,6 @@
 name = "ReTestItems"
 uuid = "817f1d60-ba6b-4fd5-9520-3cf149f6a823"
-version = "1.28.0"
+version = "1.29.0"
 
 [deps]
 Dates = "ade2ca70-3891-5945-98fb-dc099432e06a"

src/ReTestItems.jl

@@ -272,6 +272,7 @@ function runtests(
     (0 ≤ memory_threshold ≤ 1) || throw(ArgumentError("`memory_threshold` must be between 0 and 1, got $(repr(memory_threshold))"))
     testitem_timeout > 0 || throw(ArgumentError("`testitem_timeout` must be a positive number, got $(repr(testitem_timeout))"))
     timeout_profile_wait >= 0 || throw(ArgumentError("`timeout_profile_wait` must be a non-negative number, got $(repr(timeout_profile_wait))"))
+    test_end_expr.head === :block || throw(ArgumentError("`test_end_expr` must be a `:block` expression, got a `$(repr(test_end_expr.head))` expression"))
     # If we were given paths but none were valid, then nothing to run.
     !isempty(paths) && isempty(paths′) && return nothing
     ti_filter = TestItemFilter(shouldrun, tags, name)

test/integrationtests.jl

@@ -1503,4 +1503,8 @@ else
 end # VERSION
 end
 
+@testset "`test_end_expr` must be `:block`" begin
+    @test_throws "`test_end_expr` must be a `:block` expression" runtests(; test_end_expr=:(@assert false))
+end
+
 end # integrationtests.jl testset