From 4271be735981cbb81debde8962f76eb6dfb1e4d0 Mon Sep 17 00:00:00 2001 From: Ewout Pieter den Ouden Date: Mon, 10 Dec 2018 23:23:54 +0100 Subject: [PATCH] Protect JUnit logger against implementations without usesDataProvider() --- src/Util/Log/JUnit.php | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/Util/Log/JUnit.php b/src/Util/Log/JUnit.php index 391ed015387..13b9509024e 100644 --- a/src/Util/Log/JUnit.php +++ b/src/Util/Log/JUnit.php @@ -290,11 +290,17 @@ public function endTestSuite(TestSuite $suite): void */ public function startTest(Test $test): void { + $usesDataprovider = false; + + if (\method_exists($test, 'usesDataProvider')) { + $usesDataprovider = $test->usesDataProvider(); + } + $testCase = $this->document->createElement('testcase'); $testCase->setAttribute('name', $test->getName()); $class = new ReflectionClass($test); - $methodName = $test->getName(!$test->usesDataProvider()); + $methodName = $test->getName(!$usesDataprovider); if ($class->hasMethod($methodName)) { $method = $class->getMethod($methodName);