| Summary: | [OMNIBUS] Remove warnings and errors from tests output | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> | 
| Component: | Test Suite | Assignee: | Jonathan Druart <jonathan.druart> | 
| Status: | ASSIGNED --- | QA Contact: | Testopia <testopia> | 
| Severity: | enhancement | ||
| Priority: | P5 - low | CC: | m.de.rooy, martin.renvoize, mtj | 
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | All | ||
| See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27126 https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25790 | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Crowdfunding goal: | 0 | 
| Patch complexity: | --- | Documentation contact: | |
| Documentation submission: | Text to go in the release notes: | ||
| Version(s) released in: | Circulation function: | ||
| Bug Depends on: | 40375, 40382, 40442, 40443, 40448, 40449, 40315 | ||
| Bug Blocks: | |||
| 
        
          Description
        
        
          Jonathan Druart
        
        
        
        
          2020-05-15 13:40:43 UTC
        
       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! |