Skip to content

Fix failing local Windows tests due to ruamel parsing#3275

Merged
MMathisLab merged 1 commit intomainfrom
cy/fix-test-ruamel-parsing
Apr 12, 2026
Merged

Fix failing local Windows tests due to ruamel parsing#3275
MMathisLab merged 1 commit intomainfrom
cy/fix-test-ruamel-parsing

Conversation

@C-Achard
Copy link
Copy Markdown
Collaborator

@C-Achard C-Achard commented Apr 8, 2026

Update multianimal project fixture to produce stable YAML: import ruamel.yaml.scalarstring.SingleQuotedScalarString and use it for the video_sets key (posix path), set yaml width to avoid unwanted wrapping, and use project_dir.as_posix() for project_path.

Also write config.yaml with explicit UTF-8 encoding and ensure crop is a string.
These changes make YAML dumping deterministic and platform-independent for the test.

Update multianimal project fixture to produce stable YAML: import ruamel.yaml.scalarstring.SingleQuotedScalarString and use it for the video_sets key (posix path), set yaml width to avoid unwanted wrapping, and use project_dir.as_posix() for project_path. Also write config.yaml with explicit UTF-8 encoding and ensure crop is a string. These changes make YAML dumping deterministic and platform-independent for the test.
@C-Achard C-Achard self-assigned this Apr 8, 2026
@C-Achard C-Achard added the tests label Apr 8, 2026
@C-Achard
Copy link
Copy Markdown
Collaborator Author

C-Achard commented Apr 9, 2026

(Similar to #3140, but in tests)

@C-Achard C-Achard marked this pull request as ready for review April 9, 2026 13:19
@C-Achard C-Achard requested a review from deruyter92 April 9, 2026 13:19
Copy link
Copy Markdown
Collaborator

@deruyter92 deruyter92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good fix!

@MMathisLab MMathisLab merged commit 45fe7ce into main Apr 12, 2026
28 of 31 checks passed
@MMathisLab MMathisLab deleted the cy/fix-test-ruamel-parsing branch April 12, 2026 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants