Fix bug in property lookup. Add test.

Author: kevinpschaaf

src/micro/properties.html

@@ -65,7 +65,7 @@
       var info = this._getPropertyInfo(property, this.properties);
       if (!info) {
         this.behaviors.some(function(b) {
-          var info = this._getPropertyInfo(property, b.properties);
+          return info = this._getPropertyInfo(property, b.properties);
         }, this);
       }
       return info || Polymer.nob;

test/unit/behaviors-elements.html

@@ -17,6 +17,11 @@
       label: {
         type: String,
         observer: '_labelChanged'
+      },
+
+      hasOptionsA: {
+        readOnly: true,
+        notify: true
       }
 
     },
@@ -37,7 +42,7 @@
       this.__change = e.detail.value;
     }
 
-  }
+  };
 
 </script>
 
@@ -51,6 +56,11 @@
         type: Boolean,
         value: false,
         observer: '_disabledChanged'
+      },
+
+      hasOptionsB: {
+        readOnly: true,
+        notify: true
       }
 
     },

test/unit/behaviors.html

@@ -46,6 +46,12 @@
     assert.equal(el.__change, 'bar');
   });
 
+  test('property info from behavior A', function() {
+    assert.equal(el.getPropertyInfo('hasOptionsA').notify, true);
+    assert.equal(el.getPropertyInfo('hasOptionsA').readOnly, true);
+    assert.equal(typeof el._setHasOptionsA, 'function');
+  });
+
 });
 
 suite('multi-behaviors element', function() {
@@ -85,6 +91,18 @@
     assert.equal(el.__change, 'bar');
   });
 
+  test('property info from behavior A', function() {
+    assert.equal(el.getPropertyInfo('hasOptionsA').notify, true);
+    assert.equal(el.getPropertyInfo('hasOptionsA').readOnly, true);
+    assert.equal(typeof el._setHasOptionsA, 'function');
+  });
+
+  test('property info from behavior B', function() {
+    assert.equal(el.getPropertyInfo('hasOptionsB').readOnly, true);
+    assert.equal(el.getPropertyInfo('hasOptionsB').notify, true);
+    assert.equal(typeof el._setHasOptionsB, 'function');
+  });
+
 });
 
 </script>