diff --git a/tests/phpunit/tests/admin/includesTheme.php b/tests/phpunit/tests/admin/includesTheme.php index 8208d3fcfa..00b9501dcd 100644 --- a/tests/phpunit/tests/admin/includesTheme.php +++ b/tests/phpunit/tests/admin/includesTheme.php @@ -4,12 +4,25 @@ */ class Tests_Admin_IncludesTheme extends WP_UnitTestCase { + /** + * Theme root directory. + * + * @var string + */ + const THEME_ROOT = DIR_TESTDATA . '/themedir1'; + + /** + * Original theme directory. + * + * @var string + */ + private $orig_theme_dir; + public function set_up() { parent::set_up(); - $this->theme_root = DIR_TESTDATA . '/themedir1'; $this->orig_theme_dir = $GLOBALS['wp_theme_directories']; - $GLOBALS['wp_theme_directories'] = array( WP_CONTENT_DIR . '/themes', $this->theme_root ); + $GLOBALS['wp_theme_directories'] = array( WP_CONTENT_DIR . '/themes', self::THEME_ROOT ); add_filter( 'theme_root', array( $this, 'filter_theme_root' ) ); add_filter( 'stylesheet_root', array( $this, 'filter_theme_root' ) ); @@ -33,7 +46,7 @@ class Tests_Admin_IncludesTheme extends WP_UnitTestCase { // Replace the normal theme root directory with our premade test directory. public function filter_theme_root( $dir ) { - return $this->theme_root; + return self::THEME_ROOT; } /** diff --git a/tests/phpunit/tests/block-supports/layout.php b/tests/phpunit/tests/block-supports/layout.php index 9e28d6cd7d..3611c28efa 100644 --- a/tests/phpunit/tests/block-supports/layout.php +++ b/tests/phpunit/tests/block-supports/layout.php @@ -17,6 +17,21 @@ * @covers ::wp_restore_image_outer_container */ class Test_Block_Supports_Layout extends WP_UnitTestCase { + + /** + * Theme root directory. + * + * @var string + */ + private $theme_root; + + /** + * Original theme directory. + * + * @var string + */ + private $orig_theme_dir; + function set_up() { parent::set_up(); $this->theme_root = realpath( DIR_TESTDATA . '/themedir1' ); @@ -30,7 +45,6 @@ class Test_Block_Supports_Layout extends WP_UnitTestCase { add_filter( 'stylesheet_root', array( $this, 'filter_set_theme_root' ) ); add_filter( 'template_root', array( $this, 'filter_set_theme_root' ) ); - $this->queries = array(); // Clear caches. wp_clean_themes_cache(); unset( $GLOBALS['wp_themes'] ); diff --git a/tests/phpunit/tests/theme/themeDir.php b/tests/phpunit/tests/theme/themeDir.php index 365d225f11..82914c26ad 100644 --- a/tests/phpunit/tests/theme/themeDir.php +++ b/tests/phpunit/tests/theme/themeDir.php @@ -7,14 +7,27 @@ */ class Tests_Theme_ThemeDir extends WP_UnitTestCase { + /** + * Theme root directory. + * + * @var string + */ + const THEME_ROOT = DIR_TESTDATA . '/themedir1'; + + /** + * Original theme directory. + * + * @var string + */ + private $orig_theme_dir; + public function set_up() { parent::set_up(); - $this->theme_root = DIR_TESTDATA . '/themedir1'; $this->orig_theme_dir = $GLOBALS['wp_theme_directories']; // /themes is necessary as theme.php functions assume /themes is the root if there is only one root. - $GLOBALS['wp_theme_directories'] = array( WP_CONTENT_DIR . '/themes', $this->theme_root ); + $GLOBALS['wp_theme_directories'] = array( WP_CONTENT_DIR . '/themes', self::THEME_ROOT ); add_filter( 'theme_root', array( $this, 'filter_theme_root' ) ); add_filter( 'stylesheet_root', array( $this, 'filter_theme_root' ) ); @@ -33,7 +46,7 @@ class Tests_Theme_ThemeDir extends WP_UnitTestCase { // Replace the normal theme root directory with our premade test directory. public function filter_theme_root( $dir ) { - return $this->theme_root; + return self::THEME_ROOT; } /** @@ -57,12 +70,12 @@ class Tests_Theme_ThemeDir extends WP_UnitTestCase { $this->assertSame( 'default', $theme['Template'] ); $this->assertSame( 'default', $theme['Stylesheet'] ); - $this->assertContains( $this->theme_root . '/default/functions.php', $theme['Template Files'] ); - $this->assertContains( $this->theme_root . '/default/index.php', $theme['Template Files'] ); - $this->assertContains( $this->theme_root . '/default/style.css', $theme['Stylesheet Files'] ); + $this->assertContains( self::THEME_ROOT . '/default/functions.php', $theme['Template Files'] ); + $this->assertContains( self::THEME_ROOT . '/default/index.php', $theme['Template Files'] ); + $this->assertContains( self::THEME_ROOT . '/default/style.css', $theme['Stylesheet Files'] ); - $this->assertSame( $this->theme_root . '/default', $theme['Template Dir'] ); - $this->assertSame( $this->theme_root . '/default', $theme['Stylesheet Dir'] ); + $this->assertSame( self::THEME_ROOT . '/default', $theme['Template Dir'] ); + $this->assertSame( self::THEME_ROOT . '/default', $theme['Stylesheet Dir'] ); $this->assertSame( 'publish', $theme['Status'] ); $this->assertSame( '', $theme['Parent Theme'] ); } @@ -88,15 +101,15 @@ class Tests_Theme_ThemeDir extends WP_UnitTestCase { $template_files = $theme['Template Files']; - $this->assertSame( $this->theme_root . '/sandbox/functions.php', reset( $template_files ) ); - $this->assertSame( $this->theme_root . '/sandbox/index.php', next( $template_files ) ); + $this->assertSame( self::THEME_ROOT . '/sandbox/functions.php', reset( $template_files ) ); + $this->assertSame( self::THEME_ROOT . '/sandbox/index.php', next( $template_files ) ); $stylesheet_files = $theme['Stylesheet Files']; - $this->assertSame( $this->theme_root . '/sandbox/style.css', reset( $stylesheet_files ) ); + $this->assertSame( self::THEME_ROOT . '/sandbox/style.css', reset( $stylesheet_files ) ); - $this->assertSame( $this->theme_root . '/sandbox', $theme['Template Dir'] ); - $this->assertSame( $this->theme_root . '/sandbox', $theme['Stylesheet Dir'] ); + $this->assertSame( self::THEME_ROOT . '/sandbox', $theme['Template Dir'] ); + $this->assertSame( self::THEME_ROOT . '/sandbox', $theme['Stylesheet Dir'] ); $this->assertSame( 'publish', $theme['Status'] ); $this->assertSame( '', $theme['Parent Theme'] ); @@ -122,13 +135,13 @@ class Tests_Theme_ThemeDir extends WP_UnitTestCase { $this->assertSame( '1.0', $theme['Version'] ); $this->assertSame( 'sandbox', $theme['Template'] ); $this->assertSame( 'stylesheetonly', $theme['Stylesheet'] ); - $this->assertContains( $this->theme_root . '/sandbox/functions.php', $theme['Template Files'] ); - $this->assertContains( $this->theme_root . '/sandbox/index.php', $theme['Template Files'] ); + $this->assertContains( self::THEME_ROOT . '/sandbox/functions.php', $theme['Template Files'] ); + $this->assertContains( self::THEME_ROOT . '/sandbox/index.php', $theme['Template Files'] ); - $this->assertContains( $this->theme_root . '/stylesheetonly/style.css', $theme['Stylesheet Files'] ); + $this->assertContains( self::THEME_ROOT . '/stylesheetonly/style.css', $theme['Stylesheet Files'] ); - $this->assertSame( $this->theme_root . '/sandbox', $theme['Template Dir'] ); - $this->assertSame( $this->theme_root . '/stylesheetonly', $theme['Stylesheet Dir'] ); + $this->assertSame( self::THEME_ROOT . '/sandbox', $theme['Template Dir'] ); + $this->assertSame( self::THEME_ROOT . '/stylesheetonly', $theme['Stylesheet Dir'] ); $this->assertSame( 'publish', $theme['Status'] ); $this->assertSame( 'Sandbox', $theme['Parent Theme'] ); @@ -142,7 +155,7 @@ class Tests_Theme_ThemeDir extends WP_UnitTestCase { // Ignore themes in the default /themes directory. foreach ( $themes as $theme_name => $theme ) { - if ( $theme->get_theme_root() !== $this->theme_root ) { + if ( $theme->get_theme_root() !== self::THEME_ROOT ) { unset( $themes[ $theme_name ] ); } } @@ -198,7 +211,7 @@ class Tests_Theme_ThemeDir extends WP_UnitTestCase { } public function test_wp_get_theme_with_non_default_theme_root() { - $this->assertFalse( wp_get_theme( 'sandbox', $this->theme_root )->errors() ); + $this->assertFalse( wp_get_theme( 'sandbox', self::THEME_ROOT )->errors() ); $this->assertFalse( wp_get_theme( 'sandbox' )->errors() ); } @@ -212,7 +225,7 @@ class Tests_Theme_ThemeDir extends WP_UnitTestCase { $this->assertNotEmpty( $theme ); $templates = $theme['Template Files']; - $this->assertContains( $this->theme_root . '/page-templates/template-top-level.php', $templates ); + $this->assertContains( self::THEME_ROOT . '/page-templates/template-top-level.php', $templates ); } /** @@ -238,7 +251,7 @@ class Tests_Theme_ThemeDir extends WP_UnitTestCase { * @expectedDeprecated get_theme_data */ public function test_get_theme_data_subdir() { - $theme_data = get_theme_data( $this->theme_root . '/subdir/theme2/style.css' ); + $theme_data = get_theme_data( self::THEME_ROOT . '/subdir/theme2/style.css' ); $this->assertSame( 'My Subdir Theme', $theme_data['Name'] ); $this->assertSame( 'http://example.org/', $theme_data['URI'] ); diff --git a/tests/phpunit/tests/theme/wpGetGlobalStylesheet.php b/tests/phpunit/tests/theme/wpGetGlobalStylesheet.php index b3bd7754bd..57d3140ea8 100644 --- a/tests/phpunit/tests/theme/wpGetGlobalStylesheet.php +++ b/tests/phpunit/tests/theme/wpGetGlobalStylesheet.php @@ -7,6 +7,20 @@ */ class Tests_Theme_wpGetGlobalStylesheet extends WP_UnitTestCase { + /** + * Theme root directory. + * + * @var string + */ + private $theme_root; + + /** + * Original theme directory. + * + * @var string + */ + private $orig_theme_dir; + public function set_up() { parent::set_up(); diff --git a/tests/phpunit/tests/theme/wpTheme.php b/tests/phpunit/tests/theme/wpTheme.php index dd3855bafc..8807c0b9e1 100644 --- a/tests/phpunit/tests/theme/wpTheme.php +++ b/tests/phpunit/tests/theme/wpTheme.php @@ -10,6 +10,20 @@ */ class Tests_Theme_wpTheme extends WP_UnitTestCase { + /** + * Theme root directory. + * + * @var string + */ + private $theme_root; + + /** + * Original theme directory. + * + * @var string + */ + private $orig_theme_dir; + public function set_up() { parent::set_up(); $this->theme_root = realpath( DIR_TESTDATA . '/themedir1' ); diff --git a/tests/phpunit/tests/theme/wpThemeJsonResolver.php b/tests/phpunit/tests/theme/wpThemeJsonResolver.php index 6c88666e38..287e5d8aeb 100644 --- a/tests/phpunit/tests/theme/wpThemeJsonResolver.php +++ b/tests/phpunit/tests/theme/wpThemeJsonResolver.php @@ -12,6 +12,27 @@ */ class Tests_Theme_wpThemeJsonResolver extends WP_UnitTestCase { + /** + * Theme root directory. + * + * @var string + */ + private $theme_root; + + /** + * Original theme directory. + * + * @var string + */ + private $orig_theme_dir; + + /** + * Queries. + * + * @var array + */ + private $queries = array(); + public function set_up() { parent::set_up(); $this->theme_root = realpath( DIR_TESTDATA . '/themedir1' );