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

Run tests using why3find #1255

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Run tests using why3find #1255

wants to merge 1 commit into from

Conversation

Lysxia
Copy link
Collaborator

@Lysxia Lysxia commented Nov 26, 2024

Sequel to #1205

Run tests with why3find by default.

Some test cases must still be run with why3 because they use ad hoc transformations, they are indicated by a new header WHY3PROVE.

There is one test case that requires inline_goal (iterators/10_once.rs), enabled with the header TACTIC +inline_goal.

@Lysxia Lysxia force-pushed the why3find-tests branch 6 times, most recently from 6cbd65d to 0e81f66 Compare November 28, 2024 12:45
@Lysxia
Copy link
Collaborator Author

Lysxia commented Nov 28, 2024

There is still a detail I need to figure out about how why3find records prover times. I asked about it on the why3find issue tracker: https://git.frama-c.com/pub/why3find/-/issues/117

@jhjourdan
Copy link
Collaborator

We should update .gitattribute in order to not show diffs of the json files, just as is already done for xml files.

@jhjourdan
Copy link
Collaborator

Also, now that we are using Why3find, which is supposed to be more stable than why3 wrt. proof replay failure, would it be relevant to remove the "UNSTABLE" annotation of some tests? (Recall: this annotation tells CI to skip these tests, because they are too unstable.)

@xldenis ; anything to say?

@xldenis
Copy link
Collaborator

xldenis commented Nov 28, 2024

I would love to remove that annotation if it does become more stable!

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.

3 participants