Merge pull request #9 from bengourley/master

Precedence issue
serby · May 22, 2012 · 8689fc7 · 8689fc7
2 parents dd5fd51 + 0c266ca
commit 8689fc7
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 1 deletion.
diff --git a/lib/compact.js b/lib/compact.js
@@ -69,7 +69,7 @@ module.exports.createCompact = function(options) {
       for (var i = 0; i < paths.length; i++) {
         if (path.existsSync(paths[i])) {
           jsPath = paths[i];
-          continue;
+          break;
         }
       }
 

diff --git a/test/assets-alt/a.js b/test/assets-alt/a.js
@@ -0,0 +1 @@
+console.log("hello from alt!")
diff --git a/test/compact.test.js b/test/compact.test.js
@@ -359,5 +359,30 @@ describe('compact.js', function() {
 
 
     });
+
+    it('should give higher precedence to the added srcPath', function (done) {
+
+      compact.addNamespace('alternative', altPath);
+      compact.ns.alternative.addJs('a.js');
+
+      var app = {
+        helpers: function (helper) {},
+        configure: function(fn) {
+          fn();
+        }
+      };
+
+      compact.js(['alternative'])({ app : app }, {}, function () {
+
+        var compacted = fs.readFileSync(destPath + '/alternative.js', 'utf8')
+          , raw = fs.readFileSync(altPath + '/a.js', 'utf8');
+
+        compacted.should.equal(raw);
+        done();
+
+      });
+
+    });
+
   });
 });