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

Rename tests to correspond to use case requirements rather than the methods tested. #41

Open
BjoernKW opened this issue Sep 2, 2022 · 0 comments
Labels
task Anything related to course tasks testing Anything test-related

Comments

@BjoernKW
Copy link
Collaborator

BjoernKW commented Sep 2, 2022

Currently, most of the test methods in the sample solution are named like the methods they test, e.g., we have a test method getAllBooks() for testing a controller method getAllBooks().

Something more in line with user requirements, such as endpointShouldReturnAListOfBooks would be more explicit and contribute to a more test-driven approach as well as more explicitly document the desired behaviour.

@BjoernKW BjoernKW added testing Anything test-related task Anything related to course tasks labels Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
task Anything related to course tasks testing Anything test-related
Projects
None yet
Development

No branches or pull requests

1 participant