Feature/project tests #48
Conversation
not yet tested, might have some mistakes
Feature/savefiles
Feature/role interceptor
|
|
||
|
|
||
| // Hulpfunctie om de testen over te zetten naar de server en de database op de correcte plaats | ||
| private long saveTest(MultipartFile file, long projectId, long userId) throws IOException { |
There was a problem hiding this comment.
Misschien overdrijf ik ma kvind het persoonlijk cleaner om zo'n functies in een aparte file te zetten, mss kunt ge deze ook verplaatsen naar util/filehandler?
There was a problem hiding this comment.
Ik had problemen om een fileRepository te gebruiken binnen de Filehandler, daarom dat ik het in de klasse zelf deed.
There was a problem hiding this comment.
yeah ik heb die niet gebruikt binnen die Filehandler, je moet maar eens kijken anders naar die submission hoe ik da gedaan heb. Moogt ook gerust zeggen als je da nie beter vindt
|
|
||
| long userId = 1L; //TODO: replace with id of current user | ||
| if(!projectRepository.userPartOfProject(projectId, userId)){ | ||
| return ResponseEntity.status(HttpStatus.FORBIDDEN).body("You aren't part of this project"); |
There was a problem hiding this comment.
Hier moet denkik nog gecheckt worden of de user wel juiste rol heeft binnen project (op deze manier kan een student denkik ook de testen aanpassen). Ma mss kan da met Wout zijn decorater gemakkelijk
There was a problem hiding this comment.
Was ik totaal vergeten
There was a problem hiding this comment.
Met decorator is het inderdaad gemakkelijker, dan heb je ook de UserEntity object voor userId. Voorbeeld:
|
Ik ga alles correct maken en dan opnieuw pushen |
|
De base is nu dev ipv main |
Er is een Putmapping gemaakt van project testen, Alles zou correct moeten opgeslagen worden, zowel de file_id's op de database als de bestanden op de server.
Ik heb dit zelf al eens getest, maar een extra review kan geen kwaad.
Dit is het einde van #46