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

Unified Diff: tests/corelib/uri_test.dart

Issue 2117453002: Fix regression for the one case where we deliberately don't follow the RFC. (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Fixed Created 4 years, 6 months 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
« no previous file with comments | « sdk/lib/core/uri.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/corelib/uri_test.dart
diff --git a/tests/corelib/uri_test.dart b/tests/corelib/uri_test.dart
index 5d8ce189043fe053c0006c2572170227a3f4025d..a9166be3c11d0dd205f6be6fbfef99b9e504cc00 100644
--- a/tests/corelib/uri_test.dart
+++ b/tests/corelib/uri_test.dart
@@ -150,6 +150,11 @@ testUriPerRFCs() {
testResolve("s:bar", "bar");
testResolve("s:bar", "../bar");
+ // Special-case (deliberate non-RFC behavior).
+ base = Uri.parse("foo/bar");
+ testResolve("foo/baz", "baz");
+ testResolve("baz", "../baz");
+
base = Uri.parse("s:/foo");
testResolve("s:/bar", "bar");
testResolve("s:/bar", "../bar");
@@ -164,7 +169,7 @@ testUriPerRFCs() {
testResolve("./", "../..");
testResolve("../", "../../..");
testResolve("a/b/", ".");
- testResolve("/c", "../../c");
+ testResolve("c", "../../c"); // Deliberate non-RFC behavior.
base = Uri.parse("../../a/b/c?_#_"); // Initial ".." in base url.
testResolve("../../a/d", "../d");
testResolve("../../../d", "../../../d");
« no previous file with comments | « sdk/lib/core/uri.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698