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

Added common reliable check to detect jest,vitest and mocha testing framework. #1807

Merged
merged 13 commits into from
Jan 30, 2024

Conversation

siddhantrawal
Copy link
Contributor

@siddhantrawal siddhantrawal commented Jan 30, 2024

Part of #1784, fixes isJest/IsDev issues.

Changes

Added new checks to detect test frameworks other than jest:

Testing

Dev errors not showing. (vitest)

mocha check tested in isolation (see thread)

Docs

N/A

@siddhantrawal
Copy link
Contributor Author

Is there any other better approach to do this?

@siddhantrawal siddhantrawal marked this pull request as ready for review January 30, 2024 15:16
@siddhantrawal siddhantrawal requested review from a team as code owners January 30, 2024 15:16
@siddhantrawal siddhantrawal requested review from mayank99 and r100-stack and removed request for a team January 30, 2024 15:16
@siddhantrawal siddhantrawal changed the title Added common reliable check to detect jest,vitest and mocha testing environment. Added common reliable check to detect jest,vitest and mocha testing framework. Jan 30, 2024
Co-authored-by: Rohan Kadkol <45748283+rohan-kadkol@users.noreply.github.com>
Co-authored-by: Mayank <9084735+mayank99@users.noreply.github.com>
Co-authored-by: Mayank <9084735+mayank99@users.noreply.github.com>
@mayank99 mayank99 merged commit 98a36cf into main Jan 30, 2024
16 checks passed
@mayank99 mayank99 deleted the siddhant/detect_test_environment branch January 30, 2024 21:04
@imodeljs-admin imodeljs-admin mentioned this pull request Jan 30, 2024
@mayank99 mayank99 mentioned this pull request Feb 5, 2024
18 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants