tests for Notifications.pm send unneeded email

Bug #684957 reported by Dan MacNeil
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
MVHub
Confirmed
Medium
Keith Fader

Bug Description

lib-mvhub/t/Notifications/send_emails_contained_in.t

tests

     MVHub::Notifications::send_emails_contained_in

which is found in

    lib-mvhub/lib/MVHub/Notifications.pm

as a side effect it sends emails which are tedious to go through and make the tests take longer to run. .

To avoid sending email, you need to edit:

        lib-mvhub/t/Notifications/send_emails_contained_in.t

to override:

        MVHub::Common::sendmail($email);

...so that it doesn't actually send email.

To find other tests that do this use:

  cdw
   grep 'package main' . -rl

The perl developer's notebook (hard copy only) explains how to do this in some detail.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.