We have tons of: "Use of uninitialized value" "DBIx::Class::Storage::DBI::_dbh_execute():" "Subroutine * redefined" "Argument "" isn't numeric in numeric" We must get rid of them and provide a clean output.
Bug 26157 - Redirect expected DBI warnings is going to deal with "DBIx::Class::Storage::DBI::_dbh_execute():"
Bug 26186 - Remove "subroutine redefined" warnings from tests
*** Bug 8879 has been marked as a duplicate of this bug. ***
Bug 30731 removes a few too. I removed the keyword for 22.05 here, no patches here. Just an omnibus.
I wonder if adding Test::NoWarnings might be worthwhile in out test scripts... then a warning is actually a test fail.
(In reply to Martin Renvoize from comment #5) > I wonder if adding Test::NoWarnings might be worthwhile in out test > scripts... then a warning is actually a test fail. I always wanted to reach a point where no warnings where generated by the test suite and then we could add a test to parse the whole output and catch warnings. Looks like a never-ending process. Using Test::NoWarnings on the tests not generating warnings now sounds like a good idea!
Added a patch on bug 38944.