Chromium Code Reviews| Index: tests/standalone/io/string_decoder_test.dart |
| diff --git a/tests/standalone/io/string_decoder_test.dart b/tests/standalone/io/string_decoder_test.dart |
| index 6e96000093dd01c1d5e751766b9865bd6cefa801..b2f4241c4fe205a916dcf9e77ab8806826f2a942 100644 |
| --- a/tests/standalone/io/string_decoder_test.dart |
| +++ b/tests/standalone/io/string_decoder_test.dart |
| @@ -6,6 +6,8 @@ import "package:expect/expect.dart"; |
| import "dart:async"; |
| import "dart:io"; |
| +const _UNICODE_REPLACEMENT_CHARACTER_CODEPOINT = 0xFFFD; |
|
Anders Johnsen
2013/08/23 06:07:12
You don't really have to make it private here.
floitsch
2013/08/23 08:51:59
Done.
|
| + |
| void testTransform() { |
| // Code point U+10FFFF is the largest code point supported by Dart. |
| var controller = new StreamController(sync: true); |
| @@ -16,7 +18,7 @@ void testTransform() { |
| controller.add([0xfd, 0x80, 0x80, 0x80, 0x80, 0x80]); // U+40000000 |
| controller.close(); |
| - var decoder = new StringDecoder(Encoding.UTF_8, '?'.codeUnitAt(0)); |
| + var decoder = new StringDecoder(Encoding.UTF_8); |
| var stream = controller.stream.transform(decoder); |
| stream.fold( |
| new StringBuffer(), |
| @@ -26,17 +28,16 @@ void testTransform() { |
| }) |
| .then((b) => b.toString()) |
| .then((decoded) { |
| - Expect.equals(8, decoded.length); |
| + Expect.equals(16, decoded.length); |
| - var replacementChar = '?'.codeUnitAt(0); |
| + var replacementChar = _UNICODE_REPLACEMENT_CHARACTER_CODEPOINT; |
| Expect.equals(0xd800, decoded.codeUnitAt(0)); |
| Expect.equals(0xdc00, decoded.codeUnitAt(1)); |
| Expect.equals(0xdbff, decoded.codeUnitAt(2)); |
| Expect.equals(0xdfff, decoded.codeUnitAt(3)); |
| - Expect.equals(replacementChar, decoded.codeUnitAt(4)); |
| - Expect.equals(replacementChar, decoded.codeUnitAt(5)); |
| - Expect.equals(replacementChar, decoded.codeUnitAt(6)); |
| - Expect.equals(replacementChar, decoded.codeUnitAt(7)); |
| + for (int i = 4; i < 16; i++) { |
| + Expect.equals(replacementChar, decoded.codeUnitAt(i)); |
| + } |
| }); |
| } |
| @@ -50,20 +51,18 @@ void testDecode() { |
| controller.add([0xfd, 0x80, 0x80, 0x80, 0x80, 0x80]); // U+40000000 |
| controller.close(); |
| - StringDecoder.decode(controller.stream, |
| - Encoding.UTF_8, |
| - '?'.codeUnitAt(0)).then((decoded) { |
| - Expect.equals(8, decoded.length); |
| + StringDecoder.decode(controller.stream, Encoding.UTF_8) |
| + .then((decoded) { |
| + Expect.equals(16, decoded.length); |
| - var replacementChar = '?'.codeUnitAt(0); |
| + var replacementChar = _UNICODE_REPLACEMENT_CHARACTER_CODEPOINT; |
| Expect.equals(0xd800, decoded.codeUnitAt(0)); |
| Expect.equals(0xdc00, decoded.codeUnitAt(1)); |
| Expect.equals(0xdbff, decoded.codeUnitAt(2)); |
| Expect.equals(0xdfff, decoded.codeUnitAt(3)); |
| - Expect.equals(replacementChar, decoded.codeUnitAt(4)); |
| - Expect.equals(replacementChar, decoded.codeUnitAt(5)); |
| - Expect.equals(replacementChar, decoded.codeUnitAt(6)); |
| - Expect.equals(replacementChar, decoded.codeUnitAt(7)); |
| + for (int i = 4; i < 16; i++) { |
| + Expect.equals(replacementChar, decoded.codeUnitAt(i)); |
| + } |
| }); |
| } |
| @@ -75,7 +74,8 @@ void testInvalid() { |
| controller.stream.transform(new StringDecoder()).listen((string) { |
| Expect.equals(outputLength, string.length); |
| for (var i = 0; i < outputLength; i++) { |
| - Expect.equals(0xFFFD, string.codeUnitAt(i)); |
| + Expect.equals(_UNICODE_REPLACEMENT_CHARACTER_CODEPOINT, |
| + string.codeUnitAt(i)); |
| } |
| }); |
| } |