Remove debugging message
When MongoDB username/password is not provided, don't sent it to MongoDB::MongoClient because even the undef values tell MongoDB that it should try to authenticate
Fix compile error.
Require "use warnings" in the compile test
Require newer version of Perl::Tidy
Run compile test against files in "t/" too
Remove usage of "done_testing()" and instead provide Test::More with the exact number of tests that are going to be run; this is both the recommended way by Test::More, and the test results get presented more nicely when test die()s in the middle
Get rid of compiler warning
In all unit tests, use a testing database via MediaWords::Test::DB instead of connecting to the production database
Remove unused includes
When $content_downloads is undef, say so in the die() message
Make MediaWords::Test::LocalServer an instance-based class (as in "initialized with new()") because users might want to start multiple servers, and also the class can then get to know when it's being deinitialized; save PID of a worker child and kill the child in "sub DESTROY()" because if the user (a unit test) fails somewhere between startup and killing the server, the child becomes a zombie process and the whole program hangs (and thus Vagrant test gets stuck and has to be killed manually)
Remove unused / misused variable
Fix typos caught by the "use warnings"
Merge branch 'improve_unit_tests'