Don't use @expectedException and @expectedExceptionMessage

Problem/Motivation

Per #2827160: @expectedException and @expectedException tags are replaced with best practice alternatives., we shouldn't use @expectedException or @expectedExceptionMessage in tests any more.

Proposed resolution

We should use something like this instead:

$this->setExpectedException(MailHandlerException::class, 'Unable to determine message recipient');