PhpOffice\PhpSpreadsheetTests\Functional\StreamTest::testAllWritersCanWriteToStream()

Complexity

Conditions	1
Paths	1

Size

Total Lines	14
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    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?

    }