fix(init): Make the requirejs config template normalize paths

lukeapage · lukeapage · commit 54dcce31ea59 · 2015-06-18T21:36:10.000+01:00
By removing the base path and file extension of files, this normalizes files to be in a standard requirejs format and means mean sub dependencies of the test files will not be loaded as-is. See #513 (comment)
diff --git a/requirejs.config.tpl.coffee b/requirejs.config.tpl.coffee
@@ -1,9 +1,14 @@
 allTestFiles = []
 TEST_REGEXP = /(spec|test)(\.coffee)?(\.js)?$/i
 
+# Get a list of all the test files to include
 Object.keys(window.__karma__.files).forEach (file) ->
-  # Normalize paths to RequireJS module names.
-  allTestFiles.push file  if TEST_REGEXP.test(file)
+
+  if TEST_REGEXP.test(file)
+    # Normalize paths to RequireJS module names.
+    # If you require sub-dependencies of test files to be loaded as-is (requiring file extension)
+    # then do not normalize the paths
+    allTestFiles.push file.replace(/^\/base\/|\.js$/g, '')
   return
 
 require.config
diff --git a/requirejs.config.tpl.js b/requirejs.config.tpl.js
@@ -1,10 +1,14 @@
 var allTestFiles = [];
 var TEST_REGEXP = /(spec|test)\.js$/i;
 
+// Get a list of all the test files to include
 Object.keys(window.__karma__.files).forEach(function(file) {
   if (TEST_REGEXP.test(file)) {
     // Normalize paths to RequireJS module names.
-    allTestFiles.push(file);
+    // If you require sub-dependencies of test files to be loaded as-is (requiring file extension)
+    // then do not normalize the paths
+    var normalizedTestModule = file.replace(/^\/base\/|\.js$/g, '');
+    allTestFiles.push(normalizedTestModule);
   }
 });
 
