Skip to content
This repository has been archived by the owner on Jan 19, 2021. It is now read-only.

Show passed test results if debug == true #248

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

smccarthy
Copy link
Contributor

@smccarthy smccarthy commented Dec 20, 2017

I think it could be helpful to be able to see the passed tests logs as well.

I was between creating a new method, logPassedTests, or updating logFailedTests to logTests and having it be passed in test results.

I also didn't know if I should create a new flag, or use the existing debug flag. I went with the existing debug flag since it was easier.

I made the passedTests logs appear first, so the failed tests were easier to see (since they'd be at the bottom of the console).

@archlichking
Copy link
Contributor

@CLAassistant
Copy link

CLAassistant commented Aug 13, 2019

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Shawn Mccarthy seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

5 participants