Skip to content

unit-tests: add test to NCZMD_set_metadata_handler - #3254

Draft
mannreis wants to merge 11 commits into
Unidata:mainfrom
mannreis:zarr-csl-unit-test
Draft

unit-tests: add test to NCZMD_set_metadata_handler#3254
mannreis wants to merge 11 commits into
Unidata:mainfrom
mannreis:zarr-csl-unit-test

Conversation

@mannreis

Copy link
Copy Markdown
Contributor

Simple unit test to exercise the NCZMD_set_metadata_handler logic and fallback on the metadata handler selection.

@WardF

WardF commented Feb 11, 2026

Copy link
Copy Markdown
Member

@mannreis I'm not certain what the source of the issue is, but I can dig in and take a look. I'll convert this to a draft for the time being.

@WardF
WardF marked this pull request as draft February 11, 2026 23:04
@mannreis

Copy link
Copy Markdown
Contributor Author

I found the problem with respect to the MSVC build but the Docker regression testing I can't yet understand.

@edhartnett

Copy link
Copy Markdown
Contributor

Try merging in the changes from #3262 and see if that resolves this.

@mannreis
mannreis force-pushed the zarr-csl-unit-test branch from 1b32117 to 940846b Compare March 17, 2026 10:10
@mannreis

mannreis commented Mar 17, 2026

Copy link
Copy Markdown
Contributor Author

I'm not understanding why the docker regression tests are failing on the test added by this PR. It is passing in other workflows. Help is welcomed!


On Mac the test passes: https://github.com/mannreis/netcdf-c/actions/runs/23185334066/job/67369700395?pr=8#step:19:4647
On MSVC as well: https://github.com/mannreis/netcdf-c/actions/runs/23185333987/job/67367298326?pr=8#step:19:384
But on docker fails: https://github.com/mannreis/netcdf-c/actions/runs/23185333994/job/67367298258?pr=8#step:4:4891

@WardF

WardF commented Jul 15, 2026

Copy link
Copy Markdown
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants