Merge 269a3a8 into 589fcb0

invertase · Apr 27, 2021 · f9b81f9 · f9b81f9
2 parents 589fcb0 + 269a3a8
commit f9b81f9
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 16 deletions.
diff --git a/packages/database/e2e/reference/update.e2e.js b/packages/database/e2e/reference/update.e2e.js
@@ -34,16 +34,6 @@ describe('database().ref().update()', function () {
     }
   });
 
-  it('throws if values does not contain any values', async function () {
-    try {
-      await firebase.database().ref(TEST_PATH).update({});
-      return Promise.reject(new Error('Did not throw an Error.'));
-    } catch (error) {
-      error.message.should.containEql("'values' must be an object containing multiple values");
-      return Promise.resolve();
-    }
-  });
-
   it('throws if update paths are not valid', async function () {
     try {
       await firebase.database().ref(TEST_PATH).update({
@@ -83,6 +73,14 @@ describe('database().ref().update()', function () {
         foo: value,
       }),
     );
+
+    await ref.update({}); // empty update should pass, but no side effects
+    const snapshot2 = await ref.once('value');
+    snapshot2.val().should.eql(
+      jet.contextify({
+        foo: value,
+      }),
+    );
   });
 
   it('callback if function is passed', async function () {

diff --git a/packages/database/lib/DatabaseReference.js b/packages/database/lib/DatabaseReference.js
@@ -112,12 +112,6 @@ export default class DatabaseReference extends DatabaseQuery {
       throw new Error("firebase.database().ref().update(*) 'values' must be an object.");
     }
 
-    if (!Object.keys(values).length) {
-      throw new Error(
-        "firebase.database().ref().update(*) 'values' must be an object containing multiple values.",
-      );
-    }
-
     const keys = Object.keys(values);
     for (let i = 0; i < keys.length; i++) {
       if (!isValidPath(keys[i])) {