better test support for Julia <1.9

beacon-biosignals · Aug 19, 2023 · 9a447a9 · 9a447a9
1 parent ea08c3d
commit 9a447a9
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 12 deletions.
diff --git a/src/regressionmodel.jl b/src/regressionmodel.jl
@@ -147,6 +147,9 @@ end
 function _difference_method!(::Vector{T}, ::Vector{T}, 
                              ::RegressionModel, 
                              ::AutoInvLink) where {T <: AbstractFloat}
+    @static if VERSION < v"1.9"
+        @error "AutoInvLink requires extensions and is thus not available on Julia < 1.9."
+    end
     throw(ArgumentError("No appropriate extension is loaded for automatic " * 
                         "determination of the inverse link for this model type"))
 end

diff --git a/test/delta_method.jl b/test/delta_method.jl
@@ -40,6 +40,8 @@ using Test
     @test isapprox(only(eff_emm_trans.upper), 47.5; atol=0.05)
 
     @testset "AutoInvLink fails gracefully" begin
+        # this should work even pre Julia 1.9 because by definition 
+        # no extension is loaded
         @test_throws ArgumentError effects(design, model; invlink=AutoInvLink())
     end
 end
@@ -52,7 +54,11 @@ end
                  :Neuroticism => [16])
     X = [1.0 13.0 16.0 13 * 16]
     iv = Base.Fix1(GLM.linkinv, Link(model.model))
-    invlinks = [iv, AutoInvLink()]
+    @static if VERSION >= v"1.9"
+        invlinks = [iv, AutoInvLink()]
+    else
+        invlinks = [iv]
+    end
     @testset "invlink = $invlink" for invlink in invlinks
         for level in [0.68, 0.95]
             eff = effects(design, model; invlink, level)
@@ -95,17 +101,19 @@ end
     end
 end
 
-@testset "link function in a MixedModel" begin
-    model = fit(MixedModel, 
-                @formula(use ~ 1 + age + (1|urban)), 
-                MixedModels.dataset(:contra), 
-                Bernoulli(); progress=false)
-    design = Dict(:age => -10:10)
-    eff_manual = effects(design, model; 
-                         invlink=Base.Fix1(GLM.linkinv, Link(model)))
-    eff_auto = effects(design, model; invlink=AutoInvLink())
+@static if VERSION >= v"1.9"
+    @testset "link function in a MixedModel" begin
+        model = fit(MixedModel, 
+                    @formula(use ~ 1 + age + (1|urban)), 
+                    MixedModels.dataset(:contra), 
+                    Bernoulli(); progress=false)
+        design = Dict(:age => -10:10)
+        eff_manual = effects(design, model; 
+                            invlink=Base.Fix1(GLM.linkinv, Link(model)))
+        eff_auto = effects(design, model; invlink=AutoInvLink())
 
-    @test all(isapprox.(Matrix(eff_manual), Matrix(eff_auto)))
+        @test all(isapprox.(Matrix(eff_manual), Matrix(eff_auto)))
+    end
 end
 
 @testset "identity by another name" begin

diff --git a/test/runtests.jl b/test/runtests.jl
@@ -3,7 +3,7 @@ using Effects
 using Test
 
 @testset "Aqua" begin
-    Aqua.test_all(Effects; ambiguities=false)
+    Aqua.test_all(Effects; ambiguities=false, project_toml_formatting)
 end
 
 @testset "TypicalTerm" begin