Skip to content

Commit 85b206d

Browse files
[#240] Fix tests and update schema
1 parent 9bbcafd commit 85b206d

File tree

3 files changed

+17
-33
lines changed

3 files changed

+17
-33
lines changed

src/openklant/components/klantinteracties/api/serializers/partijen.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -770,8 +770,8 @@ def update(self, instance, validated_data):
770770
@transaction.atomic
771771
def create(self, validated_data):
772772
partij_identificatie = validated_data.pop("partij_identificatie", None)
773-
digitale_adressen = validated_data.pop("digitaaladres_set")
774-
rekeningnummers = validated_data.pop("rekeningnummer_set")
773+
digitale_adressen = validated_data.pop("digitaaladres_set", [])
774+
rekeningnummers = validated_data.pop("rekeningnummer_set", [])
775775

776776
if voorkeurs_digitaal_adres := validated_data.pop(
777777
"voorkeurs_digitaal_adres", None

src/openklant/components/klantinteracties/api/tests/test_partijen.py

+15-6
Original file line numberDiff line numberDiff line change
@@ -344,19 +344,28 @@ def test_create_partij_only_required(self):
344344
],
345345
)
346346

347-
digitaal_adres = DigitaalAdresFactory()
348-
349347
data = {
350-
"digitaleAdressen": [{"uuid": str(digitaal_adres.uuid)}],
351-
"voorkeursDigitaalAdres": {"uuid": str(digitaal_adres.uuid)},
352-
"rekeningnummers": [],
353-
"voorkeursRekeningnummer": None,
354348
"soortPartij": "persoon",
355349
"indicatieActief": True,
356350
}
357351

358352
response = self.client.post(list_url, data)
359353
self.assertEqual(response.status_code, status.HTTP_201_CREATED)
354+
response_data = response.json()
355+
356+
self.assertEqual(response_data["betrokkenen"], [])
357+
self.assertEqual(response_data["categorieRelaties"], [])
358+
self.assertEqual(response_data["digitaleAdressen"], [])
359+
self.assertEqual(response_data["voorkeursDigitaalAdres"], None)
360+
self.assertEqual(response_data["vertegenwoordigden"], [])
361+
self.assertEqual(response_data["rekeningnummers"], [])
362+
self.assertEqual(response_data["voorkeursRekeningnummer"], None)
363+
self.assertEqual(response_data["partijIdentificatoren"], [])
364+
self.assertEqual(response_data["soortPartij"], "persoon")
365+
self.assertEqual(response_data["indicatieGeheimhouding"], None)
366+
self.assertEqual(response_data["voorkeurstaal"], "")
367+
self.assertEqual(response_data["indicatieActief"], True)
368+
self.assertEqual(response_data["partijIdentificatie"], None)
360369

361370
def test_create_persoon(self):
362371
list_url = reverse("klantinteracties:partij-list")

src/openklant/components/klantinteracties/openapi.yaml

