Merge pull request #28 from GoogleWebComponents/fix-firebase-document-update

Ensure that the document is mutable.

Author: addyosmani

firebase-document.html

@@ -56,8 +56,7 @@
        */
       data: {
         type: Object,
-        notify: true,
-        readOnly: true
+        notify: true
       }
     },
 
@@ -81,7 +80,7 @@
 
     _onFirebaseValue: function(event) {
       this._applyRemoteDataChange(function() {
-        this._setData(event.detail.val());
+        this.set('data', event.detail.val());
       });
     },
 

test/firebase-document.html

@@ -31,6 +31,14 @@
       </firebase-document>
     </template>
   </test-fixture>
+  <test-fixture id="UpdateableDocument">
+    <template>
+      <firebase-document
+        location="https://fb-element-demo.firebaseio.com/test/updateable_document"
+        log>
+      </firebase-document>
+    </template>
+  </test-fixture>
   <test-fixture id="MalleableDocument">
     <template>
       <firebase-document
@@ -69,6 +77,33 @@
         });
       });
 
+      suite('document updating', function() {
+        setup(function(done) {
+          firebase = fixture('UpdateableDocument');
+          waitForEvent(firebase, 'firebase-value').then(function() {
+            done();
+          });
+        });
+
+        test('setting data property updates the document', function(done) {
+          var data = {};
+          var newValue = Math.random().toString().split('.').pop();
+
+          data[newValue] = newValue;
+
+          waitForEvent(firebase, 'firebase-value').then(function() {
+            expect(firebase.data[newValue]).to.be.eql(newValue);
+            done();
+          }).catch(function(e) {
+            done(e);
+          }).then(function() {
+            firebase.set('data.' + newValue, null);
+          });
+
+          firebase.set('data', data);
+        });
+      });
+
       suite('document manipulation', function() {
         var localFirebase;
         var remoteFirebase;