Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ci fails on frontend tests #134

Closed
Dassderdie opened this issue Jan 27, 2022 · 2 comments · Fixed by #137
Closed

Ci fails on frontend tests #134

Dassderdie opened this issue Jan 27, 2022 · 2 comments · Fixed by #137
Labels
bug Something isn't working CI This is about the CI
Projects

Comments

@Dassderdie
Copy link
Collaborator

> digital-fuesim-manv-frontend@0.0.0 test
> jest --runInBand --coverage --verbose

PASS Frontend src/app/core/optimistic-action-handler.spec.ts
  OptimisticActionHandler
    ✓ should correctly apply not optimistic proposed actions when response is true (5 ms)
    ✓ should correctly apply not optimistic proposed actions when response is false (2 ms)
    ✓ should correctly perform actions (1 ms)
    ✓ should correctly apply optimistic actions (2 ms)
    ✓ should keep the right order when applying optimistic updates and performed updates (1 ms)
    ✓ should perform already proposed actions after the optimistic update is done (*** ms)

------------------------------|---------|----------|---------|---------|-------------------
File                          | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s 
------------------------------|---------|----------|---------|---------|-------------------
All files                     |     100 |      100 |     100 |     100 |                   
 optimistic-action-handler.ts |     100 |      100 |     100 |     100 |                   
------------------------------|---------|----------|---------|---------|-------------------

=============================== Coverage summary ===============================
Statements   : 100% ( ***/*** )
Branches     : 100% ( 4/4 )
Functions    : 100% ( 5/5 )
Lines        : 100% ( ***1/***1 )
================================================================================
Test Suites: 1 passed, 1 total
Tests:       6 passed, 6 total
Snapshots:   0 total
Time:        4.71*** s
Ran all test suites.
node:internal/event_target:777
  process.nextTick(() => { throw err; });
                           ^

Error: Unknown worker message type message
This is caused by either a bug in Node.js or incorrect usage of Node.js internals.
Please open an issue with this stack trace at https://github.com/nodejs/node/issues

    at new NodeError (node:internal/errors:***71:5)
    at Function.fail (node:internal/assert:20:9)
    at Worker.[kOnMessage] (node:internal/worker:***17:12)
    at MessagePort.<anonymous> (node:internal/worker:201:57)
    at ZoneDelegate.Object.<anonymous>.ZoneDelegate.invokeTask (/home/runner/work/digital-fuesim-manv/digital-fuesim-manv/frontend/node_modules/zone.js/bundles/zone-testing-bundle.umd.js:441:***5)
    at Zone.Object.<anonymous>.Zone.runTask (/home/runner/work/digital-fuesim-manv/digital-fuesim-manv/frontend/node_modules/zone.js/bundles/zone-testing-bundle.umd.js:212:51)
    at ZoneTask.Object.<anonymous>.ZoneTask.invokeTask [as invoke] (/home/runner/work/digital-fuesim-manv/digital-fuesim-manv/frontend/node_modules/zone.js/bundles/zone-testing-bundle.umd.js:52***:***8)
    at invokeTask (/home/runner/work/digital-fuesim-manv/digital-fuesim-manv/frontend/node_modules/zone.js/bundles/zone-testing-bundle.umd.js:166***:18)
    at MessagePort.globalZoneAwareCallback (/home/runner/work/digital-fuesim-manv/digital-fuesim-manv/frontend/node_modules/zone.js/bundles/zone-testing-bundle.umd.js:1689:21)
    at MessagePort.[nodejs.internal.kHybridDispatch] (node:internal/event_target:562:20) {
  code: 'ERR_INTERNAL_ASSERTION'
}
@Dassderdie Dassderdie added bug Something isn't working CI This is about the CI labels Jan 27, 2022
@Dassderdie Dassderdie added this to To do in BP2021HG1 via automation Jan 27, 2022
@Dassderdie
Copy link
Collaborator Author

The MessagePort stuff seems to come from Node workers. Maybe we could try to limit jest in the frontend to one worker via https://jestjs.io/docs/cli#--maxworkersnumstring

@Dassderdie Dassderdie linked a pull request Jan 27, 2022 that will close this issue
@Dassderdie
Copy link
Collaborator Author

Currently fixed via --forceExit

BP2021HG1 automation moved this from To do to Done Jan 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working CI This is about the CI
Projects
BP2021HG1
  
Done
Development

Successfully merging a pull request may close this issue.

1 participant