Merge pull request #235 from marcandre/comments

Comments
babel · Oct 9, 2015 · f35fdd4 · f35fdd4
2 parents a62e1e0 + be1f813
commit f35fdd4
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -32,7 +32,7 @@ can be changed](https://github.com/babel/generator-babel-boilerplate#i-want-to-c
 
 Run `gulp build` to compile the source into a distributable format.
 
-Put your unit tests in `test/unit`. The `gulp` command runs the tests.
+Put your unit tests in `test/unit`. The `gulp` command runs the tests using Node. If your library / tests uses features of the DOM, see the `test/setup/node.js` file.
 
 ### Gulp tasks
 

diff --git a/app/templates/gulpfile.babel.js b/app/templates/gulpfile.babel.js
@@ -197,7 +197,7 @@ gulp.task('test', ['lint-src', 'lint-test'], test);
 // Set up coverage and run tests
 gulp.task('coverage', ['lint-src', 'lint-test'], coverage);
 
-// Set up a livereload environment for our spec runner
+// Set up a livereload environment for our spec runner `test/runner.html`
 gulp.task('test-browser', testBrowser);
 
 // Run the headless unit tests as you make changes.

diff --git a/app/templates/test/runner.html b/app/templates/test/runner.html
@@ -1,4 +1,5 @@
 <!doctype html>
+<!-- You need to run `gulp test-browser` before opening this in your browser -->
 <html lang='en'>
 <head>
   <meta charset='utf-8'>

diff --git a/app/templates/test/setup/node.js b/app/templates/test/setup/node.js
@@ -4,3 +4,11 @@ global.chai.use(require('sinon-chai'));
 
 require('babel-core/register');
 require('./setup')();
+
+/*
+	Uncomment the following if your library uses features of the DOM,
+	for example if writing a jQuery extension, and
+	add 'simple-jsdom' to the `devDependencies` of your package.json
+*/
+// import simpleJSDom from 'simple-jsdom';
+// simpleJSDom.install();