-
Notifications
You must be signed in to change notification settings - Fork 2
PIN-7916 add personal data filter #2485
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
Conversation
…ation for personalData
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Just some comments for the tests
packages/backend-for-frontend/test/api/catalogRouter/getCatalog.test.ts
Outdated
Show resolved
Hide resolved
...ges/backend-for-frontend/test/api/eserviceTemplateRouter/getCatalogEServiceTemplates.test.ts
Outdated
Show resolved
Hide resolved
packages/eservice-template-process/test/api/getEServiceTemplates.test.ts
Outdated
Show resolved
Hide resolved
…getCatalogEServiceTemplates.test.ts Co-authored-by: shuyec <[email protected]>
…g.test.ts Co-authored-by: shuyec <[email protected]>
…es.test.ts Co-authored-by: shuyec <[email protected]>
); | ||
}); | ||
|
||
it.each([true, false])( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a test where you check for undefined
and return all results?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since there's already a test for the /eservices
endpoint without query parameters, the undefined case is already implicitly covered, so adding it here would be redundant.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved, just propose to add a test if not present.
This PR os for adding the personalData filter in
getEservices
andgetEserviceTemplates