fix: check if callback is a function (#54)

* fix: check if callback is a function

* refactor: add more check points for callback

Co-authored-by: Pablo Rey Sobral <pablo.rey@adevinta.com>

src/main/application/TcfApiV2.js

@@ -33,11 +33,13 @@ class TcfApiV2 {
   }
 
   getTCData(callback, vendorIds) {
-    callback(this._getTCDataUseCase.execute({vendorIds}), true)
+    typeof callback === 'function' &&
+      callback(this._getTCDataUseCase.execute({vendorIds}), true)
   }
 
   ping(callback) {
-    callback(this._pingUseCase.execute(), true)
+    typeof callback === 'function' &&
+      callback(this._pingUseCase.execute(), true)
   }
 
   addEventListener(callback) {
@@ -51,10 +53,13 @@ class TcfApiV2 {
   getVendorList(callback, vendorListVersion) {
     return this._getVendorListUseCase
       .execute({vendorListVersion})
-      .then(vendorList => callback(vendorList, true))
+      .then(
+        vendorList =>
+          typeof callback === 'function' && callback(vendorList, true)
+      )
       .catch(error => {
         console.log('Error obtaining the vendor list', error)
-        callback(null, false)
+        typeof callback === 'function' && callback(null, false)
       })
   }
 }

src/main/domain/service/EventStatusService.js

@@ -65,22 +65,22 @@ export class EventStatusService {
         observer: callback
       })
     } catch (error) {
-      callback(null, false)
+      typeof callback === 'function' && callback(null, false)
       return
     }
 
     const tcData = this._getTCDataUseCase.execute()
     tcData.listenerId = reference
 
-    callback(tcData, true)
+    typeof callback === 'function' && callback(tcData, true)
   }
 
   removeEventListener({callback, listenerId}) {
     const success = this._domainEventBus.unregister({
       eventName: EVENT_STATUS,
       reference: listenerId
     })
-    callback(success)
+    typeof callback === 'function' && callback(success)
   }
 }
 

src/main/infrastructure/controller/TcfApiController.js

@@ -24,7 +24,7 @@ class TcfApiController {
 
   _reject(callback) {
     try {
-      callback(null, false)
+      typeof callback === 'function' && callback(null, false)
     } catch (ignored) {}
   }