Unbreak .info/serverTimestampOffset (#2061)

Author: schmidt-sebastian

packages/database/CHANGELOG.md

@@ -1,4 +1,9 @@
 # Unreleased
+- [fixed] Fixed an issue that caused `.info/serverTimeOffset` events not to fire (#2043).
+
+# 6.3.4
 - [fixed] Fixed an issue where multi-byte UTF-8 characters would not be written correctly when using `firebase.js` or `firebase-database.js` (#2035).
+
+# 6.0.0
 - [changed] Improved consistency between the type annotations for `Query.on`/`Reference.on`, 
   `Query.off`/`Reference.off` and `Query.once`/`Reference.once` (#1188, #1204).

packages/database/src/core/Repo.ts

@@ -274,7 +274,7 @@ export class Repo {
   }
 
   private onServerInfoUpdate_(updates: Object) {
-    each(updates, (value: any, key: string) => {
+    each(updates, (key: string, value: any) => {
       this.updateInfo_(key, value);
     });
   }

packages/database/test/info.test.ts

@@ -105,9 +105,7 @@ describe('.info Tests', function() {
     return ea.promise;
   });
 
-  // Skipping this test as it is expecting a server time diff from a
-  // local Firebase
-  it.skip('.info/serverTimeOffset', async function() {
+  it('.info/serverTimeOffset', async function() {
     const ref = getRootNode() as Reference;
 
     // make sure push works