matrex32
commented
1 year ago -Am rezolvat bug ul creand o noua clasa MultipleOxygenAccountException care primeste ca parametru o lista ce are fieldName, mesajul de eroare si id ul mesajului. Dupa aruncarea exceptiei de catre metoda validate voi prinde aceasta exceptie in GlobalExceptionHandler, unde formulez raspunsul JSON. -Am eliminat identificatorul de acces "public" din teste pentru a nu mai fi probleme cu Sonarul. -Am creat un test in UserControllerTest care sa verifice daca serverul arunca mai multe exceptii, acesta verificand si fieldName ul. In plus am facut ca fiecare test care testeaza daca este aruncata exceptia MultipleOxygenAccountException, sa se asigure ca este aruncata pe fieldname ul corect.
In issue-ul #27 s-a adaugat un serviciu de validare pentru input-urile de pe REST API. Momentan acesta trateaza doar prima eroare de validare returnata de validator, insa este posibil ca pe un input validatorul sa returneze mai multe erori. Service-ul nostru de validare trebuie sa intoarca catre client toate aceste probleme, nu doar pe prima.