Skip to content

Commit

Permalink
[#309] Fixed huisnummer as int
Browse files Browse the repository at this point in the history
  • Loading branch information
@danielmursa-dev
danielmursa-dev committed Feb 28, 2025
1 parent 0379206 commit 57c7acc
Show file tree
Hide file tree
Showing 11 changed files with 204 additions and 175 deletions.
44 changes: 22 additions & 22 deletions src/openklant/components/contactgegevens/api/tests/test_apis.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ def test_persoon_detail(self):
voornamen="John",
adres_nummeraanduiding_id="1234567890000001",
adres_straatnaam="straat",
adres_huisnummer="10",
adres_huisnummer=10,
adres_huisnummertoevoeging="A2",
adres_postcode="1008 DG",
adres_stad="Amsterdam",
Expand All @@ -36,7 +36,7 @@ def test_persoon_detail(self):
expected_adres = {
"nummeraanduidingId": "1234567890000001",
"straatnaam": "straat",
"huisnummer": "10",
"huisnummer": 10,
"huisnummertoevoeging": "A2",
"postcode": "1008 DG",
"stad": "Amsterdam",
Expand Down Expand Up @@ -74,7 +74,7 @@ def test_create_persoon(self):
"adres": {
"nummeraanduidingId": "1234567890000001",
"straatnaam": "straat",
"huisnummer": "10",
"huisnummer": 10,
"huisnummertoevoeging": "A2",
"postcode": "1008 DG",
"stad": "Amsterdam",
Expand Down Expand Up @@ -102,7 +102,7 @@ def test_create_persoon(self):
{
"nummeraanduidingId": "1234567890000001",
"straatnaam": "straat",
"huisnummer": "10",
"huisnummer": 10,
"huisnummertoevoeging": "A2",
"postcode": "1008 DG",
"stad": "Amsterdam",
Expand All @@ -123,7 +123,7 @@ def test_update_persoon(self):
voornamen="Devin",
adres_nummeraanduiding_id="1234567890000001",
adres_straatnaam="straat",
adres_huisnummer="10",
adres_huisnummer=10,
adres_huisnummertoevoeging="A2",
adres_postcode="1008 DG",
adres_stad="Amsterdam",
Expand Down Expand Up @@ -154,7 +154,7 @@ def test_update_persoon(self):
{
"nummeraanduidingId": "1234567890000001",
"straatnaam": "straat",
"huisnummer": "10",
"huisnummer": 10,
"huisnummertoevoeging": "A2",
"postcode": "1008 DG",
"stad": "Amsterdam",
Expand All @@ -175,7 +175,7 @@ def test_update_persoon(self):
"adres": {
"nummeraanduidingId": "1234567890000002",
"straatnaam": "changed",
"huisnummer": "10",
"huisnummer": 10,
"huisnummertoevoeging": "changed",
"postcode": "1001 AB",
"stad": "Amsterdam",
Expand All @@ -199,7 +199,7 @@ def test_update_persoon(self):
{
"nummeraanduidingId": "1234567890000002",
"straatnaam": "changed",
"huisnummer": "10",
"huisnummer": 10,
"huisnummertoevoeging": "changed",
"postcode": "1001 AB",
"stad": "Amsterdam",
Expand All @@ -220,7 +220,7 @@ def test_update_partial_persoon(self):
voornamen="Devin",
adres_nummeraanduiding_id="1234567890000001",
adres_straatnaam="straat",
adres_huisnummer="10",
adres_huisnummer=10,
adres_huisnummertoevoeging="A2",
adres_postcode="1008 DG",
adres_stad="Amsterdam",
Expand Down Expand Up @@ -251,7 +251,7 @@ def test_update_partial_persoon(self):
{
"nummeraanduidingId": "1234567890000001",
"straatnaam": "straat",
"huisnummer": "10",
"huisnummer": 10,
"huisnummertoevoeging": "A2",
"postcode": "1008 DG",
"stad": "Amsterdam",
Expand Down Expand Up @@ -279,7 +279,7 @@ def test_update_partial_persoon(self):
{
"nummeraanduidingId": "1234567890000001",
"straatnaam": "straat",
"huisnummer": "10",
"huisnummer": 10,
"huisnummertoevoeging": "A2",
"postcode": "1008 DG",
"stad": "Amsterdam",
Expand Down Expand Up @@ -312,7 +312,7 @@ def test_organisatie_detail(self):
opheffingsdatum="2020-09-05",
adres_nummeraanduiding_id="1234567890000001",
adres_straatnaam="straat",
adres_huisnummer="10",
adres_huisnummer=10,
adres_huisnummertoevoeging="A2",
adres_postcode="1008 DG",
adres_stad="Amsterdam",
Expand All @@ -329,7 +329,7 @@ def test_organisatie_detail(self):
expected_adres = {
"nummeraanduidingId": "1234567890000001",
"straatnaam": "straat",
"huisnummer": "10",
"huisnummer": 10,
"huisnummertoevoeging": "A2",
"postcode": "1008 DG",
"stad": "Amsterdam",
Expand Down Expand Up @@ -362,7 +362,7 @@ def test_create_organisatie(self):
"adres": {
"nummeraanduidingId": "1234567890000001",
"straatnaam": "straat",
"huisnummer": "10",
"huisnummer": 10,
"huisnummertoevoeging": "A2",
"postcode": "1008 DG",
"stad": "Amsterdam",
Expand All @@ -387,7 +387,7 @@ def test_create_organisatie(self):
{
"nummeraanduidingId": "1234567890000001",
"straatnaam": "straat",
"huisnummer": "10",
"huisnummer": 10,
"huisnummertoevoeging": "A2",
"postcode": "1008 DG",
"stad": "Amsterdam",
Expand All @@ -405,7 +405,7 @@ def test_update_organisatie(self):
opheffingsdatum=None,
adres_nummeraanduiding_id="1234567890000001",
adres_straatnaam="straat",
adres_huisnummer="10",
adres_huisnummer=10,
adres_huisnummertoevoeging="A2",
adres_postcode="1008 DG",
adres_stad="Amsterdam",
Expand Down Expand Up @@ -433,7 +433,7 @@ def test_update_organisatie(self):
{
"nummeraanduidingId": "1234567890000001",
"straatnaam": "straat",
"huisnummer": "10",
"huisnummer": 10,
"huisnummertoevoeging": "A2",
"postcode": "1008 DG",
"stad": "Amsterdam",
Expand All @@ -451,7 +451,7 @@ def test_update_organisatie(self):
"adres": {
"nummeraanduidingId": "1234567890000002",
"straatnaam": "changed",
"huisnummer": "10",
"huisnummer": 10,
"huisnummertoevoeging": "changed",
"postcode": "1001 AB",
"stad": "Amsterdam",
Expand All @@ -471,7 +471,7 @@ def test_update_organisatie(self):
{
"nummeraanduidingId": "1234567890000002",
"straatnaam": "changed",
"huisnummer": "10",
"huisnummer": 10,
"huisnummertoevoeging": "changed",
"postcode": "1001 AB",
"stad": "Amsterdam",
Expand All @@ -489,7 +489,7 @@ def test_update_partial_organisatie(self):
opheffingsdatum=None,
adres_nummeraanduiding_id="1234567890000001",
adres_straatnaam="straat",
adres_huisnummer="10",
adres_huisnummer=10,
adres_huisnummertoevoeging="A2",
adres_postcode="1008 DG",
adres_stad="Amsterdam",
Expand Down Expand Up @@ -517,7 +517,7 @@ def test_update_partial_organisatie(self):
{
"nummeraanduidingId": "1234567890000001",
"straatnaam": "straat",
"huisnummer": "10",
"huisnummer": 10,
"huisnummertoevoeging": "A2",
"postcode": "1008 DG",
"stad": "Amsterdam",
Expand All @@ -543,7 +543,7 @@ def test_update_partial_organisatie(self):
{
"nummeraanduidingId": "1234567890000001",
"straatnaam": "straat",
"huisnummer": "10",
"huisnummer": 10,
"huisnummertoevoeging": "A2",
"postcode": "1008 DG",
"stad": "Amsterdam",
Expand Down
20 changes: 13 additions & 7 deletions ...s/contactgegevens/migrations/0006_remove_organisatie_land_remove_persoon_land_and_more.py
View file
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# Generated by Django 4.2.17 on 2025-02-27 10:22
# Generated by Django 4.2.19 on 2025-02-28 13:57

import django.core.validators
from django.db import migrations, models
Expand Down Expand Up @@ -26,11 +26,14 @@ class Migration(migrations.Migration):
migrations.AddField(
model_name="organisatie",
name="adres_huisnummer",
field=models.CharField(
field=models.IntegerField(
blank=True,
help_text="Huisnummer van het adres (indien het een Nederlands adres betreft zonder BAG-id).",
max_length=5,
validators=[django.core.validators.validate_integer],
null=True,
validators=[
django.core.validators.MinValueValidator(1),
django.core.validators.MaxValueValidator(99999),
],
verbose_name="huisnummer",
),
),
Expand Down Expand Up @@ -82,11 +85,14 @@ class Migration(migrations.Migration):
migrations.AddField(
model_name="persoon",
name="adres_huisnummer",
field=models.CharField(
field=models.IntegerField(
blank=True,
help_text="Huisnummer van het adres (indien het een Nederlands adres betreft zonder BAG-id).",
max_length=5,
validators=[django.core.validators.validate_integer],
null=True,
validators=[
django.core.validators.MinValueValidator(1),
django.core.validators.MaxValueValidator(99999),
],
verbose_name="huisnummer",
),
),
Expand Down
10 changes: 6 additions & 4 deletions src/openklant/components/contactgegevens/openapi.yaml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -384,10 +384,11 @@ components:
zonder BAG-id).
maxLength: 255
huisnummer:
type: string
type: integer
maximum: 99999
minimum: 1
description: Huisnummer van het adres (indien het een Nederlands adres betreft
zonder BAG-id).
maxLength: 5
huisnummertoevoeging:
type: string
description: Huisnummertoevoeging van het adres (indien het een Nederlands
Expand Down Expand Up @@ -653,10 +654,11 @@ components:
zonder BAG-id).
maxLength: 255
huisnummer:
type: string
type: integer
maximum: 99999
minimum: 1
description: Huisnummer van het adres (indien het een Nederlands adres betreft
zonder BAG-id).
maxLength: 5
huisnummertoevoeging:
type: string
description: Huisnummertoevoeging van het adres (indien het een Nederlands
Expand Down
4 changes: 2 additions & 2 deletions src/openklant/components/klantinteracties/api/tests/factories.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ class KlantContactDataFactory(factory.DictFactory):
class BezoekAdresDataFactory(factory.DictFactory):
nummeraanduidingId = "1234567890000001"
straatnaam = "straat"
huisnummer = "10"
huisnummer = 10
huisnummertoevoeging = "A2"
postcode = "1008 DG"
stad = "Amsterdam"
Expand All @@ -28,7 +28,7 @@ class BezoekAdresDataFactory(factory.DictFactory):
class CorrespondentieAdresDataFactory(factory.DictFactory):
nummeraanduidingId = "1234567890000002"
straatnaam = "straat"
huisnummer = "10"
huisnummer = 10
huisnummertoevoeging = "A2"
postcode = "1008 DG"
stad = "Amsterdam"
Expand Down
Loading

0 comments on commit 57c7acc

Please sign in to comment.