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

feat: integration tests, builtin tests, backtracking #35

Merged
merged 3 commits into from
Apr 25, 2024
Merged

Conversation

guybedford
Copy link
Contributor

@guybedford guybedford commented Apr 23, 2024

This adds a new integration test runner variant, which takes a single multi-route component and then tests it against the expectations using the same expectation checking system.

Integration tests for the base64 builtins are then ported.

In addition, proper segment handling is added for path resolution in the loader, to now support backtracking.

Updates the CTests to run in parallel, supporting port assignment for running multiple servers at once. Also updates the CI to run tests in parallel.

This was referenced Apr 24, 2024
Copy link
Member

@tschneidereit tschneidereit left a comment

Choose a reason for hiding this comment

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

This LGTM, thanks! I asked for one change, but r=me with that addressed.

runtime/script_loader.cpp Outdated Show resolved Hide resolved
@guybedford guybedford merged commit d3ae487 into main Apr 25, 2024
1 check passed
@guybedford guybedford deleted the builtin-tests branch April 25, 2024 17:33
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