Skip to content

Merge branch 'main' into improve-group-update-performance #2

Merge branch 'main' into improve-group-update-performance

Merge branch 'main' into improve-group-update-performance #2

Triggered via push January 23, 2024 14:46
Status Success
Total duration 13m 38s
Artifacts 3

js-ci.yml

on: push
Check conditional workflows and jobs
7s
Check conditional workflows and jobs
Matrix: Admin UI E2E
Account UI E2E
2m 2s
Account UI E2E
Status Check - Keycloak JavaScript CI
10s
Status Check - Keycloak JavaScript CI
Fit to window
Zoom out
Zoom in

Annotations

1 error, 28 warnings, and 1 notice
[chromium] › my-resources.spec.ts:47:3 › My resources page › One is shared with alice: js/apps/account-ui/test/my-resources.spec.ts#L53
1) [chromium] › my-resources.spec.ts:47:3 › My resources page › One is shared with alice ───────── Error: expect(received).toEqual(expected) // deep equality - Expected - 3 + Received + 1 - Array [ - "one", - ] + Array [] 51 | await page.getByTestId("sharedWithMe").click(); 52 | const rowData = await page.getByTestId("row[0].name").allTextContents(); > 53 | expect(rowData).toEqual(["one"]); | ^ 54 | }); 55 | }); 56 | at /home/runner/work/keycloak/keycloak/js/apps/account-ui/test/my-resources.spec.ts:53:21
Admin Client: js/libs/keycloak-admin-client/src/resources/agent.ts#L151
Unnecessary conditional, value is always truthy
Admin Client: js/libs/keycloak-admin-client/src/resources/agent.ts#L304
This callback should return a conditional, but return is always falsy
Admin Client: js/libs/keycloak-admin-client/src/resources/clients.ts#L701
Unnecessary conditional, value is always truthy
Admin Client: js/libs/keycloak-admin-client/src/utils/auth.ts#L78
Unnecessary conditional, value is always truthy
Keycloak Masthead: js/libs/keycloak-masthead/src/Masthead.tsx#L58
Unnecessary optional chain on a non-nullish value
Keycloak Masthead: js/libs/keycloak-masthead/src/Masthead.tsx#L66
Unnecessary optional chain on a non-nullish value
Keycloak Masthead: js/libs/keycloak-masthead/src/Masthead.tsx#L72
Unnecessary optional chain on a non-nullish value
Keycloak Masthead: js/libs/keycloak-masthead/src/Masthead.tsx#L107
Unnecessary optional chain on a non-nullish value
UI Shared: js/libs/ui-shared/src/user-profile/MultiInputComponent.tsx#L66
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#L89
React Hook useEffect has a missing dependency: 'name'. Either include it or remove the dependency array
UI Shared: js/libs/ui-shared/src/user-profile/UserProfileFields.tsx#L187
React Hook useMemo has a missing dependency: 'value'. Either include it or remove the dependency array
UI Shared: js/libs/ui-shared/src/utils/useStorageItem.ts#L23
React Hook useCallback has missing dependencies: 'keyName' and 'storageArea'. Either include them or remove the dependency array
UI Shared: js/libs/ui-shared/src/utils/useStorageItem.ts#L48
React Hook useEffect has a missing dependency: 'defaultValue'. Either include it or remove the dependency array. If 'setInnerValue' needs the current value of 'defaultValue', you can also switch to useReducer instead of useState and read 'defaultValue' in the reducer
UI Shared: js/libs/ui-shared/src/utils/useStoredState.ts#L34
React Hook useCallback has a missing dependency: 'setStoredValue'. Either include it or remove the dependency array
Account UI: js/apps/account-ui/src/resources/EditTheResource.tsx#L36
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#L64
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
Admin UI: js/apps/admin-ui/cypress/e2e/client_scopes_test.spec.ts#L181
Do not wait for arbitrary time periods
Admin UI: js/apps/admin-ui/cypress/e2e/clients_test.spec.ts#L199
Do not wait for arbitrary time periods
Admin UI: js/apps/admin-ui/cypress/e2e/clients_test.spec.ts#L209
Do not wait for arbitrary time periods
Admin UI: js/apps/admin-ui/cypress/e2e/clients_test.spec.ts#L538
Do not wait for arbitrary time periods
Admin UI: js/apps/admin-ui/cypress/e2e/events_test.spec.ts#L33
Unexpected skipped mocha test
Admin UI: js/apps/admin-ui/cypress/e2e/events_test.spec.ts#L71
Do not wait for arbitrary time periods
Admin UI: js/apps/admin-ui/cypress/e2e/events_test.spec.ts#L106
Do not wait for arbitrary time periods
Admin UI: js/apps/admin-ui/cypress/e2e/group_test.spec.ts#L266
Unexpected skipped mocha test
Admin UI: js/apps/admin-ui/cypress/e2e/i18n_test.spec.ts#L43
Unnecessary conditional, value is always truthy
🎭 Playwright Run Summary
1 flaky [chromium] › my-resources.spec.ts:47:3 › My resources page › One is shared with alice ────────── 1 skipped 11 passed (22.8s)

Artifacts

Produced during runtime
Name Size
account-ui-playwright-report Expired
2.33 MB
account-ui-server-log Expired
3.33 KB
keycloak Expired
174 MB