Removing the redundant testcase

Author: RamIndia

lib/utils.js

@@ -195,11 +195,6 @@ function buildCustomNotification(_) {
 function buildNotification(notificationData) {
   let notification;
 
-  if(!notificationData) {
-    LOG._warn && LOG.warn(messages.NO_OBJECT_FOR_NOTIFY);
-    return;
-  }
-
   if (Object.keys(notificationData).length === 0) {
     LOG._warn && LOG.warn(messages.EMPTY_OBJECT_FOR_NOTIFY);
     return;

srv/service.js

@@ -10,7 +10,7 @@ class NotificationService extends cds.Service {
    * @param {object} message - The message object
    */
   emit (event, message) {
-    if(!event) {
+    if (!event) {
       LOG._warn && LOG.warn(messages.NO_OBJECT_FOR_NOTIFY);
       return;
     }

test/lib/utils.test.js

@@ -553,11 +553,6 @@ describe("Test utils", () => {
   //   ).toBeFalsy();
   // });
 
-  test("When no notification data for custom notification is passed to buildNotification", () => {
-    expect(buildNotification(undefined)).toBeFalsy();
-    expect(buildNotification(null)).toBeFalsy();
-  });
-
   test("Given invalid NTypes | When validateNotificationTypes is called | Then false is returned", () => {
     expect(validateNotificationTypes([{ NotificationTypeKey: "Test" }, { blabla: "Test2" }])).toEqual(false);
   });