PhpSchool\TerminalTest\IO\ResourceInputStreamTest

Complexity

Total Complexity

3

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	testNonStream()	0	5	1
A	testNotReadable()	0	5	1
A	testRead()	0	16	1

<?php

namespace PhpSchool\TerminalTest\IO;

use PhpSchool\Terminal\IO\ResourceInputStream;
use PHPUnit\Framework\TestCase;

/**
 * @author Aydin Hassan <aydin@hotmail.co.uk>
 */
class ResourceInputStreamTest extends TestCase
{
    public function testNonStream() : void
    {
        $this->expectException(\InvalidArgumentException::class);
        $this->expectExceptionMessage('Expected a valid stream');
        new ResourceInputStream(42);
    }

    public function testNotReadable() : void
    {
        $this->expectException(\InvalidArgumentException::class);
        $this->expectExceptionMessage('Expected a readable stream');
        new ResourceInputStream(\STDOUT);
    }

    public function testRead() : void
    {
        $stream = fopen('php://memory', 'r+');
        fwrite($stream, '1234');

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

        rewind($stream);

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

        $inputStream = new ResourceInputStream($stream);

        $input = '';
        $inputStream->read(4, function ($buffer) use (&$input) {
            $input .= $buffer;
        });

        static::assertSame('1234', $input);

        fclose($stream);

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

    }
}