Merge pull request #2080 from goldim/pass-noparse-files-to-mdeps

goto-bus-stop · web-flow · commit c8f43c74d8e0 · 2024-10-03T20:38:00.000+01:00
Relative paths in noparse option
diff --git a/index.js b/index.js
@@ -574,6 +574,7 @@ Browserify.prototype._createDeps = function (opts) {
     }).map(function (x) {
         return path.resolve(basedir, x);
     });
+    mopts.noParse = absno;
     
     function globalTr (file) {
         if (opts.detectGlobals === false) return through();
diff --git a/test/noparse.js b/test/noparse.js
@@ -29,3 +29,31 @@ test('noParse array', function (t) {
         t.deepEqual(actual, expected);
     });
 });
+
+test('noParse array with relative paths', function (t) {
+    process.chdir(__dirname);
+    
+    t.plan(2);
+    
+    var actual = [];
+    var expected = [
+        'noparse/a.js',
+        'noparse/b.js',
+        'noparse/dir1/1.js',
+        'noparse/node_modules/robot/main.js'
+    ].map(function (x) { return path.resolve(x); }).sort();
+    
+    var b = browserify({
+        entries: [__dirname + '/noparse/a.js'],
+        noParse: [
+            path.join('noparse/dir1/1.js'),
+            path.join('noparse/node_modules/robot/main.js')
+        ]
+    });
+    b.on('dep', function (dep) { actual.push(dep.file); });
+    b.bundle(function (err, src) {
+        actual.sort();
+        t.ifError(err);
+        t.deepEqual(actual, expected);
+    });
+});
