Skip to content

Conversation

@trivikr
Copy link
Member

@trivikr trivikr commented Jan 5, 2022

Issue

Attempt to fix Out of memory error emitted by jest in #3145 (comment)

Description

Call lerna run test in test:all

Testing

CI


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@trivikr trivikr requested a review from a team as a code owner January 5, 2022 23:32
@trivikr
Copy link
Member Author

trivikr commented Jan 6, 2022

Error:

lerna ERR! yarn run test exited 139 in '@aws-sdk/polly-request-presigner'
lerna ERR! yarn run test stdout:
$ jest
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

lerna ERR! yarn run test stderr:
PASS src/getSignedUrls.spec.ts (6.14 s)
  getSignedUrl
    ✓ should call SignatureV4.sign (91 ms)
    ✓ should presign with signing region and service in context if exists (5 ms)
    ✓ should presign with parameters from presign options if set (3 ms)

Test Suites: 1 passed, 1 total
Tests:       3 passed, 3 total
Snapshots:   0 total
Time:        6.223 s
Ran all test suites.
Segmentation fault (core dumped)
error Command failed with exit code 139.

Issue discussed upstream on jest repo at jestjs/jest#10662
The issue is not reproducible in local workspace.

@trivikr
Copy link
Member Author

trivikr commented Jan 6, 2022

Closing as jest throws segmentation fault on polly-request-presigner, which is not reproducible in workspace.
Trivial fixes in polly-request-presigner tests didn't help.

@trivikr trivikr closed this Jan 6, 2022
@trivikr trivikr deleted the run-test-each-package branch January 6, 2022 01:25
@github-actions
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant