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

Unified Diff: pkg/analysis_server/test/index/page_node_manager_test.dart

Issue 327003004: Tweaks for review comments. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 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
Index: pkg/analysis_server/test/index/page_node_manager_test.dart
diff --git a/pkg/analysis_server/test/index/page_node_manager_test.dart b/pkg/analysis_server/test/index/page_node_manager_test.dart
index b2e2eaeeaeab777284d1dd93a79b084f3b2ce3be..78b1281314ab33e79b0233e31d0a5d8c27153803 100644
--- a/pkg/analysis_server/test/index/page_node_manager_test.dart
+++ b/pkg/analysis_server/test/index/page_node_manager_test.dart
@@ -53,7 +53,7 @@ _treeWithPageNodeManager() {
int tryCount = 1000;
Set<int> keys = new Set<int>();
{
- Random random = new Random();
+ Random random = new Random(37);
for (int i = 0; i < tryCount; i++) {
int key = random.nextInt(maxKey);
keys.add(key);
@@ -66,7 +66,7 @@ _treeWithPageNodeManager() {
}
// remove random keys
{
- Random random = new Random();
+ Random random = new Random(37);
for (int key in new Set<int>.from(keys)) {
if (random.nextBool()) {
keys.remove(key);
@@ -229,6 +229,14 @@ class _FixedStringCodecTest {
// decode
expect(codec.decode(buffer), 'AB');
}
+
+ test_russian() {
+ // encode
+ codec.encode(buffer, 'ЩУКА');
+ expect(bytes, [0, 4, 4, 41, 4, 35, 4, 26, 4, 16]);
+ // decode
+ expect(codec.decode(buffer), 'ЩУКА');
+ }
}
@@ -286,6 +294,14 @@ class _MemoryPageManagerTest {
manager.write(42, page);
}, throws);
}
+
+ test_write_invalidLenght() {
Paul Berry 2014/06/10 20:23:40 "Length" misspelled
scheglov 2014/06/10 20:36:55 Done.
+ int id = manager.alloc();
+ Uint8List page = new Uint8List(0);
+ expect(() {
+ manager.write(id, page);
+ }, throws);
+ }
}

Powered by Google App Engine
This is Rietveld 408576698