fix($firebaseRef): $firebaseRef vs. firebaseRef

Author: davideast

src/firebaseAuthService.js

@@ -1,10 +1,10 @@
 (function() {
   "use strict";
 
-  function FirebaseAuthService($firebaseAuth, firebaseRef) {
-    return $firebaseAuth(firebaseRef.default);
+  function FirebaseAuthService($firebaseAuth, $firebaseRef) {
+    return $firebaseAuth($firebaseRef.default);
   }
-  FirebaseAuthService.$inject = ['$firebaseAuth', 'firebaseRef'];
+  FirebaseAuthService.$inject = ['$firebaseAuth', '$firebaseRef'];
 
   angular.module('firebase')
     .factory('$firebaseAuthService', FirebaseAuthService);

src/firebaseRef.js

@@ -41,6 +41,6 @@
   }
 
   angular.module('firebase')
-    .provider('firebaseRef', FirebaseRef);
+    .provider('$firebaseRef', FirebaseRef);
 
 })();

tests/unit/FirebaseAuthService.spec.js

@@ -1,11 +1,11 @@
 'use strict';
 describe('$firebaseAuthService', function () {
-  var firebaseRefProvider;
+  var $firebaseRefProvider;
   var MOCK_URL = 'https://stub.firebaseio-demo.com/'
 
-  beforeEach(module('firebase', function(_firebaseRefProvider_) {
-    firebaseRefProvider = _firebaseRefProvider_;
-    firebaseRefProvider.registerUrl(MOCK_URL);
+  beforeEach(module('firebase', function(_$firebaseRefProvider_) {
+    $firebaseRefProvider = _$firebaseRefProvider_;
+    $firebaseRefProvider.registerUrl(MOCK_URL);
   }));
 
   describe('<constructor>', function() {
@@ -18,7 +18,7 @@ describe('$firebaseAuthService', function () {
       });
     });
 
-    it('should exist because we called firebaseRefProvider.registerUrl()', inject(function() {
+    it('should exist because we called $firebaseRefProvider.registerUrl()', inject(function() {
       expect($firebaseAuthService).not.toBe(null);
     }));
 

tests/unit/firebaseRef.spec.js

@@ -1,50 +1,50 @@
 'use strict';
 describe('firebaseRef', function () {
 
-  var firebaseRefProvider;
+  var $firebaseRefProvider;
   var MOCK_URL = 'https://stub.firebaseio-demo.com/'
 
-  beforeEach(module('firebase', function(_firebaseRefProvider_) {
-    firebaseRefProvider = _firebaseRefProvider_;
+  beforeEach(module('firebase', function(_$firebaseRefProvider_) {
+    $firebaseRefProvider = _$firebaseRefProvider_;
   }));
 
   describe('registerUrl', function() {
 
     it('creates a single reference with a url', inject(function() {
-      firebaseRefProvider.registerUrl(MOCK_URL);
-      expect(firebaseRefProvider.$get().default).toBeAFirebaseRef();
+      $firebaseRefProvider.registerUrl(MOCK_URL);
+      expect($firebaseRefProvider.$get().default).toBeAFirebaseRef();
     }));
 
     it('creates a default reference with a config object', inject(function() {
-      firebaseRefProvider.registerUrl({
+      $firebaseRefProvider.registerUrl({
         default: MOCK_URL
       });
-      var firebaseRef = firebaseRefProvider.$get();
+      var firebaseRef = $firebaseRefProvider.$get();
       expect(firebaseRef.default).toBeAFirebaseRef();
     }));
 
     it('creates multiple references with a config object', inject(function() {
-      firebaseRefProvider.registerUrl({
+      $firebaseRefProvider.registerUrl({
         default: MOCK_URL,
         messages: MOCK_URL + 'messages'
       });
-      var firebaseRef = firebaseRefProvider.$get();
+      var firebaseRef = $firebaseRefProvider.$get();
       expect(firebaseRef.default).toBeAFirebaseRef();
       expect(firebaseRef.messages).toBeAFirebaseRef();
     }));
 
     it('should throw an error when no url is provided', inject(function () {
       function errorWrapper() {
-        firebaseRefProvider.registerUrl();
-        firebaseRefProvider.$get();
+        $firebaseRefProvider.registerUrl();
+        $firebaseRefProvider.$get();
       }
       expect(errorWrapper).toThrow();
     }));
 
     it('should throw an error when no default url is provided', inject(function() {
       function errorWrapper() {
-        firebaseRefProvider.registerUrl({ messages: MOCK_URL + 'messages' });
-        firebaseRefProvider.$get();
+        $firebaseRefProvider.registerUrl({ messages: MOCK_URL + 'messages' });
+        $firebaseRefProvider.$get();
       }
       expect(errorWrapper).toThrow();      
     }));