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

Unified Diff: tests/corelib/iterable_min_max_test.dart

Issue 13685004: Reduce usage of ItearbleMixinWorkaround. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Fix bug in set-base Created 7 years, 8 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 | « sdk/lib/collection/queue.dart ('k') | tools/dom/src/WrappedList.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/corelib/iterable_min_max_test.dart
diff --git a/tests/corelib/iterable_min_max_test.dart b/tests/corelib/iterable_min_max_test.dart
index 1a8c4fd3aeb1e9edace0a9d95145cd0dc7d44e65..97b4a49c777639040fcbd17f8fcc662ec05df663 100644
--- a/tests/corelib/iterable_min_max_test.dart
+++ b/tests/corelib/iterable_min_max_test.dart
@@ -26,14 +26,10 @@ int compareC(C a, C b) => a.x.compareTo(b.x);
testMinMax(iterable, min, max) {
Expect.equals(min, iterable.min());
Expect.equals(min, iterable.min(Comparable.compare));
- Expect.equals(min, IterableMixinWorkaround.min(iterable));
- Expect.equals(min, IterableMixinWorkaround.min(iterable, Comparable.compare));
Expect.equals(max, iterable.min((a, b) => Comparable.compare(b, a)));
Expect.equals(max, iterable.max());
Expect.equals(max, iterable.max(Comparable.compare));
- Expect.equals(max, IterableMixinWorkaround.max(iterable));
- Expect.equals(max, IterableMixinWorkaround.max(iterable, Comparable.compare));
Expect.equals(min, iterable.max((a, b) => Comparable.compare(b, a)));
}
@@ -57,18 +53,12 @@ main() {
// Objects that are not Comparable.
Expect.equals(const C(0), cList.min(compareC));
- Expect.equals(const C(0), IterableMixinWorkaround.min(cList, compareC));
Expect.equals(const C(0), new List.from(cList).min(compareC));
- Expect.equals(const C(0), IterableMixinWorkaround.min(new List.from(cList), compareC));
Expect.equals(const C(0), new Set.from(cList).min(compareC));
- Expect.equals(const C(0), IterableMixinWorkaround.min(new Set.from(cList), compareC));
Expect.equals(const C(10), cList.max(compareC));
- Expect.equals(const C(10), IterableMixinWorkaround.max(cList, compareC));
Expect.equals(const C(10), new List.from(cList).max(compareC));
- Expect.equals(const C(10), IterableMixinWorkaround.max(new List.from(cList), compareC));
Expect.equals(const C(10), new Set.from(cList).max(compareC));
- Expect.equals(const C(10), IterableMixinWorkaround.max(new Set.from(cList), compareC));
bool checkedMode = false;
assert(checkedMode = true);
@@ -77,4 +67,3 @@ main() {
Expect.throws(cList.max, (e) => checkedMode ? e is TypeError
: e is NoSuchMethodError);
}
-
« no previous file with comments | « sdk/lib/collection/queue.dart ('k') | tools/dom/src/WrappedList.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698