PhpUnit and Jpgraph (#2307)

oleibman · web-flow · commit f2cd62a9efe6 · 2021-09-26T09:39:15.000-07:00
35_Char_render.php had previously been a problem only for PHP8+. It is now a problem for PHP7.4, and will therefore be skipped all the time.
diff --git a/tests/PhpSpreadsheetTests/Helper/SampleTest.php b/tests/PhpSpreadsheetTests/Helper/SampleTest.php
@@ -30,6 +30,7 @@ public function providerSample(): array
         $skipped = [
             'Chart/32_Chart_read_write_PDF.php', // Unfortunately JpGraph is not up to date for latest PHP and raise many warnings
             'Chart/32_Chart_read_write_HTML.php', // idem
+            'Chart/35_Chart_render.php', // idem
         ];
         // TCPDF and DomPDF libraries don't support PHP8 yet
         if (\PHP_VERSION_ID >= 80000) {
@@ -38,7 +39,6 @@ public function providerSample(): array
                 [
                     'Pdf/21_Pdf_Domdf.php',
                     'Pdf/21_Pdf_TCPDF.php',
-                    'Chart/35_Chart_render.php', // idem
                 ]
             );
         }

Original file line number	Diff line number	Diff line change
`@@ -30,6 +30,7 @@ public function providerSample(): array`
`30`	`30`	`$skipped = [`
`31`	`31`	`'Chart/32_Chart_read_write_PDF.php', // Unfortunately JpGraph is not up to date for latest PHP and raise many warnings`
`32`	`32`	`'Chart/32_Chart_read_write_HTML.php', // idem`
	`33`	`+ 'Chart/35_Chart_render.php', // idem`
`33`	`34`	`];`
`34`	`35`	`// TCPDF and DomPDF libraries don't support PHP8 yet`
`35`	`36`	`if (\PHP_VERSION_ID >= 80000) {`
`@@ -38,7 +39,6 @@ public function providerSample(): array`
`38`	`39`	`[`
`39`	`40`	`'Pdf/21_Pdf_Domdf.php',`
`40`	`41`	`'Pdf/21_Pdf_TCPDF.php',`
`41`		`- 'Chart/35_Chart_render.php', // idem`
`42`	`42`	`]`
`43`	`43`	`);`
`44`	`44`	`}`