Changeset 814 in tests

Timestamp:

06/30/2012 09:07:53 PM (14 years ago)

Author:

nacin

Message:

Fix TestThemeSupport tests. see #92.

We can't say for sure what the current theme's support is. So add/remove/add to make sure we can test both.

Add some grouping. see #56, #92, #72.

Location:

trunk/wp-testcase

Files:

• test_admin_includes_theme.php (modified) (1 diff)
• test_includes_class-wp-theme.php (modified) (1 diff)
• test_includes_theme.php (modified) (6 diffs)

trunk/wp-testcase/test_admin_includes_theme.php

@@ -1 +1 @@
 <?php
-// Test functions that fetch stuff from the theme directory
+/**
+ * @group themes
+ */
 class TestPageTemplates extends WP_UnitTestCase {
         function setUp() {

trunk/wp-testcase/test_includes_class-wp-theme.php

@@ -1 +1 @@
 <?php
+/**
+ * @group themes
+ */
 class Test_WP_Theme extends WP_UnitTestCase {
         function setUp() {

trunk/wp-testcase/test_includes_theme.php

@@ -1 +1 @@
 <?php
 
-// test wp-includes/theme.php
+/**
+ * test wp-includes/theme.php
+ *
+ * @group themes
+ */
 class TestDefaultThemes extends WP_UnitTestCase {
 
@@ -236 +240 @@
 }
 
-// Test functions that fetch stuff from the theme directory
+/**
+ * Test functions that fetch stuff from the theme directory
+ *
+ * @group themes
+ */
 class TestThemeDir extends WP_UnitTestCase {
         function setUp() {
@@ -431 +439 @@
 }
 
+/**
+ * @group themes
+ */
 class TestLargeThemeDir extends WP_UnitTestCase {
         function setUp() {
@@ -492 +503 @@
 }
 
+/**
+ * @group themes
+ */
 class TestThemeSupport extends WP_UnitTestCase {
         function setUp() {
@@ -502 +516 @@
 
         function test_the_basics() {
-                $this->assertFalse( current_theme_supports( 'automatic-feed-links' ) );
                 add_theme_support( 'automatic-feed-links' );
                 $this->assertTrue( current_theme_supports( 'automatic-feed-links' ) );
                 remove_theme_support( 'automatic-feed-links' );
                 $this->assertFalse( current_theme_supports( 'automatic-feed-links' ) );
+                add_theme_support( 'automatic-feed-links' );
+                $this->assertTrue( current_theme_supports( 'automatic-feed-links' ) );
         }
 
         function test_admin_bar() {
-                $this->assertFalse( current_theme_supports( 'admin-bar' ) );
                 add_theme_support( 'admin-bar' );
                 $this->assertTrue( current_theme_supports( 'admin-bar' ) );
                 remove_theme_support( 'admin-bar' );
                 $this->assertFalse( current_theme_supports( 'admin-bar' ) );
+                add_theme_support( 'admin-bar' );
+                $this->assertTrue( current_theme_supports( 'admin-bar' ) );
 
                 add_theme_support( 'admin-bar', array( 'callback' => '__return_false' ) );
@@ -529 +545 @@
 
         function test_post_thumbnails() {
-                $this->assertFalse( current_theme_supports( 'post-thumbnails' ) );
                 add_theme_support( 'post-thumbnails' );
                 $this->assertTrue( current_theme_supports( 'post-thumbnails' ) );
                 remove_theme_support( 'post-thumbnails' );
                 $this->assertFalse( current_theme_supports( 'post-thumbnails' ) );
+                add_theme_support( 'post-thumbnails' );
+                $this->assertTrue( current_theme_supports( 'post-thumbnails' ) );
 
                 // simple array of post types.