Switched Test::NoWarnings to Test::FailWarnings.

[kimmel]

My editor also stripped some trailing whitespace as well.

[karenetheridge]

I'm not sure if the jury has settled on Test::FailWarnings over Test::Warnings yet ;) they have similar but not identical feature sets

[kimmel]

My entire decision was based on being able to do whitelist specific modules like this:

use Test::FailWarnings -allow_from => [ qw/Annoying::Module/ ];

[karenetheridge]

I'd be content to recommend both ::FailWarnings and ::Warnings - xdg and I developed those mostly in tandem, following a conversation we had about how ::NoWarnings was missing things we needed.

[kimmel]

I am going to redo this on its own branch so I can have multiple PRs at the same time.