✅ [open-formulieren/open-forms#4380] Update tests with paymentId

open-formulieren · Aug 5, 2024 · 3abebe4 · 3abebe4
1 parent 35fce5c
commit 3abebe4
Show file tree

Hide file tree

Showing 5 changed files with 36 additions and 20 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -39,6 +39,8 @@ jobs:
         with:
           repository: open-formulieren/open-forms
           path: open-forms
+          # TODO remove after this is merged
+          ref: feature/4380-ogone-payment-id
 
       - name: Checkout StUF-ZDS payments extension
         uses: actions/checkout@v4

diff --git a/...onTestCase/StufZDSPaymentsRegistrationTestCase.test_register_submission_with_payment.yaml b/...onTestCase/StufZDSPaymentsRegistrationTestCase.test_register_submission_with_payment.yaml
@@ -7,7 +7,7 @@ interactions:
       \   xmlns:BG=\"http://www.egem.nl/StUF/sector/bg/0310\"\n    xmlns:GML=\"http://www.opengis.net/gml\"\n>\n
       \   <ZKN:stuurgegevens>\n        <StUF:berichtcode>Lk01</StUF:berichtcode>\n
       \       <StUF:zender>\n    <StUF:organisatie>zender_organisatie-0</StUF:organisatie>\n<StUF:applicatie>zender_applicatie-0</StUF:applicatie>\n<StUF:administratie>zender_administratie-0</StUF:administratie>\n<StUF:gebruiker>zender_gebruiker-0</StUF:gebruiker>\n\n</StUF:zender>\n<StUF:ontvanger>\n
-      \   <StUF:organisatie>ontvanger_organisatie-0</StUF:organisatie>\n<StUF:applicatie>ontvanger_applicatie-0</StUF:applicatie>\n<StUF:administratie>ontvanger_administratie-0</StUF:administratie>\n<StUF:gebruiker>ontvanger_gebruiker-0</StUF:gebruiker>\n\n</StUF:ontvanger>\n<StUF:referentienummer>58f3d63c-37ab-4b65-844d-fdb9da043b17</StUF:referentienummer>\n<StUF:tijdstipBericht>20240805124256</StUF:tijdstipBericht>\n\n
+      \   <StUF:organisatie>ontvanger_organisatie-0</StUF:organisatie>\n<StUF:applicatie>ontvanger_applicatie-0</StUF:applicatie>\n<StUF:administratie>ontvanger_administratie-0</StUF:administratie>\n<StUF:gebruiker>ontvanger_gebruiker-0</StUF:gebruiker>\n\n</StUF:ontvanger>\n<StUF:referentienummer>71257f65-be1b-4d9a-bc67-16dfd5a3e1a8</StUF:referentienummer>\n<StUF:tijdstipBericht>20240805140054</StUF:tijdstipBericht>\n\n
       \       <StUF:entiteittype>ZAK</StUF:entiteittype>\n    </ZKN:stuurgegevens>\n
       \   <ZKN:parameters>\n        <StUF:mutatiesoort>T</StUF:mutatiesoort>\n        <StUF:indicatorOvername>V</StUF:indicatorOvername>\n
       \   </ZKN:parameters>\n    <ZKN:object StUF:verwerkingssoort=\"T\" StUF:entiteittype=\"ZAK\">\n
@@ -19,11 +19,13 @@ interactions:
       \       \n        <ZKN:startdatum>20240805</ZKN:startdatum>\n        <ZKN:registratiedatum>20240805</ZKN:registratiedatum>\n
       \       <ZKN:betalingsIndicatie>N.v.t.</ZKN:betalingsIndicatie>\n        \n\n
       \       <ZKN:zaakniveau>1</ZKN:zaakniveau>\n        <ZKN:deelzakenIndicatie>N</ZKN:deelzakenIndicatie>\n
-      \       <StUF:tijdstipRegistratie>20240805124256</StUF:tijdstipRegistratie>\n
+      \       <StUF:tijdstipRegistratie>20240805140054</StUF:tijdstipRegistratie>\n
       \       <StUF:extraElementen>\n\n<StUF:extraElement naam=\"language_code\">en</StUF:extraElement>\n\n<StUF:extraElement
       naam=\"extra\">BuzzBazz</StUF:extraElement>\n\n<StUF:extraElement naam=\"payment_completed\">true</StUF:extraElement>\n\n<StUF:extraElement
       naam=\"payment_amount\">40.0</StUF:extraElement>\n\n<StUF:extraElement naam=\"payment_public_order_ids.0\">foo</StUF:extraElement>\n\n<StUF:extraElement
-      naam=\"payment_public_order_ids.1\">bar</StUF:extraElement>\n\n</StUF:extraElementen>\n\n\n
+      naam=\"payment_public_order_ids.1\">bar</StUF:extraElement>\n\n<StUF:extraElement
+      naam=\"provider_payment_ids.0\">123456</StUF:extraElement>\n\n<StUF:extraElement
+      naam=\"provider_payment_ids.1\">654321</StUF:extraElement>\n\n</StUF:extraElementen>\n\n\n
       \       <ZKN:isVan StUF:entiteittype=\"ZAKZKT\" StUF:verwerkingssoort=\"T\">\n
       \           <ZKN:gerelateerde StUF:verwerkingssoort=\"I\" StUF:entiteittype=\"ZKT\">\n
       \               \n                    <ZKN:omschrijving>bar</ZKN:omschrijving>\n
@@ -39,7 +41,7 @@ interactions:
       \                           \n                            <BG:geboortedatum
       StUF:indOnvolledigeDatum=\"V\">20001231</BG:geboortedatum>\n\n                            \n
       \                       </ZKN:natuurlijkPersoon>\n                    \n                </ZKN:gerelateerde>\n
-      \               <StUF:tijdstipRegistratie>20240805124256</StUF:tijdstipRegistratie>\n
+      \               <StUF:tijdstipRegistratie>20240805140054</StUF:tijdstipRegistratie>\n
       \           </ZKN:heeftAlsInitiator>\n        \n        \n        \n            <ZKN:heeft
       StUF:entiteittype=\"ZAKSTT\" StUF:verwerkingssoort=\"T\">\n                <ZKN:gerelateerde
       StUF:entiteittype=\"STT\" StUF:verwerkingssoort=\"I\">\n                    \n
@@ -56,7 +58,7 @@ interactions:
       Connection:
       - keep-alive
       Content-Length:
-      - '5155'
+      - '5309'
       Content-Type:
       - application/soap+xml
       SOAPAction:
@@ -89,7 +91,7 @@ interactions:
       Content-Type:
       - text/xml
       Date:
-      - Mon, 05 Aug 2024 12:42:56 GMT
+      - Mon, 05 Aug 2024 14:00:54 GMT
       Server:
       - Werkzeug/3.0.3 Python/3.12.4
     status:
@@ -102,7 +104,7 @@ interactions:
       \       \n\n<ZKN:genereerDocumentIdentificatie_Di02 xmlns:ZKN=\"http://www.egem.nl/StUF/sector/zkn/0310\">\n
       \   <ZKN:stuurgegevens>\n        <StUF:berichtcode>Di02</StUF:berichtcode>\n
       \       <StUF:zender>\n    <StUF:organisatie>zender_organisatie-0</StUF:organisatie>\n<StUF:applicatie>zender_applicatie-0</StUF:applicatie>\n<StUF:administratie>zender_administratie-0</StUF:administratie>\n<StUF:gebruiker>zender_gebruiker-0</StUF:gebruiker>\n\n</StUF:zender>\n<StUF:ontvanger>\n
-      \   <StUF:organisatie>ontvanger_organisatie-0</StUF:organisatie>\n<StUF:applicatie>ontvanger_applicatie-0</StUF:applicatie>\n<StUF:administratie>ontvanger_administratie-0</StUF:administratie>\n<StUF:gebruiker>ontvanger_gebruiker-0</StUF:gebruiker>\n\n</StUF:ontvanger>\n<StUF:referentienummer>e4bcac4d-5fab-4a01-a28a-76d168bd7180</StUF:referentienummer>\n<StUF:tijdstipBericht>20240805124256</StUF:tijdstipBericht>\n\n
+      \   <StUF:organisatie>ontvanger_organisatie-0</StUF:organisatie>\n<StUF:applicatie>ontvanger_applicatie-0</StUF:applicatie>\n<StUF:administratie>ontvanger_administratie-0</StUF:administratie>\n<StUF:gebruiker>ontvanger_gebruiker-0</StUF:gebruiker>\n\n</StUF:ontvanger>\n<StUF:referentienummer>1f00fc6e-a068-4e91-9d66-840e23d0af57</StUF:referentienummer>\n<StUF:tijdstipBericht>20240805140054</StUF:tijdstipBericht>\n\n
       \       <StUF:functie>genereerDocumentidentificatie</StUF:functie>\n    </ZKN:stuurgegevens>\n</ZKN:genereerDocumentIdentificatie_Di02>\n
       \   </soapenv:Body>\n</soapenv:Envelope>\n"
     headers:
@@ -136,7 +138,7 @@ interactions:
         \               <StUF:functie>genereerDocumentidentificatie</StUF:functie>\n
         \           </ZKN:stuurgegevens>\n            <ZKN:melding>melding</ZKN:melding>\n
         \           <ZKN:document StUF:entiteittype=\"EDC\" StUF:functie=\"entiteit\">\n
-        \               <ZKN:identificatie>b3c8b5a4d32fb168</ZKN:identificatie>\n
+        \               <ZKN:identificatie>8eec122ce1010d29</ZKN:identificatie>\n
         \           </ZKN:document>\n        </ZKN:genereerDocumentIdentificatie_Du02>\n
         \   </soapenv:Body>\n</soapenv:Envelope>"
     headers:
@@ -147,7 +149,7 @@ interactions:
       Content-Type:
       - text/xml
       Date:
-      - Mon, 05 Aug 2024 12:42:56 GMT
+      - Mon, 05 Aug 2024 14:00:54 GMT
       Server:
       - Werkzeug/3.0.3 Python/3.12.4
     status:
@@ -160,11 +162,11 @@ interactions:
       \       \n\n<ZKN:edcLk01 xmlns:ZKN=\"http://www.egem.nl/StUF/sector/zkn/0310\"
       xmlns:xmime=\"http://www.w3.org/2005/05/xmlmime\">\n    <ZKN:stuurgegevens>\n
       \       <StUF:berichtcode>Lk01</StUF:berichtcode>\n        <StUF:zender>\n    <StUF:organisatie>zender_organisatie-0</StUF:organisatie>\n<StUF:applicatie>zender_applicatie-0</StUF:applicatie>\n<StUF:administratie>zender_administratie-0</StUF:administratie>\n<StUF:gebruiker>zender_gebruiker-0</StUF:gebruiker>\n\n</StUF:zender>\n<StUF:ontvanger>\n
-      \   <StUF:organisatie>ontvanger_organisatie-0</StUF:organisatie>\n<StUF:applicatie>ontvanger_applicatie-0</StUF:applicatie>\n<StUF:administratie>ontvanger_administratie-0</StUF:administratie>\n<StUF:gebruiker>ontvanger_gebruiker-0</StUF:gebruiker>\n\n</StUF:ontvanger>\n<StUF:referentienummer>3cac727d-3782-4905-b516-8ecc109a2c6c</StUF:referentienummer>\n<StUF:tijdstipBericht>20240805124256</StUF:tijdstipBericht>\n\n
+      \   <StUF:organisatie>ontvanger_organisatie-0</StUF:organisatie>\n<StUF:applicatie>ontvanger_applicatie-0</StUF:applicatie>\n<StUF:administratie>ontvanger_administratie-0</StUF:administratie>\n<StUF:gebruiker>ontvanger_gebruiker-0</StUF:gebruiker>\n\n</StUF:ontvanger>\n<StUF:referentienummer>66ff8468-958d-4909-b57f-9b114d60429e</StUF:referentienummer>\n<StUF:tijdstipBericht>20240805140054</StUF:tijdstipBericht>\n\n
       \       <StUF:entiteittype>EDC</StUF:entiteittype>\n    </ZKN:stuurgegevens>\n
       \   <ZKN:parameters>\n        <StUF:mutatiesoort>T</StUF:mutatiesoort>\n        <StUF:indicatorOvername>I</StUF:indicatorOvername>\n
       \   </ZKN:parameters>\n    <ZKN:object StUF:entiteittype=\"EDC\" StUF:verwerkingssoort=\"T\">\n
-      \       <ZKN:identificatie>b3c8b5a4d32fb168</ZKN:identificatie>\n        <ZKN:dct.omschrijving>foo</ZKN:dct.omschrijving>\n
+      \       <ZKN:identificatie>8eec122ce1010d29</ZKN:identificatie>\n        <ZKN:dct.omschrijving>foo</ZKN:dct.omschrijving>\n
       \       <ZKN:creatiedatum>20240805</ZKN:creatiedatum>\n        <ZKN:ontvangstdatum>20240805</ZKN:ontvangstdatum>\n
       \       <ZKN:titel>inzending</ZKN:titel>\n        <ZKN:beschrijving>Ingezonden
       formulier</ZKN:beschrijving>\n        <ZKN:formaat>application/pdf</ZKN:formaat>\n
@@ -174,11 +176,11 @@ interactions:
       StUF:bestandsnaam=\"open-forms-inzending.pdf\"></ZKN:inhoud>\n        <StUF:tijdvakGeldigheid>\n
       \           <StUF:beginGeldigheid>20240805</StUF:beginGeldigheid>\n            <StUF:eindGeldigheid
       StUF:noValue=\"geenWaarde\" xsi:nil=\"true\"/>\n        </StUF:tijdvakGeldigheid>\n
-      \       <StUF:tijdstipRegistratie>20240805124256</StUF:tijdstipRegistratie>\n
+      \       <StUF:tijdstipRegistratie>20240805140054</StUF:tijdstipRegistratie>\n
       \       <ZKN:isRelevantVoor StUF:entiteittype=\"EDCZAK\" StUF:verwerkingssoort=\"T\">\n
       \           <ZKN:gerelateerde StUF:entiteittype=\"ZAK\" StUF:verwerkingssoort=\"I\">\n
       \               <ZKN:identificatie>abc123</ZKN:identificatie>\n                <ZKN:omschrijving>my-form</ZKN:omschrijving>\n
-      \           </ZKN:gerelateerde>\n            <StUF:tijdstipRegistratie>20240805124256</StUF:tijdstipRegistratie>\n
+      \           </ZKN:gerelateerde>\n            <StUF:tijdstipRegistratie>20240805140054</StUF:tijdstipRegistratie>\n
       \       </ZKN:isRelevantVoor>\n    </ZKN:object>\n</ZKN:edcLk01>\n    </soapenv:Body>\n</soapenv:Envelope>\n"
     headers:
       Accept:
@@ -221,7 +223,7 @@ interactions:
       Content-Type:
       - text/xml
       Date:
-      - Mon, 05 Aug 2024 12:42:56 GMT
+      - Mon, 05 Aug 2024 14:00:54 GMT
       Server:
       - Werkzeug/3.0.3 Python/3.12.4
     status:

diff --git a/...ymentsRegistrationTestCase/StufZDSPaymentsRegistrationTestCase.test_set_zaak_payment.yaml b/...ymentsRegistrationTestCase/StufZDSPaymentsRegistrationTestCase.test_set_zaak_payment.yaml
@@ -6,7 +6,7 @@ interactions:
       \       \n\n<ZKN:zakLk01 xmlns:ZKN=\"http://www.egem.nl/StUF/sector/zkn/0310\">\n
       \   <ZKN:stuurgegevens>\n        <StUF:berichtcode>Lk01</StUF:berichtcode>\n
       \       <StUF:zender>\n    <StUF:organisatie>zender_organisatie-0</StUF:organisatie>\n<StUF:applicatie>zender_applicatie-0</StUF:applicatie>\n<StUF:administratie>zender_administratie-0</StUF:administratie>\n<StUF:gebruiker>zender_gebruiker-0</StUF:gebruiker>\n\n</StUF:zender>\n<StUF:ontvanger>\n
-      \   <StUF:organisatie>ontvanger_organisatie-0</StUF:organisatie>\n<StUF:applicatie>ontvanger_applicatie-0</StUF:applicatie>\n<StUF:administratie>ontvanger_administratie-0</StUF:administratie>\n<StUF:gebruiker>ontvanger_gebruiker-0</StUF:gebruiker>\n\n</StUF:ontvanger>\n<StUF:referentienummer>a62fad4a-01c0-46ad-98a7-49592c88b21e</StUF:referentienummer>\n<StUF:tijdstipBericht>20240805124256</StUF:tijdstipBericht>\n\n
+      \   <StUF:organisatie>ontvanger_organisatie-0</StUF:organisatie>\n<StUF:applicatie>ontvanger_applicatie-0</StUF:applicatie>\n<StUF:administratie>ontvanger_administratie-0</StUF:administratie>\n<StUF:gebruiker>ontvanger_gebruiker-0</StUF:gebruiker>\n\n</StUF:ontvanger>\n<StUF:referentienummer>5be0d3ae-c1f5-484e-842d-b978f79eed2c</StUF:referentienummer>\n<StUF:tijdstipBericht>20240805140054</StUF:tijdstipBericht>\n\n
       \       <StUF:entiteittype>ZAK</StUF:entiteittype>\n    </ZKN:stuurgegevens>\n
       \   <ZKN:parameters>\n        <StUF:mutatiesoort>W</StUF:mutatiesoort>\n        <StUF:indicatorOvername>V</StUF:indicatorOvername>\n
       \   </ZKN:parameters>\n    <ZKN:object StUF:verwerkingssoort=\"W\" StUF:entiteittype=\"ZAK\">\n
@@ -15,7 +15,9 @@ interactions:
       naam=\"language_code\">en</StUF:extraElement>\n\n<StUF:extraElement naam=\"extra\">BuzzBazz</StUF:extraElement>\n\n<StUF:extraElement
       naam=\"payment_completed\">true</StUF:extraElement>\n\n<StUF:extraElement naam=\"payment_amount\">40.0</StUF:extraElement>\n\n<StUF:extraElement
       naam=\"payment_public_order_ids.0\">foo</StUF:extraElement>\n\n<StUF:extraElement
-      naam=\"payment_public_order_ids.1\">bar</StUF:extraElement>\n\n</StUF:extraElementen>\n\n\n
+      naam=\"payment_public_order_ids.1\">bar</StUF:extraElement>\n\n<StUF:extraElement
+      naam=\"provider_payment_ids.0\">123456</StUF:extraElement>\n\n<StUF:extraElement
+      naam=\"provider_payment_ids.1\">654321</StUF:extraElement>\n\n</StUF:extraElementen>\n\n\n
       \   </ZKN:object>\n</ZKN:zakLk01>\n    </soapenv:Body>\n</soapenv:Envelope>\n"
     headers:
       Accept:
@@ -25,7 +27,7 @@ interactions:
       Connection:
       - keep-alive
       Content-Length:
-      - '2171'
+      - '2325'
       Content-Type:
       - application/soap+xml
       SOAPAction:
@@ -58,7 +60,7 @@ interactions:
       Content-Type:
       - text/xml
       Date:
-      - Mon, 05 Aug 2024 12:42:56 GMT
+      - Mon, 05 Aug 2024 14:00:54 GMT
       Server:
       - Werkzeug/3.0.3 Python/3.12.4
     status:

diff --git a/...tsRegistrationTestCase.test_set_zaak_payment_incorrect_payment_status_update_mapping.yaml b/...tsRegistrationTestCase.test_set_zaak_payment_incorrect_payment_status_update_mapping.yaml
@@ -6,7 +6,7 @@ interactions:
       \       \n\n<ZKN:zakLk01 xmlns:ZKN=\"http://www.egem.nl/StUF/sector/zkn/0310\">\n
       \   <ZKN:stuurgegevens>\n        <StUF:berichtcode>Lk01</StUF:berichtcode>\n
       \       <StUF:zender>\n    <StUF:organisatie>zender_organisatie-0</StUF:organisatie>\n<StUF:applicatie>zender_applicatie-0</StUF:applicatie>\n<StUF:administratie>zender_administratie-0</StUF:administratie>\n<StUF:gebruiker>zender_gebruiker-0</StUF:gebruiker>\n\n</StUF:zender>\n<StUF:ontvanger>\n
-      \   <StUF:organisatie>ontvanger_organisatie-0</StUF:organisatie>\n<StUF:applicatie>ontvanger_applicatie-0</StUF:applicatie>\n<StUF:administratie>ontvanger_administratie-0</StUF:administratie>\n<StUF:gebruiker>ontvanger_gebruiker-0</StUF:gebruiker>\n\n</StUF:ontvanger>\n<StUF:referentienummer>59dedae3-715d-429f-ada3-e66c3b3fe3a9</StUF:referentienummer>\n<StUF:tijdstipBericht>20240805124256</StUF:tijdstipBericht>\n\n
+      \   <StUF:organisatie>ontvanger_organisatie-0</StUF:organisatie>\n<StUF:applicatie>ontvanger_applicatie-0</StUF:applicatie>\n<StUF:administratie>ontvanger_administratie-0</StUF:administratie>\n<StUF:gebruiker>ontvanger_gebruiker-0</StUF:gebruiker>\n\n</StUF:ontvanger>\n<StUF:referentienummer>40a76860-3e96-4fe3-b35f-ca5f134e9d4b</StUF:referentienummer>\n<StUF:tijdstipBericht>20240805140054</StUF:tijdstipBericht>\n\n
       \       <StUF:entiteittype>ZAK</StUF:entiteittype>\n    </ZKN:stuurgegevens>\n
       \   <ZKN:parameters>\n        <StUF:mutatiesoort>W</StUF:mutatiesoort>\n        <StUF:indicatorOvername>V</StUF:indicatorOvername>\n
       \   </ZKN:parameters>\n    <ZKN:object StUF:verwerkingssoort=\"W\" StUF:entiteittype=\"ZAK\">\n
@@ -56,7 +56,7 @@ interactions:
       Content-Type:
       - text/xml
       Date:
-      - Mon, 05 Aug 2024 12:42:56 GMT
+      - Mon, 05 Aug 2024 14:00:54 GMT
       Server:
       - Werkzeug/3.0.3 Python/3.12.4
     status:

diff --git a/stuf_zds_payments/tests/test_plugin.py b/stuf_zds_payments/tests/test_plugin.py
@@ -113,12 +113,14 @@ def setUpTestData(cls):
             amount=Decimal("25.00"),
             public_order_id="foo",
             status=PaymentStatus.completed,
+            provider_payment_id="123456",
         )
         SubmissionPaymentFactory.create(
             submission=cls.submission,
             amount=Decimal("15.00"),
             public_order_id="bar",
             status=PaymentStatus.completed,
+            provider_payment_id="654321",
         )
 
     def test_set_zaak_payment(self):
@@ -142,6 +144,10 @@ def test_set_zaak_payment(self):
 
             <StUF:extraElement naam="payment_public_order_ids.1">bar</StUF:extraElement>
 
+            <StUF:extraElement naam="provider_payment_ids.0">123456</StUF:extraElement>
+
+            <StUF:extraElement naam="provider_payment_ids.1">654321</StUF:extraElement>
+
             </StUF:extraElementen>"""
         )
 
@@ -207,6 +213,10 @@ def test_register_submission_with_payment(self):
 
             <StUF:extraElement naam="payment_public_order_ids.1">bar</StUF:extraElement>
 
+            <StUF:extraElement naam="provider_payment_ids.0">123456</StUF:extraElement>
+
+            <StUF:extraElement naam="provider_payment_ids.1">654321</StUF:extraElement>
+
             </StUF:extraElementen>"""
         )