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

Clean up Tact's testsuite #346

Closed
anton-trunov opened this issue May 19, 2024 · 0 comments · Fixed by #402
Closed

Clean up Tact's testsuite #346

anton-trunov opened this issue May 19, 2024 · 0 comments · Fixed by #402
Assignees
Milestone

Comments

@anton-trunov
Copy link
Member

anton-trunov commented May 19, 2024

For instance, the grammar and typechecker tests should have suggestive names instead of case-XX.tact. The test file names should reflect the feature they test. If there is a lot of tests for one feature, we should consider grouping those into folders.

This will also help with merge conflicts, because name clashes between tests will be less likely for PRs that add different features.

@anton-trunov anton-trunov self-assigned this Jun 10, 2024
@anton-trunov anton-trunov added this to the v1.4.0 milestone Jun 10, 2024
@anton-trunov anton-trunov changed the title Tests should have suggestive names instead of case-XX.tact Clean up Tact's testsuite Jun 11, 2024
@anton-trunov anton-trunov mentioned this issue Jun 11, 2024
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant