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.

Sign up for GitHub

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

Jump to bottom

PIN-6263 Compare readmodels #1601

Draft
wants to merge 557 commits into
base: PIN_6218-eservice-queries
Choose a base branch
from
Draft

PIN-6263 Compare readmodels #1601

wants to merge 557 commits into from

Conversation

taglioni-r
Copy link
Contributor

No description provided.

shuyec and others added 30 commits February 21, 2025 10:35
@shuyec
Merge branch 'PIN-6073_scaffolding-readmodel-package' into PIN-6093_c…
da2fe19 
…lient-splitter
@shuyec
Merge branch 'PIN-6073_scaffolding-readmodel-package' into PIN-6087_a…
38315d0 
…ttribute-splitter
@paolomanca-pagopa
Merge branch 'PIN-6073_scaffolding-readmodel-package' into PIN-6095_c…
df91373 
…lient-jwk-key-splitter
@paolomanca-pagopa
Merge branch 'PIN-6073_scaffolding-readmodel-package' into PIN-6096_p…
ae82529 
…roducer-jwk-key-splitter
@paolomanca-pagopa
Merge branch 'PIN-6073_scaffolding-readmodel-package' into PIN-6089_a…
09e668b 
…greement-splitter
@shuyec
Merge branch 'PIN-6073_scaffolding-readmodel-package' into PIN-6094_p…
405f7b2 
…roducer-keychain-splitter
@shuyec
Merge branch 'PIN-6073_scaffolding-readmodel-package' into PIN-6091_a…
a7904b9 
…dd-delegation-splitter
@shuyec
Merge branch 'PIN-6073_scaffolding-readmodel-package' into PIN-6092_a…
bebae24 
…dd-purpose-splitters
@paolomanca-pagopa
fix import
63a68c8
@taglioni-r
Add client aggregator
44885d5
@shuyec
Squashed commit of the following:
9749f77 
commit 5ee25bb
Author: Roberto Taglioni <[email protected]>
Date:   Fri Feb 21 10:53:34 2025 +0100

    Update test titles

commit 4021d18
Author: Roberto Taglioni <[email protected]>
Date:   Fri Feb 21 10:24:24 2025 +0100

    Refactor null check

commit 0d3e4f3
Author: Roberto Taglioni <[email protected]>
Date:   Fri Feb 21 10:17:58 2025 +0100

    Fix aggregator

commit 7634fbe
Author: Roberto Taglioni <[email protected]>
Date:   Fri Feb 21 10:17:51 2025 +0100

    Add aggregator test

commit 963ea0e
Merge: e79ca9e 6fa6581
Author: Roberto Taglioni <[email protected]>
Date:   Fri Feb 21 09:43:51 2025 +0100

    Merge branch 'PIN-6090_tenant-splitters' into PIN-6101_tenant-aggregators

commit e79ca9e
Author: Roberto Taglioni <[email protected]>
Date:   Thu Feb 20 15:17:15 2025 +0100

    Add tenant array aggregator

commit fcca8e8
Author: Roberto Taglioni <[email protected]>
Date:   Thu Feb 20 12:34:33 2025 +0100

    Fix test

commit 9c722d1
Author: Roberto Taglioni <[email protected]>
Date:   Thu Feb 20 12:31:44 2025 +0100

    Fix

commit 88e4ad9
Author: Roberto Taglioni <[email protected]>
Date:   Thu Feb 20 12:30:16 2025 +0100

    Fix features

commit 0a0146d
Merge: 2c6b2a4 7e53ea4
Author: Roberto Taglioni <[email protected]>
Date:   Thu Feb 20 12:23:20 2025 +0100

    Merge branch 'PIN-6090_tenant-splitters' into PIN-6101_tenant-aggregators

commit 2c6b2a4
Author: Roberto Taglioni <[email protected]>
Date:   Wed Feb 19 15:26:53 2025 +0100

    Fix import

commit d43d356
Merge: d2b1cc7 58ccee9
Author: Roberto Taglioni <[email protected]>
Date:   Wed Feb 19 15:26:32 2025 +0100

    Merge branch 'PIN-6090_tenant-splitters' into PIN-6101_tenant-aggregators

commit d2b1cc7
Author: Roberto Taglioni <[email protected]>
Date:   Mon Feb 17 15:08:14 2025 +0100

    Fix version

commit f673531
Author: Roberto Taglioni <[email protected]>
Date:   Mon Feb 17 14:38:51 2025 +0100

    Reorder properties

commit d908c55
Author: Roberto Taglioni <[email protected]>
Date:   Mon Feb 17 14:37:03 2025 +0100

    Add tenant aggregator

commit 9fc09d8
Author: Roberto Taglioni <[email protected]>
Date:   Mon Feb 17 12:15:29 2025 +0100

    Add todo
@taglioni-r
Add aggregator
f8dd1f7
@shuyec
Squashed commit of the following:
01e948b 
commit 30b3bbb
Author: Stefano Hu <[email protected]>
Date:   Fri Feb 21 15:44:14 2025 +0100

    Add aggregateDelegationArray

commit 8fe6f60
Author: Stefano Hu <[email protected]>
Date:   Fri Feb 21 12:08:15 2025 +0100

    Remove console.log

commit 4e95fa2
Author: Stefano Hu <[email protected]>
Date:   Fri Feb 21 12:07:59 2025 +0100

    Update optional properties logic in getMockDelegation

commit 88dcae6
Author: Stefano Hu <[email protected]>
Date:   Fri Feb 21 11:26:23 2025 +0100

    Replace toEqual with toMatchObject

commit 3fad9ab
Merge: 9f3f492 a7904b9
Author: shuyec <[email protected]>
Date:   Fri Feb 21 11:25:47 2025 +0100

    Merge branch 'PIN-6091_add-delegation-splitter' into PIN-6102_delegation-aggregator

commit 9f3f492
Author: Roberto Taglioni <[email protected]>
Date:   Thu Feb 20 15:09:00 2025 +0100

    Add array aggregator

commit 7f4a33e
Author: Roberto Taglioni <[email protected]>
Date:   Thu Feb 20 12:52:21 2025 +0100

    Fix test

commit 97d8de0
Author: Roberto Taglioni <[email protected]>
Date:   Thu Feb 20 12:51:59 2025 +0100

    Fix aggregator

commit c525553
Merge: ee91570 b483d78
Author: Roberto Taglioni <[email protected]>
Date:   Thu Feb 20 12:50:56 2025 +0100

    Merge branch 'PIN-6091_add-delegation-splitter' into PIN-6102_delegation-aggregator

commit ee91570
Merge: 5dcb759 9fa0a59
Author: Roberto Taglioni <[email protected]>
Date:   Thu Feb 20 12:35:51 2025 +0100

    Merge branch 'PIN-6091_add-delegation-splitter' into PIN-6102_delegation-aggregator

commit 5dcb759
Author: Stefano Hu <[email protected]>
Date:   Wed Feb 19 18:06:42 2025 +0100

    Add delegation aggregator tests

commit c0de07c
Author: Stefano Hu <[email protected]>
Date:   Wed Feb 19 18:06:19 2025 +0100

    Fix optional properties in delegation aggregator

commit 39f3adb
Author: Roberto Taglioni <[email protected]>
Date:   Wed Feb 19 16:02:19 2025 +0100

    Fix import

commit c4c52e7
Merge: e3b646e 7372641
Author: Roberto Taglioni <[email protected]>
Date:   Wed Feb 19 16:01:31 2025 +0100

    Merge branch 'PIN-6091_add-delegation-splitter' into PIN-6102_delegation-aggregator

commit e3b646e
Author: Roberto Taglioni <[email protected]>
Date:   Mon Feb 17 15:29:44 2025 +0100

    Add delegation aggregator

commit a644a8d
Author: Roberto Taglioni <[email protected]>
Date:   Mon Feb 17 14:57:36 2025 +0100

    Add todo
@shuyec
Squashed commit of the following:
3556c60 
commit 5548e58
Author: Stefano Hu <[email protected]>
Date:   Fri Feb 21 15:18:25 2025 +0100

    Add aggregatePurposeArray

commit f860220
Author: Stefano Hu <[email protected]>
Date:   Fri Feb 21 14:48:22 2025 +0100

    Rename purposeSQLToPurpose to aggregatePurpose

commit d39fe58
Author: Stefano Hu <[email protected]>
Date:   Fri Feb 21 12:17:56 2025 +0100

    Fix purpose aggregator "null -> undefined" test

commit f5ef2f8
Author: Stefano Hu <[email protected]>
Date:   Fri Feb 21 12:09:51 2025 +0100

    Replace toEqual with toMatchObject

commit 0b01ab7
Merge: 92468af bebae24
Author: shuyec <[email protected]>
Date:   Fri Feb 21 12:09:01 2025 +0100

    Merge branch 'PIN-6092_add-purpose-splitters' into PIN-6103_add-purpose-aggregators

commit 92468af
Author: Stefano Hu <[email protected]>
Date:   Wed Feb 19 18:11:47 2025 +0100

    Remove unnecessary variables in aggregator test

commit a886f66
Author: Stefano Hu <[email protected]>
Date:   Wed Feb 19 17:01:58 2025 +0100

    Add purpose aggregator tests

commit 468984e
Merge: c3598a0 991a5f2
Author: shuyec <[email protected]>
Date:   Wed Feb 19 16:54:58 2025 +0100

    Merge branch 'PIN-6092_add-purpose-splitters' into PIN-6103_add-purpose-aggregators

commit c3598a0
Author: Stefano Hu <[email protected]>
Date:   Wed Feb 19 16:52:38 2025 +0100

    Fix optional properties in purpose aggregator

commit f284536
Author: Stefano Hu <[email protected]>
Date:   Wed Feb 19 15:46:35 2025 +0100

    Fix readmodel-models imports

commit 6c86adc
Merge: 3ab1303 6cc5506
Author: shuyec <[email protected]>
Date:   Wed Feb 19 15:45:01 2025 +0100

    Merge branch 'PIN-6092_add-purpose-splitters' into PIN-6103_add-purpose-aggregators

commit 3ab1303
Author: Stefano Hu <[email protected]>
Date:   Wed Feb 19 11:09:22 2025 +0100

    Fix missing properties in purpose aggregator

commit 9e71f4a
Author: Stefano Hu <[email protected]>
Date:   Wed Feb 19 09:51:51 2025 +0100

    Fix reduce in purposeRiskAnalysisFormSQLToPurposeRiskAnalysisForm

commit 19c829b
Author: Stefano Hu <[email protected]>
Date:   Mon Feb 17 17:24:03 2025 +0100

    Fix riskAnalysisId for purposeRiskAnalysisForm

commit a41eb2e
Merge: b972339 2ad2be0
Author: shuyec <[email protected]>
Date:   Mon Feb 17 17:21:24 2025 +0100

    Merge branch 'PIN-6092_add-purpose-splitters' into PIN-6103_add-purpose-aggregators

commit b972339
Author: Stefano Hu <[email protected]>
Date:   Mon Feb 17 17:16:11 2025 +0100

    Replace .parse with explicit objects

commit 87f0d56
Author: Stefano Hu <[email protected]>
Date:   Mon Feb 17 15:32:16 2025 +0100

    Add parse for PurposeRiskAnalysisForm

commit 15d6f2a
Author: Stefano Hu <[email protected]>
Date:   Mon Feb 17 15:27:34 2025 +0100

    Add purpose aggregator logic
@taglioni-r
Add aggregator
2748915
@taglioni-r
Add aggregator
b32f93c
@taglioni-r
Add aggregator
5fce14e
@shuyec
Squashed commit of the following:
e603456 
commit eafedcf
Author: Paolo Manca <[email protected]>
Date:   Fri Feb 21 19:08:48 2025 +0100

    adding agreement array aggregator

commit 825c4cf
Author: Paolo Manca <[email protected]>
Date:   Fri Feb 21 18:41:06 2025 +0100

    adding agreement test aggregator

commit 226fd52
Merge: 4ec7ce2 63a68c8
Author: Paolo Manca <[email protected]>
Date:   Fri Feb 21 13:52:41 2025 +0100

    Merge branch 'PIN-6089_agreement-splitter' into PIN-6100_agreement-aggregator

commit 4ec7ce2
Author: Paolo Manca <[email protected]>
Date:   Fri Feb 21 12:32:29 2025 +0100

    agreement aggregator

commit b3e7373
Merge: b42eef0 09e668b
Author: Paolo Manca <[email protected]>
Date:   Fri Feb 21 11:03:14 2025 +0100

    Merge branch 'PIN-6089_agreement-splitter' into PIN-6100_agreement-aggregator

commit b42eef0
Author: Paolo Manca <[email protected]>
Date:   Wed Feb 19 19:52:25 2025 +0100

    draft version agreement aggregator

commit a35363d
Author: Paolo Manca <[email protected]>
Date:   Wed Feb 19 17:18:23 2025 +0100

    add scaffolding for agreement aggreagator and test
@taglioni-r
Renaming
2a1b99c
@taglioni-r
Add attribute queries
b387372
@shuyec
Add readModelServiceBuilder for client JWK key
f0057e2
@taglioni-r
Refactor
17371bb
@taglioni-r
Add agreement queries
a010e8a
@shuyec
Add readModelServiceBuilder for delegation
8d89b78
@shuyec
Revert aggregateDelegation
cdf106f
@taglioni-r
Refactor
264112e
@taglioni-r
Add tenant queries
c682d7c
@shuyec
Fix typo
c2c8f79
@taglioni-r
Add producer JWK key queries
5ae42c3
@shuyec
Add readModelServiceBuilder for purpose
6b99cb5
shuyec and others added 30 commits March 4, 2025 14:29
@shuyec
Fix typos
3277ad5
@shuyec
Add getAllAgreements
b25b937
@taglioni-r
Merge branch 'PIN-6219_agreement-queries' into PIN-6263_compare-readm…
ac0a900 
…odels
@taglioni-r
Add agreement checks
1afaadb
@taglioni-r
Merge branch 'PIN-6222_purpose-queries' into PIN-6263_compare-readmodels
7025d2f
@taglioni-r
Add purpose checks
8e75e83
@taglioni-r
Merge branch 'PIN-6223_client-queries' into PIN-6263_compare-readmodels
30630b7
@taglioni-r
Add client checks
c21acfb
@taglioni-r
Merge branch 'PIN-6225_client-jwk-key-CRUD' into PIN-6263_compare-rea…
84e6b6a 
…dmodels
@taglioni-r
Add key checks
7acfbe3
@taglioni-r
Merge branch 'PIN-6224_producer-keychain-queries' into PIN-6263_compa…
8b57114 
…re-readmodels
@taglioni-r
Add producer keychain checks
80d461f
@taglioni-r
Merge branch 'PIN-6226_add-producer-jwk-key-queries' into PIN-6263_co…
c1bebf8 
…mpare-readmodels
@taglioni-r
Add producer keys checks
357e365
@taglioni-r
Merge branch 'PIN-6221_delegation-queries' into PIN-6263_compare-read…
e518ec8 
…models
@taglioni-r
Add delegation checks
2ebf5a3
@taglioni-r
Naming
9f0adee
@taglioni-r
Add agreement test
9241e37
@taglioni-r
Add tests
835fb7a
@paolomanca-pagopa
fix Set non-uniquenesse producer keychain
17e3325
@taglioni-r
Add client checks
9424bf7
@taglioni-r
Merge branch 'PIN-6224_producer-keychain-queries' into PIN-6263_compa…
db47ff7 
…re-readmodels
@taglioni-r
Add test
e099b53
@taglioni-r
Add tests
3c342f8
@taglioni-r
Fix
95577f2
@taglioni-r
Fix undefined
741da12
@taglioni-r
Fix producer keychain
02c4c9d
@taglioni-r
Fix tests
019a5e0
@taglioni-r
Use logger
7627927
@taglioni-r
Remove skip
f03246b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@beetlecrunch beetlecrunch Awaiting requested review from beetlecrunch beetlecrunch will be requested when the pull request is marked ready for review beetlecrunch is a code owner

@galales galales Awaiting requested review from galales galales will be requested when the pull request is marked ready for review galales is a code owner

@AsterITA AsterITA Awaiting requested review from AsterITA AsterITA will be requested when the pull request is marked ready for review AsterITA is a code owner

@Carminepo2 Carminepo2 Awaiting requested review from Carminepo2 Carminepo2 will be requested when the pull request is marked ready for review Carminepo2 is a code owner

Assignees
No one assigned
Labels
readmodel-sql
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@taglioni-r @shuyec @paolomanca-pagopa