From e66ba56cc02b619a2afe28c5d531ff4234bdde5b Mon Sep 17 00:00:00 2001 From: Thomas Ploch Date: Tue, 16 Dec 2014 13:35:22 +0100 Subject: [PATCH] Fixed `array()` usage and added `HHVM_VERSION` check --- tests/Monolog/Formatter/NormalizerFormatterTest.php | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tests/Monolog/Formatter/NormalizerFormatterTest.php b/tests/Monolog/Formatter/NormalizerFormatterTest.php index e5983f56..0da32ca7 100644 --- a/tests/Monolog/Formatter/NormalizerFormatterTest.php +++ b/tests/Monolog/Formatter/NormalizerFormatterTest.php @@ -169,6 +169,10 @@ class NormalizerFormatterTest extends \PHPUnit_Framework_TestCase public function testExceptionTraceWithArgs() { + if (defined('HHVM_VERSION')) { + $this->markTestSkipped('Not supported in HHVM since it detects errors differently'); + } + // This happens i.e. in React promises or Guzzle streams where stream wrappers are registered // and no file or line are included in the trace because it's treated as internal function set_error_handler(function ($errno, $errstr, $errfile, $errline) { @@ -187,7 +191,7 @@ class NormalizerFormatterTest extends \PHPUnit_Framework_TestCase } $formatter = new NormalizerFormatter(); - $record = ['context' => ['exception' => $e]]; + $record = array('context' => array('exception' => $e)); $result = $formatter->format($record); $this->assertRegExp(