Skip to content

fix: Correctly handle timed out tests in Jest#55

Merged
dividedmind merged 1 commit intodevelopfrom
fix/jest-timeouts
Dec 11, 2023
Merged

fix: Correctly handle timed out tests in Jest#55
dividedmind merged 1 commit intodevelopfrom
fix/jest-timeouts

Conversation

@dividedmind
Copy link
Copy Markdown
Collaborator

When a test times out, Jest passes a message string back as an error instead of an Error instace. Handle this case.

Fixes #46

@ahtrotta, note the test referred to in #46 still fails because of recording overhead — adjusting timeouts allows it to pass.

When a test times out, Jest passes a message string back as an
error instead of an Error instace. Handle this case.

Fixes #46
@ahtrotta
Copy link
Copy Markdown

ahtrotta commented Dec 5, 2023

How do I build a local version of this from source so I can test it manually?

@dividedmind
Copy link
Copy Markdown
Collaborator Author

How do I build a local version of this from source so I can test it manually?

Check out, then yarn prepack and you can use bin/appmap-node.js to run it.

@dividedmind dividedmind changed the base branch from main to develop December 11, 2023 14:08
@dividedmind dividedmind merged commit 216c4f1 into develop Dec 11, 2023
@dividedmind dividedmind deleted the fix/jest-timeouts branch December 11, 2023 14:08
@appland-release
Copy link
Copy Markdown

🎉 This PR is included in version 2.7.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ReferenceError: You are trying to import a file after the Jest environment has been torn down. From test/cli/scan.spec.ts.

4 participants