big update to observe, template_binding, polymer

pkg/polymer/test/property_observe_test.dart

Index: pkg/polymer/test/property_observe_test.dart
diff --git a/pkg/polymer/test/property_observe_test.dart b/pkg/polymer/test/property_observe_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..3ee21eac2a8f8ebbfd537301c1ac59cc34c69b1e
--- /dev/null
+++ b/pkg/polymer/test/property_observe_test.dart
@@ -0,0 +1,78 @@
+// Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+
+library polymer.test.property_change_test;
+
+import 'dart:async';
+import 'dart:html';
+import 'package:polymer/polymer.dart';
+import 'package:unittest/unittest.dart';
+import 'package:unittest/html_config.dart';
+import 'package:unittest/matcher.dart';
+
+var _changes = 0;
+final _done = new Completer();
+
+checkDone() {
+  if (4 == ++_changes) {
+    _done.complete();
+  }
+}
+
+@CustomTag('x-test')
+class XTest extends PolymerElement {
+  @observable String bar = '';
+  @observable String pie;
+  @observable Map a;
+
+  XTest.created() : super.created();
+
+  ready() {
+    bar = 'bar';
+    pie = 'pie';
+    a = {'b': {'c': 'exists'}};
+  }
+
+  barChanged() {
+    _done.completeError('barChanged should not be called.');

[Siggi Cherem (dart-lang), 2014/02/03 22:52:48]

seems a bit strange that this is not called. Is that because there is an
@ObserveProperty('bar') below? I assumed it was OK to use the same property in
multiple places.

[Jennifer Messerly, 2014/02/04 00:33:06]

it isn't in Polymer.js. It is now supported in Dart (for better or worse).

Updated test.

+  }
+
+  @ObserveProperty('bar pie')
+  validate() {
+    window.console.log('validate');
+    expect('bar', 'bar', reason: 'custom change observer called');
+    expect('pie', 'pie', reason: 'custom change observer called');
+    checkDone();
+  }
+
+  @ObserveProperty('a.b.c')
+  validateSubPath(oldValue, newValue) {
+    window.console.log('validateSubPath $oldValue $newValue');
+    expect(newValue, 'exists', reason: 'subpath change observer called');
+    checkDone();
+  }
+}
+
+@CustomTag('x-test2')
+class XTest2 extends XTest {
+  @observable String noogle;
+
+  XTest2.created() : super.created();
+
+  @ObserveProperty('noogle')
+  validate() => super.validate();
+
+  ready() {
+    super.ready();
+    noogle = 'noogle';
+  }
+}
+
+main() => initPolymer().run(() {
+  useHtmlConfiguration();
+
+  setUp(() => Polymer.onReady);
+
+  test('changes detected', () => _done.future);
+});