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

Add --outfile arg #7

Merged
merged 3 commits into from
Jul 18, 2016
Merged

Conversation

jonnyreeves
Copy link
Contributor

See mantoni/mochify.js#140 for justification.

  • Add --outfile arg: writes the html file to a given path as well as passing it through to browserify.
  • Add file-system sandbox for testcases: removes the manual fs cleanup
  • Resolve the bundle relative to the outfile.

Allow a destination path to be supplied; the standalone HTML file will be written here as well as being passed through to browserify.
All tests that write to the file system are now sandboxed so the files generated during the testcase are automatically removed at the end of the run without requiring manual cleanup
@jonnyreeves jonnyreeves changed the title Feature/destination Add --outfile arg Jul 16, 2016
if (done) {
done();
out.on('end', done);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to wait for the async chain of fs.writeFile calls to compete

@mantoni
Copy link
Owner

mantoni commented Jul 17, 2016

Awesome work! Thanks for the contribution. As an alternative to the custom lib/util.js implementation, have you looked at listen.js? I usually use that for cases like these.

@jonnyreeves
Copy link
Contributor Author

jonnyreeves commented Jul 18, 2016

@mantoni would you like me to remove the series util and replace it with a listen.js based implementation before this can land? Happy to do so, but will need to make some time for it.

@mantoni mantoni merged commit 27fdf34 into mantoni:master Jul 18, 2016
@mantoni
Copy link
Owner

mantoni commented Jul 18, 2016

Sorry. This is good to land as is. Easy enough to change to listen.js later. Thanks again.

@mantoni
Copy link
Owner

mantoni commented Jul 18, 2016

Released in v2.2.0 🍻

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

Successfully merging this pull request may close these issues.

2 participants