ContentReviewBaseTest::setUp() - Code Metrics - Inspection of "Merge pull request #61 from creative-commoners/pul..." - silverstripe/silverstripe-contentreview - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 4a6357...521c8c )

by Franco

created 2017-09-07 04:20 UTC

ContentReviewBaseTest::setUp() A

↳ Parent: ContentReviewBaseTest

Complexity

Conditions	2
Paths	2

Size

Total Lines	19
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	19
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	5
nc	2
nop	0

<?php

namespace SilverStripe\ContentReview\Tests;

use SilverStripe\Dev\FunctionalTest;
use SilverStripe\CMS\Model\SiteTree;
// @todo add translatable namespace
use Translatable;

/**
 * Extend this class when writing unit tests which are compatible with other modules.
 * All compatibility code goes here.
 */
abstract class ContentReviewBaseTest extends FunctionalTest
{
    /**
     * @var bool
     */
    protected $translatableEnabledBefore;

    protected function setUp()
    {
        parent::setUp();

        /*
         *  We set the locale for pages explicitly, because if we don't, then we get into a situation
         *  where the page takes on the tester's (your) locale, and any calls to simulate subsequent requests
         *  (e.g. $this->post()) do not seem to get passed the tester's locale, but instead fallback to the default locale.
         *
         *  So we set the pages locale to be the default locale, which will then match any subsequent requests.
         *
         *  If creating pages in your unit tests (rather than reading from the fixtures file), you must explicitly call
         *  self::compat() on the page, for the same reasons as above.
         */
        if (class_exists(Translatable::class)) {
            $this->translatableEnabledBefore = SiteTree::has_extension(Translatable::class);
            SiteTree::remove_extension(Translatable::class);
        }
    }

    protected function tearDown()
    {
        if (class_exists(Translatable::class)) {
            if ($this->translatableEnabledBefore) {
                SiteTree::add_extension(Translatable::class);
            }
        }

        parent::tearDown();
    }
}


1			<?php
2
3			namespace SilverStripe\ContentReview\Tests;
4
5			use SilverStripe\Dev\FunctionalTest;
6			use SilverStripe\CMS\Model\SiteTree;
7			// @todo add translatable namespace
8			use Translatable;
9
10			/**
11			* Extend this class when writing unit tests which are compatible with other modules.
12			* All compatibility code goes here.
13			*/
14			abstract class ContentReviewBaseTest extends FunctionalTest
15			{
16			/**
17			* @var bool
18			*/
19			protected $translatableEnabledBefore;
20
21			protected function setUp()
22			{
23			parent::setUp();
24
25			/*
26			* We set the locale for pages explicitly, because if we don't, then we get into a situation
27			* where the page takes on the tester's (your) locale, and any calls to simulate subsequent requests
28			* (e.g. $this->post()) do not seem to get passed the tester's locale, but instead fallback to the default locale.
29			*
30			* So we set the pages locale to be the default locale, which will then match any subsequent requests.
31			*
32			* If creating pages in your unit tests (rather than reading from the fixtures file), you must explicitly call
33			* self::compat() on the page, for the same reasons as above.
34			*/
35			if (class_exists(Translatable::class)) {
36			$this->translatableEnabledBefore = SiteTree::has_extension(Translatable::class);
37			SiteTree::remove_extension(Translatable::class);
38			}
39			}
40
41			protected function tearDown()
42			{
43			if (class_exists(Translatable::class)) {
44			if ($this->translatableEnabledBefore) {
45			SiteTree::add_extension(Translatable::class);
46			}
47			}
48
49			parent::tearDown();
50			}
51			}
52

silverstripe / silverstripe-contentreview

Push — master ( 4a6357...521c8c )

ContentReviewBaseTest::setUp() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like