Better way of fixing attribute_changed_callback_test, doesn't require a timer

BUG=

Review URL: https://codereview.chromium.org//1061403002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@44946 260f80e4-7a28-3924-810f-c04153c831b5
This commit is contained in:
alanknight@google.com 2015-04-07 20:52:54 +00:00
parent 8c4caa642b
commit 4b119a94d8

View file

@ -32,8 +32,14 @@ class B extends HtmlElement {
static var invocations = [];
Completer completer;
void attributeChanged(name, oldValue, newValue) {
invocations.add('$name: $oldValue => $newValue');
if (completer != null) {
completer.complete('value changed to $newValue');
completer = null;
}
}
}
@ -90,16 +96,19 @@ main() {
B.invocations = [];
var b = new B();
var completer = new Completer();
b.completer = completer;
b.id = 'x';
return new Future.delayed(new Duration(milliseconds: 1))
return completer.future
.then((_) => expect(B.invocations, ['created', 'id: null => x']))
.then((_) {
B.invocations = [];
b.attributes.remove('id');
})
.then((_) => new Future.delayed(new Duration(milliseconds: 1)))
.then((_) => expect(B.invocations, ['id: x => null']));
B.invocations = [];
var secondCompleter = new Completer();
b.completer = secondCompleter;
b.attributes.remove('id');
return secondCompleter.future;
})
.then((_) => expect(B.invocations, ['id: x => null']));
});
});