Fix the resolution of uri-based part-of directives in the SDK (issue 29598)

pkg/analyzer/test/src/context/source_test.dart

 // Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
-library analyzer.test.src.context.source_test;
-
 import 'package:analyzer/file_system/file_system.dart';
+import 'package:analyzer/file_system/memory_file_system.dart';
 import 'package:analyzer/src/context/source.dart';
+import 'package:analyzer/src/dart/sdk/sdk.dart';
+import 'package:analyzer/src/generated/java_engine_io.dart';
 import 'package:analyzer/src/generated/source.dart';
 import 'package:package_config/packages.dart';
 import 'package:test/test.dart';
 import 'package:test_reflective_loader/test_reflective_loader.dart';
 
 import 'abstract_context.dart';
 
 main() {
   defineReflectiveSuite(() {
     defineReflectiveTests(SourceFactoryImplTest);
   });
 }
 
 @reflectiveTest
 class SourceFactoryImplTest extends AbstractContextTest {
+  @soloTest
+  void test_resolveUri_coreFromPart() {
+    SourceFactoryImpl sourceFactory = new SourceFactoryImpl(<UriResolver>[
+      new ResourceUriResolver(resourceProvider),
+      new DartUriResolver(_createDartSdk())
+    ]);
+    Source coreSource = sourceFactory.forUri('dart:core');
+    Source numSource = sourceFactory.resolveUri(coreSource, 'num.dart');
+    expect(sourceFactory.resolveUri(numSource, 'core.dart'), coreSource);

[scheglov, 2017/06/29 20:19:24]

We test functionality of DartSdk, so it would be logical to put the test into
the corresponding DartSdk tests, not into SourceFactory test.

[Brian Wilkerson, 2017/06/29 20:47:15]

Done

+  }
+
   void test_restoreUri() {
     String libPath = resourceProvider.convertPath('/pkgs/somepkg/lib/');
     Uri libUri = resourceProvider.getFolder(libPath).toUri();
     Map<String, Uri> packageUriMap = <String, Uri>{'foo': libUri};
     SourceFactoryImpl sourceFactory = new SourceFactoryImpl(
       <UriResolver>[new ResourceUriResolver(resourceProvider)],
       new _MockPackages(packageUriMap),
     );
     Source libSource = newSource('/pkgs/somepkg/lib');
     Uri uri = sourceFactory.restoreUri(libSource);
     try {
       expect(uri, Uri.parse('package:foo/'));
     } catch (e) {
       print('=== debug info ===');
       print('libPath: $libPath');
       print('libUri: $libUri');
       print('libSource: ${libSource?.fullName}');
       rethrow;
     }
   }
+
+  FolderBasedDartSdk _createDartSdk() {
+    resourceProvider = new MemoryResourceProvider();
+    Folder sdkDirectory =
+        resourceProvider.getFolder(resourceProvider.convertPath('/sdk'));
+    _createFile(sdkDirectory,
+        ['lib', '_internal', 'sdk_library_metadata', 'lib', 'libraries.dart'],
+        content: _librariesFileContent());
+    _createFile(sdkDirectory, ['bin', 'dart']);
+    _createFile(sdkDirectory, ['bin', 'dart2js']);
+    _createFile(sdkDirectory, ['bin', 'pub']);
+    _createFile(sdkDirectory, ['lib', 'async', 'async.dart']);
+    _createFile(sdkDirectory, ['lib', 'core', 'core.dart']);
+    _createFile(sdkDirectory, ['lib', 'core', 'num.dart']);

[scheglov, 2017/06/29 20:19:24]

Hm... Why do we need all the rest if we use only core?

+    _createFile(sdkDirectory,
+        ['lib', 'html', 'html_common', 'html_common_dart2js.dart']);
+    _createFile(sdkDirectory, ['lib', 'html', 'dartium', 'html_dartium.dart']);
+    _createFile(
+        sdkDirectory, ['bin', (OSUtilities.isWindows() ? 'pub.bat' : 'pub')]);
+    return new FolderBasedDartSdk(resourceProvider, sdkDirectory);
+  }
+
+  void _createFile(Folder directory, List<String> segments,
+      {String content: ''}) {
+    Folder parent = directory;
+    int last = segments.length - 1;
+    for (int i = 0; i < last; i++) {
+      parent = parent.getChildAssumingFolder(segments[i]);
+    }
+    File file = parent.getChildAssumingFile(segments[last]);
+    resourceProvider.newFile(file.path, content);
+  }
+
+  String _librariesFileContent() => '''
+final Map<String, LibraryInfo> LIBRARIES = const <String, LibraryInfo> {
+  "async": const LibraryInfo(
+      "async/async.dart",
+      categories: "Client,Server",
+      maturity: Maturity.STABLE,
+      dart2jsPatchPath: "_internal/js_runtime/lib/async_patch.dart"),
+
+  "core": const LibraryInfo(
+      "core/core.dart",
+      categories: "Client,Server,Embedded",
+      maturity: Maturity.STABLE,
+      dart2jsPatchPath: "_internal/js_runtime/lib/core_patch.dart"),
+
+  "html": const LibraryInfo(
+      "html/dartium/html_dartium.dart",
+      categories: "Client",
+      maturity: Maturity.WEB_STABLE,
+      dart2jsPath: "html/dart2js/html_dart2js.dart"),
+
+  "html_common": const LibraryInfo(
+      "html/html_common/html_common.dart",
+      categories: "Client",
+      maturity: Maturity.WEB_STABLE,
+      dart2jsPath: "html/html_common/html_common_dart2js.dart",
+      documented: false,
+      implementation: true),
+};
+''';
 }
 
 /**
  * An implementation of [Packages] used for testing.
  */
 class _MockPackages implements Packages {
   final Map<String, Uri> map;
 
   _MockPackages(this.map);
 
   @override
   Iterable<String> get packages => map.keys;
 
   @override
   Map<String, Uri> asMap() => map;
 
   @override
   Uri resolve(Uri packageUri, {Uri notFound(Uri packageUri)}) {
     fail('Unexpected invocation of resolve');
     return null;
   }
 }