Merge branch 'main' into feat/verblijfplaatsen-aflopend-sorteren

BRP-API · Jul 4, 2024 · 270a60a · 270a60a
2 parents 6ba0ecb + ce0316e
commit 270a60a
Show file tree

Hide file tree

Showing 10 changed files with 472 additions and 449 deletions.
diff --git a/.docker/historie-data-service.yml b/.docker/historie-data-service.yml
@@ -3,7 +3,7 @@ version: '3.7'
 services:
   historie-data-service:
     container_name: historie-data-service
-    image: haalcentraal.azurecr.io/gba-historie-api:2.0.2-202406060648
+    image: haalcentraal.azurecr.io/gba-historie-api:2.0.4-202407040735
     environment:
       - ASPNETCORE_URLS=http://+;
       - Database__Host=postgres

diff --git a/.github/actions/haal-build-run-identifier-uit-docker-manifest/action.yml b/.github/actions/haal-build-run-identifier-uit-docker-manifest/action.yml
@@ -16,5 +16,5 @@ runs:
     - name: Get build run identifier
       id: get-build-run
       run: |
-        echo "BUILD_RUN=$(sed -n 's/image:.*:.*-\(.*\)/\1/p' ${{ inputs.path-docker-manifest }})" >> $GITHUB_OUTPUT
+        echo "BUILD_RUN=$(sed -n 's/.*image:.*:.*-\(.*\)/\1/p' ${{ inputs.path-docker-manifest }})" >> $GITHUB_OUTPUT
       shell: bash
diff --git a/.github/actions/haal-versie-uit-docker-manifest/action.yml b/.github/actions/haal-versie-uit-docker-manifest/action.yml
@@ -16,5 +16,5 @@ runs:
     - name: Get version
       id: get-version
       run: |
-        echo "APP_VERSION=$(sed -n 's/image:.*:\(.*\)-.*/\1/p' ${{ inputs.path-docker-manifest }})" >> $GITHUB_OUTPUT
+        echo "APP_VERSION=$(sed -n 's/.*image:.*:\(.*\)-.*/\1/p' ${{ inputs.path-docker-manifest }})" >> $GITHUB_OUTPUT
       shell: bash
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -11,13 +11,16 @@ on:
 env:
   NODE_VERSION: '20'
   DATA_SERVICE_DOCKER_FILE_PATH: .docker/historie-data-service.yml
+  DATA_SERVICE_CONTAINER_IMAGE: haalcentraal.azurecr.io/gba-historie-api
+  APP_CONTAINER_IMAGE: ghcr.io/brp-api/historie-data-service
 
 jobs:
   continuous-integration:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout repo
         uses: actions/checkout@v4
+
       - name: Haal build run identifier uit Docker manifest
         id: build-run-id
         uses: ./.github/actions/haal-build-run-identifier-uit-docker-manifest
@@ -28,22 +31,26 @@ jobs:
         uses: ./.github/actions/haal-versie-uit-docker-manifest
         with:
           path-docker-manifest: ${{ env.DATA_SERVICE_DOCKER_FILE_PATH }}
+
       - name: Setup Node.js
         uses: actions/setup-node@v4
         with:
           node-version: ${{ env.NODE_VERSION }}
       - name: Install dependencies
         run: npm ci
+
       - name: Login to P5 Container Registry
         uses: docker/[email protected]
         with:
           registry: haalcentraal.azurecr.io
           username: ${{ secrets.P5_CR_USER }}
           password: ${{ secrets.P5_CR_PWD }}
