Continuous Integration

[#IOPID-2556] `io-lollipop` integration tests #486

Sign in to view logs

Summary
Jobs
- code_review
- e2e
Run details
- Usage
- Workflow file

Triggered via pull request December 18, 2024 15:56

edited #143

IOPID-2556

Status Success

Total duration 5m 17s

Artifacts –

code_review.yaml

on: pull_request

Annotations

12 warnings

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/services/__tests__/redis-session-storage.spec.ts#L528

Unexpected let, use const instead

code_review: apps/io-session-manager/src/services/__tests__/redis-session-storage.spec.ts#L538

Unexpected let, use const instead

code_review: apps/io-session-manager/src/services/__tests__/redis-session-storage.spec.ts#L563

Unexpected let, use const instead

code_review: apps/io-session-manager/src/services/__tests__/redis-session-storage.spec.ts#L572

Unexpected let, use const instead

code_review: apps/io-session-manager/src/services/__tests__/redis-session-storage.spec.ts#L584

Unexpected let, use const instead

code_review: apps/io-session-manager/src/utils/__tests__/lollipop.spec.ts#L298

Define a constant instead of duplicating this literal 3 times

code_review: apps/io-session-manager/src/utils/__tests__/redis-trace-wrapper.spec.ts#L1

'beforeAll' is defined but never used

ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636