-25
Original file line numberDiff line numberDiff line change
@@ -650,7 +650,6 @@ paths:
650650
application/json:
651651
schema:
652652
$ref: '#/components/schemas/Bijlage'
653-
required: true
654653
security:
655654
- tokenAuth: []
656655
responses:
@@ -703,7 +702,6 @@ paths:
703702
application/json:
704703
schema:
705704
$ref: '#/components/schemas/Bijlage'
706-
required: true
707705
security:
708706
- tokenAuth: []
709707
responses:
@@ -1939,7 +1937,6 @@ paths:
19391937
application/json:
19401938
schema:
19411939
$ref: '#/components/schemas/Onderwerpobject'
1942-
required: true
19431940
security:
19441941
- tokenAuth: []
19451942
responses:
@@ -1992,7 +1989,6 @@ paths:
19921989
application/json:
19931990
schema:
19941991
$ref: '#/components/schemas/Onderwerpobject'
1995-
required: true
19961992
security:
19971993
- tokenAuth: []
19981994
responses:
@@ -2136,7 +2132,6 @@ paths:
21362132
application/json:
21372133
schema:
21382134
$ref: '#/components/schemas/PartijIdentificator'
2139-
required: true
21402135
security:
21412136
- tokenAuth: []
21422137
responses:
@@ -2189,7 +2184,6 @@ paths:
21892184
application/json:
21902185
schema:
21912186
$ref: '#/components/schemas/PartijIdentificator'
2192-
required: true
21932187
security:
21942188
- tokenAuth: []
21952189
responses:
@@ -3105,16 +3099,12 @@ components:
31053099
required:
31063100
- betrokkenen
31073101
- categorieRelaties
3108-
- digitaleAdressen
31093102
- indicatieActief
31103103
- partijIdentificatoren
3111-
- rekeningnummers
31123104
- soortPartij
31133105
- url
31143106
- uuid
31153107
- vertegenwoordigden
3116-
- voorkeursDigitaalAdres
3117-
- voorkeursRekeningnummer
31183108
Betrokkene:
31193109
type: object
31203110
description: |-
@@ -3196,7 +3186,6 @@ components:
31963186
- url
31973187
- uuid
31983188
- volledigeNaam
3199-
- wasPartij
32003189
BetrokkeneCorrespondentieadres:
32013190
type: object
32023191
properties:
@@ -3327,7 +3316,6 @@ components:
33273316
- url
33283317
- uuid
33293318
- volledigeNaam
3330-
- wasPartij
33313319
Bezoekadres:
33323320
type: object
33333321
properties:
@@ -3391,7 +3379,6 @@ components:
33913379
required:
33923380
- url
33933381
- uuid
3394-
- wasBijlageVanKlantcontact
33953382
BijlageForeignKey:
33963383
type: object
33973384
properties:
@@ -3524,8 +3511,6 @@ components:
35243511
Een voorbeeld: 2022-02-21'
35253512
required:
35263513
- beginDatum
3527-
- categorie
3528-
- partij
35293514
- url
35303515
- uuid
35313516
CategorieRelatieForeignKey:
@@ -3640,7 +3625,6 @@ components:
36403625
description: De voledige naam van het constact persoon.
36413626
required:
36423627
- volledigeNaam
3643-
- werkteVoorPartij
36443628
ContactpersoonPersoon:
36453629
type: object
36463630
properties:
@@ -3718,8 +3702,6 @@ components:
37183702
- soortDigitaalAdres
37193703
- url
37203704
- uuid
3721-
- verstrektDoorBetrokkene
3722-
- verstrektDoorPartij
37233705
DigitaalAdresForeignKey:
37243706
type: object
37253707
properties:
@@ -4105,10 +4087,8 @@ components:
41054087
description: Gegevens die een onderwerpobject in een extern register uniek
41064088
identificeren.
41074089
required:
4108-
- klantcontact
41094090
- url
41104091
- uuid
4111-
- wasKlantcontact
41124092
OnderwerpobjectForeignKey:
41134093
type: object
41144094
properties:
@@ -4166,7 +4146,6 @@ components:
41664146
- klantcontact
41674147
- url
41684148
- uuid
4169-
- wasKlantcontact
41704149
Onderwerpobjectidentificator:
41714150
type: object
41724151
properties:
@@ -4755,8 +4734,6 @@ components:
47554734
description: Gegevens die een partij in een basisregistratie of ander extern
47564735
register uniek identificeren.
47574736
required:
4758-
- identificeerdePartij
4759-
- partijIdentificator
47604737
- url
47614738
- uuid
47624739
PartijIdentificatorForeignkey:
@@ -5373,7 +5350,6 @@ components:
53735350
readOnly: true
53745351
description: De voledige naam van het persoon.
53755352
required:
5376-
- contactnaam
53775353
- volledigeNaam
53785354
PersoonContact:
53795355
type: object
@@ -5436,7 +5412,6 @@ components:
54365412
minLength: 8
54375413
required:
54385414
- iban
5439-
- partij
54405415
- url
54415416
- uuid
54425417
RekeningnummerForeignKey:

0 commit comments

Comments
 (0)