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

Refactor tests #453

Merged
merged 25 commits into from
May 6, 2022
Merged

Refactor tests #453

merged 25 commits into from
May 6, 2022

Conversation

sjakobi
Copy link
Member

@sjakobi sjakobi commented May 4, 2022

…by moving the test definitions directly into the trees.

Context: #284.

This is also preparation for the addition for more invariant-checking tests (#449).


TODO:

  • HashSet tests
  • More test suites?! Nah

@sjakobi sjakobi added the tests label May 4, 2022
@sjakobi sjakobi force-pushed the sjakobi/284-refactor-tests branch from 25a901f to 5c8c238 Compare May 4, 2022 21:14
@sjakobi sjakobi force-pushed the sjakobi/284-refactor-tests branch from 5a58d14 to 9f97d5a Compare May 6, 2022 00:46
@sjakobi sjakobi force-pushed the sjakobi/284-refactor-tests branch from 45c51bb to 3414dbd Compare May 6, 2022 09:37
@sjakobi sjakobi marked this pull request as ready for review May 6, 2022 09:51
@sjakobi
Copy link
Member Author

sjakobi commented May 6, 2022

I'll spare the other test-suites for now.

In the case of Properties.List I suspect that we won't need to do any changes and can hopefully get rid of them when #155 is implemented.

The regression test cases are mostly a bit larger, so I don't think they'd benefit from being inlined into the test tree.

I'm not very familiar with the strictness tests yet. I'll look at them at a different time.

@sjakobi sjakobi merged commit 4d84602 into master May 6, 2022
@sjakobi sjakobi deleted the sjakobi/284-refactor-tests branch May 6, 2022 10:01
@sjakobi sjakobi mentioned this pull request May 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant