8000 Make user attribute option for SAML mappers required · keycloak/keycloak@3794647 · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Make user attribute option for SAML mappers required #28241

Make user attribute option for SAML mappers required

Make user attribute option for SAML mappers required #28241

Triggered via pull request April 25, 2025 10:26
Status Success
Total duration 20m 19s
Artifacts 5

js-ci.yml

on: pull_request
8000
Check conditional workflows and jobs
11s
Check conditional workflows and jobs
Matrix: Admin UI E2E
Status Check - Keycloak JavaScript CI
8s
Status Check - Keycloak JavaScript CI
Fit to window
Zoom out
Zoom in

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
0