Add tests for PrescribedOcean, ERA5PrescribedAtmosphere, and GLORYS Column

- PrescribedOcean: add display/summary and net_fluxes tests
- ERA5PrescribedAtmosphere: add construction test in test_cds_downloading
- GLORYS: expand test_glorys_downloading with Column Nearest/Linear
  downloads, BoundingBox Field construction
- CI: add test_glorys_downloading to Data Downloading workflow

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: glwagner

.github/workflows/ci.yml

@@ -189,7 +189,7 @@ jobs:
               using Pkg;
               Pkg.test(; coverage=true,
                         julia_args=["--check-bounds=yes", "--compiled-modules=yes", "-O0"],
-                        test_args=["--verbose", "test_cds_downloading", "test_downloading"])
+                        test_args=["--verbose", "test_cds_downloading", "test_downloading", "test_glorys_downloading"])
           '
       - uses: julia-actions/julia-processcoverage@v1
       - uses: codecov/codecov-action@v5

test/test_cds_downloading.jl

@@ -7,6 +7,7 @@ using NCDatasets
 using NumericalEarth.DataWrangling.ERA5
 using NumericalEarth.DataWrangling.ERA5: ERA5Hourly, ERA5Monthly, ERA5_dataset_variable_names
 using NumericalEarth.DataWrangling: metadata_path, download_dataset
+using NumericalEarth.Atmospheres: PrescribedAtmosphere
 
 # Test date: Kyoto Protocol ratification date, February 16, 2005
 start_date = DateTime(2005, 2, 16, 12)
@@ -195,3 +196,24 @@ start_date = DateTime(2005, 2, 16, 12)
         end
     end
 end
+
+@testset "ERA5PrescribedAtmosphere" begin
+    for arch in test_architectures
+        A = typeof(arch)
+
+        @testset "Construction with BoundingBox on $A" begin
+            region = NumericalEarth.DataWrangling.BoundingBox(longitude=(0, 5), latitude=(40, 45))
+            atmos = ERA5PrescribedAtmosphere(arch;
+                                             start_date = DateTime(2005, 2, 16),
+                                             end_date = DateTime(2005, 2, 16, 6),
+                                             region)
+
+            @test atmos isa PrescribedAtmosphere
+            @test length(atmos.times) > 0
+            @test atmos.tracers.T isa FieldTimeSeries
+            @test atmos.tracers.q isa FieldTimeSeries
+            @test atmos.velocities.u isa FieldTimeSeries
+            @test atmos.velocities.v isa FieldTimeSeries
+        end
+    end
+end

test/test_glorys_downloading.jl

@@ -1,15 +1,54 @@
 include("runtests_setup.jl")
 
 using CopernicusMarine
+using NumericalEarth.DataWrangling: metadata_path, download_dataset,
+                                    BoundingBox, Column, Nearest, Linear
+using NumericalEarth.DataWrangling.GLORYS: GLORYSDaily
 
 @testset "Downloading GLORYS data" begin
     variables = (:temperature, :salinity, :u_velocity, :v_velocity)
-    region = NumericalEarth.DataWrangling.BoundingBox(longitude=(200, 202), latitude=(35, 37))
-    dataset = NumericalEarth.DataWrangling.GLORYS.GLORYSDaily()
-    for variable in variables
-        metadatum = Metadatum(variable; dataset, region)
-        filepath = NumericalEarth.DataWrangling.metadata_path(metadatum)
+    region = BoundingBox(longitude=(200, 202), latitude=(35, 37))
+    dataset = GLORYSDaily()
+
+    @testset "BoundingBox download" begin
+        for variable in variables
+            metadatum = Metadatum(variable; dataset, region)
+            filepath = metadata_path(metadatum)
+            isfile(filepath) && rm(filepath; force=true)
+            download_dataset(metadatum)
+            @test isfile(filepath)
+        end
+    end
+
+    @testset "Column Nearest download" begin
+        col = Column(201.0, 36.0; interpolation=Nearest())
+        metadatum = Metadatum(:temperature; dataset, region=col)
+        filepath = metadata_path(metadatum)
+        isfile(filepath) && rm(filepath; force=true)
+        download_dataset(metadatum)
+        @test isfile(filepath)
+        rm(filepath; force=true)
+    end
+
+    @testset "Column Linear download" begin
+        col = Column(201.0, 36.0; interpolation=Linear())
+        metadatum = Metadatum(:temperature; dataset, region=col)
+        filepath = metadata_path(metadatum)
         isfile(filepath) && rm(filepath; force=true)
-        NumericalEarth.DataWrangling.download_dataset(metadatum)
+        download_dataset(metadatum)
+        @test isfile(filepath)
+        rm(filepath; force=true)
+    end
+
+    for arch in test_architectures
+        A = typeof(arch)
+        @testset "GLORYS Field with BoundingBox on $A" begin
+            metadatum = Metadatum(:temperature; dataset, region)
+            filepath = metadata_path(metadatum)
+            isfile(filepath) || download_dataset(metadatum)
+            field = Field(metadatum, arch)
+            @test field isa Field
+            @allowscalar @test any(!=(0), interior(field))
+        end
     end
 end

test/test_prescribed_ocean.jl

@@ -30,6 +30,20 @@ include("runtests_setup.jl")
             end
         end
 
+        @testset "Display and net_fluxes on $A" begin
+            grid = RectilinearGrid(arch; size = (), topology = (Flat, Flat, Flat))
+            ocean = PrescribedOcean(grid, NamedTuple())
+
+            str = summary(ocean)
+            @test occursin("PrescribedOcean", str)
+
+            buf = IOBuffer()
+            show(buf, ocean)
+            @test occursin("PrescribedOcean", String(take!(buf)))
+
+            @test NumericalEarth.EarthSystemModels.InterfaceComputations.net_fluxes(ocean) === nothing
+        end
+
         @testset "EarthSystemModel interface on $A" begin
             grid = RectilinearGrid(arch; size = (), topology = (Flat, Flat, Flat))