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

Changing tests to not use dafny_0 and /countVerificationErrors:0 #3160

Merged
merged 123 commits into from
Dec 9, 2022

Conversation

davidcok
Copy link
Collaborator

@davidcok davidcok commented Dec 8, 2022

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.

davidcok and others added 30 commits January 29, 2021 12:41
@davidcok davidcok added the run-deep-tests Tells CI to run all tests label Dec 8, 2022
Copy link
Member

@MikaelMayer MikaelMayer left a comment

Choose a reason for hiding this comment

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

I think one test's output is problematic, otherwise, great addition to make tests stricter !

Test/errortests/InvalidArgument1.dfy.expect Outdated Show resolved Hide resolved
Test/git-issues/github-issue-2563.dfy Show resolved Hide resolved
Test/lit.site.cfg Show resolved Hide resolved
Scripts/test-exit.py Outdated Show resolved Hide resolved
Scripts/test-exit.py Show resolved Hide resolved
Copy link
Member

@robin-aws robin-aws left a comment

Choose a reason for hiding this comment

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

I especially like that some tests had %dafny_0 directly replaced by %dafny, meaning they didn't need the former in the first place. :)

@davidcok davidcok enabled auto-merge (squash) December 8, 2022 23:47
@davidcok davidcok merged commit f981089 into dafny-lang:master Dec 9, 2022
@davidcok davidcok deleted the cok-2487E branch December 9, 2022 00:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
run-deep-tests Tells CI to run all tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants