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

Upgrade test suite to ui_test #1417

Open
2 tasks
fpoli opened this issue Jun 26, 2023 · 2 comments
Open
2 tasks

Upgrade test suite to ui_test #1417

fpoli opened this issue Jun 26, 2023 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@fpoli
Copy link
Member

fpoli commented Jun 26, 2023

We should upgrade the test suite, replacing compiletest-rs with the new and less error-prone ui_test crate. A good example to start from is here.

While doing so, I believe that we should

  • Merge all verify/verify_overflow/... folders into one, adding //@compile-flags: ... headers where needed. The separate folders were originally needed only because Prusti didn't support the -P... configuration flags that we now have.
  • Merge all ui/pass/fail folders into one, adding //@check-pass and //@run headers where needed. This makes it easier to group and maintain related tests, because we no longer have to duplicate the folder structure under both pass and fail.
@fpoli fpoli added enhancement New feature or request good first issue Good for newcomers labels Jun 26, 2023
@fpoli
Copy link
Member Author

fpoli commented Jun 26, 2023

ui_test might also be used to test the crates in prusti-tests/tests/cargo_verify/, replacing cargotest.rs. We should check.

@fpoli
Copy link
Member Author

fpoli commented Oct 27, 2023

This is blocked by oli-obk/ui_test#147, because we need to test on Windows.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant