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

Unified Diff: client/tests/client/observable/AbstractObservableTests.dart

Issue 8297009: Get rid of "==" in Expectation. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 9 years, 2 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 | « client/testing/unittest/unittestsuite.dart ('k') | client/tests/client/observable/ChangeEventTests.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: client/tests/client/observable/AbstractObservableTests.dart
diff --git a/client/tests/client/observable/AbstractObservableTests.dart b/client/tests/client/observable/AbstractObservableTests.dart
index 54d50180d101cd9df2f02daa404e5c24f92cf982..83a24dd377c167154108ef494e35b0a4bafdcd95 100644
--- a/client/tests/client/observable/AbstractObservableTests.dart
+++ b/client/tests/client/observable/AbstractObservableTests.dart
@@ -15,8 +15,8 @@ class AbstractObservableTests extends ObservableTestSetBase {
final target = new AbstractObservable();
final listener = (e) { };
- expect(target.addChangeListener(listener)) == true;
- expect(target.addChangeListener(listener)) == false;
+ expect(target.addChangeListener(listener)).isTrue();
+ expect(target.addChangeListener(listener)).isFalse();
});
});
}
@@ -60,24 +60,24 @@ class AbstractObservableTests extends ObservableTestSetBase {
final target = new AbstractObservable();
EventSummary res = null;
target.addChangeListener((summary) {
- expect(res) == null;
+ expect(res).isNull();
res = summary;
expect(res).isNotNull();
});
- target.recordPropertyUpdate("pM", 10, 11);
+ target.recordPropertyUpdate('pM', 10, 11);
expect(res).isNotNull();
- expect(res.events.length) == 1;
+ expect(res.events.length).equals(1);
checkEvent(res.events[0],
- target, "pM", null, ChangeEvent.UPDATE, 10, 11);
+ target, 'pM', null, ChangeEvent.UPDATE, 10, 11);
res = null;
- target.recordPropertyUpdate("pL", "11", "13");
+ target.recordPropertyUpdate('pL', '11', '13');
expect(res).isNotNull();
- expect(res.events.length) == 1;
+ expect(res.events.length).equals(1);
checkEvent(res.events[0],
- target, "pL", null, ChangeEvent.UPDATE, "11", "13");
+ target, 'pL', null, ChangeEvent.UPDATE, '11', '13');
}
}
« no previous file with comments | « client/testing/unittest/unittestsuite.dart ('k') | client/tests/client/observable/ChangeEventTests.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698