Merge pull request #64 from mvayngrib/alt-browser

fall back to 'browser' in deps without alt-browser field

Author: defunctzombie

index.js

@@ -312,7 +312,7 @@ function normalizeBrowserFieldName(browser) {
 
 function getReplacements(info, browser) {
     browser = normalizeBrowserFieldName(browser);
-    var replacements = info[browser];
+    var replacements = info[browser] || info.browser;
 
     // support legacy browserify field for easier migration from legacy
     // many packages used this field historically

test/fixtures/node_modules/alt-browser-field/foo.js

@@ -263,3 +263,19 @@ test('alt-browser deep module reference mapping', function(done) {
         done();
     });
 });
+
+test('alt-browser fallback to "browser" on deps of deps', function(done) {
+    var parent = {
+        filename: fixtures_dir + '/alt-browser-field/foo.js',
+        package: { main: './index.js' },
+        browser: 'chromeapp'
+    };
+
+    resolve('foobar', parent, function(err, path, pkg) {
+        assert.ifError(err);
+        assert.equal(path, require.resolve('./fixtures/node_modules/module-k/browser'));
+        assert.equal(pkg.main, './browser.js');
+        assert.equal(pkg.browser['./index.js'], './browser.js');
+        done();
+    });
+});