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

Doesn't output to stdout #5

Closed
morganrallen opened this issue Sep 9, 2017 · 6 comments · Fixed by #6
Closed

Doesn't output to stdout #5

morganrallen opened this issue Sep 9, 2017 · 6 comments · Fixed by #6
Assignees

Comments

@morganrallen
Copy link

When looking at bin/tap-html one would think this outputs to stdout but instead tap-html.html is always created. Looking at lib/generate.js:71 it can be seen it is in fact hardcoded to ./tap-html.html.

@gabrielcsapo
Copy link
Owner

An extra option can output the generated HTML to stdout, would that be a good addition?

@morganrallen
Copy link
Author

My thinking is that should be the default option. Pipeline in, pipeline out. Then have an optional -f/--file or -o/--out/--output/--outfile if you want to write directly to a file.

@gabrielcsapo
Copy link
Owner

@morganrallen so my main motivation for building this was to generate an html report much like http://adamgruber.github.io/mochawesome/ for mocha. You make a good point that since these should be pipeable, it should have a default of behavior as such.

@gabrielcsapo
Copy link
Owner

Another problem with this requirement is that in order to generate the html test report, I am using webpack, in order for webpack to work it must write to disk. @morganrallen should it write to a virtual disk and only dump the contents to the physical disk if the output is wanted as such?

@morganrallen
Copy link
Author

I've never used webpack so I don't know what the best course with that is.

@gabrielcsapo
Copy link
Owner

I think I found a solution @morganrallen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants