Webfactory\ShortcodeBundle\Test\EndToEndTestHelper

Complexity

Total Complexity

4

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	createFromContainer()	0	3	1
A	processShortcode()	0	8	2

<?php

namespace Webfactory\ShortcodeBundle\Test;

use Symfony\Component\DependencyInjection\ContainerInterface;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\RequestStack;
use Thunder\Shortcode\Processor\Processor;

/**
 * Helper class that you can use in functional (end-to-end) tests to verify that a given
 * content with shortcodes is processed as expected.
 */
class EndToEndTestHelper
{
    /**
     * @var Processor
     */
    private $processor;

    /**
     * @var RequestStack
     */
    private $requestStack;

    public static function createFromContainer(ContainerInterface $container): self
    {
        return new self($container->get(Processor::class), $container->get(RequestStack::class));

It seems like $container->get(Symfony\Component\HttpFoundation\RequestStack::class) can also be of type null; however, parameter $requestStack of Webfactory\ShortcodeBundle\Test\EndToEndTestHelper::__construct() does only seem to accept Symfony\Component\HttpFoundation\RequestStack, maybe add an additional type check?

It seems like $container->get(Thunder\Shortcode\Processor\Processor::class) can also be of type null; however, parameter $processor of Webfactory\ShortcodeBundle\Test\EndToEndTestHelper::__construct() does only seem to accept Thunder\Shortcode\Processor\Processor, maybe add an additional type check?

    }

    public function __construct(Processor $processor, RequestStack $requestStack)
    {
        $this->processor = $processor;
        $this->requestStack = $requestStack;
    }

    public function processShortcode(string $shortcode): string
    {
        // The fragment handler used by EmbeddedShortcodeHandler requires a request to be active, so let's make sure that is the case
        if (null === $this->requestStack->getCurrentRequest()) {
            $this->requestStack->push(new Request());
        }

        return $this->processor->process($shortcode);
    }
}