Issue #424

Corretto problema nell'annullamento di una pendenza gia' annullata, ora l'operazione viene consentita e viene aggiornata la descrizione dell'operazione.
link-it · Nov 4, 2021 · 36653e3 · 36653e3
1 parent da3e92c
commit 36653e3
Show file tree

Hide file tree

Showing 2 changed files with 54 additions and 3 deletions.
diff --git a/...t/src/test/java/test/api/backoffice/v1/pendenze/patch/pendenza-patch-annullamento.feature b/...t/src/test/java/test/api/backoffice/v1/pendenze/patch/pendenza-patch-annullamento.feature
@@ -48,4 +48,54 @@ And request
 ]
 """
 When method patch
-Then status 200
+Then status 200
+
+Scenario: Annullamento di una pendenza annullata
+
+* def backofficeBasicBaseurl = getGovPayApiBaseUrl({api: 'backoffice', versione: 'v1', autenticazione: 'spid'})
+
+Given url backofficeBasicBaseurl
+And path '/pendenze', idA2A, idPendenza
+And headers operatoreSpid2AutenticationHeader
+And request
+"""
+[
+  { "op":"REPLACE","path":"/stato","value":"ANNULLATA"},
+  {"op":"REPLACE","path":"/descrizioneStato","value":"Test annullamento"}
+]
+"""
+When method patch
+Then status 200
+
+Given url backofficeBasicBaseurl
+And path '/pendenze', idA2A, idPendenza
+And headers operatoreSpid2AutenticationHeader
+When method get
+Then status 200
+And match response.stato == 'ANNULLATA'
+And match response.descrizioneStato == 'Test annullamento'
+
+
+Given url backofficeBasicBaseurl
+And path '/pendenze', idA2A, idPendenza
+And headers operatoreSpid2AutenticationHeader
+And request
+"""
+[
+  { "op":"REPLACE","path":"/stato","value":"ANNULLATA"},
+  {"op":"REPLACE","path":"/descrizioneStato","value":"Test annullamento 2"}
+]
+"""
+When method patch
+Then status 200
+
+Given url backofficeBasicBaseurl
+And path '/pendenze', idA2A, idPendenza
+And headers operatoreSpid2AutenticationHeader
+When method get
+Then status 200
+And match response.stato == 'ANNULLATA'
+And match response.descrizioneStato == 'Test annullamento 2'
+
+
+
diff --git a/jars/core/src/main/java/it/govpay/core/dao/pagamenti/PendenzeDAO.java b/jars/core/src/main/java/it/govpay/core/dao/pagamenti/PendenzeDAO.java
@@ -767,14 +767,15 @@ private void patchStato(Authentication authentication, it.govpay.bd.model.Versam
 
 		switch (nuovoStato) {
 		case ANNULLATO:
-			if(versamentoLetto.getStatoVersamento().equals(StatoVersamento.NON_ESEGUITO)) {
+			if(versamentoLetto.getStatoVersamento().equals(StatoVersamento.NON_ESEGUITO)
+					|| versamentoLetto.getStatoVersamento().equals(StatoVersamento.ANNULLATO)) {
 				versamentoLetto.setStatoVersamento(StatoVersamento.ANNULLATO);
 				versamentoLetto.setAvvisoNotificato(null);
 				versamentoLetto.setAvvAppIOPromemoriaScadenzaNotificato(null); 
 				versamentoLetto.setAvvMailPromemoriaScadenzaNotificato(null);
 				//				eventoUtente.setDettaglioEsito("Pendenza annullata");
 			} else {
-				throw new ValidationException("Non e' consentito aggiornare lo stato di una pendenza ad ANNULLATO da uno stato diverso da NON_ESEGUITO");
+				throw new ValidationException("Non e' consentito aggiornare lo stato di una pendenza ad ANNULLATO da uno stato diverso da NON_ESEGUITO o ANNULLATO");
 			}
 			break;
 		case NON_ESEGUITO: