Use resolvePackage instead of resolveModulePath in `resolveHasteN…

…ame` (#1136)

Summary:
This fixes #1128 by calling the `resolvePackage` instead of `resolveModulePath` in `resolveHasteName`.
Only `resolvePackage` has the code to resolve package "exports" and it calls `resolveModulePath` as a fallback.

Changelog: [Experimental] When enabled, the `"exports"` field is now considered for Haste packages (which could include local monorepo packages)

Pull Request resolved: #1136

Test Plan: I've added a failing test which passed as the fix got implemented.

Reviewed By: huntie

Differential Revision: D51346769

Pulled By: robhogan

fbshipit-source-id: 8a003d5b147b73d344365db7cff8187ff946013d

packages/metro-resolver/src/__tests__/package-exports-test.js

@@ -455,6 +455,24 @@ describe('with package exports resolution enabled', () => {
  expect(logWarning).not.toHaveBeenCalled();
  });
  });
+
+ describe('haste package', () => {
+ test('should resolve subpath in "exports"', () => {
+ const context = {
+ ...baseContext,
+ resolveHastePackage(name: string) {
+ if (name === 'test-pkg') {
+ return '/root/node_modules/test-pkg/package.json';
+ }
+ return null;
+ },
+ };
+ expect(Resolver.resolve(context, 'test-pkg/foo.js', null)).toEqual({
+ type: 'sourceFile',
+ filePath: '/root/node_modules/test-pkg/lib/foo.js',
+ });
+ });
+ });
  });
 
  describe('subpath patterns', () => {

packages/metro-resolver/src/resolve.js

@@ -206,7 +206,7 @@ function resolveHasteName(
  const packageDirPath = path.dirname(packageJsonPath);
  const pathInModule = moduleName.substring(packageName.length + 1);
  const potentialModulePath = path.join(packageDirPath, pathInModule);
- const result = resolveModulePath(context, potentialModulePath, platform);
+ const result = resolvePackage(context, potentialModulePath, platform);
  if (result.type === 'resolved') {
  return result;
  }