FirebaseStub: refactoring

This commit is contained in:
gianantoniopini
2020-12-24 09:53:55 +01:00
parent 445d1e37d9
commit 458aab4e8d
3 changed files with 192 additions and 170 deletions

View File

@ -1,5 +1,9 @@
import FirebaseStub from '../gatsby-plugin-firebase'; import FirebaseStub from '../gatsby-plugin-firebase';
describe('FirebaseStub', () => {
const resumesPath = FirebaseStub.database().resumesPath;
const usersPath = FirebaseStub.database().usersPath;
describe('auth', () => { describe('auth', () => {
afterEach(() => { afterEach(() => {
FirebaseStub.auth().dispose(); FirebaseStub.auth().dispose();
@ -66,8 +70,8 @@ describe('database', () => {
}); });
it('reuses existing Reference instance', () => { it('reuses existing Reference instance', () => {
const ref1 = FirebaseStub.database().ref('resumes/123'); const ref1 = FirebaseStub.database().ref(`${resumesPath}/123`);
const ref2 = FirebaseStub.database().ref('resumes/123'); const ref2 = FirebaseStub.database().ref(`${resumesPath}/123`);
expect(ref1.uuid).toBeTruthy(); expect(ref1.uuid).toBeTruthy();
expect(ref2.uuid).toBeTruthy(); expect(ref2.uuid).toBeTruthy();
@ -83,7 +87,7 @@ describe('database', () => {
}); });
it("can spy on Reference 'update' function", async () => { it("can spy on Reference 'update' function", async () => {
const referencePath = 'resumes/123456'; const referencePath = `${resumesPath}/123456`;
const functionSpy = jest.spyOn( const functionSpy = jest.spyOn(
FirebaseStub.database().ref(referencePath), FirebaseStub.database().ref(referencePath),
'update', 'update',
@ -101,7 +105,7 @@ describe('database', () => {
it('initializing data sets up resumes and users', async () => { it('initializing data sets up resumes and users', async () => {
FirebaseStub.database().initializeData(); FirebaseStub.database().initializeData();
const resumesRef = FirebaseStub.database().ref('resumes'); const resumesRef = FirebaseStub.database().ref(resumesPath);
const resumesDataSnapshot = await resumesRef.once('value'); const resumesDataSnapshot = await resumesRef.once('value');
const resumes = resumesDataSnapshot.val(); const resumes = resumesDataSnapshot.val();
expect(resumes).toBeTruthy(); expect(resumes).toBeTruthy();
@ -113,7 +117,7 @@ describe('database', () => {
expect(emptyResume).toBeTruthy(); expect(emptyResume).toBeTruthy();
expect(emptyResume.id).toEqual(FirebaseStub.database().emptyResumeId); expect(emptyResume.id).toEqual(FirebaseStub.database().emptyResumeId);
const usersRef = FirebaseStub.database().ref('users'); const usersRef = FirebaseStub.database().ref(usersPath);
const usersDataSnapshot = await usersRef.once('value'); const usersDataSnapshot = await usersRef.once('value');
const users = usersDataSnapshot.val(); const users = usersDataSnapshot.val();
expect(users).toBeTruthy(); expect(users).toBeTruthy();
@ -128,7 +132,7 @@ describe('database', () => {
const resume = ( const resume = (
await FirebaseStub.database() await FirebaseStub.database()
.ref(`resumes/${FirebaseStub.database().demoResumeId}`) .ref(`${resumesPath}/${FirebaseStub.database().demoResumeId}`)
.once('value') .once('value')
).val(); ).val();
@ -141,7 +145,9 @@ describe('database', () => {
const resumeId = 'invalidResumeId'; const resumeId = 'invalidResumeId';
const resume = ( const resume = (
await FirebaseStub.database().ref(`resumes/${resumeId}`).once('value') await FirebaseStub.database()
.ref(`${resumesPath}/${resumeId}`)
.once('value')
).val(); ).val();
expect(resume).toBeNull(); expect(resume).toBeNull();
@ -152,7 +158,7 @@ describe('database', () => {
const user = ( const user = (
await FirebaseStub.database() await FirebaseStub.database()
.ref(`users/${FirebaseStub.database().anonymousUser.uid}`) .ref(`${usersPath}/${FirebaseStub.database().anonymousUser.uid}`)
.once('value') .once('value')
).val(); ).val();
@ -165,9 +171,12 @@ describe('database', () => {
const userId = 'invalidUserId'; const userId = 'invalidUserId';
const user = ( const user = (
await FirebaseStub.database().ref(`users/${userId}`).once('value') await FirebaseStub.database()
.ref(`${usersPath}/${userId}`)
.once('value')
).val(); ).val();
expect(user).toBeNull(); expect(user).toBeNull();
}); });
}); });
});

View File

@ -60,13 +60,13 @@ class Auth {
} }
class Database { class Database {
static resumesPath = 'resumes';
static usersPath = 'users';
static #instance = undefined; static #instance = undefined;
#uuid = ''; #uuid = '';
#data = {}; #data = {};
#references = {}; #references = {};
#anonymousUser = undefined; #anonymousUser = undefined;
#demoResumeId = 'demore';
#emptyResumeId = 'mtre01';
constructor() { constructor() {
if (Database.#instance) { if (Database.#instance) {
@ -82,15 +82,23 @@ class Database {
}; };
} }
get resumesPath() {
return Reference.resumesPath;
}
get usersPath() {
return Reference.usersPath;
}
get anonymousUser() { get anonymousUser() {
return this.#anonymousUser; return this.#anonymousUser;
} }
get demoResumeId() { get demoResumeId() {
return 'demore'; return this.#demoResumeId;
} }
get emptyResumeId() { get emptyResumeId() {
return 'mtre01'; return this.#emptyResumeId;
} }
get uuid() { get uuid() {
@ -124,11 +132,11 @@ class Database {
resume.createdAt = date.valueOf(); resume.createdAt = date.valueOf();
} }
this.#data[Database.resumesPath] = resumes; this.#data[this.resumesPath] = resumes;
const users = {}; const users = {};
users[this.anonymousUser.uid] = this.anonymousUser; users[this.anonymousUser.uid] = this.anonymousUser;
this.#data[Database.usersPath] = users; this.#data[this.usersPath] = users;
} }
ref(path) { ref(path) {
@ -144,6 +152,8 @@ class Database {
} }
class Reference { class Reference {
static resumesPath = 'resumes';
static usersPath = 'users';
#rootPath = '.'; #rootPath = '.';
#path = ''; #path = '';
#uuid = ''; #uuid = '';
@ -188,15 +198,15 @@ class Reference {
} }
if ( if (
this.#path === Database.resumesPath || this.#path === Reference.resumesPath ||
this.#path === Database.usersPath this.#path === Reference.usersPath
) { ) {
return this.#path in databaseData ? databaseData[this.#path] : null; return this.#path in databaseData ? databaseData[this.#path] : null;
} }
if ( if (
this.#path.startsWith(`${Database.resumesPath}/`) || this.#path.startsWith(`${Reference.resumesPath}/`) ||
this.#path.startsWith(`${Database.usersPath}/`) this.#path.startsWith(`${Reference.usersPath}/`)
) { ) {
const databaseLocationId = this.#path.substring( const databaseLocationId = this.#path.substring(
this.#path.indexOf('/') + 1, this.#path.indexOf('/') + 1,

View File

@ -26,6 +26,7 @@ beforeEach(() => {
afterEach(cleanup); afterEach(cleanup);
describe('builder', () => { describe('builder', () => {
const resumesPath = FirebaseStub.database().resumesPath;
let resumeId = null; let resumeId = null;
let resume = null; let resume = null;
let mockUpdateFunction = null; let mockUpdateFunction = null;
@ -34,10 +35,12 @@ describe('builder', () => {
beforeEach(async () => { beforeEach(async () => {
resumeId = FirebaseStub.database().demoResumeId; resumeId = FirebaseStub.database().demoResumeId;
resume = ( resume = (
await FirebaseStub.database().ref(`resumes/${resumeId}`).once('value') await FirebaseStub.database()
.ref(`${resumesPath}/${resumeId}`)
.once('value')
).val(); ).val();
mockUpdateFunction = jest.spyOn( mockUpdateFunction = jest.spyOn(
FirebaseStub.database().ref(`resumes/${resumeId}`), FirebaseStub.database().ref(`${resumesPath}/${resumeId}`),
'update', 'update',
); );