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

Unified Diff: tests/corelib/uri_scheme_test.dart

Issue 23766030: Return the correct port from Uri, when port is 0 and scheme is either http or https. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Add tests and check for first charecter is alphabetic in scheme. Created 7 years, 3 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 | « tests/corelib/uri_path_test.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_scheme_test.dart
diff --git a/tests/corelib/uri_scheme_test.dart b/tests/corelib/uri_scheme_test.dart
index 2a7fd764e126beca2a0c9d05eee14523e47fef0f..2565e70ab427372c705300c32c1ccc267d75e755 100644
--- a/tests/corelib/uri_scheme_test.dart
+++ b/tests/corelib/uri_scheme_test.dart
@@ -7,6 +7,8 @@ import "package:expect/expect.dart";
void testInvalidArguments() {
Expect.throws(() => new Uri(scheme: "_"), (e) => e is ArgumentError);
Expect.throws(() => new Uri(scheme: "http_s"), (e) => e is ArgumentError);
+ Expect.throws(() => new Uri(scheme: "127.0.0.1:80"),
+ (e) => e is ArgumentError);
}
void testScheme() {
@@ -21,6 +23,7 @@ void testScheme() {
test("http+ssl", "http+ssl:", "HTTP+ssl");
test("urn", "urn:", "urn");
test("urn", "urn:", "UrN");
+ test("a123.432", "a123.432:", "a123.432");
}
main() {
« no previous file with comments | « tests/corelib/uri_path_test.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698