Update auth.service.ts

Ensure that we return an error message when rejecting the sign in

Author: AXeL-dev

src/app/services/auth.service.ts

@@ -3,6 +3,7 @@ import { AngularFireAuth } from '@angular/fire/auth';
 import { Authentication, AuthenticationType } from '../models/auth.model';
 import * as firebase from 'firebase/app';
 import 'firebase/auth'; // required when importing firebase from 'firebase/app'
+import { DebugService } from './debug.service';
 
 @Injectable({
   providedIn: 'root'
@@ -12,9 +13,12 @@ export class AuthService {
   currentUser: firebase.default.User = null;
   lastError: firebase.default.FirebaseError = null;
 
-  constructor(private auth: AngularFireAuth) {
+  constructor(
+    private auth: AngularFireAuth,
+    private debugService: DebugService
+  ) {
     this.auth.onAuthStateChanged((user) => {
-      // console.log(user);
+      // this.debugService.log(user);
       this.currentUser = user;
     });
   }
@@ -24,10 +28,10 @@ export class AuthService {
   }
 
   signIn(authentication: Authentication): Promise<void> {
-    // console.log('sign in', authentication);
+    this.debugService.log('sign in', authentication);
     return new Promise((resolve, reject) => {
       if (this.isSignedIn()) {
-        console.log('already signed in!');
+        this.debugService.log('already signed in!');
         resolve();
       } else {
         // Set authentication function
@@ -61,14 +65,16 @@ export class AuthService {
             reject(error);
           });
         } else {
-          reject();
+          reject({
+            message: 'Unknown authentication type'
+          });
         }
       }
     });
   }
 
   signOut(force: boolean = false): void {
-    // console.log('sign out', this.isSignedIn());
+    this.debugService.log('sign out', this.isSignedIn());
     if (force || this.isSignedIn()) {
       this.auth.signOut().catch((error: firebase.default.FirebaseError) => {
         this.setLastError(error);