(PC-33129)[API] feat: adding isOnboardinOngoing to /offerer/<id> #7720
dev_on_pull_request_workflow.yml
on: pull_request
Init job
39s
Tests pro
/
...
/
Style quality check
1m 33s
Tests pro
/
Unit tests
8m 12s
[pcapi-tests] build docker image.
/
Build pcapi-tests:1e0f916e7ae1236a1a9368552adcae20f468cd04
3m 58s
[pcapi] build docker image.
/
Build pcapi:1e0f916e7ae1236a1a9368552adcae20f468cd04
2m 23s
[pcapi-console] build docker image.
/
Build pcapi-console:1e0f916e7ae1236a1a9368552adcae20f468cd04
3m 39s
MyPy cop
/
MyPy cop test
35s
Tests API documentation
/
Check API documentation can be built
[PRO] Deploy PR version for validation
/
deploy_on_firebase
1m 24s
Dependabot
/
dependabot
Matrix: Tests api / Pytest
Tests api
/
Quality checks
1m 50s
Tests api
/
Pylint
3m 45s
Tests api
/
Test database schema
3m 18s
Update api client template
/
Update API Client template
1m 50s
Push pcapi docker image to registry
/
push-docker-image
Push pcapi-console docker image to registry
/
push-docker-image
Annotations
2 errors, 10 warnings, and 4 notices
tests/routes/backoffice/accounts_test.py.test_get_public_account_history:
tests/routes/backoffice/accounts_test.py#L961
AssertionError: assert False
+ where False = <built-in method startswith of str object at 0x7ffab5ce7690>('Le 26/10/2024 à ')
+ where <built-in method startswith of str object at 0x7ffab5ce7690> = 'Le 27/10/2024 à 00h12'.startswith
+ and 'Le 26/10/2024 à ' = <built-in method strftime of datetime.date object at 0x7ffab5b77d90>('Le %d/%m/%Y à ')
+ where <built-in method strftime of datetime.date object at 0x7ffab5b77d90> = (datetime.date(2024, 11, 25) - relativedelta(days=+30)).strftime
+ where datetime.date(2024, 11, 25) = <built-in method today of type object at 0x7ffad3757ec0>()
+ where <built-in method today of type object at 0x7ffad3757ec0> = <class 'datetime.date'>.today
+ where <class 'datetime.date'> = datetime.date
+ and relativedelta(days=+30) = relativedelta(days=30)
|
Tests api / Pytest (tests/routes/backoffice -m 'backoffice' --ignore=tests/routes/backoffice/collective_booki...
❌ Tests reported 1 failures
|
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/commons/utils/sortCollectiveOffers.ts#L19
Switch is not exhaustive. Cases not matched: null
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/commons/utils/timezone.ts#L56
Switch is not exhaustive. Cases not matched: undefined | null
|
Update api client template / Update API Client template
Running update api client template with europe-west1-docker.pkg.dev/passculture-infra-prod/pass-culture-artifact-registry/pcapi:1e0f916e7ae1236a1a9368552adcae20f468cd04
|
Tests api / Quality checks
Running tests api with europe-west1-docker.pkg.dev/passculture-infra-prod/pass-culture-artifact-registry/pcapi-tests:1e0f916e7ae1236a1a9368552adcae20f468cd04
|
[PRO] Deploy PR version for validation / deploy_on_firebase
Firebase deployment is available at : https://pc-pro-testing--pr15210-anougbele-pass-pc-33-jzrg7v85.web.app
|
Tests pro E2E / E2E tests and notifications
Running e2e-tests with europe-west1-docker.pkg.dev/passculture-infra-prod/pass-culture-artifact-registry/pcapi:1e0f916e7ae1236a1a9368552adcae20f468cd04
|
Artifacts
Produced during runtime
Name | Size | |
---|---|---|
pass-culture~pass-culture-main~KUPTMI.dockerbuild
|
157 KB |
|
pass-culture~pass-culture-main~SUW83U.dockerbuild
|
94.5 KB |
|
pass-culture~pass-culture-main~Y1RN39.dockerbuild
|
83.7 KB |
|
pcapi-1e0f916e7ae1236a1a9368552adcae20f468cd04.tar
Expired
|
299 MB |
|
pcapi-console-1e0f916e7ae1236a1a9368552adcae20f468cd04.tar
Expired
|
482 MB |
|
pcapi-tests-1e0f916e7ae1236a1a9368552adcae20f468cd04.tar
Expired
|
529 MB |
|