WordPress.org

Make WordPress Core

Changeset 327 in tests


Ignore:
Timestamp:
01/20/2011 02:50:29 PM (7 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.