Fix #561: Corretta definizione di prossimiRisultati rimuovendo il for…

…mat integer
link-it · Jan 14, 2023 · 512d349 · 512d349
2 parents 9d9ed2c + 5bb2ba5
commit 512d349
Show file tree

Hide file tree

Showing 142 changed files with 6,068 additions and 2,862 deletions.
diff --git a/docs/cruscotto/configurazione/enti/pendenza_dovuto.rst b/docs/cruscotto/configurazione/enti/pendenza_dovuto.rst
@@ -40,11 +40,7 @@ Trasformazione dati
 ~~~~~~~~~~~~~~~~~~~
 
 In questa sezione può essere configurato il processo che, dai dati inseriti nella form, produce una pendenza. Il dato di Input è il JSON prodotto
-<<<<<<< HEAD
 dalla form di inserimento dati, mentre l'output deve essere un JSON che rispetta lo schema di una `PendenzaPost <https://redocly.github.io/redoc/?url=https://raw.githubusercontent.com/link-it/govpay/master/wars/api-backoffice/src/main/webapp/v1/govpay-api-backoffice-v1.yaml#model-pendenzaPost&nocors>`_.
-=======
-dalla form di inserimento dati, mentre l'output deve essere un JSON che rispetta lo schema di una `PendenzaPost <https://redocly.github.io/redoc/?url=https://raw.githubusercontent.com/link-it/govpay/3.6.x/wars/api-backoffice/src/main/webapp/v1/govpay-api-backoffice-v1.yaml#model-pendenzaPost&nocors>`_.
->>>>>>> 3.6.x
 oppure un JSON compatibile con il successivo servizio di inoltro, se configurato. 
 
 Inoltro

diff --git a/ear/pom.xml b/ear/pom.xml
@@ -5,7 +5,7 @@
 	<parent>
 		<artifactId>bom</artifactId>
 		<groupId>it.govpay</groupId>
-		<version>3.6.4</version>
+		<version>3.6.5</version>
 	</parent>
 
 	<artifactId>ear</artifactId>

diff --git a/ear/src/main/application/properties/govpay.properties b/ear/src/main/application/properties/govpay.properties
@@ -205,7 +205,8 @@ it.govpay.client.numeroMassimoConnessioniPerPool=${it.govpay.client.numeroMassim
 # Indica il numero massimo di connessioni per Route di default
 it.govpay.client.numeroMassimoConnessioniPerRouteDefault=${it.govpay.client.numeroMassimoConnessioniPerRouteDefault}
 
-
+# Indica il nome dell'header di autenticazione Subscription Key da inviare a PagoPA
+it.govpay.client.pagopa.autenticazione.subscriptionkey.header.name=${it.govpay.client.pagopa.autenticazione.subscriptionkey.header.name}
 
 
 # Configurazione CORS

diff --git a/integration-test/src/test/java/configurazione/v1/msg/dominio.json b/integration-test/src/test/java/configurazione/v1/msg/dominio.json
diff --git a/...ation-test/src/test/java/test/api/backoffice/v1/intermediari/put/intermediari-put.feature b/...ation-test/src/test/java/test/api/backoffice/v1/intermediari/put/intermediari-put.feature
@@ -10,6 +10,7 @@ Background:
 * def intermediarioBasicAuth = read('classpath:test/api/backoffice/v1/intermediari/put/msg/intermediarioBasicAuth.json')
 * def intermediarioServerAuth = read('classpath:test/api/backoffice/v1/intermediari/put/msg/intermediarioServerAuth.json')
 * def intermediarioClientAuth = read('classpath:test/api/backoffice/v1/intermediari/put/msg/intermediarioClientAuth.json')
+* def intermediarioSubscriptionKeyAuth = read('classpath:test/api/backoffice/v1/intermediari/put/msg/intermediarioSubscriptionKeyAuth.json')
 
 Scenario: Configurazione intermediario senza autenticazione verso pagoPA
 
@@ -87,6 +88,25 @@ When method get
 Then status 200
 And match response == intermediarioClientAuth
 
+Scenario: Configurazione intermediario con autenticazione subscription key verso pagoPA
+
+Given url backofficeBaseurl
+And path 'intermediari', idIntermediario
+And headers basicAutenticationHeader
+And request intermediarioSubscriptionKeyAuth
+When method put
+Then assert responseStatus == 200 || responseStatus == 201
+
+* set intermediarioSubscriptionKeyAuth.idIntermediario = idIntermediario
+* set intermediarioSubscriptionKeyAuth.stazioni = '#ignore'
+
+Given url backofficeBaseurl
+And path 'intermediari', idIntermediario
+And headers basicAutenticationHeader
+When method get
+Then status 200
+And match response == intermediarioSubscriptionKeyAuth
+
 
 
 
diff --git a/...st/java/test/api/backoffice/v1/intermediari/put/msg/intermediarioSubscriptionKeyAuth.json b/...st/java/test/api/backoffice/v1/intermediari/put/msg/intermediarioSubscriptionKeyAuth.json
@@ -0,0 +1,11 @@
+{
+	"denominazione": "Soggetto Intermediario",
+	"principalPagoPa": '#(ndpsym_user)',
+	"servizioPagoPa": {
+		"urlRPT": '#(ndpsym_url + "/pagopa/PagamentiTelematiciRPTservice")',
+		"auth": {
+			"subscriptionKey": "8daebdf9-558c-4203-aade-0dee45bfc08d"
+		}
+	},
+	"abilitato": true
+}
diff --git a/.../java/test/api/backoffice/v1/pendenze/get/msg/notifica-attivazione-multibeneficiario.json b/.../java/test/api/backoffice/v1/pendenze/get/msg/notifica-attivazione-multibeneficiario.json
@@ -32,15 +32,17 @@
                   "fiscalCodePA":"#(idDominio)",
                   "IBAN":"#(ibanAccredito)",
                   "remittanceInformation":"#('/RFB/' + iuv + '/100.99/TXT/Pagamento TARI n.' + idPendenza + '')",
-                  "transferCategory":"9/SEGRETERIA"
+                  "transferCategory":"9/SEGRETERIA",
+                  "metadata":null
                },
                {
                   "idTransfer":2,
                   "transferAmount":"#? _ == pendenzaPut.voci[1].importo || _ == pendenzaPut.voci[0].importo.toString()",
                   "fiscalCodePA":"#(idDominio_2)",
                   "IBAN":"#(ibanAccreditoEnteNonIntermediato)",
                   "remittanceInformation":"#('/RFB/' + iuv + '/10.00/TXT/Pagamento TEFA n.' + idPendenza + '')",
-                  "transferCategory":"9/CodiceContabilita"
+                  "transferCategory":"9/CodiceContabilita",
+                  "metadata":null
                }
             ]
          },

