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

Unified Diff: tests/corelib/list_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/iterable_to_list_test.dart ('k') | tests/corelib/num_parse_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/corelib/list_test.dart
diff --git a/tests/corelib/list_test.dart b/tests/corelib/list_test.dart
index bba0e8775ae737ddf41ad5e8d6933211ee9cf6ef..8d6df271327737e0179e6c585493e7b2e350b528 100644
--- a/tests/corelib/list_test.dart
+++ b/tests/corelib/list_test.dart
@@ -63,7 +63,6 @@ void testErrors() {
Expect.equals(0, err.start, "$name[$index] start");
}
}
-
testIndex(list, name) {
testIndexError(list, list.length, name); // Just too big.
testIndexError(list, -1, name); // Negative.
@@ -88,7 +87,7 @@ void testErrors() {
Expect.isTrue(actualError is RangeError, "$name is-error: $actualError");
Expect.equals(realError.name, actualError.name, "$name name");
Expect.equals(realError.invalidValue, actualError.invalidValue,
- "$name[0:l+1] value");
+ "$name[0:l+1] value");
Expect.equals(realError.start, actualError.start, "$name[0:l+1] start");
Expect.equals(realError.end, actualError.end, "$name[0:l+1] end");
return;
@@ -99,7 +98,7 @@ void testErrors() {
}
testSlice(list, name) {
- testSliceError(list, 0, list.length, name); // Should not fail.
+ testSliceError(list, 0, list.length, name); // Should not fail.
testSliceError(list, 0, list.length + 1, name);
testSliceError(list, 0, 0x123456789, name);
testSliceError(list, -1, list.length, name);
@@ -115,7 +114,6 @@ void testErrors() {
testIndex(list, "$name#${list.length} index");
testSlice(list, "$name#${list.length} slice");
}
-
// Empty lists.
testRangeErrors([], "list");
testRangeErrors(new List(0), "fixed-list");
@@ -202,7 +200,7 @@ void testTypedLengthInvariantOperations(List list) {
list.sort();
Expect.listEquals([0, 1, 2, 3], list);
list.setRange(0, 4, [1, 3, 0, 2]);
- list.sort((a, b) => b - a); // reverse compare.
+ list.sort((a, b) => b - a); // reverse compare.
Expect.listEquals([3, 2, 1, 0], list);
list.setRange(0, 4, [1, 2, 3, 0]);
list.sort((a, b) => b - a);
@@ -212,10 +210,7 @@ void testTypedLengthInvariantOperations(List list) {
list.setRange(0, 4, [0, 1, 2, 3]);
// map.
- testMap(val) {
- return val * 2 + 10;
- }
-
+ testMap(val) {return val * 2 + 10; }
List mapped = list.map(testMap).toList();
Expect.equals(mapped.length, list.length);
for (var i = 0; i < list.length; i++) {
@@ -223,18 +218,9 @@ void testTypedLengthInvariantOperations(List list) {
}
matchAll(val) => true;
- matchSome(val) {
- return (val == 1 || val == 2);
- }
-
- matchSomeFirst(val) {
- return val == 0;
- }
-
- matchSomeLast(val) {
- return val == 3;
- }
-
+ matchSome(val) { return (val == 1 || val == 2); }
+ matchSomeFirst(val) { return val == 0; }
+ matchSomeLast(val) { return val == 3; }
matchNone(val) => false;
// where.
@@ -325,7 +311,6 @@ void testCannotChangeLength(List list) {
isUnsupported(action()) {
Expect.throws(action, (e) => e is UnsupportedError);
}
-
isUnsupported(() => list.add(0));
isUnsupported(() => list.addAll([0]));
isUnsupported(() => list.removeLast());
@@ -486,7 +471,6 @@ void testGrowableListOperations(List list) {
}
}, (e) => e is ConcurrentModificationError);
}
-
testForEach(int when) {
list.length = 4;
list.setAll(0, [0, 1, 2, 3]);
@@ -496,7 +480,6 @@ void testGrowableListOperations(List list) {
});
}, (e) => e is ConcurrentModificationError);
}
-
// Test the change at different points of the iteration.
testIterator(0);
testIterator(1);
@@ -583,50 +566,26 @@ class MyList<E> extends ListBase<E> {
List<E> _source;
MyList(this._source);
int get length => _source.length;
- void set length(int length) {
- _source.length = length;
- }
-
- E operator [](int index) => _source[index];
- void operator []=(int index, E value) {
- _source[index] = value;
- }
+ void set length(int length) { _source.length = length; }
+ E operator[](int index) => _source[index];
+ void operator[]=(int index, E value) { _source[index] = value; }
}
class MyFixedList<E> extends ListBase<E> {
List<E> _source;
MyFixedList(this._source);
int get length => _source.length;
- void set length(int length) {
- throw new UnsupportedError("Fixed length!");
- }
-
- E operator [](int index) => _source[index];
- void operator []=(int index, E value) {
- _source[index] = value;
- }
+ void set length(int length) { throw new UnsupportedError("Fixed length!"); }
+ E operator[](int index) => _source[index];
+ void operator[]=(int index, E value) { _source[index] = value; }
}
void testListConstructor() {
- // Is fixed-length.
- Expect.throws(() {
- new List(0).add(4);
+ Expect.throws(() { new List(0).add(4); }); // Is fixed-length.
Expect.throws(() { new List(-2); }); // Not negative. //# 01: ok
- // Not null.
- Expect.throws(() {
- new List(null);
- });
+ Expect.throws(() { new List(null); }); // Not null.
Expect.listEquals([4], new List()..add(4));
- // Is fixed-length.
- Expect.throws(() {
- new List.filled(0, 42).add(4);
- });
- // Not negative.
- Expect.throws(() {
- new List.filled(-2, 42);
- });
- // Not null.
- Expect.throws(() {
- new List.filled(null, 42);
- });
+ Expect.throws(() { new List.filled(0, 42).add(4); }); // Is fixed-length.
+ Expect.throws(() { new List.filled(-2, 42); }); // Not negative.
+ Expect.throws(() { new List.filled(null, 42); }); // Not null.
}
« no previous file with comments | « tests/corelib/iterable_to_list_test.dart ('k') | tests/corelib/num_parse_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698