Make user attribute option for SAML mappers required #28241
js-ci.yml
on: pull_request
8000
Admin Client
UI Shared
Account UI
Admin UI
Admin Client
38s
UI Shared
47s
Account UI
52s
Admin UI
2m 11s
Matrix: Admin UI E2E
Annotations
2 errors, 29 warnings, and 1 notice
[chromium] › test/realm-roles/main.spec.ts:121:3 › Realm roles test › Add associated roles test:
js/apps/admin-ui/test/utils/table.ts#L103
1) [chromium] › test/realm-roles/main.spec.ts:121:3 › Realm roles test › Add associated roles test
Error: Row manage-consent not found: ,accountmanage-account-links,False,role_manage-account-links,,,offline_access,False,role_offline-access,
at utils/table.ts:103
101 | const rowIndex = rows.findIndex((r) => r.includes(row as string));
102 | if (rowIndex === -1) {
> 103 | throw new Error(`Row ${row} not found: ${rows}`);
| ^
104 | }
105 | row = rowIndex;
106 | }
at clickSelectRow (/home/runner/work/keycloak/keycloak/js/apps/admin-ui/test/utils/table.ts:103:13)
at pickRole (/home/runner/work/keycloak/keycloak/js/apps/admin-ui/test/utils/roles.ts:26:3)
at /home/runner/work/keycloak/keycloak/js/apps/admin-ui/test/realm-roles/main.spec.ts:146:5
|
[chromium] › test/realm-roles/main.spec.ts:121:3 › Realm roles test › Add associated roles test:
js/apps/admin-ui/test/utils/table.ts#L71
1) [chromium] › test/realm-roles/main.spec.ts:121:3 › Realm roles test › Add associated roles test
Retry #1 ───────────────────────────────────────────────────────────────────────────────────────
Error: locator.waitFor: Error: strict mode violation: getByLabel('Associated roles', { exact: true }).locator('tbody') resolved to 2 elements:
1) <tbody role="rowgroup" class="pf-v5-c-table__tbody">…</tbody> aka getByTestId('assigned-roles').locator('tbody')
2) <tbody role="rowgroup" class="pf-v5-c-table__tbody">…</tbody> aka getByText('delete-accountaccountrole_delete-accountmanage-accountaccountrole_manage-')
Call log:
- waiting for getByLabel('Associated roles', { exact: true }).locator('tbody') to be visible
at utils/table.ts:71
69 | export async function getTableData(page: Page, name: string) {
70 | const table = page.getByLabel(name, { exact: true });
> 71 | await table.locator("tbody").waitFor();
| ^
72 | const rows = await table.locator("tbody tr").elementHandles();
73 |
74 | const tableData = await Promise.all(
at getTableData (/home/runner/work/keycloak/keycloak/js/apps/admin-ui/test/utils/table.ts:71:32)
at clickSelectRow (/home/runner/work/keycloak/keycloak/js/apps/admin-ui/test/utils/table.ts:100:24)
at pickRole (/home/runner/work/keycloak/keycloak/js/apps/admin-ui/test/utils/roles.ts:26:9)
at /home/runner/work/keycloak/keycloak/js/apps/admin-ui/test/realm-roles/main.spec.ts:140:11
|
UI Shared:
js/libs/ui-shared/src/context/KeycloakContext.tsx#L46
Unexpected reassignment of a variable which was defined outside of the component. Components and hooks should be pure and side-effect free, but variable reassignment is a form of side-effect. If this variable is used in rendering, use useState instead. (https://react.dev/reference/rules/components-and-hooks-must-be-pure#side-effects-must-run-outside-of-render)
|
UI Shared:
js/libs/ui-shared/src/context/KeycloakContext.tsx#L81
React Hook useEffect has a missing dependency: 'environment.scope'. Either include it or remove the dependency array
|
UI Shared:
js/libs/ui-shared/src/controls/table/KeycloakDataTable.tsx#L151
React Hook useEffect has a missing dependency: 'rowsSelectedOnPage.length'. Either include it or remove the dependency array
|
UI Shared:
js/libs/ui-shared/src/controls/table/KeycloakDataTable.tsx#L489
React Hook useMemo has missing dependencies: 'convertToColumns', 'getNodeText', 'isPaginated', and 'unPaginatedData'. Either include them or remove the dependency array
|
UI Shared:
js/libs/ui-shared/src/select/SingleSelect.tsx#L52
This function accesses a ref value (the `current` property), which may not be accessed during render. (https://react.dev/reference/react/useRef)
|
UI Shared:
js/libs/ui-shared/src/select/SingleSelect.tsx#L52
Ref values (the `current` property) may not be accessed during render. (https://react.dev/reference/react/useRef)
|
UI Shared:
js/libs/ui-shared/src/user-profile/MultiInputComponent.tsx#L72
React Hook useMemo has a missing dependency: 'defaultValue'. Either include it or remove the dependency array
|
UI Shared:
js/libs/ui-shared/src/user-profile/MultiInputComponent.tsx#L99
React Hook useEffect has a missing dependency: 'name'. Either include it or remove the dependency array
|
UI Shared:
js/libs/ui-shared/src/utils/useFetch.ts#L43
React Hook useEffect was passed a dependency list that is not an array literal. This means we can't statically verify whether you've passed the correct dependencies
|
UI Shared:
js/libs/ui-shared/src/utils/useFetch.ts#L43
React Hook useEffect has missing dependencies: 'adminClientCall', 'callback', and 'showBoundary'. Either include them or remove the dependency array. If 'adminClientCall' changes too often, find the parent component that defines it and wrap that definition in useCallback
|
Account UI:
js/apps/account-ui/src/oid4vci/Oid4Vci.tsx#L69
React Hook useEffect has missing dependencies: 'context', 'credentialsIssuer', 'initialSelected', and 'selectOptions'. Either include them or remove the dependency array
|
Account UI:
js/apps/account-ui/src/resources/EditTheResource.tsx#L42
React Hook useEffect has missing dependencies: 'permissions' and 'reset'. Either include them or remove the dependency array
|
Account UI:
js/apps/account-ui/src/resources/ShareTheResource.tsx#L72
React Hook useEffect has a missing dependency: 'append'. Either include it or remove the dependency array
|
Account UI:
js/apps/account-ui/src/utils/usePromise.ts#L73
React Hook useEffect was passed a dependency list that is not an array literal. This means we can't statically verify whether you've passed the correct dependencies
|
Account UI:
js/apps/account-ui/src/utils/usePromise.ts#L73
React Hook useEffect has missing dependencies: 'callback' and 'factory'. Either include them or remove the dependency array. If 'callback' changes too often, find the parent component that defines it and wrap that definition in useCallback
|
Account UI:
js/apps/account-ui/test/oid4vci/oid4vci.spec.ts#L7
Unexpected use of the `.skip()` annotation
|
Account UI:
js/apps/account-ui/test/personal-info/personal-info.spec.ts#L70
Unexpected use of page.waitForSelector()
|
Account UI:
js/apps/account-ui/test/personal-info/personal-info.spec.ts#L84
Unexpected use of page.waitForSelector()
|
Admin UI:
js/apps/admin-ui/src/App.tsx#L56
React Hook useEffect has missing dependencies: 'environment' and 'keycloak'. Either include them or remove the dependency array
|
Admin UI:
js/apps/admin-ui/src/authentication/AuthenticationSection.tsx#L102
Hooks must be called at the top level in the body of a function component or custom hook, and may not be called within function expressions. See the Rules of Hooks (https://react.dev/warnings/invalid-hook-call-warning)
|
Admin UI:
js/apps/admin-ui/src/authentication/AuthenticationSection.tsx#L104
Hooks must be the same function on every render, but this value may change over time to a different function. See https://react.dev/reference/rules/react-calls-components-and-hooks#dont-dynamically-use-hooks
|
Admin UI:
js/apps/admin-ui/src/authentication/AuthenticationSection.tsx#L105
Hooks must be the same function on every render, but this value may change over time to a different function. See https://react.dev/reference/rules/react-calls-components-and-hooks#dont-dynamically-use-hooks
|
Admin UI:
js/apps/admin-ui/src/authentication/AuthenticationSection.tsx#L106
Hooks must be the same function on every render, but this value may change over time to a different function. See https://react.dev/reference/rules/react-calls-components-and-hooks#dont-dynamically-use-hooks
|
Admin UI:
js/apps/admin-ui/src/authentication/DuplicateFlowModal.tsx#L45
React Hook useEffect has missing dependencies: 'setValue' and 't'. Either include them or remove the dependency array
|
Admin UI:
js/apps/admin-ui/src/authentication/EditFlowModal.tsx#L31
React Hook useEffect has a missing dependency: 'reset'. Either include it or remove the dependency array
|
Admin UI:
js/apps/admin-ui/src/authentication/components/DraggableTable.tsx#L62
React Hook useMemo has a missing dependency: 'keyField'. Either include it or remove the dependency array
|
Admin UI:
js/apps/admin-ui/src/authentication/components/EditFlow.tsx#L32
React Hook useEffect has a missing dependency: 'form'. Either include it or remove the dependency array
|
Admin UI:
js/apps/admin-ui/src/authentication/components/ExecutionConfigModal.tsx#L110
React Hook useEffect has a missing dependency: 'setupForm'. Either include it or remove the dependency array
|
Account UI E2E
No files were found with the provided path: js/apps/account-ui/playwright-report. No artifacts will be uploaded.
|
🎭 Playwright Run Summary
1 flaky
[chromium] › test/realm-roles/main.spec.ts:121:3 › Realm roles test › Add associated roles test
1 skipped
342 passed (12.3m)
|
Artifacts
Produced during runtime
Name | Size | Digest | |
---|---|---|---|
account-ui-server-log
|
1 KB |
sha256:30861a59737bad7c832661dd6a17fd0670d4226038ded09ae8642c3070713ca2
|
|
admin-ui-playwright-report-chromium
Expired
|
2.3 MB |
sha256:aed8de33a59e06eb2be37375cc102d42f1fe64cd94e68158c597f8e47939f896
|
|
admin-ui-server-log-chromium
|
3.8 KB |
sha256:fe83af09999d35f8d100ae87cd34800eb29cba118d65bdd388a8051f87a6266e
|
|
keycloak
|
148 MB |
sha256:3f02a8b09a43ec67d15b61e5722937f2e86995777c4867191493b4b1a40634ed
|
|
m2-keycloak.tzts
Expired
|
366 MB |
sha256:0b6237520e8d0795b8c16f45d2a517debf0be8ad7bac6f116e10ca843d4f73fb
|
|