Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revamp test suite. #59

Merged
merged 22 commits into from
Feb 12, 2024
Merged

Revamp test suite. #59

merged 22 commits into from
Feb 12, 2024

Conversation

LaurenzV
Copy link
Collaborator

This PR makes the following changes:

  • General cleanup of tests (remove unused reference images, remove test cases that don't exist in resvg, add all remaining resvg test cases without reference images)
  • Instead of using a custom test runner, we generate tests using Python. While it removes some flexibility in how we run the tests, it allows us to use cargo's native test runner, which has many advantages, such as being more consistent overall and also not quitting instantly if a test case panics.

@LaurenzV LaurenzV changed the title Revamp test suite Revamp test suite. Feb 12, 2024
@LaurenzV LaurenzV merged commit 78d14ef into typst:main Feb 12, 2024
2 checks passed
@LaurenzV LaurenzV deleted the test-suite branch February 12, 2024 13:56
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