Skip to content
Lint pull requests

fix BB-343 : Integrated the modification of identifiers and aliases into the primary editing workflow #1029

Sign in to view logs

fix BB-343 : Integrated the modification of identifiers and aliases into the primary editing workflow

fix BB-343 : Integrated the modification of identifiers and aliases into the primary editing workflow #1029

Triggered via pull request April 24, 2024 06:20
@Tarunmeena0901Tarunmeena0901
synchronize #1043
Status Success
Total duration 44s
Artifacts –

lint-pull-requests.yml

on: pull_request_target
Run linters for pull requests
35s
Run linters for pull requests
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 5 warnings
test/src/server/routes/entity/author.js ► Author routes with entity editing priv ► should not throw error while seeding author: test-results.json#L0
Failed test found in: test-results.json Error: Timeout of 50000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/runner/work/bookbrainz-site/bookbrainz-site/test/src/server/routes/entity/author.js)
test/src/server/routes/entity/edition-group.js ► Edition Group routes with entity editing priv ► should not throw error while seeding edition group: test-results.json#L0
Failed test found in: test-results.json Error: Timeout of 50000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/runner/work/bookbrainz-site/bookbrainz-site/test/src/server/routes/entity/edition-group.js)
test/src/server/routes/entity/edition.js ► Edition routes with entity editing priv ► should not throw error while seeding edition: test/src/server/routes/entity/edition.js#L63
Failed test found in: test-results.json Error: expected false to be true
test/src/server/routes/entity/publisher.js ► Publisher routes with entity editing priv ► should not throw error while seeding publisher: test-results.json#L0
Failed test found in: test-results.json Error: Timeout of 50000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/runner/work/bookbrainz-site/bookbrainz-site/test/src/server/routes/entity/publisher.js)
test/src/server/routes/entity/series.js ► Series routes with entity editing priv ► should not throw error while seeding series: test-results.json#L0
Failed test found in: test-results.json Error: Timeout of 50000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/runner/work/bookbrainz-site/bookbrainz-site/test/src/server/routes/entity/series.js)
test/src/server/routes/entity/work.js ► Work routes with entity editing priv ► should not throw error while seeding work: test/src/server/routes/entity/work.js#L60
Failed test found in: test-results.json Error: expected false to be true
test/src/server/routes/merge.js ► Merge routes with entity editing priv /add route ► "after all" hook for "should recreate the merge queue if adding another entity type": test-results.json#L0
Failed test found in: test-results.json Error: Timeout of 50000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/runner/work/bookbrainz-site/bookbrainz-site/test/src/server/routes/merge.js)
test/src/server/routes/merge.js ► Merge routes with entity editing priv /cancel route ► "after all" hook for "should clear the merge queue": test-results.json#L0
Failed test found in: test-results.json Error: Timeout of 50000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/runner/work/bookbrainz-site/bookbrainz-site/test/src/server/routes/merge.js)
test/src/server/routes/merge.js ► Merge routes with entity editing priv /cancel route ► should clear the merge queue: test-results.json#L0
Failed test found in: test-results.json Error: Timeout of 50000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/runner/work/bookbrainz-site/bookbrainz-site/test/src/server/routes/merge.js)
test/src/server/routes/relationship.js ► Relationship ► "after all" hook for "should be able to add new relationships without having previous relationships": test-results.json#L0
Failed test found in: test-results.json Error: Timeout of 50000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/runner/work/bookbrainz-site/bookbrainz-site/test/src/server/routes/relationship.js)
src/client/components/pages/revision.js#L177
Unexpected 'todo' comment: 'TODO: Add proper error handling.' (no-warning-comments)
src/server/helpers/collectionRouteUtils.js#L22
'log' is defined but never used (@typescript-eslint/no-unused-vars)
src/server/routes/register.js#L22
'handler' is defined but never used (@typescript-eslint/no-unused-vars)
Run linters for pull requests
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, wearerequired/lint-action@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Run linters for pull requests
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, wearerequired/lint-action@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/