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): refactor test folder structure #109

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

lu-pl
Copy link
Contributor

@lu-pl lu-pl commented Oct 23, 2024

This branch implements a restructuring of the test suite.

Rationale: Over-modularization of test parameter definitions clutters up the test
suite and leads to lengthy/repetitive import statements.

For now, the structure of the test suite should only differentiate

  1. unit tests and
  2. all other tests.

Additional test categories may be introduced at a later point.

The test category should be indicated in the test module doc string.

Closes #100

@lu-pl lu-pl marked this pull request as draft October 23, 2024 13:15
@lu-pl lu-pl force-pushed the lupl/restructure-tests branch 7 times, most recently from 7974e28 to 25acb21 Compare October 29, 2024 09:17
@lu-pl lu-pl marked this pull request as ready for review October 29, 2024 09:36
@lu-pl lu-pl requested a review from b1rger October 29, 2024 09:36
@lu-pl lu-pl marked this pull request as draft November 3, 2024 16:16
@lu-pl lu-pl force-pushed the lupl/restructure-tests branch 2 times, most recently from f8d44fc to 51f8e53 Compare November 5, 2024 08:14
@lu-pl lu-pl marked this pull request as ready for review November 5, 2024 08:16
This branch implements a restructuring of the test suite.

Rationale: Over-modularization of test parameter definitions clutters up the test
suite and leads to lengthy/repetitive import statements.

For now, the structure of the test suite should only differentiate
1. unit tests and 2. all other tests. Additional test categories may
be introduced at a later point.

The test category should be indicated in the test module doc string.

Closes #100
@lu-pl lu-pl force-pushed the lupl/restructure-tests branch from 51f8e53 to 5e14939 Compare November 6, 2024 09:39
@lu-pl lu-pl merged commit 2b6ff37 into main Nov 6, 2024
9 checks passed
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.

Structure/Organize tests section
2 participants