diff --git a/.../api/backoffice/v1/pendenze/get/msg/notifica-terminazione-eseguito-multibeneficiario.json b/.../api/backoffice/v1/pendenze/get/msg/notifica-terminazione-eseguito-multibeneficiario.json
@@ -32,15 +32,17 @@
                   "fiscalCodePA":"#(idDominio)",
                   "IBAN":"#(ibanAccredito)",
                   "remittanceInformation":"#('/RFB/' + iuv + '/100.99/TXT/Pagamento TARI n.' + idPendenza + '')",
-                  "transferCategory":"9/SEGRETERIA"
+                  "transferCategory":"9/SEGRETERIA",
+                  "metadata":null
                },
                {
                   "idTransfer":2,
                   "transferAmount":"#? _ == pendenzaPut.voci[1].importo || _ == pendenzaPut.voci[0].importo.toString()",
                   "fiscalCodePA":"#(idDominio_2)",
                   "IBAN":"#(ibanAccreditoEnteNonIntermediato)",
                   "remittanceInformation":"#('/RFB/' + iuv + '/10.00/TXT/Pagamento TEFA n.' + idPendenza + '')",
-                  "transferCategory":"9/CodiceContabilita"
+                  "transferCategory":"9/CodiceContabilita",
+                  "metadata":null
                }
             ]
          },
@@ -78,15 +80,17 @@
                   "fiscalCodePA":"#(idDominio)",
                   "IBAN":"#(ibanAccredito)",
                   "remittanceInformation":"#('/RFB/' + iuv + '/100.99/TXT/Pagamento TARI n.' + idPendenza + '')",
-                  "transferCategory":"9/SEGRETERIA"
+                  "transferCategory":"9/SEGRETERIA",
+                  "metadata":null
                },
                {
                   "idTransfer":2,
                   "transferAmount":"#? _ == pendenzaPut.voci[1].importo || _ == pendenzaPut.voci[0].importo.toString()",
                   "fiscalCodePA":"#(idDominio_2)",
                   "IBAN":"#(ibanAccreditoEnteNonIntermediato)",
                   "remittanceInformation":"#('/RFB/' + iuv + '/10.00/TXT/Pagamento TEFA n.' + idPendenza + '')",
-                  "transferCategory":"9/CodiceContabilita"
+                  "transferCategory":"9/CodiceContabilita",
+                  "metadata":null
                }
             ]
          },
@@ -117,7 +121,7 @@
 	      "tipo":"ENTRATA",
 	      "importo": "#(pendenzaPut.voci[0].importo)",
 	      "data":"#regex\\d\\d\\d\\d-\\d\\d-\\d\\d",
-	      "commissioni":null,
+	      "commissioni":1.00,
 	      "allegato":null
 	   },
 	   {
@@ -132,7 +136,7 @@
 	      "tipo":"ENTRATA_PA_NON_INTERMEDIATA",
 	      "importo": "#(pendenzaPut.voci[1].importo)",
 	      "data":"#regex\\d\\d\\d\\d-\\d\\d-\\d\\d",
-	      "commissioni":null,
+	      "commissioni":1.00,
 	      "allegato":null
 	   }
 	]

