Skip to content

Commit

Permalink
Merge pull request #1139 from insidewarehouse/resolve-exposed-folders
Browse files Browse the repository at this point in the history
Tests for problems introduced in ~resolve@1.1.5 and fixed in resolve@1.1.6
  • Loading branch information
ljharb committed Feb 8, 2019
2 parents cfd3af2 + f13b713 commit 7ad39ce
Show file tree
Hide file tree
Showing 2 changed files with 70 additions and 1 deletion.
70 changes: 69 additions & 1 deletion test/resolve_exposed.js
Expand Up @@ -4,7 +4,7 @@ var test = require('tap').test;

test('resolve exposed files', function (t) {
t.plan(2);

var b = browserify(__dirname + '/resolve_exposed/main.js', {
basedir: __dirname + '/resolve_exposed'
});
Expand All @@ -18,3 +18,71 @@ test('resolve exposed files', function (t) {
}
});
});

test('resolve exposed files without extension', function (t) {
t.plan(2);

var b = browserify(__dirname + '/resolve_exposed/main.js', {
basedir: __dirname + '/resolve_exposed'
});
b.require('./x', {expose: 'xyz'});
b.bundle(function (err, src) {
t.ifError(err);
var c = {console: {log: log}};
vm.runInNewContext(src, c);
function log(x) {
t.equal(x, 333);
}
});
});

test('resolve exposed directories', function (t) {
t.plan(2);

var b = browserify(__dirname + '/resolve_exposed/main.js', {
basedir: __dirname + '/resolve_exposed'
});
b.require('./y', {expose: 'xyz'});
b.bundle(function (err, src) {
t.ifError(err);
var c = {console: {log: log}};
vm.runInNewContext(src, c);
function log(x) {
t.equal(x, 555);
}
});
});

test('resolve exposed index from directories', function (t) {
t.plan(2);

var b = browserify(__dirname + '/resolve_exposed/main.js', {
basedir: __dirname + '/resolve_exposed'
});
b.require('./y/index', {expose: 'xyz'});
b.bundle(function (err, src) {
t.ifError(err);
var c = {console: {log: log}};
vm.runInNewContext(src, c);
function log(x) {
t.equal(x, 555);
}
});
});

test('resolve exposed index.js from directories', function (t) {
t.plan(2);

var b = browserify(__dirname + '/resolve_exposed/main.js', {
basedir: __dirname + '/resolve_exposed'
});
b.require('./y/index.js', {expose: 'xyz'});
b.bundle(function (err, src) {
t.ifError(err);
var c = {console: {log: log}};
vm.runInNewContext(src, c);
function log(x) {
t.equal(x, 555);
}
});
});
1 change: 1 addition & 0 deletions test/resolve_exposed/y/index.js
@@ -0,0 +1 @@
module.exports = 5

0 comments on commit 7ad39ce

Please sign in to comment.