Main Tests & Analysis

Release v0.6.2 #41

Triggered via push February 23, 2024 20:53

Chingling152

pushed 016c287

main

Status Success

Total duration 5m 15s

Artifacts –

branch_main_ci.yml

on: push

Matrix: Tests / Code Tests

Code Analysis / Code coverage

2m 49s

Annotations

46 warnings and 3 notices

Tests / Code Tests (core)

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions/setup-dotnet@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

Tests / Code Tests (core)

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Tests / Code Tests (core): src/api/common/FinancialHub.Common/Results/Errors/ServiceError.cs#L7

#warning: 'it can be changed to an enum or use a different value from https codes'

Tests / Code Tests (core): src/api/core/FinancialHub.Core.Application/Services/TransactionBalanceService.cs#L124

This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.

Tests / Code Tests (core): src/api/core/FinancialHub.Core.Infra.Data/Repositories/TransactionsRepository.cs#L14

#warning: 'This is not a good practice, remove this later'

Tests / Code Tests (core): src/api/auth/FinancialHub.Auth.Presentation/Controllers/SignupController.cs#L34

Dereference of a possibly null reference.

Tests / Code Tests (core): src/api/auth/FinancialHub.Auth.Presentation/Controllers/SignupController.cs#L38

Possible null reference argument for parameter 'data' in 'ItemResponse<UserModel>.ItemResponse(UserModel data)'.

Tests / Code Tests (core): src/api/auth/FinancialHub.Auth.Presentation/Controllers/SigninController.cs#L36

Dereference of a possibly null reference.

Tests / Code Tests (core): src/api/auth/FinancialHub.Auth.Presentation/Controllers/SigninController.cs#L41

Possible null reference argument for parameter 'data' in 'ItemResponse<TokenModel>.ItemResponse(TokenModel data)'.

Tests / Code Tests (core): src/api/auth/FinancialHub.Auth.Presentation/Controllers/UsersController.cs#L35

Dereference of a possibly null reference.

Tests / Code Tests (core): src/api/auth/FinancialHub.Auth.Presentation/Controllers/UsersController.cs#L41

Possible null reference argument for parameter 'data' in 'ItemResponse<UserModel>.ItemResponse(UserModel data)'.

Tests / Code Tests (core): src/api/auth/FinancialHub.Auth.Presentation/Controllers/UsersController.cs#L64

Dereference of a possibly null reference.

Tests / Code Tests (auth)

Tests / Code Tests (auth): src/api/common/FinancialHub.Common/Results/Errors/ServiceError.cs#L7

#warning: 'it can be changed to an enum or use a different value from https codes'

Tests / Code Tests (auth): src/api/core/FinancialHub.Core.Infra.Data/Repositories/TransactionsRepository.cs#L14

#warning: 'This is not a good practice, remove this later'

Tests / Code Tests (auth): src/api/core/FinancialHub.Core.Application/Services/TransactionBalanceService.cs#L124

Tests / Code Tests (auth): src/api/auth/FinancialHub.Auth.Presentation/Controllers/UsersController.cs#L35

Dereference of a possibly null reference.

Tests / Code Tests (auth): src/api/auth/FinancialHub.Auth.Presentation/Controllers/UsersController.cs#L41

Possible null reference argument for parameter 'data' in 'ItemResponse<UserModel>.ItemResponse(UserModel data)'.

Tests / Code Tests (auth): src/api/auth/FinancialHub.Auth.Presentation/Controllers/UsersController.cs#L64

Dereference of a possibly null reference.

Tests / Code Tests (auth): src/api/auth/FinancialHub.Auth.Presentation/Controllers/UsersController.cs#L69

Possible null reference argument for parameter 'data' in 'SaveResponse<UserModel>.SaveResponse(UserModel data)'.

Tests / Code Tests (auth): src/api/auth/FinancialHub.Auth.Presentation/Controllers/SigninController.cs#L36

Dereference of a possibly null reference.

Tests / Code Tests (auth): src/api/auth/FinancialHub.Auth.Presentation/Controllers/SigninController.cs#L41

Possible null reference argument for parameter 'data' in 'ItemResponse<TokenModel>.ItemResponse(TokenModel data)'.

Tests / Code Tests (auth): src/api/auth/FinancialHub.Auth.Presentation/Controllers/SignupController.cs#L34

Dereference of a possibly null reference.

Code Analysis / Code coverage

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions/setup-dotnet@v2, actions/setup-java@v3, actions/cache@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

Code Analysis / Code coverage

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/cache@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Code Analysis / Code coverage: src/api/common/FinancialHub.Common/Results/Errors/ServiceError.cs#L7

#warning: 'it can be changed to an enum or use a different value from https codes'

Code Analysis / Code coverage: src/api/auth/FinancialHub.Auth.Domain/Interfaces/Helpers/IPasswordHelper.cs#L3

Complete the task associated to this 'TODO' comment.

Code Analysis / Code coverage: src/api/core/FinancialHub.Core.Domain/Queries/TransactionQuery.cs#L20

Complete the task associated to this 'TODO' comment.

Code Analysis / Code coverage: src/api/core/FinancialHub.Core.Domain/Interfaces/Services/IBalancesService.cs#L16

Do not forget to remove this deprecated code someday.

Code Analysis / Code coverage: src/api/core/FinancialHub.Core.Domain/Queries/TransactionQuery.cs#L35

Provide the "DateTimeKind" when creating this object.