build(deps): bump react-router-dom from 6.27.0 to 6.28.0 in /pro #6901
dependabot[bot]dev_on_pull_request_workflow.yml
on: pull_request
Init job
38s
Tests pro
/
...
/
Style quality check
9m 32s
Tests pro
/
Unit tests
6m 48s
[pcapi-tests] build docker image.
/
build-docker-image
[pcapi-console] build docker image.
/
Build pcapi-console:650d6123df030d0e2f7a9fab3c9fb48541ef7b81
3m 36s
MyPy cop
/
MyPy cop test
Tests API documentation
/
Check API documentation can be built
[PRO] Deploy PR version for validation
/
deploy_on_firebase
0s
Dependabot
/
dependabot
Matrix: Tests api / Pytest
Waiting for pending jobs
Tests api
/
Pylint
Tests api
/
Quality checks
Tests api
/
Test database schema
Update api client template
/
Update API Client template
Push pcapi docker image to registry
/
push-docker-image
Push pcapi-console docker image to registry
/
push-docker-image
Annotations
10 errors, 14 warnings, and 1 notice
|
app/__specs__/App.spec.tsx > App > should render the cookie banner:
pro/src/commons/utils/renderWithProviders.tsx#L74
Error: Should not already be working.
❯ performConcurrentWorkOnRoot ../node_modules/react-dom/cjs/react-dom.development.js:25742:11
❯ flushActQueue ../node_modules/react/cjs/react.development.js:2667:24
❯ act ../node_modules/react/cjs/react.development.js:2582:11
❯ ../node_modules/@testing-library/react/dist/act-compat.js:47:25
❯ renderRoot ../node_modules/@testing-library/react/dist/pure.js:180:26
❯ Proxy.render ../node_modules/@testing-library/react/dist/pure.js:271:10
❯ Module.renderWithProviders commons/utils/renderWithProviders.tsx:74:46
❯ renderApp app/__specs__/App.spec.tsx:32:3
❯ app/__specs__/App.spec.tsx:83:5
|
|
app/__specs__/App.spec.tsx > App > should render the cookie banner:
pro/node_modules/@testing-library/react/dist/index.js#L28
Error: Should not already be working.
❯ performConcurrentWorkOnRoot ../node_modules/react-dom/cjs/react-dom.development.js:25742:11
❯ flushActQueue ../node_modules/react/cjs/react.development.js:2667:24
❯ act ../node_modules/react/cjs/react.development.js:2582:11
❯ ../node_modules/@testing-library/react/dist/act-compat.js:47:25
❯ ../node_modules/@testing-library/react/dist/pure.js:285:28
❯ cleanup ../node_modules/@testing-library/react/dist/pure.js:281:22
❯ ../node_modules/@testing-library/react/dist/index.js:28:25
|
|
app/__specs__/App.spec.tsx > App > should redirect to login if not logged in on a private page:
pro/src/commons/utils/renderWithProviders.tsx#L74
Error: Should not already be working.
❯ performConcurrentWorkOnRoot ../node_modules/react-dom/cjs/react-dom.development.js:25742:11
❯ flushActQueue ../node_modules/react/cjs/react.development.js:2667:24
❯ act ../node_modules/react/cjs/react.development.js:2582:11
❯ ../node_modules/@testing-library/react/dist/act-compat.js:47:25
❯ renderRoot ../node_modules/@testing-library/react/dist/pure.js:180:26
❯ Proxy.render ../node_modules/@testing-library/react/dist/pure.js:271:10
❯ Module.renderWithProviders commons/utils/renderWithProviders.tsx:74:46
❯ renderApp app/__specs__/App.spec.tsx:32:3
❯ app/__specs__/App.spec.tsx:92:5
|
|
app/__specs__/App.spec.tsx > App > should redirect to login if not logged in on a private page:
pro/node_modules/@testing-library/react/dist/index.js#L28
Error: Should not already be working.
❯ performConcurrentWorkOnRoot ../node_modules/react-dom/cjs/react-dom.development.js:25742:11
❯ flushActQueue ../node_modules/react/cjs/react.development.js:2667:24
❯ act ../node_modules/react/cjs/react.development.js:2582:11
❯ ../node_modules/@testing-library/react/dist/act-compat.js:47:25
❯ ../node_modules/@testing-library/react/dist/pure.js:285:28
❯ cleanup ../node_modules/@testing-library/react/dist/pure.js:281:22
❯ ../node_modules/@testing-library/react/dist/index.js:28:25
|
|
app/__specs__/App.spec.tsx > App > should redirect to onboarding if has no user_offerer on private page:
pro/src/commons/utils/renderWithProviders.tsx#L74
Error: Should not already be working.
❯ performConcurrentWorkOnRoot ../node_modules/react-dom/cjs/react-dom.development.js:25742:11
❯ flushActQueue ../node_modules/react/cjs/react.development.js:2667:24
❯ act ../node_modules/react/cjs/react.development.js:2582:11
❯ ../node_modules/@testing-library/react/dist/act-compat.js:47:25
❯ renderRoot ../node_modules/@testing-library/react/dist/pure.js:180:26
❯ Proxy.render ../node_modules/@testing-library/react/dist/pure.js:271:10
❯ Module.renderWithProviders commons/utils/renderWithProviders.tsx:74:46
❯ renderApp app/__specs__/App.spec.tsx:32:3
❯ app/__specs__/App.spec.tsx:98:5
|
|
app/__specs__/App.spec.tsx > App > should redirect to onboarding if has no user_offerer on private page:
pro/node_modules/@testing-library/react/dist/index.js#L28
Error: Should not already be working.
❯ performConcurrentWorkOnRoot ../node_modules/react-dom/cjs/react-dom.development.js:25742:11
❯ flushActQueue ../node_modules/react/cjs/react.development.js:2667:24
❯ act ../node_modules/react/cjs/react.development.js:2582:11
❯ ../node_modules/@testing-library/react/dist/act-compat.js:47:25
❯ ../node_modules/@testing-library/react/dist/pure.js:285:28
❯ cleanup ../node_modules/@testing-library/react/dist/pure.js:281:22
❯ ../node_modules/@testing-library/react/dist/index.js:28:25
|
|
app/__specs__/App.spec.tsx > App > should not initialize firebase on the adage iframe:
pro/src/commons/utils/renderWithProviders.tsx#L74
Error: Should not already be working.
❯ performConcurrentWorkOnRoot ../node_modules/react-dom/cjs/react-dom.development.js:25742:11
❯ flushActQueue ../node_modules/react/cjs/react.development.js:2667:24
❯ act ../node_modules/react/cjs/react.development.js:2582:11
❯ ../node_modules/@testing-library/react/dist/act-compat.js:47:25
❯ renderRoot ../node_modules/@testing-library/react/dist/pure.js:180:26
❯ Proxy.render ../node_modules/@testing-library/react/dist/pure.js:271:10
❯ Module.renderWithProviders commons/utils/renderWithProviders.tsx:74:46
❯ renderApp app/__specs__/App.spec.tsx:32:3
❯ app/__specs__/App.spec.tsx:116:5
|
|
app/__specs__/App.spec.tsx > App > should not initialize firebase on the adage iframe:
pro/node_modules/@testing-library/react/dist/index.js#L28
Error: Should not already be working.
❯ performConcurrentWorkOnRoot ../node_modules/react-dom/cjs/react-dom.development.js:25742:11
❯ flushActQueue ../node_modules/react/cjs/react.development.js:2667:24
❯ act ../node_modules/react/cjs/react.development.js:2582:11
❯ ../node_modules/@testing-library/react/dist/act-compat.js:47:25
❯ ../node_modules/@testing-library/react/dist/pure.js:285:28
❯ cleanup ../node_modules/@testing-library/react/dist/pure.js:281:22
❯ ../node_modules/@testing-library/react/dist/index.js:28:25
|
|
app/__specs__/App.spec.tsx > App > should redirect to page 404 when api has not found:
pro/src/commons/utils/renderWithProviders.tsx#L74
Error: Should not already be working.
❯ performConcurrentWorkOnRoot ../node_modules/react-dom/cjs/react-dom.development.js:25742:11
❯ flushActQueue ../node_modules/react/cjs/react.development.js:2667:24
❯ act ../node_modules/react/cjs/react.development.js:2582:11
❯ ../node_modules/@testing-library/react/dist/act-compat.js:47:25
❯ renderRoot ../node_modules/@testing-library/react/dist/pure.js:180:26
❯ Proxy.render ../node_modules/@testing-library/react/dist/pure.js:271:10
❯ Module.renderWithProviders commons/utils/renderWithProviders.tsx:74:46
❯ renderApp app/__specs__/App.spec.tsx:32:3
❯ app/__specs__/App.spec.tsx:133:5
|
|
app/__specs__/App.spec.tsx > App > should redirect to page 404 when api has not found:
pro/node_modules/@testing-library/react/dist/index.js#L28
Error: Should not already be working.
❯ performConcurrentWorkOnRoot ../node_modules/react-dom/cjs/react-dom.development.js:25742:11
❯ flushActQueue ../node_modules/react/cjs/react.development.js:2667:24
❯ act ../node_modules/react/cjs/react.development.js:2582:11
❯ ../node_modules/@testing-library/react/dist/act-compat.js:47:25
❯ ../node_modules/@testing-library/react/dist/pure.js:285:28
❯ cleanup ../node_modules/@testing-library/react/dist/pure.js:281:22
❯ ../node_modules/@testing-library/react/dist/index.js:28:25
|
|
Tests pro / Unit tests
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/cache@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Tests pro / Type check / Quality check / Style quality check
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/cache@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/app/App/App.tsx#L75
React Hook useEffect has a missing dependency: 'dispatch'. Either include it or remove the dependency array
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/app/App/hook/useLogExtraProData.ts#L25
React Hook useEffect has missing dependencies: 'location.pathname', 'logEvent', and 'previousOfferer'. Either include them or remove the dependency array
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/app/App/hook/useLogNavigation.ts#L32
React Hook useEffect has missing dependencies: 'params' and 'previousPage'. Either include them or remove the dependency array
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/app/App/hook/usePageTitle.ts#L16
React Hook useEffect has a missing dependency: 'location'. Either include it or remove the dependency array
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/commons/hooks/useGetBitmap.ts#L21
React Hook useEffect has a missing dependency: 'file'. Either include it or remove the dependency array
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/commons/hooks/useIsElementVisible.ts#L18
The 'observer' object construction makes the dependencies of useEffect Hook (at line 32) change on every render. Move it inside the useEffect callback. Alternatively, wrap the initialization of 'observer' in its own useMemo() Hook
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/commons/hooks/useNotification.ts#L50
React Hook useMemo has a missing dependency: 'notify'. Either include it or remove the dependency array
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/commons/hooks/useRedirectLoggedUser.ts#L53
React Hook useEffect has missing dependencies: 'dispatch', 'navigate', and 'redirectToUrl'. Either include them or remove the dependency array
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/components/Address/Address.tsx#L43
React Hook useEffect has a missing dependency: 'selectedField.value'. Either include it or remove the dependency array
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/components/Address/Address.tsx#L85
React Hook useEffect has missing dependencies: 'addressesMap', 'searchField', and 'setFieldValue'. Either include them or remove the dependency array
|
|
Tests pro E2E / E2E tests and notifications
The following actions use a deprecated Node.js version and will be forced to run on node20: technote-space/workflow-conclusion-action@v3, actions/cache@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Tests pro E2E / E2E tests and notifications
The "process_gcloudignore" option is true, but no .gcloudignore file was found. If you do not intend to process a gcloudignore file, set "process_gcloudignore" to false.
|
|
Tests pro E2E / E2E tests and notifications
Running e2e-tests with europe-west1-docker.pkg.dev/passculture-infra-prod/pass-culture-artifact-registry/pcapi:latest
|
Artifacts
Produced during runtime
| Name | Size | |
|---|---|---|
|
pass-culture~pass-culture-main~UWI2EP.dockerbuild
|
89.7 KB |
|
|
pcapi-console-650d6123df030d0e2f7a9fab3c9fb48541ef7b81.tar
|
480 MB |
|