Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(421)

Unified Diff: compiler/javatests/com/google/dart/compiler/PackageLibraryManagerTest.java

Issue 11365196: Move JSSyntaxRegExp to core as a private member. This removes the last refrences to dart:coreimpl. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Fix two pending TODO's. Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: compiler/javatests/com/google/dart/compiler/PackageLibraryManagerTest.java
diff --git a/compiler/javatests/com/google/dart/compiler/PackageLibraryManagerTest.java b/compiler/javatests/com/google/dart/compiler/PackageLibraryManagerTest.java
index bbd43c1a75de302bcb67644913b514c73eb7dc79..f455ee5d4800347a0eb1f99258753905848574c3 100644
--- a/compiler/javatests/com/google/dart/compiler/PackageLibraryManagerTest.java
+++ b/compiler/javatests/com/google/dart/compiler/PackageLibraryManagerTest.java
@@ -34,25 +34,6 @@ public class PackageLibraryManagerTest extends TestCase {
}
public void testExpand2() throws Exception {
- URI shortUri = new URI("dart:coreimpl");
- URI fullUri = packageLibraryManager.expandRelativeDartUri(shortUri);
- assertNotNull(fullUri);
- assertEquals("dart", fullUri.getScheme());
- assertEquals("coreimpl", fullUri.getHost());
- assertTrue(getPath(fullUri).endsWith("/coreimpl.dart"));
- }
-
- public void testExpand3() throws Exception {
- URI shortUri = new URI("dart:coreimpl");
- URI fullUri1 = packageLibraryManager.expandRelativeDartUri(shortUri);
- URI fullUri2 = packageLibraryManager.expandRelativeDartUri(fullUri1);
- assertNotNull(fullUri2);
- assertEquals("dart", fullUri2.getScheme());
- assertEquals("coreimpl", fullUri2.getHost());
- assertTrue(getPath(fullUri2).endsWith("/coreimpl.dart"));
- }
-
- public void testExpand4() throws Exception {
URI shortUri = new URI("dart:doesnotexist");
URI fullUri = packageLibraryManager.expandRelativeDartUri(shortUri);
assertNull(fullUri);
@@ -69,16 +50,6 @@ public class PackageLibraryManagerTest extends TestCase {
}
public void testTranslate2() throws Exception {
- URI shortUri = new URI("dart:coreimpl");
- URI fullUri = packageLibraryManager.expandRelativeDartUri(shortUri);
- URI translatedURI = packageLibraryManager.resolveDartUri(fullUri);
- assertNotNull(translatedURI);
- String scheme = translatedURI.getScheme();
- assertTrue(scheme.equals("file"));
- assertTrue(getPath(translatedURI).endsWith("/coreimpl.dart"));
- }
-
- public void testTranslate3() throws Exception {
URI fullUri = new URI("dart://doesnotexist/some/file.dart");
URI translatedURI = packageLibraryManager.resolveDartUri(fullUri);
assertNotNull(translatedURI);

Powered by Google App Engine
This is Rietveld 408576698