package:observe fix various api issues

pkg/mdv/test/custom_element_bindings_test.dart

 // 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 custom_element_bindings_test;
 
 import 'dart:html';
 import 'package:mdv/mdv.dart' as mdv;
+import 'package:observe/observe.dart' show toObservable;
 import 'package:unittest/html_config.dart';
 import 'package:unittest/unittest.dart';
 import 'mdv_test_utils.dart';
 
 main() {
   mdv.initialize();
   useHtmlConfiguration();
   group('Custom Element Bindings', customElementBindingsTest);
 }
 
@@ skipping 17 matching lines @@
     for (var node in div.queryAll('*')) {
       if (node.isTemplate) TemplateElement.decorate(node);
     }
 
     return div;
   }
 
 
   observeTest('override bind/unbind/unbindAll', () {
     var element = new MyCustomElement();
-    var model = toSymbolMap({'a': new Point(123, 444), 'b': new Monster(100)});
+    var model = toObservable({'a': new Point(123, 444), 'b': new Monster(100)});
 
     element.bind('my-point', model, 'a');
     element.bind('scary-monster', model, 'b');
 
     expect(element.attributes, isNot(contains('my-point')));
     expect(element.attributes, isNot(contains('scary-monster')));
 
-    expect(element.myPoint, model[#a]);
-    expect(element.scaryMonster, model[#b]);
+    expect(element.myPoint, model['a']);
+    expect(element.scaryMonster, model['b']);
 
-    model[#a] = null;
+    model['a'] = null;
     performMicrotaskCheckpoint();
     expect(element.myPoint, null);
     element.unbind('my-point');
 
-    model[#a] = new Point(1, 2);
-    model[#b] = new Monster(200);
+    model['a'] = new Point(1, 2);
+    model['b'] = new Monster(200);
     performMicrotaskCheckpoint();
-    expect(element.scaryMonster, model[#b]);
+    expect(element.scaryMonster, model['b']);
     expect(element.myPoint, null, reason: 'a was unbound');
 
     element.unbindAll();
-    model[#b] = null;
+    model['b'] = null;
     performMicrotaskCheckpoint();
     expect(element.scaryMonster.health, 200);
   });
 
   observeTest('override attribute setter', () {
     var element = new WithAttrsCustomElement().real;
-    var model = toSymbolMap({'a': 1, 'b': 2});
+    var model = toObservable({'a': 1, 'b': 2});
     element.bind('hidden?', model, 'a');
     element.bind('id', model, 'b');
 
     expect(element.attributes, contains('hidden'));
     expect(element.attributes['hidden'], '');
     expect(element.id, '2');
 
-    model[#a] = null;
+    model['a'] = null;
     performMicrotaskCheckpoint();
     expect(element.attributes, isNot(contains('hidden')),
         reason: 'null is false-y');
 
-    model[#a] = false;
+    model['a'] = false;
     performMicrotaskCheckpoint();
     expect(element.attributes, isNot(contains('hidden')));
 
-    model[#a] = 'foo';
+    model['a'] = 'foo';
     // TODO(jmesserly): this is here to force an ordering between the two
     // changes. Otherwise the order depends on what order StreamController
     // chooses to fire the two listeners in.
     performMicrotaskCheckpoint();
 
-    model[#b] = 'x';
+    model['b'] = 'x';
     performMicrotaskCheckpoint();
     expect(element.attributes, contains('hidden'));
     expect(element.attributes['hidden'], '');
     expect(element.id, 'x');
 
     expect(element.xtag.attributes.log, [
       ['remove', 'hidden?'],
       ['[]=', 'hidden', ''],
       ['[]=', 'id', '2'],
       ['remove', 'hidden'],
       ['remove', 'hidden'],
       ['[]=', 'hidden', ''],
       ['[]=', 'id', 'x'],
     ]);
   });
 
   observeTest('template bind uses overridden custom element bind', () {
 
-    var model = toSymbolMap({'a': new Point(123, 444), 'b': new Monster(100)});
+    var model = toObservable({'a': new Point(123, 444), 'b': new Monster(100)});
 
     var div = createTestHtml('<template bind>'
           '<my-custom-element my-point="{{a}}" scary-monster="{{b}}">'
           '</my-custom-element>'
         '</template>');
 
     callback(fragment) {
       for (var e in fragment.queryAll('my-custom-element')) {
         new MyCustomElement.attach(e);
       }
     }
     mdv.instanceCreated.add(callback);
 
     div.query('template').model = model;
     performMicrotaskCheckpoint();
 
     var element = div.nodes[1];
 
     expect(element.xtag is MyCustomElement, true,
         reason: '${element.xtag} should be a MyCustomElement');
 
-    expect(element.xtag.myPoint, model[#a]);
-    expect(element.xtag.scaryMonster, model[#b]);
+    expect(element.xtag.myPoint, model['a']);
+    expect(element.xtag.scaryMonster, model['b']);
 
     expect(element.attributes, isNot(contains('my-point')));
     expect(element.attributes, isNot(contains('scary-monster')));
 
-    model[#a] = null;
+    model['a'] = null;
     performMicrotaskCheckpoint();
     expect(element.xtag.myPoint, null);
 
     div.query('template').model = null;
     performMicrotaskCheckpoint();
 
     expect(element.parentNode, null, reason: 'element was detached');
 
-    model[#a] = new Point(1, 2);
-    model[#b] = new Monster(200);
+    model['a'] = new Point(1, 2);
+    model['b'] = new Monster(200);
     performMicrotaskCheckpoint();
 
     expect(element.xtag.myPoint, null, reason: 'model was unbound');
     expect(element.xtag.scaryMonster.health, 100, reason: 'model was unbound');
 
     mdv.instanceCreated.remove(callback);
   });
 
 }
 
@@ skipping 104 matching lines @@
   bool get isEmpty => _map.isEmpty;
   bool get isNotEmpty => _map.isNotEmpty;
 }
 
 /**
  * Sanitizer which does nothing.
  */
 class NullTreeSanitizer implements NodeTreeSanitizer {
   void sanitizeTree(Node node) {}
 }