modified multi bundle unique test for removal of hashing

browserify · Jul 23, 2014 · 32d4054 · 32d4054
1 parent 5ea1412
commit 32d4054
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/changelog.markdown b/changelog.markdown
@@ -19,6 +19,10 @@ for path masking
 
 removes derequire for performance reasons
 
+.external(bundle) has been removed for now
+
+semicolon at end of output
+
 # 4.2.2
 
 fixes a bug applying transforms to symlinked files by resolving the realpath

diff --git a/test/multi_bundle.js b/test/multi_bundle.js
@@ -43,7 +43,8 @@ test('multi bundle', function (t) {
     });
 });
 
-test('multi bundle', function (t) {
+// re-enable this in future releases
+if (false) test('multi bundle', function (t) {
     t.plan(8);
 
     var core = browserify({ exposeAll: true });

diff --git a/test/multi_bundle_unique.js b/test/multi_bundle_unique.js
@@ -12,14 +12,14 @@ test('unique require', function (t) {
         externalRequireName: 'unique_require',
         prelude: prelude
     });
-    core.require(__dirname + '/multi_bundle/b.js', { expose: 'b' });
+    core.require(__dirname + '/multi_bundle/b.js', { expose: './b' });
 
     var app = browserify(
         [__dirname + '/multi_bundle/a.js'],
         { prelude: prelude }
     );
     // inform this bundle that b exists in another bundle
-    app.external(__dirname + '/multi_bundle/b.js');
+    app.external('./b');
 
     core.bundle(function (err, src) {
         var c = {
@@ -49,7 +49,7 @@ test('unique require', function (t) {
             // because it was part of the core bundle
             t.equal(c.baton.times, 1);
 
-            t.equal(c.unique_require('b'), 'foo');
+            t.equal(c.unique_require('./b'), 'foo');
         });
     });
 });