Skip to content

[#IOPID-2458] io-functions-lollipop migration into io-auth-n-identity-domain #431

[#IOPID-2458] io-functions-lollipop migration into io-auth-n-identity-domain

[#IOPID-2458] io-functions-lollipop migration into io-auth-n-identity-domain #431

Triggered via pull request December 6, 2024 14:48
Status Failure
Total duration 9m 45s
Artifacts

code_review.yaml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

2 errors and 12 warnings
code_review
io-lollipop#test:coverage: command (/home/runner/work/io-auth-n-identity-domain/io-auth-n-identity-domain/apps/io-lollipop) /tmp/xfs-570cd474/yarn run test:coverage exited (1)
code_review
Process completed with exit code 1.
code_review
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
code_review: apps/io-session-manager/src/__mocks__/appinsights.mocks.ts#L12
Unexpected any. Specify a different type
code_review: apps/io-session-manager/src/controllers/__tests__/authentication.spec.ts#L24
'TelemetryClient' is defined but never used
code_review: apps/io-session-manager/src/controllers/__tests__/pagopa.spec.ts#L176
Update this function so that its implementation is not identical to the one on line 141
code_review: apps/io-session-manager/src/utils/__tests__/lollipop.spec.ts#L298
Define a constant instead of duplicating this literal 3 times
e2e
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636