diff options
author | Andreas Fischer <bantu@phpbb.com> | 2011-09-18 00:55:24 +0200 |
---|---|---|
committer | Andreas Fischer <bantu@phpbb.com> | 2011-09-18 22:23:41 +0200 |
commit | 7b3f6cb219cac448ba470f016ed5068bdc7ffc56 (patch) | |
tree | 3430398dfb2211a838b9b36a38cf0311fe6d2bc4 /tests/error_collector_test.php | |
parent | eb8b1c333fac415a4c4db22559d625cc50a9f9b6 (diff) | |
download | forums-7b3f6cb219cac448ba470f016ed5068bdc7ffc56.tar forums-7b3f6cb219cac448ba470f016ed5068bdc7ffc56.tar.gz forums-7b3f6cb219cac448ba470f016ed5068bdc7ffc56.tar.bz2 forums-7b3f6cb219cac448ba470f016ed5068bdc7ffc56.tar.xz forums-7b3f6cb219cac448ba470f016ed5068bdc7ffc56.zip |
[ticket/10369] Always include errfile and errline in format_errors().
We remove the phpBB root path from errfile. This is consistent with how
msg_handler handles E_WARNING messages etc.
PHPBB3-10369
Diffstat (limited to 'tests/error_collector_test.php')
-rw-r--r-- | tests/error_collector_test.php | 35 |
1 files changed, 35 insertions, 0 deletions
diff --git a/tests/error_collector_test.php b/tests/error_collector_test.php new file mode 100644 index 0000000000..e1ac32f5ac --- /dev/null +++ b/tests/error_collector_test.php @@ -0,0 +1,35 @@ +<?php +/** +* +* @package testing +* @copyright (c) 2011 phpBB Group +* @license http://opensource.org/licenses/gpl-license.php GNU Public License +* +*/ + +require_once dirname(__FILE__) . '/../phpBB/includes/functions.php'; +require_once dirname(__FILE__) . '/../phpBB/includes/error_collector.php'; + +class phpbb_error_collector_test extends phpbb_test_case +{ + public function test_collection() + { + $collector = new phpbb_error_collector; + $collector->install(); + + // Cause a warning + 1/0; $line = __LINE__; + + $collector->uninstall(); + + list($errno, $msg_text, $errfile, $errline) = $collector->errors[0]; + $error_contents = $collector->format_errors(); + + $this->assertEquals($errno, 2); + + // Unfortunately $error_contents will contain the full path here, + // because the tests directory is outside of phpbb root path. + $this->assertStringStartsWith('Errno 2: Division by zero at ', $error_contents); + $this->assertStringEndsWith(" line $line", $error_contents); + } +} |