Skip to content

Make YAML serialization test format-independent - #53

Open
vaishali824 wants to merge 1 commit into
INCF:masterfrom
vaishali824:fix/yaml-serialization-test
Open

vaishali824 wants to merge 1 commit into
INCF:masterfrom
vaishali824:fix/yaml-serialization-test

Conversation

@vaishali824

Copy link
Copy Markdown

Summary

Make test_flat_body resilient to YAML formatting differences across PyYAML versions. The YAML case now compares the parsed NineML objects rather than serialized text; XML and JSON serialization checks remain unchanged.

Validation

  • The focused test passed with the installed YAML block-style output.
  • The focused test also passed with simulated compact (flow-style) YAML output.
  • JSON serialization was excluded during these focused runs because the existing code passes the removed encoding argument on Python 3.14 (tracked separately in Fix reading JSON on Python 3.9+ (removed encoding kwarg) #52).

Compare parsed NineML objects rather than serialized YAML text, which can differ between YAML library versions.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

1 participant