Merge pull request mochajs#2669 from mochajs/avoid-dist-file-modifica…

…tion-in-test

Use temporary test-only build of mocha.js for browsers tests

.eslintignore

@@ -1,3 +1,4 @@
 coverage/
 lib/to-iso-string/**/*.js
 mocha.js
+BUILDTMP

.gitignore

@@ -12,3 +12,4 @@ npm-debug.log*
 *.orig
 .nyc_output/
 coverage/
+BUILDTMP/

Makefile

@@ -18,8 +18,9 @@ TESTS = $(shell find test -name "*.js" -type f | sort)
 
 all: mocha.js
 
-mocha.js: $(SRC) browser-entry.js
+mocha.js BUILDTMP/mocha.js: $(SRC) browser-entry.js
 	@printf "==> [Browser :: build]\n"
+	mkdir -p ${@D}
 	$(BROWSERIFY) ./browser-entry \
 		--plugin ./scripts/dedefine \
 		--ignore 'fs' \
@@ -29,21 +30,21 @@ mocha.js: $(SRC) browser-entry.js
 
 clean:
 	@printf "==> [Clean]\n"
-	rm -f mocha.js
+	rm -rf BUILDTMP
 
 lint:
 	@printf "==> [Test :: Lint]\n"
 	$(ESLINT) . bin/*
 
 test-node: test-bdd test-tdd test-qunit test-exports test-unit test-integration test-jsapi test-compilers test-glob test-requires test-reporters test-only test-global-only
 
-test-browser: clean mocha.js test-browser-unit test-browser-bdd test-browser-qunit test-browser-tdd test-browser-exports
+test-browser: clean BUILDTMP/mocha.js test-browser-unit test-browser-bdd test-browser-qunit test-browser-tdd test-browser-exports
 
 test: lint test-node test-browser
 
 test-browser-unit:
 	@printf "==> [Test :: Browser]\n"
-	NODE_PATH=. $(KARMA) start --single-run
+	NODE_PATH=BUILDTMP $(KARMA) start --single-run
 
 test-browser-bdd:
 	@printf "==> [Test :: Browser :: BDD]\n"

package.json

@@ -299,10 +299,11 @@
     "npm": ">= 1.4.x"
   },
   "scripts": {
-    "test": "make test",
+    "test": "make test && make clean",
     "precoverage": "rm -rf coverage",
     "coverage": "COVERAGE=true npm run test",
-    "postcoverage": "istanbul-combine -d coverage -r lcov -r html coverage/reports/*/*.json"
+    "postcoverage": "istanbul-combine -d coverage -r lcov -r html coverage/reports/*/*.json",
+    "preversion": "make test && make mocha.js && git add mocha.js"
   },
   "dependencies": {
     "browser-stdout": "1.3.0",