Skip to content
This repository was archived by the owner on Mar 17, 2025. It is now read-only.

Commit a7554c6

Browse files
committed
fix(tests): Check for default key in firebaseRefProvider.registerUrl()
1 parent 052f4a6 commit a7554c6

File tree

2 files changed

+13
-1
lines changed

2 files changed

+13
-1
lines changed

src/firebaseRef.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,9 @@
2020
if (!urlConfig) {
2121
return new Error('No Firebase URL registered. Use firebaseRefProvider.registerUrl() in the config phase. This is required if you are using $firebaseAuthService.');
2222
}
23+
if (!urlConfig.default) {
24+
return new Error('No default Firebase URL registered. Use firebaseRefProvider.registerUrl({ default: "https://<my-firebase-app>.firebaseio.com/"}).');
25+
}
2326
};
2427

2528
this.$$createRefsFromUrlConfig = function $$createMultipleRefs(urlConfig) {

tests/unit/firebaseRef.spec.js

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ describe('firebaseRef', function () {
3232
expect(firebaseRef.default).toBeAFirebaseRef();
3333
expect(firebaseRef.messages).toBeAFirebaseRef();
3434
}));
35-
35+
3636
it('should throw an error when no url is provided', inject(function () {
3737
function errorWrapper() {
3838
firebaseRefProvider.registerUrl();
@@ -41,6 +41,15 @@ describe('firebaseRef', function () {
4141
expect(errorWrapper).toThrow();
4242
}));
4343

44+
it('should throw an error when no default url is provided', inject(function() {
45+
function errorWrapper() {
46+
firebaseRefProvider.registerUrl({ messages: MOCK_URL + 'messages' });
47+
firebaseRefProvider.$get();
48+
}
49+
expect(errorWrapper).toThrow();
50+
}));
51+
52+
4453
});
4554

4655
});

0 commit comments

Comments
 (0)