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.

Sign up for GitHub

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

Jump to bottom

Reconsider Integration Testing #515

Open
knoppiks opened this issue Jan 9, 2025 · 0 comments · May be fixed by #516
Open

Reconsider Integration Testing #515

knoppiks opened this issue Jan 9, 2025 · 0 comments · May be fixed by #516
Assignees
@knoppiks
Labels
aspect/Tests Everything related to testing (unit, integration, etc.) kind/Refactoring Code changes without altering behavior, focused on improvement
Milestone
5.2

Comments

@knoppiks
Copy link
Collaborator

knoppiks commented Jan 9, 2025

Integration testing in it's current form is very cumbersome to implement, error prone.

  • Our CDA and RDA ITs are kind-of e2e tests
  • They may not contribute to test coverage in a way that corresponds to the implementation efforts
  • We need at least some ITs of the API to check auth configuration
@knoppiks knoppiks added kind/Refactoring Code changes without altering behavior, focused on improvement aspect/Tests Everything related to testing (unit, integration, etc.) labels Jan 9, 2025
@knoppiks knoppiks self-assigned this Jan 9, 2025
@knoppiks knoppiks linked a pull request Jan 9, 2025 that will close this issue
Reconsider Integration Testing #516
Draft
@knoppiks knoppiks linked a pull request Jan 9, 2025 that will close this issue
Draft
@knoppiks knoppiks added this to the 5.2 milestone Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@knoppiks knoppiks

Labels
aspect/Tests Everything related to testing (unit, integration, etc.) kind/Refactoring Code changes without altering behavior, focused on improvement
Projects
None yet
Milestone
5.2
Development

Successfully merging a pull request may close this issue.

Reconsider Integration Testing medizininformatik-initiative/fts-next
1 participant
@knoppiks