mirror of
https://github.com/PHPOffice/PhpSpreadsheet.git
synced 2026-09-14 20:16:34 +00:00
Simplify Test Bootstrap
This commit is contained in:
@@ -17,6 +17,11 @@ use PhpOffice\PhpSpreadsheet\Writer\Html;
|
||||
use PhpOffice\PhpSpreadsheet\Writer\Xlsx as XlsxWriter;
|
||||
use PhpOffice\PhpSpreadsheetTests\Functional\AbstractFunctional;
|
||||
|
||||
/**
|
||||
* Seems to need to be separate for Php8.4 PhpUnit9 combination.
|
||||
*
|
||||
* @runTestsInSeparateProcesses
|
||||
*/
|
||||
class ChartsDynamicTitleTest extends AbstractFunctional
|
||||
{
|
||||
protected function tearDown(): void
|
||||
|
||||
@@ -25,20 +25,6 @@ function phpunit10ErrorHandler(int $errno, string $errstr, string $filename, int
|
||||
return true; // message suppressed - stop error handling
|
||||
}
|
||||
|
||||
// This code applies only when running release210 with Php8.4.
|
||||
// I don't get it at all. I think mitoteam is the victim of circumstance.
|
||||
// We need to run PhpUnit 9 because we need to support Php8.0.
|
||||
// We are at the highest release of PhpUnit9,
|
||||
// but it refers to E_STRICT,
|
||||
// which is deprecated in 8.4.
|
||||
if (
|
||||
str_contains($errstr, 'Constant ')
|
||||
&& str_contains($errstr, ' already defined')
|
||||
&& str_contains($filename, 'mitoteam')
|
||||
) {
|
||||
return true;
|
||||
}
|
||||
|
||||
if (!method_exists(PHPUnit\Framework\TestCase::class, 'setOutputCallback')) {
|
||||
throw new Exception("$errstr $filename $lineno");
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user