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

Structure/Organize tests section #100

Closed
lu-pl opened this issue Oct 18, 2024 · 1 comment · Fixed by #109
Closed

Structure/Organize tests section #100

lu-pl opened this issue Oct 18, 2024 · 1 comment · Fixed by #109
Assignees
Labels

Comments

@lu-pl
Copy link
Contributor

lu-pl commented Oct 18, 2024

The tests section needs urgent restructuring/cleaning up.

Currently the tests folder structure (and e.g. also tests.utils._types) is tightly coupled with instantiate_model_from_bindings tests.

@lu-pl lu-pl self-assigned this Oct 18, 2024
@lu-pl lu-pl added the tests label Oct 22, 2024
lu-pl added a commit that referenced this issue Oct 29, 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 (loosely) mirror the
rdfproxy package structure, e.g. sparql_utils should be tested in
tests_sparql_utils etc.

Test types (unit, integration, end2end, etc) should be indicated in
docstrings or a separate tests README.

Closes #100
@lu-pl
Copy link
Contributor Author

lu-pl commented Nov 3, 2024

Consider separating the test suite into unit, integration and end2end tests.

What would be the exact distinction between integration and end2end testing for rdfproxy?

lu-pl added a commit that referenced this issue Nov 5, 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 indiciated in the test module doc string.

Closes #100
lu-pl added a commit that referenced this issue Nov 5, 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 added a commit that referenced this issue Nov 6, 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 closed this as completed in #109 Nov 6, 2024
@lu-pl lu-pl closed this as completed in 2b6ff37 Nov 6, 2024
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 a pull request may close this issue.

1 participant