Tests\Koded\Http\FileStreamTest

Complexity

Total Complexity

3

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	test_should_create_only_writable_stream()	0	14	1
A	test_should_create_read_write_stream_by_default()	0	11	1
A	tearDown()	0	3	1

<?php

namespace Tests\Koded\Http;

use Koded\Http\FileStream;
use PHPUnit\Framework\TestCase;
use RuntimeException;

class FileStreamTest extends TestCase
{
    private string $file = '/tmp/test';

    public function test_should_create_only_writable_stream()
    {
        $this->expectException(RuntimeException::class);
        $this->expectExceptionMessage('The stream is not readable');

        $stream = new FileStream($this->file, 'w');
        $stream->write('hello world');

        $this->assertSame('w', $stream->getMetadata('mode'));
        $this->assertSame('', $stream->getContents());

        $this->assertFalse($stream->isReadable());
        $this->assertFalse($stream->isSeekable());
        $this->assertTrue($stream->isWritable());
    }

    public function test_should_create_read_write_stream_by_default()
    {
        $stream = new FileStream($this->file, 'w+');
        $stream->write('hello world');

        $this->assertSame('w+', $stream->getMetadata('mode'));
        $this->assertSame('hello world', $stream->getContents());

        $this->assertTrue($stream->isReadable());
        $this->assertTrue($stream->isSeekable());
        $this->assertTrue($stream->isWritable());
    }

    protected function tearDown(): void
    {
        @unlink($this->file);

It seems like you do not handle an error condition for unlink(). This can introduce security issues, and is generally not recommended.

    }
}