+
       - name: Set run permissions on script folder
         uses: ./.github/actions/zet-run-bash-scripts-permission
         with:
           path-bash-file-or-folder: ./scripts/*
+
       - name: Start containers
         run: |
           ./scripts/containers-start.sh ci
@@ -57,6 +64,7 @@ jobs:
         if: always()
         run: |
           ./scripts/containers-stop.sh ci
+
       - name: Genereer test rapportage
         if: always()
         run: |
@@ -78,3 +86,18 @@ jobs:
           destination-github-username: BRP-API
           target-branch: test-reports/historie
           commit-message: "test rapporten van build: ${{ steps.build-run-id.outputs.build-run }}"
+
+      - name: Login to GitHub Container Registry
+        if: always()
+        uses: docker/[email protected]
+        with:
+          registry: ghcr.io
+          username: ${{ github.actor }}
+          password: ${{ secrets.GITHUB_TOKEN }}
+      - name: Tag & push app container images naar registry
+        if: always()
+        run: |
+          docker tag ${{ env.DATA_SERVICE_CONTAINER_IMAGE }}:${{ steps.app-version.outputs.version }}-${{ steps.build-run-id.outputs.build-run }} ${{ env.APP_CONTAINER_IMAGE }}:${{ steps.app-version.outputs.version }}-${{ steps.build-run-id.outputs.build-run }}
+          docker tag ${{ env.DATA_SERVICE_CONTAINER_IMAGE }}:${{ steps.app-version.outputs.version }}-${{ steps.build-run-id.outputs.build-run }} ${{ env.APP_CONTAINER_IMAGE }}:latest
+          docker push ${{ env.APP_CONTAINER_IMAGE }}:latest
+          docker push ${{ env.APP_CONTAINER_IMAGE }}:${{ steps.app-version.outputs.version }}-${{ steps.build-run-id.outputs.build-run }}
diff --git a/...g-verblijfplaats-met-peildatum/test/peildatum-filtering-opeenvolgend-onbekend-gba.feature b/...g-verblijfplaats-met-peildatum/test/peildatum-filtering-opeenvolgend-onbekend-gba.feature
diff --git a/...pleeg-verblijfplaats-met-peildatum/test/peildatum-filtering-opeenvolgend-onbekend.feature b/...pleeg-verblijfplaats-met-peildatum/test/peildatum-filtering-opeenvolgend-onbekend.feature
diff --git a/features/raadpleeg-verblijfplaats-met-periode/opschorting-bijhouding.feature b/features/raadpleeg-verblijfplaats-met-periode/opschorting-bijhouding.feature
@@ -9,7 +9,7 @@ Functionaliteit: raadplegen historie met periode op persoonslijst met opschortin
       | 0800                 | Korte straatnaam   |
 
   Regel: Voor een persoon op een logisch verwijderde persoonslijst wordt geen verblijfplaatshistorie geleverd
-    Een persoonslijst is logisch verwijderd wanneer reden opschorting bijhouding is gelijk aan "W" (wissen)
+         Een persoonslijst is logisch verwijderd wanneer reden opschorting bijhouding is gelijk aan "W" (wissen)
 
     Abstract Scenario: Gevraagde persoon heeft een logisch verwijderde persoonslijst en <sub scenario>
       Gegeven de persoon met burgerservicenummer '000000024' is ingeschreven op adres 'A1' met de volgende gegevens
@@ -54,7 +54,7 @@ Functionaliteit: raadplegen historie met periode op persoonslijst met opschortin
 
 
   Regel: Voor een persoon met afgevoerde persoonslijst wordt geen verblijfplaatshistorie geleverd
-    Een persoonslijst is afgevoerd wanneer reden opschorting bijhouding is gelijk aan "F" (fout)
+         Een persoonslijst is afgevoerd wanneer reden opschorting bijhouding is gelijk aan "F" (fout)
 
     Abstract Scenario: Gevraagde persoon heeft een afgevoerde persoonslijst en <sub scenario>
       Gegeven de persoon met burgerservicenummer '000000024' is ingeschreven op adres 'A1' met de volgende gegevens

diff --git a/features/raadpleeg-verblijfplaats-met-periode/periode-filtering.feature b/features/raadpleeg-verblijfplaats-met-periode/periode-filtering.feature
@@ -263,10 +263,10 @@ Functionaliteit: alleen verblijfplaatsen die binnen de gevraagde periode vallen
       | datumVan            | 2008-10-14          |
       | datumTot            | 2020-07-30          |
       Dan heeft de response verblijfplaatsen met de volgende gegevens
-      | gemeenteVanInschrijving.code | gemeenteVanInschrijving.omschrijving | adresseerbaarObjectIdentificatie | straat        | datumAanvangAdreshouding | datumAanvangVolgendeAdreshouding |functieAdres.code|functieAdres.omschrijving|
-      | 0800                         | Hoogeloon, Hapert en Casteren        | 0800010000000003                 | Derde straat  | 20181014                 |                                  |W|woonadres|
-      | 0800                         | Hoogeloon, Hapert en Casteren        | 0800010000000002                 | Tweede straat | 20160526                 | 20181014                         |W|woonadres|
-      | 0800                         | Hoogeloon, Hapert en Casteren        | 0800010000000001                 | Eerste straat | 20100818                 | 20160526                         |W|woonadres|
+      | gemeenteVanInschrijving.code | gemeenteVanInschrijving.omschrijving | adresseerbaarObjectIdentificatie | straat        | datumAanvangAdreshouding | datumAanvangVolgendeAdreshouding | functieAdres.code | functieAdres.omschrijving |
+      | 0800                         | Hoogeloon, Hapert en Casteren        | 0800010000000003                 | Derde straat  | 20181014                 |                                  | W                 | woonadres                 |
+      | 0800                         | Hoogeloon, Hapert en Casteren        | 0800010000000002                 | Tweede straat | 20160526                 | 20181014                         | W                 | woonadres                 |
+      | 0800                         | Hoogeloon, Hapert en Casteren        | 0800010000000001                 | Eerste straat | 20100818                 | 20160526                         | W                 | woonadres                 |
 
     Scenario: periode ligt geheel tussen datum aanvang en datum aanvang volgende
       Gegeven de persoon met burgerservicenummer '000000012' is ingeschreven op adres 'A1' met de volgende gegevens
@@ -285,5 +285,5 @@ Functionaliteit: alleen verblijfplaatsen die binnen de gevraagde periode vallen
       | datumVan            | 2016-07-01          |
       | datumTot            | 2017-01-01          |
       Dan heeft de response verblijfplaatsen met de volgende gegevens
-      | gemeenteVanInschrijving.code | gemeenteVanInschrijving.omschrijving | adresseerbaarObjectIdentificatie | straat | datumAanvangAdreshouding | datumAanvangVolgendeAdreshouding | functieAdres.code | functieAdres.omschrijving |
-      | 0800 | Hoogeloon, Hapert en Casteren | 0800010000000002 | Tweede straat | 20160526 | 20181014 | W | woonadres |
+      | gemeenteVanInschrijving.code | gemeenteVanInschrijving.omschrijving | adresseerbaarObjectIdentificatie | straat        | datumAanvangAdreshouding | datumAanvangVolgendeAdreshouding | functieAdres.code | functieAdres.omschrijving |
+      | 0800                         | Hoogeloon, Hapert en Casteren        | 0800010000000002                 | Tweede straat | 20160526                 | 20181014                         | W                 | woonadres                 |
diff --git a/...ltering-opeenvolgend-onbekend-gba.feature → ...e-filtering-opeenvolgend-onbekend.feature b/...ltering-opeenvolgend-onbekend-gba.feature → ...e-filtering-opeenvolgend-onbekend.feature
diff --git a/features/step_definitions/world.js b/features/step_definitions/world.js
@@ -35,7 +35,7 @@ function configureSqlSettings(context) {
 
 function configureOAuthSettings(context) {
     context.oAuth = {
-        enable: true,
+        enable: false,
         accessTokenUrl: 'http://identityserver:6000/connect/token',
         clients:[
             {