diff --git a/...ion-test/src/test/java/test/api/backoffice/v1/pendenze/put/pendenza-put-semantica.feature b/...ion-test/src/test/java/test/api/backoffice/v1/pendenze/put/pendenza-put-semantica.feature
@@ -411,5 +411,23 @@ And match response ==
 }
 """
 
+Scenario: Caricamento con numero avviso con cifra iniziale non coincidente con auxdigit dominio
 
+* set pendenzaPutMono.numeroAvviso = '123456789248953761'
+
+Given url pendenzeBaseurl
+And path '/pendenze', idA2A, idPendenza
+And headers idA2ABasicAutenticationHeader
+And request pendenzaPutMono
+When method put
+Then status 422
+And match response == 
+"""
+{ 
+	categoria: 'RICHIESTA',
+	codice: 'VER_039',
+	descrizione: 'Richiesta non valida',
+	dettaglio: '#("Il carattere iniziale del numero avviso ("+ pendenzaPutMono.numeroAvviso +") indicato per la pendenza (IdA2A:"+idA2A+" Id:"+idPendenza+") non coincide con l\'aux digit ("+dominio.auxDigit+") impostato per l\'Ente Creditore ("+idDominio+").")'
+}
+"""
 
diff --git a/...ation-test/src/test/java/test/api/pagamento/v2/avvisi/get/verifica-avviso-anonimo.feature b/...ation-test/src/test/java/test/api/pagamento/v2/avvisi/get/verifica-avviso-anonimo.feature
@@ -188,6 +188,12 @@ And header cookie = cookie1
 When method get
 Then status 200
 
+Given url pagamentiBaseurl
+And path '/avvisi', idDominio, numeroAvviso
+And header cookie = cookie1
+When method get
+Then status 406
+
 
 @test3
 Scenario: Verifica avviso non presente anonimo

diff --git a/...gration-test/src/test/java/test/api/pagamento/v2/avvisi/get/verifica-avviso-basic.feature b/...gration-test/src/test/java/test/api/pagamento/v2/avvisi/get/verifica-avviso-basic.feature
@@ -51,6 +51,12 @@ And header Accept = 'application/pdf'
 When method get
 Then status 200
 
+Given url pagamentiBaseurl
+And path '/avvisi', idDominio, numeroAvviso
+And headers basicAutenticationHeader
+When method get
+Then status 406
+
 @test2
 Scenario: Verifica avviso non presente basic
 

diff --git a/integration-test/src/test/java/test/api/pagamento/v2/avvisi/get/verifica-avviso-spid.feature b/integration-test/src/test/java/test/api/pagamento/v2/avvisi/get/verifica-avviso-spid.feature
@@ -52,6 +52,11 @@ And header Accept = 'application/pdf'
 When method get
 Then status 200
 
+Given url pagamentiBaseurl
+And path '/avvisi', idDominio, numeroAvviso
+And headers spidHeaders
+When method get
+Then status 406
 
 @test2
 Scenario: Verifica avviso non presente cittadino

diff --git a/integration-test/src/test/java/test/api/pendenza/v2/pendenze/get/pendenza-get-avvisi.feature b/integration-test/src/test/java/test/api/pendenza/v2/pendenze/get/pendenza-get-avvisi.feature
@@ -56,6 +56,22 @@ Then status 201
 Given url pendenzeBaseurl
 And path '/avvisi', response.idDominio, response.numeroAvviso
 And headers basicAutenticationHeader
+And header Accept = 'application/json'
 When method get
 Then status 200
 
+Scenario: Lettura di un avviso senza specificare l'header Accept
+
+Given url pendenzeBaseurl
+And path '/pendenze', idA2A, idPendenza
+And headers basicAutenticationHeader
+And request pendenzaPut
+When method put
+Then status 201
+
+Given url pendenzeBaseurl
+And path '/avvisi', response.idDominio, response.numeroAvviso
+And headers basicAutenticationHeader
+When method get
+Then status 406
+
diff --git a/...ation-test/src/test/java/test/api/pendenza/v2/pendenze/get/pendenza-get-documenti.feature b/...ation-test/src/test/java/test/api/pendenza/v2/pendenze/get/pendenza-get-documenti.feature
@@ -149,13 +149,21 @@ And headers basicAutenticationHeader
 When method get
 Then status 200
 
+* def idDominioToCheck = response.dominio.idDominio
+
 Given url pendenzeBaseurl
-And path 'documenti', response.dominio.idDominio, idDocumento, 'avvisi'
+And path 'documenti', idDominioToCheck, idDocumento, 'avvisi'
 And headers basicAutenticationHeader
 And header Accept = 'application/pdf'
 When method get
 Then status 200
 
+Given url pendenzeBaseurl
+And path 'documenti', idDominioToCheck, idDocumento, 'avvisi'
+And headers basicAutenticationHeader
+When method get
+Then status 406
+
 
 Scenario: Lettura di un avviso pdf per un documento con violazione CDS