Skip to content

Fix failing local Windows tests due to ruamel parsing - #3275

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

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

Conversation

@C-Achard

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

Copy link
Copy Markdown
Collaborator

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

C-Achard commented Apr 9, 2026

Copy link
Copy Markdown
Collaborator Author

(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

@deruyter92 deruyter92 left a comment

Copy link
Copy Markdown
Collaborator

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
@deruyter92 deruyter92 mentioned this pull request May 21, 2026
1 task
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