Firebase Stub: leading slash in reference path is ignored

__mocks__/__tests__/gatsby-plugin-firebase.test.js

@@ -74,6 +74,7 @@ describe('FirebaseStub', () => {
       expect(database1.uuid).toEqual(database2.uuid);
     });
 
+    describe('ref function', () => {
       it('reuses existing Reference instance', () => {
         const ref1 = FirebaseStub.database().ref(
           `${DatabaseConstants.resumesPath}/123`,
@@ -82,9 +83,22 @@ describe('FirebaseStub', () => {
           `${DatabaseConstants.resumesPath}/123`,
         );
 
-      expect(ref1.uuid).toBeTruthy();
-      expect(ref2.uuid).toBeTruthy();
-      expect(ref1.uuid).toEqual(ref2.uuid);
+        expect(ref1).toBeTruthy();
+        expect(ref2).toBeTruthy();
+        expect(ref1).toEqual(ref2);
+      });
+
+      it('leading slash in reference path is ignored', () => {
+        const path = `${DatabaseConstants.resumesPath}/123`;
+
+        const ref1 = FirebaseStub.database().ref(path);
+        expect(ref1).toBeTruthy();
+        expect(ref1.path).toEqual(path);
+
+        const ref2 = FirebaseStub.database().ref(`/${path}`);
+        expect(ref2).toBeTruthy();
+        expect(ref2).toEqual(ref1);
+      });
     });
 
     it('ServerValue.TIMESTAMP returns current time in milliseconds', () => {

__mocks__/gatsby-plugin-firebase/constants/database.js

@@ -5,7 +5,7 @@ const childRemovedEventType = 'child_removed';
 
 const resumesPath = 'resumes';
 const usersPath = 'users';
-const connectedPath = '/.info/connected';
+const connectedPath = '.info/connected';
 
 const demoStateResume1Id = 'demo_1';
 const demoStateResume2Id = 'demo_2';

__mocks__/gatsby-plugin-firebase/database/database.js

@@ -135,18 +135,19 @@ class Database {
   }
 
   ref(referencePath) {
-    const existingRef = this._getReference(referencePath);
-    if (existingRef) {
-      existingRef.initializeQueryParameters();
-      return existingRef;
-    }
-
     const newRef = new Reference(
       referencePath,
       (dataPath) => this._getData(dataPath),
       (dataPath, value) => this._setData(dataPath, value),
       (refPath) => this._getReference(refPath),
     );
+
+    const existingRef = this._getReference(newRef.path);
+    if (existingRef) {
+      existingRef.initializeQueryParameters();
+      return existingRef;
+    }
+
     this._references[newRef.path] = newRef;
     return newRef;
   }

__mocks__/gatsby-plugin-firebase/database/reference.js

@@ -5,16 +5,26 @@ import { debounce } from 'lodash';
 import DatabaseConstants from '../constants/database';
 import DataSnapshot from './dataSnapshot';
 
-class Reference {
-  constructor(path, getDatabaseData, setDatabaseData, getReference) {
+const parsePath = (path) => {
   if (!path) {
     throw new Error('path must be provided.');
   } else if (typeof path !== 'string') {
     throw new Error('path should be a string.');
   } else {
-      this._path = path;
+    let parsedPath = path.trim();
+
+    if (parsedPath[0] === '/') {
+      parsedPath = parsedPath.substring(1);
     }
 
+    return parsedPath;
+  }
+};
+
+class Reference {
+  constructor(path, getDatabaseData, setDatabaseData, getReference) {
+    this._path = parsePath(path);
+
     this._uuid = uuidv4();
 
     this._dataSnapshot = null;