DanchukAS\Mock\Type\UnknownTypeMock

Complexity

Total Complexity

3

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	newTempFile()	0	5	1
A	getSample()	0	14	2

<?php
declare(strict_types=1);
/**
 * Created by PhpStorm.
 * User: danchukas
 * Date: 2017-07-18 08:46
 */

namespace DanchukAS\Mock\Type;


use DanchukAS\Mock\TypeMock;

/**
 * Class UnknownTypeMock
 * @package DanchukAS\Mock\Type
 */
class UnknownTypeMock extends TypeMock
{

    /**
     * @return \Generator|resource
     */
    public static function getSample()
    {
        $file_name = self::newTempFile();
        $resource = fopen($file_name, 'r');
        fclose($resource);
        $unknown = $resource;
        yield [$unknown];
        @unlink($file_name);

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

        if (function_exists("imagecreate")) {
            $resource = \imagecreate(1, 1);
            \imagedestroy($resource);
            $unknown = $resource;
            yield [$unknown];
        }
    }

    /**
     * @return bool|string
     */
    private static function newTempFile()
    {
        $file_name = tempnam(sys_get_temp_dir(), 'vo_');

        return $file_name;
    }
}