Fix cellarea tests

+
rafaqz · Oct 11, 2024 · bcd7fb1 · bcd7fb1
1 parent a07be84
commit bcd7fb1
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/test/cellarea.jl b/test/cellarea.jl
@@ -75,10 +75,10 @@ include(joinpath(dirname(pathof(Rasters)), "../test/test_utils.jl"))
     @testset "Unitful cellarea" begin
          # Case 1: cellarea with unitful manifold
          # This is the simplest case
-         unitful_manifold = Spherical(; radius = 6.3710088u"m")
+         unitful_manifold = Spherical(; radius = Spherical().radius * Unitful.u"m")
          unitful_cellarea = cellarea(unitful_manifold, dimz_3857)
          @test unitful_cellarea isa Raster{<:Quantity}
-         @test Unitful.ustrip.(unitful_cellarea) == cellarea(Spherical(; radius = unitful_manifold.radius |> Unitful.ustrip), dimz_3857)
+         @test Unitful.ustrip.(parent(unitful_cellarea)) == cellarea(Spherical(; radius = unitful_manifold.radius |> Unitful.ustrip), dimz_3857)
          # Case 2: unitful dimensions
          ux_3857 = rebuild(x_3857; val = rebuild(val(x_3857); data = val(x_3857) .* Unitful.u"m", span = Regular(val(x_3857).span.step * Unitful.u"m")))
          uy_3857 = rebuild(y_3857; val = rebuild(val(y_3857); data = val(y_3857) .* Unitful.u"m", span = Regular(val(y_3857).span.step * Unitful.u"m")))
@@ -88,6 +88,7 @@ include(joinpath(dirname(pathof(Rasters)), "../test/test_utils.jl"))
          # Unitful lookups shouldn't work without a unitful manifold
          @test_throws Unitful.DimensionError cellarea(unitful_dimz_3857)
          # The tests below fail because Unitful apparently can't convert to Float64...
+         # (see https://github.com/PainterQubits/Unitful.jl/issues/742)
          # But we also have to re-convert back to the original unit type, which cellarea currently
          # doesn't do.