PhpOffice\PhpSpreadsheetTests\Functional\StreamTest

Complexity

Total Complexity

2

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	testAllWritersCanWriteToStream()	0	14	1
A	providerFormats()	0	11	1

<?php

namespace PhpOffice\PhpSpreadsheetTests\Functional;

use PhpOffice\PhpSpreadsheet\IOFactory;
use PhpOffice\PhpSpreadsheet\Spreadsheet;
use PHPUnit\Framework\TestCase;

class StreamTest extends TestCase
{
    public function providerFormats(): array
    {
        return [
            ['Xls'],
            ['Xlsx'],
            ['Ods'],
            ['Csv'],
            ['Html'],
            ['Tcpdf'],
            ['Dompdf'],
            ['Mpdf'],
        ];
    }

    /**
     * @dataProvider providerFormats
     *
     * @param string $format
     */
    public function testAllWritersCanWriteToStream(string $format): void
    {
        $spreadsheet = new Spreadsheet();
        $spreadsheet->getActiveSheet()->setCellValue('A1', 'foo');
        $writer = IOFactory::createWriter($spreadsheet, $format);

        $stream = fopen('php://memory', 'wb+');
        self::assertSame(0, fstat($stream)['size']);

It seems like $stream can also be of type false; however, parameter $handle of fstat() does only seem to accept resource, maybe add an additional type check?

        $writer->save($stream);

It seems like $stream can also be of type false; however, parameter $pFilename of PhpOffice\PhpSpreadsheet\Writer\IWriter::save() does only seem to accept resource|string, maybe add an additional type check?

        self::assertIsResource($stream, 'should not close the stream for further usage out of PhpSpreadsheet');
        self::assertGreaterThan(0, fstat($stream)['size'], 'something should have been written to the stream');
        self::assertGreaterThan(0, ftell($stream), 'should not be rewinded, because not all streams support it');

It seems like $stream can also be of type false; however, parameter $handle of ftell() does only seem to accept resource, maybe add an additional type check?

    }
}