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

Default reporter gets corrupted with 2>/dev/null #275

Closed
vladikoff opened this issue Jul 20, 2016 · 2 comments
Closed

Default reporter gets corrupted with 2>/dev/null #275

vladikoff opened this issue Jul 20, 2016 · 2 comments

Comments

@vladikoff
Copy link

vladikoff commented Jul 20, 2016

STR

Example: repo https://github.com/vladikoff/issue-tap-reporter-bad-formatting
clone, npm install and run ./run.sh

Expected (run node_modules/.bin/tap example-test.js)

node_modules/.bin/tap example-test.js                                               
example-test.js ....................................... 4/4
total ................................................. 4/4

  4 passing (389.584ms)

  ok

Actual (run ./run.sh)

➜  tap-rep git:(master) ./run.sh 
example-test.js ....example-test.js ....................................... 4/4
total ................................................. 4/4

  4 passing (399.759ms)

  ok

^ the test name above is repeated

@isaacs
Copy link
Member

isaacs commented Jul 20, 2016

Good catch! It was deciding to do fancy stdout stuff based on stdout being a tty, but then would only do the beginningOfLine ansi code if both stdout and stderr were tty's. Should be fixed now.

@vladikoff
Copy link
Author

Thank you!

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

No branches or pull requests

2 participants