fix(CI): Inkscape not available in CI, add check to tests

kunzaatko · Jan 9, 2025 · 50e2bde · 50e2bde
1 parent da60540
commit 50e2bde
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/src/save-fig.jl b/src/save-fig.jl
@@ -141,10 +141,8 @@ Return the set of formats to generate based on the given `backends` required `fo
 
 # Examples
 ```jldoctest
-julia> MakieMaestro.get_formats([CairoMakie], Set([MakieMaestro.PdfTex])) # Added Svg so the PdfTex is possible to create
-2-element Vector{MakieMaestro.Format}:
- Svg::Format = 1
- PdfTex::Format = 4
+julia> (Sys.which("inkscape") === nothing) || MakieMaestro.Svg ∈ MakieMaestro.get_formats([CairoMakie], Set([MakieMaestro.PdfTex])) # Added Svg so the PdfTex is possible to create
+true
 
 julia> MakieMaestro.get_formats([GLMakie], Set([MakieMaestro.Pdf]))
 ERROR: ArgumentError: None of the backends Module[GLMakie] support the formats Set(MakieMaestro.Format[MakieMaestro.Pdf])

diff --git a/test/runtests.jl b/test/runtests.jl
@@ -20,7 +20,7 @@ include("./tools.jl")
     @testset "DocTests" begin
         # NOTE: Show for `Unitful.jl` does nm⁻¹ on macOS and nm^-1 on Linux. This is necessary, since the `jldoctest` is only one
         if !haskey(ENV, "GITHUB_ACTIONS") ||
-            haskey(ENV, "RUNNER_OS") && ENV["RUNNER_OS"] == "Linux"
+           haskey(ENV, "RUNNER_OS") && ENV["RUNNER_OS"] == "Linux"
             # NOTE: Better than doc-testing in `make.jl` because, I can track the coverage
             # NOTE: When updating, must update also in `docs/make.jl` and  `test/fix_doctests.jl`<18-12-24> 
             DocMeta.setdocmeta!(
@@ -76,9 +76,11 @@ include("./tools.jl")
         @testset "theme" begin end
         @testset "`savefig`" begin
             @test MakieMaestro.get_formats([CairoMakie], Set([MakieMaestro.Png])) ==
-                [MakieMaestro.Png] # Allowed formats
-            @test MakieMaestro.Svg in
-                MakieMaestro.get_formats([CairoMakie], Set([MakieMaestro.PdfTex])) # Added necessary Svg format
+                  [MakieMaestro.Png] # Allowed formats
+            if Sys.which("inkscape") !== nothing
+                @test MakieMaestro.Svg in
+                      MakieMaestro.get_formats([CairoMakie], Set([MakieMaestro.PdfTex])) # Added necessary Svg format
+            end
             @test_throws ArgumentError MakieMaestro.get_formats(
                 [GLMakie], Set([MakieMaestro.Pdf])
             )