Codecov Report

Merging #359 (845498e) into main (4a09859) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #359      +/-   ##
==========================================
- Coverage   79.20%   79.17%   -0.04%     
==========================================
  Files          38       38              
  Lines        3121     3121              
==========================================
- Hits         2472     2471       -1     
- Misses        649      650       +1     

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Tests?

Tests?

I have no idea how to write tests, but I I'll look into it.

Tests are crucial, more important than the code in the long term. Otherwise I or someone else might break this functionality again by accident.

You can see some other tests in the tes/gdal.jl and test/ncdatasets.jl - that might be a good place to add tests where you rename a file to something custom and then force it to the right file type.

Do your best anyway! If not I can add them.

Can you recommend some good tutorial on writing tests in julia? I feel that I need to make myself more familiar with this anyway.

Maybe https://bkamins.github.io/julialang/2022/04/22/testing.html?

or https://itnext.io/organizing-tests-in-julia-c3de1e43d6ab

Ok, making tests turned out to be not so difficult, but I think that they reveal some more bugs. I copy test/data/*.nc files to *.nc4 files and try to create Raster and RasterStack objects from them. First, Raster() works without source=Rasters.NCDfile keyword, and with this keyword throws

ERROR: MethodError: no method matching _open(::Type{Array}, ::Type{Rasters.NCDfile}, ::ArchGDAL.RasterDataset{Float32, ArchGDAL.Dataset}; key=nothing)

This is inconsistent with RasterStack interface. Second, both Raster and RasterStack created from *.nc files are not equal to Raster and RasterStack created from *.nc4 files, they do not pass isequal test. Is such behavior expected?

but I think that they reveal some more bugs

Thats great. Can you push the tests so I can try to find your problems?

Not sure if I am doing the right thing, but here is my attempt.