LPS-45890 Fix tests - Please read comment

cc /@sergiogonzalez

Hey @johnnyhowey ,

I've introduced some logic to fix the userNotification test. The key point is that with your logic the fw is creating a notificationEvent for mail events, wich is not necessary, because all the email sending logic is in SubscriptionSender, and the fw only handles if it should be deliver or not.

Anyway, I've agreed with Sergio that it's weird that  a method called sendUserNotificationEvents should not do anything related with mailEvents. So there are 2 possible solutions to this:

1. Change the fw logic to handle the email sending
2. Change the method name to explictly explain that this method is going to send the notifications just for mobile and website and not for emails

What do you think??