fix(auth): check for null user before attempting to transform

Fixes angular#251

Author: jeffbcross

src/providers/auth.spec.ts

@@ -528,6 +528,20 @@ describe('FirebaseAuth', () => {
         expect(app.auth().signOut).toHaveBeenCalled();
       });
     });
+
+
+    describe('getAuth()', () => {
+      it('should return null when no user is logged in', () => {
+        authSpy['currentUser'] = null;
+        expect(afAuth.getAuth()).toBe(null);
+      });
+
+
+      it('should return authState if user is logged in', () => {
+        authSpy['currentUser'] = firebaseUser;
+        expect(afAuth.getAuth().uid).toEqual(AngularFireAuthState.uid);
+      })
+    });
   });
 });
 

src/providers/auth_backend.ts

@@ -70,6 +70,7 @@ export interface TwitterCredential extends CommonOAuthCredential {
 export type OAuthCredential = CommonOAuthCredential | GoogleCredential | TwitterCredential;
 
 export function authDataToAuthState(authData: firebase.User, providerData?: OAuthCredential): FirebaseAuthState {
+  if (!authData) return null;
   let providerId;
   let { uid } = authData;
   let authState: FirebaseAuthState = { auth: authData, uid, provider: null };