Closed bas-inh closed 2 years ago
Heb je daar ook een test voor? Bijvoorbeeld een cucumber? Welke branch hebben we het over?
Nee we hebben nog geen tests, we gaan eerst de code verbeteren en de feedback verwerken. Het staat op mijn branch, die heeft mijn naam, Bas, als naam.
De enige files die er mee te maken hebben zijn de User entity en UserApiController denk ik.
Ik heb ook nog nooit via een GitHub issue gewerkt, dus kan zijn dat ik niet weet wat je gewent bent.
Ik stel voor dat je eerst een cucumber test schrijft. Dat moet niet al te moeilijk zijn. Die gaat zeker falen, maar dan kun je wel met debug doorheen stappen om te kijken waar het fout gaat. En als het dan oplost, slaagt die test. Probeer die aanpak eens. Bovendien als je er niet uitkomt en ik er alsnog naar moet kijken, dan heb ik zelf ook een middel om de situatie te triggeren...
I have not received a comment in 5 days. I will close this issue now.
Ik krijg de bovenstaande exception maar ik kan hem zelf niet opvangen in een catch blok. Dit gebeurt tijdens het aanmaken van Users die al bestaande informatie bevatten, waar ook een constraint op staat via de User Entity (username, email en phone nr).
Het is de bedoeling dat ik deze exception vang en afhandel in een try-catch en aan de frontend een HTTP Status Code en een bericht meegeef.