Skip to content

Add test for image roundtrip#971

Merged
DJMcNab merged 5 commits intolinebender:mainfrom
sagudev:preserve-image-test
May 7, 2025
Merged

Add test for image roundtrip#971
DJMcNab merged 5 commits intolinebender:mainfrom
sagudev:preserve-image-test

Conversation

@sagudev
Copy link
Contributor

@sagudev sagudev commented May 6, 2025

Repro for #972. It's currently set to should fail as fixup is still in the works.

Why that particular image? Because it makes vello not work (due too math errors). It was generated by WPT (while testing servo) and has encoded JSON about cross origin stuff.

Signed-off-by: sagudev <16504129+sagudev@users.noreply.github.com>
sagudev added 2 commits May 6, 2025 21:19
Signed-off-by: sagudev <16504129+sagudev@users.noreply.github.com>
Signed-off-by: sagudev <16504129+sagudev@users.noreply.github.com>
@sagudev sagudev requested review from DJMcNab May 6, 2025 19:41
Signed-off-by: sagudev <16504129+sagudev@users.noreply.github.com>
Copy link
Member

@DJMcNab DJMcNab left a comment

Choose a reason for hiding this comment

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

Sorry to keep coming back with nits - this should be the last one.

This test should actually be in known_issues.rs, not regression.rs. Once it becomes fixed (and stops panicking), we'd move it to regression.rs.

Signed-off-by: sagudev <16504129+sagudev@users.noreply.github.com>
Copy link
Member

@DJMcNab DJMcNab left a comment

Choose a reason for hiding this comment

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

Thank you!

@DJMcNab DJMcNab added this pull request to the merge queue May 7, 2025
Merged via the queue into linebender:main with commit 349117d May 7, 2025
17 checks passed
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.

2 participants