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

Unified Diff: client/tests/client/observable/EventBatchTests.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
Index: client/tests/client/observable/EventBatchTests.dart
diff --git a/client/tests/client/observable/EventBatchTests.dart b/client/tests/client/observable/EventBatchTests.dart
index 8f627faee7b903d538663af7eafd1ac429f655a7..515f72cc71f7c18556ab0fe0e8bb285facd5bf94 100644
--- a/client/tests/client/observable/EventBatchTests.dart
+++ b/client/tests/client/observable/EventBatchTests.dart
@@ -16,39 +16,39 @@ class EventBatchTests extends ObservableTestSetBase {
final target = new AbstractObservable();
EventSummary res = null;
target.addChangeListener((summary) {
- expect(res) == null;
+ expect(res).isNull();
res = summary;
expect(res).isNotNull();
});
final f = EventBatch.wrap((e) {
- target.recordPropertyUpdate("pM", 10, 11);
- target.recordPropertyUpdate("pL", "11", "13");
- target.recordListUpdate(2, "a", "b");
- target.recordListInsert(5, "a");
- target.recordListRemove(4, "c");
+ target.recordPropertyUpdate('pM', 10, 11);
+ target.recordPropertyUpdate('pL', '11', '13');
+ target.recordListUpdate(2, 'a', 'b');
+ target.recordListInsert(5, 'a');
+ target.recordListRemove(4, 'c');
target.recordGlobalChange();
});
- expect(res) == null;
+ expect(res).isNull();
f(null);
expect(res).isNotNull();
- expect(res.events.length) == 6;
+ expect(res.events.length).equals(6);
checkEvent(res.events[0],
- target, "pM", null, ChangeEvent.UPDATE, 10, 11);
+ target, 'pM', null, ChangeEvent.UPDATE, 10, 11);
checkEvent(res.events[1],
- target, "pL", null, ChangeEvent.UPDATE, "11", "13");
+ target, 'pL', null, ChangeEvent.UPDATE, '11', '13');
checkEvent(res.events[2],
- target, null, 2, ChangeEvent.UPDATE, "a", "b");
+ target, null, 2, ChangeEvent.UPDATE, 'a', 'b');
checkEvent(res.events[3],
- target, null, 5, ChangeEvent.INSERT, "a", null);
+ target, null, 5, ChangeEvent.INSERT, 'a', null);
checkEvent(res.events[4],
- target, null, 4, ChangeEvent.REMOVE, null, "c");
+ target, null, 4, ChangeEvent.REMOVE, null, 'c');
checkEvent(res.events[5],
target, null, null, ChangeEvent.GLOBAL, null, null);
« no previous file with comments | « client/tests/client/observable/ChangeEventTests.dart ('k') | client/tests/client/observable/ObservableListTests.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698