The assignment says that if a video has not yet been uploaded, but meta-data has been created, it should respond with a 404 response code.
If no mpeg data has been uploaded for the specified video, then the server should return a 404 status code.
However, Spring does not by default associate a 404 response with the FileNotFoundException, which is thrown by the VideoFileManager. Therefore a 500 response will be generated if one doesn't specifically handle this with a handler or try-catch. The tests do not check that they get a 404 for missing uploads, only for invalid video IDs.
I suggest adding an additional test to future iterations of the course, to ensure the programmer has properly handled this either by adding a handler or by try-catching.
The assignment says that if a video has not yet been uploaded, but meta-data has been created, it should respond with a 404 response code.
However, Spring does not by default associate a 404 response with the FileNotFoundException, which is thrown by the VideoFileManager. Therefore a 500 response will be generated if one doesn't specifically handle this with a handler or try-catch. The tests do not check that they get a 404 for missing uploads, only for invalid video IDs.
I suggest adding an additional test to future iterations of the course, to ensure the programmer has properly handled this either by adding a handler or by try-catching.