Skip to content

Commit

Permalink
Update node-haste from upstream
Browse files Browse the repository at this point in the history
Reviewed By: zpao

Differential Revision: D2747183

fb-gh-sync-id: f1b963b19cb6ea16945b16370d1bee26110bb329
  • Loading branch information
cpojer authored and facebook-github-bot-3 committed Dec 11, 2015
1 parent f1e32ae commit a04e2be
Show file tree
Hide file tree
Showing 4 changed files with 39 additions and 54 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1362,19 +1362,19 @@ describe('DependencyGraph', function() {
});
});

testBrowserField('browser')
testBrowserField('react-native')
testBrowserField('browser');
testBrowserField('react-native');

function replaceBrowserField (json, fieldName) {
function replaceBrowserField(json, fieldName) {
if (fieldName !== 'browser') {
json[fieldName] = json.browser
delete json.browser
json[fieldName] = json.browser;
delete json.browser;
}

return json
return json;
}

function testBrowserField (fieldName) {
function testBrowserField(fieldName) {
pit('should support simple browser field in packages ("' + fieldName + '")', function() {
var root = '/root';
fs.__setMockFilesystem({
Expand All @@ -1393,8 +1393,8 @@ describe('DependencyGraph', function() {
}, fieldName)),
'main.js': 'some other code',
'client.js': 'some code',
}
}
},
},
});

var dgraph = new DependencyGraph({
Expand Down Expand Up @@ -1448,8 +1448,8 @@ describe('DependencyGraph', function() {
}, fieldName)),
'main.js': 'some other code',
'client.js': 'some code',
}
}
},
},
});

var dgraph = new DependencyGraph({
Expand Down Expand Up @@ -1503,8 +1503,8 @@ describe('DependencyGraph', function() {
}, fieldName)),
'main.js': 'some other code',
'client.js': 'some code',
}
}
},
},
});

var dgraph = new DependencyGraph({
Expand Down Expand Up @@ -1559,8 +1559,8 @@ describe('DependencyGraph', function() {
}, fieldName)),
'main.js': 'some other code',
'client.js': 'some code',
}
}
},
},
});

var dgraph = new DependencyGraph({
Expand Down Expand Up @@ -1630,15 +1630,15 @@ describe('DependencyGraph', function() {
},
'hello.js': 'hello',
'bye.js': 'bye',
}
}
},
},
});

var dgraph = new DependencyGraph({
...defaults,
roots: [root],
});
return getOrderedDependenciesAsJSON(dgraph, '/root/index.js').then(function(deps) {
const dgraph = new DependencyGraph({
...defaults,
roots: [root],
});
return getOrderedDependenciesAsJSON(dgraph, '/root/index.js').then(function(deps) {
expect(deps)
.toEqual([
{ id: 'index',
Expand Down Expand Up @@ -1716,7 +1716,7 @@ describe('DependencyGraph', function() {
name: 'aPackage',
browser: {
'node-package': 'browser-package',
}
},
}, fieldName)),
'index.js': 'require("node-package")',
'node-package': {
Expand All @@ -1731,8 +1731,8 @@ describe('DependencyGraph', function() {
}),
'index.js': 'some browser code',
},
}
}
},
},
});

var dgraph = new DependencyGraph({
Expand Down Expand Up @@ -1788,7 +1788,7 @@ describe('DependencyGraph', function() {
name: 'aPackage',
browser: {
'node-package': 'browser-package',
}
},
}, fieldName)),
'index.js': 'require("node-package")',
'node-package': {
Expand All @@ -1803,8 +1803,8 @@ describe('DependencyGraph', function() {
}),
'index.js': 'some browser code',
},
}
}
},
},
});

var dgraph = new DependencyGraph({
Expand Down Expand Up @@ -1861,34 +1861,34 @@ describe('DependencyGraph', function() {
name: 'aPackage',
'react-native': {
'node-package': 'rn-package',
}
},
}),
'index.js': 'require("node-package")',
'node_modules': {
'node-package': {
'package.json': JSON.stringify({
'name': 'node-package'
'name': 'node-package',
}),
'index.js': 'some node code',
},
'rn-package': {
'package.json': JSON.stringify({
'name': 'rn-package',
browser: {
'nested-package': 'nested-browser-package'
}
'nested-package': 'nested-browser-package',
},
}),
'index.js': 'require("nested-package")',
},
'nested-browser-package': {
'package.json': JSON.stringify({
'name': 'nested-browser-package',
}),
'index.js': 'some code'
}
}
}
}
'index.js': 'some code',
},
},
},
},
});

var dgraph = new DependencyGraph({
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ function extract(contents) {
}


var commentStartRe = /^\/\*\*?/;
var commentStartRe = /^\/\*\*/;
var commentEndRe = /\*\/$/;
var wsRe = /[\t ]+/g;
var stringStartRe = /(\r?\n|^) *\*/g;
Expand Down
2 changes: 1 addition & 1 deletion packager/react-packager/src/DependencyResolver/Package.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ class Package {

getMain() {
return this._read().then(json => {
var replacements = getReplacements(json)
var replacements = getReplacements(json);
if (typeof replacements === 'string') {
return path.join(this.root, replacements);
}
Expand Down
15 changes: 0 additions & 15 deletions packager/react-packager/src/DependencyResolver/replacePatterns.js

This file was deleted.

0 comments on commit a04e2be

Please sign in to comment.