-
Notifications
You must be signed in to change notification settings - Fork 2
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
325-refactor-model_binding-in-get-student_photo-endpoint #334
base: main
Are you sure you want to change the base?
325-refactor-model_binding-in-get-student_photo-endpoint #334
Conversation
…into 325-refactor-model_binding-in-get-student_photo-endpoint
…into 325-refactor-model_binding-in-get-student_photo-endpoint
… to model binding
…into 325-refactor-model_binding-in-get-student_photo-endpoint
|
||
} | ||
|
||
public function testCanInstantiateAnUser(): void |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove the fixtures tests (only check the instance of what you are testing, if this has sense)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review the proposed changes
} | ||
|
||
public function test_can_return_a_500_on_internal_server_error(): void | ||
public function testCanReturns_404WithInvalidStudentUuid(): void |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
beware the underscore
We want to apply Model Binding to the GET 'student.photo.get' endpoint