Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SUPPORT-13459] fix vb_verblijfsobject_adres voor Oracle #1694

Merged
merged 2 commits into from
Feb 4, 2023

Conversation

geertoff
Copy link
Contributor

@geertoff geertoff commented Feb 3, 2023

Het attribuut 'maaktdeeluitvan' bevatte ook de gebruiksdoelen waar normaal gesproken het ID van het pand wordt opgeslagen. Door het script te vergelijken met het postgresql script, heb ik deze aanpassingen verricht. De aanleiding voor deze aanpassing was ticket SUPPORT-13459.

@geertoff geertoff requested a review from matthijsln February 3, 2023 09:19
@mprins mprins changed the title bewerk de vb_verblijfsobject_adres in oracle [SUPPORT-13459] fix vb_verblijfsobject_adres in oracle Feb 3, 2023
Copy link
Member

@mprins mprins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ik denk dat er iets scheef staat; de views worden niet aangemaakt.

Error:  Failures: 
Error:    BAGViewsTest.testBasisViewsBAG:107 lijsten met basis views zijn ongelijk ==> expected: <[vb_adres, vb_adresseerbaar_object_geometrie, vb_ligplaats_adres, vb_standplaats_adres, vb_verblijfsobject_adres]> but was: <[vb_adres, vb_ligplaats_adres, vb_standplaats_adres]>
Error:    MaterializedViewsTest.testBasisMViews:190 lijsten met materialized views zijn ongelijk ==> expected: <[mb_adres, mb_adres_bag, mb_adresseerbaar_object_geometrie_bag, mb_avg_koz_rechth, mb_avg_koz_rechth_bag, mb_avg_subject, mb_avg_zr_rechth, mb_ben_obj_nevenadres, mb_benoemd_obj_adres, mb_kad_onrrnd_zk_adres, mb_kad_onrrnd_zk_adres_bag, mb_kad_onrrnd_zk_archief, mb_koz_rechth, mb_koz_rechth_bag, mb_pand, mb_percelenkaart, mb_subject, mb_util_app_re_kad_perceel, mb_zr_rechth]> but was: <[mb_adres, mb_adres_bag, mb_avg_koz_rechth, mb_avg_subject, mb_avg_zr_rechth, mb_ben_obj_nevenadres, mb_benoemd_obj_adres, mb_kad_onrrnd_zk_adres, mb_kad_onrrnd_zk_archief, mb_koz_rechth, mb_pand, mb_percelenkaart, mb_subject, mb_util_app_re_kad_perceel, mb_zr_rechth]>

@mprins mprins force-pushed the edit-208-bag2-oracle branch from bba09fd to a5cf188 Compare February 3, 2023 12:28
@mprins mprins added this to the 3.0.0 milestone Feb 3, 2023
@geertoff geertoff requested a review from mprins February 3, 2023 15:14
@codecov
Copy link

codecov bot commented Feb 3, 2023

Codecov Report

Merging #1694 (9ae8a13) into master (d544ae8) will decrease coverage by 0%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##             master   #1694    +/-   ##
=========================================
- Coverage        37%     37%    -0%     
- Complexity      810     948   +138     
=========================================
  Files           414     414            
  Lines         21167   21167            
  Branches       2004    2004            
=========================================
- Hits           7771    7770     -1     
  Misses        12774   12774            
- Partials        622     623     +1     
Impacted Files Coverage Δ
...nl/b3p/brmo/util/http/HttpSeekableByteChannel.java 70% <0%> (-1%) ⬇️

@mprins mprins changed the title [SUPPORT-13459] fix vb_verblijfsobject_adres in oracle [SUPPORT-13459] fix vb_verblijfsobject_adres voor Oracle Feb 4, 2023
@mprins mprins merged commit 5634ce4 into master Feb 4, 2023
@mprins mprins deleted the edit-208-bag2-oracle branch February 4, 2023 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants