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

Testing : exitOnFail #307

Merged
merged 4 commits into from Mar 26, 2019

Conversation

2 participants
@zekth
Copy link
Contributor

commented Mar 26, 2019

Following : #306
Add a parameter to handle exitOnFail in runTests() with default value: true to not change the API. Updated documentation also.

Also i think maybe a refactor of the test runtime to a Class in a further PR would be a good idea.

zekth added some commits Mar 26, 2019

Show resolved Hide resolved testing/mod.ts Outdated
@ry

ry approved these changes Mar 26, 2019

Copy link
Contributor

left a comment

LGTM

@ry ry merged commit d9e8953 into denoland:master Mar 26, 2019

2 checks passed

denoland.deno_std #20190326.10 succeeded
Details
license/cla Contributor License Agreement is signed.
Details

@bartlomieju bartlomieju referenced this pull request Mar 26, 2019

Merged

api fix #309

@zekth zekth referenced this pull request Mar 28, 2019

Open

[Discussion] Testing #306

@zekth zekth deleted the zekth:testing_exit branch Mar 28, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.