SilverStripe\CMS\Tests\Behaviour\ThemeContext

Complexity

Total Complexity

16

Size/Duplication

Total Lines	102
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	stepCreateTheme()	0	9	2
A	requireDir()	0	6	2
A	cleanThemesAfterScenario()	0	26	6
A	requireFile()	0	14	3
A	stepCreateTemplate()	0	11	3

<?php

namespace SilverStripe\CMS\Tests\Behaviour;

use Behat\Behat\Context\Context;

The type Behat\Behat\Context\Context was not found. Maybe you did not declare it correctly or list all dependencies?

use SilverStripe\BehatExtension\Context\MainContextAwareTrait;

The type SilverStripe\BehatExtension\Context\MainContextAwareTrait was not found. Maybe you did not declare it correctly or list all dependencies?

/**
 * Context used to create fixtures in the SilverStripe ORM.
 */
class ThemeContext implements Context
{
    use MainContextAwareTrait;

    protected $restoreFiles = array();
    protected $restoreDirectories = array();

    /**
     * Create a test theme
     *
     * @Given /^a theme "(?<theme>[^"]+)"/
     * @param string $theme
     */
    public function stepCreateTheme($theme)
    {
        if (!preg_match('/^[0-9a-zA-Z_-]+$/', $theme)) {
            throw new \InvalidArgumentException("Bad theme '$theme'");
        }

        $this->requireDir(BASE_PATH . '/themes');
        $this->requireDir(BASE_PATH . '/themes/' . $theme);
        $this->requireDir(BASE_PATH . '/themes/' . $theme . '/templates');
    }

    /**
     * Create a template within a test theme
     *
     * @Given /^a template "(?<template>[^"]+)" in theme "(?<theme>[^"]+)" with content "(?<content>[^"]+)"/
     * @param string $template
     * @param string $theme
     * @param string $content
     */
    public function stepCreateTemplate($template, $theme, $content)
    {
        if (!preg_match('/^[0-9a-zA-Z_-]+$/', $theme)) {
            throw new \InvalidArgumentException("Bad theme '$theme'");
        }
        if (!preg_match('/^(Layout\/)?[0-9a-zA-Z_-]+\.ss$/', $template)) {
            throw new \InvalidArgumentException("Bad template '$template'");
        }

        $this->stepCreateTheme($theme);
        $this->requireFile(BASE_PATH . '/themes/' . $theme . '/templates/' . $template, $content);
    }

    protected function requireFile($filename, $content)
    {
        // Already exists
        if (file_exists($filename)) {
            // If the content is different, remember old content for restoration
            $origContent = file_get_contents($filename);
            if ($origContent != $content) {
                file_put_contents($filename, $content);
                $this->restoreFiles[$filename] = $origContent;
            }
        // Doesn't exist, mark it for deletion after test
        } else {
            file_put_contents($filename, $content);
            $this->restoreFiles[$filename] = null;
        }
    }

    protected function requireDir($dirname)
    {
        // Directory doesn't exist, create it and mark it for deletion
        if (!file_exists($dirname)) {
            mkdir($dirname);
            $this->restoreDirectories[] = $dirname;
        }
    }

    /**
     * Clean up any theme manipulation
     *
     * @AfterScenario
     */
    public function cleanThemesAfterScenario()
    {
        // Restore any created/modified files.
        //  - If modified, revert then to original contnet
        //  - If created, delete them
        if ($this->restoreFiles) {

The expression $this->restoreFiles of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            foreach ($this->restoreFiles as $file => $origContent) {
                if ($origContent === null) {
                    unlink($file);
                } else {
                    file_put_contents($file, $origContent);
                }
            }

            $this->restoreFiles = array();
        }

        // Restore any created directories: that is, delete them
        if ($this->restoreDirectories) {

The expression $this->restoreDirectories of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            // Flip the order so that nested direcotires are unlinked() first
            $this->restoreDirectories = array_reverse($this->restoreDirectories);
            foreach ($this->restoreDirectories as $dir) {
                rmdir($dir);
            }

            $this->restoreDirectories = array();
        }
    }
}