Issue #423

Corretto problema nell'aggiornamento di una applicazione a cui viene cambiato il principal.
link-it · Nov 5, 2021 · 6780319 · 6780319
1 parent 36653e3
commit 6780319
Show file tree

Hide file tree

Showing 2 changed files with 38 additions and 0 deletions.
diff --git a/...ation-test/src/test/java/test/api/backoffice/v1/applicazioni/put/applicazioni-put.feature b/...ation-test/src/test/java/test/api/backoffice/v1/applicazioni/put/applicazioni-put.feature
@@ -192,3 +192,40 @@ Examples:
 | domini | [ '*' ] |
 | tipiPendenza | [ '*' ] |
 | tipiPendenza | [ 'autodeterminazione' ] |
+
+
+
+Scenario: Modifica del principal di una applicazione non deve modificare le autorizzazione sulle API
+
+* def applicazione = read('classpath:test/api/backoffice/v1/applicazioni/put/msg/applicazione.json')
+* def idComune = getCurrentTimeMillis()
+* def idAppl1 = 'PROVA_' + idComune
+
+* set applicazione.principal = idAppl1
+
+Given url backofficeBaseurl
+And path 'applicazioni', idAppl1
+And headers basicAutenticationHeader
+And request applicazione
+When method put
+Then assert responseStatus == 200 || responseStatus == 201
+
+* set applicazione.principal = 'PRINCIPAL_MODIFICATO'
+
+Given url backofficeBaseurl
+And path 'applicazioni', idAppl1
+And headers basicAutenticationHeader
+And request applicazione
+When method put
+Then assert responseStatus == 200 || responseStatus == 201
+
+Given url backofficeBaseurl
+And path 'applicazioni', idAppl1
+And headers basicAutenticationHeader
+When method get
+Then status 200
+And match response.apiPagamenti == true
+And match response.apiPendenze == true
+And match response.apiRagioneria == true
+
+
diff --git a/jars/orm/src/main/java/it/govpay/bd/anagrafica/UtenzeBD.java b/jars/orm/src/main/java/it/govpay/bd/anagrafica/UtenzeBD.java
@@ -668,6 +668,7 @@ public void insertUtenza(Utenza utenza) throws  ServiceException{
 				aclBD.setAtomica(false);
 				for(Acl aclNuova : utenza.getAclPrincipal()) {
 					aclNuova.setIdUtenza(utenza.getId());
+					aclNuova.setUtenza(utenza);
 					aclBD.insertAcl(aclNuova);
 				}
 			}