fix(local-notifications): Adding check for new Notification support (…

…#295)

* Adding check to see if `new Notification` is supported

* Adding additional check to prevent test from sending empty notification on supported browsers

* switching to `('Notification' in window)` for consistency

* Renaming `checkNotificationSupport` to `hasNotificationSupport`

Co-authored-by: jcesarmobile <jcesarmobile@gmail.com>

local-notifications/src/web.ts

@@ -30,7 +30,7 @@ export class LocalNotificationsWeb
   }
 
   async schedule(options: ScheduleOptions): Promise<ScheduleResult> {
-    if (!('Notification' in window)) {
+    if (!this.hasNotificationSupport()) {
       throw this.unavailable('Notifications not supported in this browser.');
     }
 
@@ -71,7 +71,7 @@ export class LocalNotificationsWeb
   }
 
   async requestPermissions(): Promise<PermissionStatus> {
-    if (!('Notification' in window)) {
+    if (!this.hasNotificationSupport()) {
       throw this.unavailable('Notifications not supported in this browser.');
     }
 
@@ -83,7 +83,7 @@ export class LocalNotificationsWeb
   }
 
   async checkPermissions(): Promise<PermissionStatus> {
-    if (!('Notification' in window)) {
+    if (!this.hasNotificationSupport()) {
       throw this.unavailable('Notifications not supported in this browser.');
     }
 
@@ -94,6 +94,26 @@ export class LocalNotificationsWeb
     return { display };
   }
 
+  protected hasNotificationSupport = (): boolean => {
+    if (!('Notification' in window) || !Notification.requestPermission) {
+      return false;
+    }
+
+    if (Notification.permission !== 'granted') {
+      // don't test for `new Notification` if permission has already been granted
+      // otherwise this sends a real notification on supported browsers
+      try {
+        new Notification('');
+      } catch (e) {
+        if (e.name == 'TypeError') {
+          return false;
+        }
+      }
+    }
+
+    return true;
+  };
+
   protected transformNotificationPermission(
     permission: NotificationPermission,
   ): PermissionState {