Skip to content

Upgrade npm

Upgrade npm #1424

Triggered via pull request July 19, 2024 14:46
Status Failure
Total duration 1m 14s
Artifacts

js-tests.yml

on: pull_request
Matrix: Tests
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 12 warnings
Tests (20.x): src/lib/authentication/pages/Login/LoginWithLocalAccount/LoginWithLocalAccount.js#L84
Replace `⏎········await·authenticationService.loginWithLocalAccount(values` with `·await·authenticationService.loginWithLocalAccount(⏎········values⏎······`
Tests (20.x): src/lib/modules/Loan/backoffice/OverdueLoanSendNotificationModal/state/actions.js#L18
Replace `⏎········await·loanApi.sendOverdueLoansNotificationReminder(loanPid` with `·await·loanApi.sendOverdueLoansNotificationReminder(⏎········loanPid⏎······`
Tests (20.x): src/lib/pages/backoffice/ILL/BorrowingRequest/BorrowingRequestDetails/BorrowingRequestPatronLoan/BorrowingRequestLoanExtension/actions.js#L56
Replace `⏎········await·borrowingRequestApi.declineExtension(borrowingRequestPid` with `·await·borrowingRequestApi.declineExtension(⏎········borrowingRequestPid⏎······`
Tests (20.x)
Process completed with exit code 1.
Tests (20.x)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Tests (20.x)
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v2, actions/setup-node@v1. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Tests (20.x): src/lib/components/Pagination/Pagination.js#L47
propType "currentSize" is not required, but has no corresponding defaultProps declaration
Tests (20.x): src/lib/components/Pagination/Pagination.js#L48
propType "totalPages" is not required, but has no corresponding defaultProps declaration
Tests (20.x): src/lib/components/Pagination/Pagination.js#L51
propType "loading" is not required, but has no corresponding defaultProps declaration
Tests (20.x): src/lib/components/Pagination/Pagination.js#L52
propType "firstItem" is not required, but has no corresponding defaultProps declaration
Tests (20.x): src/lib/components/Pagination/Pagination.js#L53
propType "lastItem" is not required, but has no corresponding defaultProps declaration
Tests (20.x): src/lib/components/SearchBar/SearchBarILS.js#L8
inputRef should be placed after componentDidMount
Tests (20.x): src/lib/components/backoffice/UrlList/UrlList.js#L15
Do not use Array index in keys
Tests (20.x): src/lib/components/backoffice/buttons/ViewDetailsButtons/PatronDetailsLink.js#L18
Fragments should contain more than one child - otherwise, there’s no need for a Fragment at all
Tests (20.x): src/lib/forms/rjsf/RJSFCustomTemplates.js#L44
propType "children" is not required, but has no corresponding defaultProps declaration
Tests (20.x): src/lib/forms/rjsf/RJSFCustomTemplates.js#L140
Do not use Array index in keys