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

Unified Diff: tests/corelib_strong/hash_set_test.dart

Issue 2770063002: Revert "Format all multitests" (Closed)
Patch Set: Created 3 years, 9 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_strong/double_parse_test.dart ('k') | tests/corelib_strong/int_modulo_arith_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/corelib_strong/hash_set_test.dart
diff --git a/tests/corelib_strong/hash_set_test.dart b/tests/corelib_strong/hash_set_test.dart
index 12c0630674222a6913406575308ee65ea96684e1..831bf0c82436fafad7cb2afa29449e3c932e1147 100644
--- a/tests/corelib_strong/hash_set_test.dart
+++ b/tests/corelib_strong/hash_set_test.dart
@@ -8,20 +8,19 @@
// modification errors.
library hash_map2_test;
-
import "package:expect/expect.dart";
import 'dart:collection';
import 'dart:math' as math;
testSet(Set newSet(), Set newSetFrom(Iterable from)) {
+
Set gen(int from, int to) =>
new Set.from(new Iterable.generate(to - from, (n) => n + from));
bool odd(int n) => (n & 1) == 1;
bool even(int n) => (n & 1) == 0;
- {
- // Test growing to largish capacity.
+ { // Test growing to largish capacity.
Set set = newSet();
for (int i = 0; i < 256; i++) {
@@ -43,8 +42,7 @@ testSet(Set newSet(), Set newSetFrom(Iterable from)) {
Expect.equals(1000, set.length);
}
- {
- // Test having many deleted elements.
+ { // Test having many deleted elements.
Set set = newSet();
set.add(0);
for (int i = 0; i < 1000; i++) {
@@ -54,8 +52,7 @@ testSet(Set newSet(), Set newSetFrom(Iterable from)) {
}
}
- {
- // Test having many elements with same hashCode
+ { // Test having many elements with same hashCode
Set set = newSet();
for (int i = 0; i < 1000; i++) {
set.add(new BadHashCode());
@@ -63,22 +60,19 @@ testSet(Set newSet(), Set newSetFrom(Iterable from)) {
Expect.equals(1000, set.length);
}
- {
- // Check concurrent modification
+ { // Check concurrent modification
Set set = newSet()..add(0)..add(1);
- {
- // Test adding before a moveNext.
+ { // Test adding before a moveNext.
Iterator iter = set.iterator;
iter.moveNext();
- set.add(1); // Updating existing key isn't a modification.
+ set.add(1); // Updating existing key isn't a modification.
iter.moveNext();
set.add(2);
Expect.throws(iter.moveNext, (e) => e is Error);
}
- {
- // Test adding after last element.
+ { // Test adding after last element.
Iterator iter = set.iterator;
Expect.equals(3, set.length);
iter.moveNext();
@@ -88,11 +82,10 @@ testSet(Set newSet(), Set newSetFrom(Iterable from)) {
Expect.throws(iter.moveNext, (e) => e is Error);
}
- {
- // Test removing during iteration.
+ { // Test removing during iteration.
Iterator iter = set.iterator;
iter.moveNext();
- set.remove(1000); // Not a modification if it's not there.
+ set.remove(1000); // Not a modification if it's not there.
iter.moveNext();
int n = iter.current;
set.remove(n);
@@ -101,8 +94,7 @@ testSet(Set newSet(), Set newSetFrom(Iterable from)) {
Expect.throws(iter.moveNext, (e) => e is Error);
}
- {
- // Test removing after last element.
+ { // Test removing after last element.
Iterator iter = set.iterator;
Expect.equals(3, set.length);
iter.moveNext();
@@ -115,8 +107,7 @@ testSet(Set newSet(), Set newSetFrom(Iterable from)) {
Expect.throws(iter.moveNext, (e) => e is Error);
}
- {
- // Test that updating value doesn't cause error.
+ { // Test that updating value doesn't cause error.
Iterator iter = set.iterator;
Expect.equals(2, set.length);
iter.moveNext();
@@ -126,8 +117,7 @@ testSet(Set newSet(), Set newSetFrom(Iterable from)) {
Expect.isTrue(set.contains(iter.current));
}
- {
- // Check adding many existing values isn't considered modification.
+ { // Check adding many existing values isn't considered modification.
Set set2 = newSet();
for (var value in set) {
set2.add(value);
@@ -139,31 +129,29 @@ testSet(Set newSet(), Set newSetFrom(Iterable from)) {
}
}
- {
- // Check that updating existing elements is not a modification.
- // This must be the case even if the underlying data structure is
- // nearly full.
- for (int i = 1; i < 128; i++) {
- // Create maps of different sizes, some of which should be
- // at a limit of the underlying data structure.
- Set set = newSetFrom(gen(0, i));
- Iterator iter = set.iterator;
- for (int j = 0; j < i; j++) {
- set.add(j);
- }
- iter.moveNext(); // Should not throw.
-
- for (int j = 1; j < i; j++) {
- set.remove(j);
- }
- iter = set.iterator;
- set.add(0);
- iter.moveNext(); // Should not throw.
- }
+ { // Check that updating existing elements is not a modification.
+ // This must be the case even if the underlying data structure is
+ // nearly full.
+ for (int i = 1; i < 128; i++) {
+ // Create maps of different sizes, some of which should be
+ // at a limit of the underlying data structure.
+ Set set = newSetFrom(gen(0, i));
+ Iterator iter = set.iterator;
+ for (int j = 0; j < i; j++) {
+ set.add(j);
+ }
+ iter.moveNext(); // Should not throw.
+
+ for (int j = 1; j < i; j++) {
+ set.remove(j);
+ }
+ iter = set.iterator;
+ set.add(0);
+ iter.moveNext(); // Should not throw.
+ }
}
- {
- // Check that null can be in the set.
+ { // Check that null can be in the set.
Set set = newSet();
set.add(null);
Expect.equals(1, set.length);
@@ -206,8 +194,7 @@ testSet(Set newSet(), Set newSetFrom(Iterable from)) {
Expect.isFalse(set.contains(null));
}
- {
- // Check that addAll and clear works.
+ { // Check that addAll and clear works.
Set set = newSet();
set.addAll([]);
Expect.isTrue(set.isEmpty);
@@ -221,8 +208,7 @@ testSet(Set newSet(), Set newSetFrom(Iterable from)) {
Expect.isTrue(set.isEmpty);
}
- {
- // Check that removeWhere and retainWhere work.
+ { // Check that removeWhere and retainWhere work.
Set set = newSetFrom([1, 2, 3]);
set.removeWhere((each) => each == 2);
Expect.equals(2, set.length);
@@ -236,8 +222,7 @@ testSet(Set newSet(), Set newSetFrom(Iterable from)) {
Expect.isTrue(set.contains(3));
}
- {
- // Test lookup
+ { // Test lookup
Set set = newSet();
var m1a = new Mutable(1);
var m1b = new Mutable(1);
@@ -255,7 +240,7 @@ testSet(Set newSet(), Set newSetFrom(Iterable from)) {
Expect.identical(m2a, set.lookup(m2a));
Expect.identical(m2a, set.lookup(m2b));
- set.add(m2b); // Adding doesn't change element.
+ set.add(m2b); // Adding doesn't change element.
Expect.identical(m2a, set.lookup(m2a));
Expect.identical(m2a, set.lookup(m2b));
@@ -270,8 +255,7 @@ testSet(Set newSet(), Set newSetFrom(Iterable from)) {
Expect.identical(-0.0, set.lookup(0.0));
}
- {
- // Test special hash codes
+ { // Test special hash codes
Set set = newSet();
List keys = [];
// Powers of two
@@ -287,23 +271,22 @@ testSet(Set newSet(), Set newSetFrom(Iterable from)) {
}
}
+
void testIdentitySet(Set create()) {
Set set = create();
set.add(1);
set.add(2);
- set.add(1); // Integers are identical if equal.
+ set.add(1); // Integers are identical if equal.
Expect.equals(2, set.length);
var complex = 4;
- complex = set.length == 2 ? complex ~/ 4 : 87; // Avoid compile-time constant.
- Expect.isTrue(set.contains(complex)); // 1 is in set, even if computed.
+ complex = set.length == 2 ? complex ~/ 4 : 87; // Avoid compile-time constant.
+ Expect.isTrue(set.contains(complex)); // 1 is in set, even if computed.
set.clear();
// All compile time constants are identical to themselves.
- var constants = [
- double.INFINITY,
+ var constants = [double.INFINITY,
double.NAN, -0.0, //# 01: ok
- 0.0, 42, "", null, false, true, #bif, testIdentitySet
- ];
+ 0.0, 42, "", null, false, true, #bif, testIdentitySet];
set.addAll(constants);
Expect.equals(constants.length, set.length);
for (var c in constants) {
@@ -315,7 +298,7 @@ void testIdentitySet(Set create()) {
var m1 = new Mutable(1);
var m2 = new Mutable(2);
var m3 = new Mutable(3);
- var m4 = new Mutable(2); // Equal to m2, but not identical.
+ var m4 = new Mutable(2); // Equal to m2, but not identical.
set.addAll([m1, m2, m3, m4]);
Expect.equals(4, set.length);
Expect.equals(3, m3.hashCode);
@@ -332,6 +315,7 @@ void testIdentitySet(Set create()) {
Expect.identical(null, set.lookup(m3));
}
+
void main() {
testSet(() => new Set(), (m) => new Set.from(m));
testSet(() => new HashSet(), (m) => new HashSet.from(m));
@@ -339,12 +323,14 @@ void main() {
testIdentitySet(() => new Set.identity());
testIdentitySet(() => new HashSet.identity());
testIdentitySet(() => new LinkedHashSet.identity());
- testIdentitySet(() => new HashSet(
- equals: (x, y) => identical(x, y), hashCode: (x) => identityHashCode(x)));
- testIdentitySet(() => new LinkedHashSet(
- equals: (x, y) => identical(x, y), hashCode: (x) => identityHashCode(x)));
+ testIdentitySet(() => new HashSet(equals: (x, y) => identical(x, y),
+ hashCode: (x) => identityHashCode(x)));
+ testIdentitySet(
+ () => new LinkedHashSet(equals: (x, y) => identical(x, y),
+ hashCode: (x) => identityHashCode(x)));
}
+
class BadHashCode {
static int idCounter = 0;
final int id;
@@ -359,5 +345,5 @@ class Mutable {
int id;
Mutable(this.id);
int get hashCode => id;
- bool operator ==(other) => other is Mutable && id == other.id;
+ bool operator==(other) => other is Mutable && id == other.id;
}
« no previous file with comments | « tests/corelib_strong/double_parse_test.dart ('k') | tests/corelib_strong/int_modulo_arith_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698