slurp-36 Catch PDO Exceptions and re-throw with additional context. #37

courtney-miles · 2019-08-18T07:30:52Z

This PR makes changes to catch any thrown PDOExceptions and instead throw a LoadRuntimeException with a more specific exception message based on the context of the original exception.

coveralls · 2019-08-18T07:36:44Z

Pull Request Test Coverage Report for Build 135

43 of 51 (84.31%) changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 91.501%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/Load/DatabaseLoader/DatabaseLoader.php	23	24	95.83%
src/Load/DatabaseLoader/StagedLoad.php	2	9	22.22%

Totals
Change from base Build 133:	-0.4%
Covered Lines:	689
Relevant Lines:	753

💛 - Coveralls

slurp-36 Catch PDO Exceptions and re-throw with additional context.

982842f

courtney-miles merged commit 6820269 into release Aug 19, 2019

courtney-miles deleted the imprv/36/CatchPDOExceptionAndReThrow branch January 17, 2022 06:45

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

slurp-36 Catch PDO Exceptions and re-throw with additional context. #37

slurp-36 Catch PDO Exceptions and re-throw with additional context. #37

courtney-miles commented Aug 18, 2019

coveralls commented Aug 18, 2019 •

edited

slurp-36 Catch PDO Exceptions and re-throw with additional context. #37

slurp-36 Catch PDO Exceptions and re-throw with additional context. #37

Conversation

courtney-miles commented Aug 18, 2019

coveralls commented Aug 18, 2019 • edited

Pull Request Test Coverage Report for Build 135

💛 - Coveralls

coveralls commented Aug 18, 2019 •

edited