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

Unified Diff: packages/collection/test/comparators_test.dart

Issue 2989763002: Update charted to 0.4.8 and roll (Closed)
Patch Set: Removed Cutch from list of reviewers Created 3 years, 5 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: packages/collection/test/comparators_test.dart
diff --git a/packages/collection/test/comparators_test.dart b/packages/collection/test/comparators_test.dart
index 4acdc2c18716c05f9ecd5f0a972e8b94a58cf4a8..aa868d95740e4242fe15ab2ad983c1a1e9cc7cd9 100644
--- a/packages/collection/test/comparators_test.dart
+++ b/packages/collection/test/comparators_test.dart
@@ -57,63 +57,63 @@ void main() {
"~"
];
- sortedBy(compare) => strings.toList()..shuffle()..sort(compare);
+ sortedBy(compare) => strings.toList()
+ ..shuffle()
+ ..sort(compare);
test("String.compareTo", () {
expect(sortedBy(null), strings);
});
test("compareAsciiLowerCase", () {
- expect(sortedBy(compareAsciiLowerCase),
- sortedBy((a, b) {
- int delta = a.toLowerCase().compareTo(b.toLowerCase());
- if (delta != 0) return delta;
- if (a == b) return 0;
- return a.compareTo(b);
- }));
+ expect(sortedBy(compareAsciiLowerCase), sortedBy((a, b) {
+ int delta = a.toLowerCase().compareTo(b.toLowerCase());
+ if (delta != 0) return delta;
+ if (a == b) return 0;
+ return a.compareTo(b);
+ }));
});
test("compareAsciiUpperCase", () {
- expect(sortedBy(compareAsciiUpperCase),
- sortedBy((a, b) {
- int delta = a.toUpperCase().compareTo(b.toUpperCase());
- if (delta != 0) return delta;
- if (a == b) return 0;
- return a.compareTo(b);
- }));
+ expect(sortedBy(compareAsciiUpperCase), sortedBy((a, b) {
+ int delta = a.toUpperCase().compareTo(b.toUpperCase());
+ if (delta != 0) return delta;
+ if (a == b) return 0;
+ return a.compareTo(b);
+ }));
});
// Replace any digit sequence by ("0", value, length) as char codes.
// This will sort alphabetically (by charcode) the way digits sort
// numerically, and the leading 0 means it sorts like a digit
// compared to non-digits.
- replaceNumbers(string) => string.replaceAllMapped(new RegExp(r"\d+"), (m) {
- var digits = m[0];
- return new String.fromCharCodes([0x30, int.parse(digits), digits.length]);
- });
+ replaceNumbers(String string) =>
+ string.replaceAllMapped(new RegExp(r"\d+"), (m) {
+ var digits = m[0];
+ return new String.fromCharCodes(
+ [0x30, int.parse(digits), digits.length]);
+ });
test("compareNatural", () {
expect(sortedBy(compareNatural),
- sortedBy((a, b) => replaceNumbers(a).compareTo(replaceNumbers(b))));
+ sortedBy((a, b) => replaceNumbers(a).compareTo(replaceNumbers(b))));
});
test("compareAsciiLowerCaseNatural", () {
- expect(sortedBy(compareAsciiLowerCaseNatural),
- sortedBy((a, b) {
- int delta = replaceNumbers(a.toLowerCase()).compareTo(
- replaceNumbers(b.toLowerCase()));
- if (delta != 0) return delta;
- if (a == b) return 0;
- return a.compareTo(b);
- }));
+ expect(sortedBy(compareAsciiLowerCaseNatural), sortedBy((a, b) {
+ int delta = replaceNumbers(a.toLowerCase())
+ .compareTo(replaceNumbers(b.toLowerCase()));
+ if (delta != 0) return delta;
+ if (a == b) return 0;
+ return a.compareTo(b);
+ }));
});
test("compareAsciiUpperCaseNatural", () {
- expect(sortedBy(compareAsciiUpperCaseNatural),
- sortedBy((a, b) {
- int delta = replaceNumbers(a.toUpperCase()).compareTo(
- replaceNumbers(b.toUpperCase()));
- if (delta != 0) return delta;
- if (a == b) return 0;
- return a.compareTo(b);
- }));
+ expect(sortedBy(compareAsciiUpperCaseNatural), sortedBy((a, b) {
+ int delta = replaceNumbers(a.toUpperCase())
+ .compareTo(replaceNumbers(b.toUpperCase()));
+ if (delta != 0) return delta;
+ if (a == b) return 0;
+ return a.compareTo(b);
+ }));
});
}
« no previous file with comments | « packages/collection/test/combined_wrapper/map_test.dart ('k') | packages/collection/test/equality_map_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698