[kbn-test] call Kibana API to fetch current user profile #186279
Conversation
|
/ci |
dmlemeshko
added
release_note:skip
|
|
||
| test(`'getUserData' should call security API and return user profile data`, async () => { | ||
| const testData: UserProfile = { | ||
| username: '6ta90xc', |
There was a problem hiding this comment.
Is this a hashed value or smth?
There was a problem hiding this comment.
Cloud generates unique string so I just put a random one. The most important to show it is not related to role/email/full name
| @@ -117,6 +134,15 @@ describe('SamlSessionManager', () => { | |||
|
|
|||
| test(`doesn't throw error when supportedRoles is not defined`, async () => { | |||
| const nonExistingRole = 'tester'; | |||
| const testData: UserProfile = { | |||
| username: '6ta90xc', | |||
There was a problem hiding this comment.
Same as above? What is this value?
There was a problem hiding this comment.
just random string, user has no control over it.
|
/ci |
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]Public APIs missing comments
Public APIs missing exports
History
To update your PR or re-run it, just comment with: |
) Related to elastic#185870 In this PR I move user_profile fetching out of SAML session creation to its `sessionManager.getUserData()`: - rely on Kibana Security API for both local/Kibana CI and MKI cases (currently it is cloud and cached on saml session creation) - do not cache profile data in test service, Kibana API is fast enough I deleted the tests that no longer relevant
dmlemeshko
deleted the
kbn-test/always-call-security-api-for-user-profile
branch
Related to #185870
In this PR I move user_profile fetching out of SAML session creation to its
sessionManager.getUserData():I deleted the tests that no longer relevant