Add resource tests.

tests/corelib/data_resource_test.dart

Index: tests/corelib/data_resource_test.dart
diff --git a/tests/corelib/data_resource_test.dart b/tests/corelib/data_resource_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..a3d4f9a7b4cce04f7d522723cdf2d975c20814f6
--- /dev/null
+++ b/tests/corelib/data_resource_test.dart
@@ -0,0 +1,52 @@
+// Copyright (c) 2015, 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.
+
+const sampleText = "Sample text file";
+
+main() async {
+  var uriEncoded = sampleText.replaceAll(' ', '%20');
+  await testUri("data:application/dart;charset=utf-8,$uriEncoded");
+  // await testUri("data:text/plain;charset=utf-8,$uriEncoded");

[Søren Gjesse, 2015/08/07 13:51:16]

Open bugs on these contentType and base64 issues.

[Lasse Reichstein Nielsen, 2015/08/10 10:40:09]

Acknowledged.

+  var base64Encoded = "U2FtcGxlIHRleHQgZmlsZQ==";
+  // await testUri("data:application/dart;charset=utf-8;base64,$base64Encoded");
+  // await testUri("data:text/plain;charset=utf-8;base64,$base64Encoded");

[Lasse Reichstein Nielsen, 2015/08/07 12:21:35]

I think we should properly support the data: scheme for resources:
- don't require application/dart for the content type (since it likely isn't for
resources)
- accept base64 encoding.

[Ivan Posva, 2015/08/10 05:39:56]

ACK
Once the Uri class understands base64 encoding/decoding, then that should become
simple in the implementations.

[Lasse Reichstein Nielsen, 2015/08/10 10:40:09]

I'm not sure the Uri class is the right place - but then again, that's where the
data:-URI ends up anyway, even if they aren't hierarchical.
That'll need some design though to not mess up :)

+}
+
+testUri(uriText) async {
+  var resource = new Resource(uriText);
+
+  if (resource.uri != Uri.parse(uriText)) {
+    throw "uriText: Incorrect URI: ${resource.uri}";
+  }
+
+  var text = await resource.readAsString();
+  if (text != sampleText) {
+    throw "uriText: Incorrect reading of text file: $text";
+  }
+
+  var bytes = await resource.readAsBytes();
+  if (!compareBytes(bytes, sampleText.codeUnits)) {
+    throw "uriText: Incorrect reading of bytes: $bytes";
+  }
+
+  var streamBytes = [];
+  await for (var byteSlice in resource.openRead()) {
+    streamBytes.addAll(byteSlice);
+  }
+  if (!compareBytes(streamBytes, sampleText.codeUnits)) {
+    throw "uriText: Incorrect reading of bytes: $bytes";
+  }
+  if (!compareBytes(streamBytes, bytes)) {

[Søren Gjesse, 2015/08/07 13:51:16]

I don't think this check adds anything.

[Lasse Reichstein Nielsen, 2015/08/07 13:55:52]

True, it made sense for the package_resource_test because that file may contain
a line terminator, but here it isn't necessary.

+    throw "uriText: Inconsistent reading of bytes: $bytes / $streamBytes";
+  }
+}
+
+/// Checks that [bytes] and [expectedBytes] have the same contents.
+bool compareBytes(bytes, expectedBytes) {
+  if (bytes.length != expectedBytes.length) return false;
+  for (int i = 0; i < expectedBytes.length; i++) {
+    if (bytes[i] != expectedBytes[i]) return false;
+  }
+  return true;
+}