SimpleSAML\Test\Web\TemplateTest

Complexity

Total Complexity

4

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	testSyntax()	0	23	4

<?php
/**
 * Simple test for syntax-checking Twig-templates.
 *
 * @author Tim van Dijen <tvdijen@gmail.com>
 * @package SimpleSAMLphp
 */
 
namespace SimpleSAML\Test\Web;

use PHPUnit\Framework\TestCase;
use SimpleSAML\Configuration;
use SimpleSAML\XHTML\Template;
use SimpleSAML\Module;
use Twig\Error\SyntaxError;

class TemplateTest extends TestCase
{
    public function testSyntax()
    {
        $config = Configuration::loadFromArray([
            'language.i18n.backend' => 'gettext/gettext',
            'module.enable' => array_fill_keys(Module::getModules(), true),
        ]);

        Configuration::setPreLoadedConfig($config);
        $basedir = dirname(dirname(dirname(__FILE__))).DIRECTORY_SEPARATOR.'templates';

        // Base templates
        $files = array_diff(scandir($basedir), ['.', '..']);

It seems like scandir($basedir) can also be of type false; however, parameter $array1 of array_diff() does only seem to accept array, maybe add an additional type check?

        foreach ($files as $file) {
            if (preg_match('/.twig$/', $file)) {
                $t = new Template($config, 'monitor:'.basename($file));
                ob_start();
                try {
                    $t->show();

The function SimpleSAML\XHTML\Template::show() has been deprecated: Do not use this method, use Twig + send() instead. Will be removed in 2.0

                    $this->addToAssertionCount(1);
                } catch (SyntaxError $e) {
                    $this->fail($e->getMessage().' in '.$e->getFile().':'.$e->getLine());
                }
                ob_end_clean();
            }
        }
    }
}