Fix for run extension tests with commonjs

m3talsmith · Jan 25, 2016 · 341122a · 341122a
1 parent 0587f20
commit 341122a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/test/all.js b/test/all.js
@@ -50,7 +50,7 @@ function loadClientTests(cb) {
 			return file.replace(/\.js$/, '');
 		});
 
-		// load all modules
+		// load all modules with the AMD loader
 		define(modules, function () {
 			cb(null);
 		}, cb);
@@ -61,13 +61,12 @@ function loadPluginTests(cb) {
 	var root = path.join(path.dirname(__dirname), 'extensions');
 	glob(TEST_GLOB, { cwd: root }, function (err, files) {
 
+		// load modules with commonjs
 		var modules = files.map(function (file) {
-			return 'extensions/' + file.replace(/\.js$/, '');
+			return '../extensions/' + file.replace(/\.js$/, '');
 		});
-
-		define(modules, function() {
-			cb(null);
-		}, cb);
+		modules.forEach(require);
+		cb(null);
 	});
 }
 
@@ -142,7 +141,7 @@ function main() {
 		}));
 	} else {
 		loadTasks.push(loadClientTests);
-		//loadTasks.push(loadPluginTests);
+		loadTasks.push(loadPluginTests);
 	}
 
 	async.parallel(loadTasks, function (err) {

diff --git a/test/mocha.opts b/test/mocha.opts
@@ -2,4 +2,3 @@
 --ui tdd
 --reporter dot
 test/all.js
-extensions/**/test/*.test.js