WordPress.org

Make WordPress Core

Changeset 327 in tests


Ignore:
Timestamp:
01/20/11 14:50:29 (4 years ago)
Author:
duck_
Message:

Fixes and cleanup in TestObjectCache and TestSSLLinks

TestObjectCache: null value is converted to empty string
TestSSLLinks: extra is_ssl checks; still todo - last invalid admin_url needs fixing

Location:
wp-testcase
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • wp-testcase/test_includes_cache.php

    r280 r327  
    1717 
    1818    function &init_cache() { 
    19         $cache = new WP_Object_cache(); 
     19        $cache = new WP_Object_Cache(); 
    2020        return $cache; 
    2121    } 
     
    4646        $val = null; 
    4747 
    48         // you can't store null in the cache 
    49         $this->assertFalse($this->cache->add($key, $val)); 
    50         $this->assertFalse($this->cache->get($key)); 
     48        $this->assertTrue( $this->cache->add($key, $val) ); 
     49        // null is converted to empty string 
     50        $this->assertEquals( '', $this->cache->get($key) ); 
    5151    } 
    5252 
     
    9393        $this->assertFalse($this->cache->get($key)); 
    9494    } 
    95  
    9695} 
    9796 
  • wp-testcase/test_link_functions.php

    r211 r327  
    22 
    33// tests for link-template.php and related URL functions 
    4  
    5  
    64class TestSSLLinks extends WPTestCase { 
    75    var $_old_server; 
     
    97        $this->_old_server = $_SERVER; 
    108    } 
    11      
     9 
    1210    function tearDown() { 
    1311        $_SERVER = $this->_old_server; 
    1412    } 
    15      
     13 
    1614    function test_is_ssl_positive() { 
    17         $this->knownWPBug(7001); 
    18          
    1915        $_SERVER['HTTPS'] = 'on'; 
    2016        $this->assertTrue( is_ssl() ); 
    21          
     17 
    2218        $_SERVER['HTTPS'] = 'ON'; 
    2319        $this->assertTrue( is_ssl() ); 
     20 
     21        $_SERVER['HTTPS'] = '1'; 
     22        $this->assertTrue( is_ssl() ); 
     23 
     24        unset( $_SERVER['HTTPS'] ); 
     25        $_SERVER['SERVER_PORT'] = '443'; 
     26        $this->assertTrue( is_ssl() ); 
    2427    } 
    25      
     28 
    2629    function test_is_ssl_negative() { 
    27         $this->knownWPBug(7001); 
    28  
    2930        $_SERVER['HTTPS'] = 'off'; 
    3031        $this->assertFalse( is_ssl() ); 
    31          
     32 
    3233        $_SERVER['HTTPS'] = 'OFF'; 
    3334        $this->assertFalse( is_ssl() ); 
    34          
     35 
    3536        unset($_SERVER['HTTPS']); 
    3637        $this->assertFalse( is_ssl() ); 
    3738    } 
    38      
     39 
    3940    function test_admin_url_valid() { 
    40         $this->knownWPBug(7001); 
    41          
    42         $siteurl = get_option('siteurl'); 
    43          
    4441        $paths = array( 
    45                 '' => "/wp-admin/", 
    46                 'foo' => "/wp-admin/foo", 
    47                 '/foo' => "/wp-admin/foo", 
    48                 '/foo/' => "/wp-admin/foo/", 
    49                 'foo.php' => "/wp-admin/foo.php", 
    50                 '/foo.php' => "/wp-admin/foo.php", 
    51                 '/foo.php?bar=1' => "/wp-admin/foo.php?bar=1", 
    52             ); 
     42            '' => "/wp-admin/", 
     43            'foo' => "/wp-admin/foo", 
     44            '/foo' => "/wp-admin/foo", 
     45            '/foo/' => "/wp-admin/foo/", 
     46            'foo.php' => "/wp-admin/foo.php", 
     47            '/foo.php' => "/wp-admin/foo.php", 
     48            '/foo.php?bar=1' => "/wp-admin/foo.php?bar=1", 
     49        ); 
    5350        $https = array('on', 'off'); 
    54          
     51 
    5552        foreach ($https as $val) { 
    5653            $_SERVER['HTTPS'] = $val; 
     
    5855            if ( $val == 'on' ) 
    5956                $siteurl = str_replace('http://', 'https://', $siteurl); 
    60                  
     57 
    6158            foreach ($paths as $in => $out) { 
    6259                $this->assertEquals( $siteurl.$out, admin_url($in), "admin_url('{$in}') should equal '{$siteurl}{$out}'"); 
     
    6663 
    6764    function test_admin_url_invalid() { 
    68         $this->knownWPBug(7001); 
    69          
    70         $siteurl = get_option('siteurl'); 
    71          
    7265        $paths = array( 
    73                 null => "/wp-admin/", 
    74                 0 => "/wp-admin/", 
    75                 -1 => "/wp-admin/", 
    76                 '../foo/' => "/wp-admin/", 
    77                 ' ' => "/wp-admin/", 
    78             ); 
     66            null => "/wp-admin/", 
     67            0 => "/wp-admin/", 
     68            -1 => "/wp-admin/", 
     69            '../foo/' => "/wp-admin/", 
     70            ' ' => "/wp-admin/", 
     71        ); 
    7972        $https = array('on', 'off'); 
    80          
     73 
    8174        foreach ($https as $val) { 
    8275            $_SERVER['HTTPS'] = $val; 
     
    8477            if ( $val == 'on' ) 
    8578                $siteurl = str_replace('http://', 'https://', $siteurl); 
    86                  
     79 
    8780            foreach ($paths as $in => $out) { 
    8881                $this->assertEquals( $siteurl.$out, admin_url($in), "admin_url('{$in}') should equal '{$siteurl}{$out}'"); 
Note: See TracChangeset for help on using the changeset viewer.