Merge pull request #255 from Sbozzolo/master

Throw informative error for unsupported vtype
JuliaGeo · Apr 10, 2024 · 4793a53 · 4793a53
2 parents d85c6ca + 8f452ad
commit 4793a53
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/src/cfvariable.jl b/src/cfvariable.jl
@@ -114,6 +114,7 @@ function defVar(ds::NCDataset,name::SymbolOrString,vtype::DataType,dimnames;
             typeid = nc_def_vlen(ds.ncid, typename, ncType[eltype(vtype)])
         else
             # base-type
+            haskey(ncType, vtype) || error("$vtype not supported")
             ncType[vtype]
         end
 

diff --git a/test/test_writevar.jl b/test/test_writevar.jl
@@ -12,6 +12,8 @@ ds = NCDataset(filename,"c")
 defDim(ds,"lon",sz[1])
 defDim(ds,"lat",sz[2])
 
+# vartype not supported
+@test_throws ErrorException defVar(ds,"var-DT",DateTime,("lon","lat"))
 
 # variables
 for T in [UInt8,Int8,UInt16,Int16,UInt32,Int32,UInt64,Int64,Float32,Float64]