Skip to content
This repository has been archived by the owner on Apr 15, 2019. It is now read-only.

Remove logging from tests #52

Closed
willclarktech opened this issue Jul 6, 2017 · 0 comments
Closed

Remove logging from tests #52

willclarktech opened this issue Jul 6, 2017 · 0 comments
Assignees
@willclarktech
Labels
*easy 📝 test standards

Comments

@willclarktech
Copy link
Contributor

willclarktech commented Jul 6, 2017 •
edited

We should swallow the vorpal output to avoid polluted test logs like this:

lisky>

  lisky get command palette @now
lisky>     ✓ should test command get account
lisky>
    ✓ should have the right parameters with block
    ✓ should have the right parameters with delegate
@willclarktech willclarktech self-assigned this Jul 6, 2017
@willclarktech willclarktech added this to In Progress in Version 0.1.0 Jul 6, 2017
@willclarktech willclarktech changed the title Remove console logging from tests Remove logging from tests Jul 6, 2017
@willclarktech willclarktech moved this from In Progress to Backlog in Version 0.1.0 Jul 6, 2017
@willclarktech willclarktech moved this from Backlog to In Progress in Version 0.1.0 Jul 6, 2017
@willclarktech willclarktech moved this from In Progress to Backlog in Version 0.1.0 Jul 6, 2017
@willclarktech willclarktech removed this from Backlog in Version 0.1.0 Jul 6, 2017
@willclarktech willclarktech added this to Backlog in Version 0.3.0 Jul 6, 2017
@willclarktech willclarktech moved this from Backlog to In Progress in Version 0.3.0 Jul 7, 2017
@willclarktech willclarktech removed this from In Progress in Version 0.3.0 Jul 7, 2017
@willclarktech willclarktech added this to In Progress in Version 0.1.0 Jul 7, 2017
@willclarktech willclarktech mentioned this issue Jul 7, 2017
Merged
toschdev pushed a commit that referenced this issue Jul 10, 2017
Merge pull request #73 from LiskHQ/52-remove_test_logging
7965a45 
Remove vorpal logging output in tests - Closes #52
@toschdev toschdev removed this from In Progress in Version 0.1.0 Jul 10, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@willclarktech willclarktech

Labels
*easy 📝 test standards
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@willclarktech