set default value for push type to alert

If defined explicitly the passed value for push type is used.

Author: funkenstrahlen

spec/APNS.spec.js

@@ -194,6 +194,29 @@ describe('APNS', () => {
     expect(notification.priority).toEqual(priority);
     done();
   });
+
+  it('sets push type to alert if not defined explicitly', (done) => {
+    //Mock request data
+    let data = {
+      'alert': 'alert',
+      'title': 'title',
+      'badge': 100,
+      'sound': 'test',
+      'content-available': 1,
+      'mutable-content': 1,
+      'category': 'INVITE_CATEGORY',
+      'threadId': 'a-thread-id',
+      'key': 'value',
+      'keyAgain': 'valueAgain'
+    };
+    let expirationTime = 1454571491354;
+    let collapseId = "collapseIdentifier";
+
+    let notification = APNS._generateNotification(data, { expirationTime: expirationTime, collapseId: collapseId });
+
+    expect(notification.pushType).toEqual('alert');
+    done();
+  });
 
     it('can generate APNS notification from raw data', (done) => {
       //Mock request data

src/APNS.js

@@ -216,7 +216,11 @@ export class APNS {
     notification.topic = headers.topic;
     notification.expiry = Math.round(headers.expirationTime / 1000);
     notification.collapseId = headers.collapseId;
-    notification.pushType = headers.pushType
+    // set alert as default push type. If push type is not set notifications are not delivered to devices running iOS 13, watchOS 6 and later.
+    notification.pushType = 'alert';
+    if (headers.pushType) {
+      notification.pushType = headers.pushType;
+    }
     if (headers.priority) {
       // if headers priority is not set 'node-apn' defaults it to 5 which is min. required value for background pushes to launch the app in background.
       notification.priority = headers.priority