WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
10/09/2016 01:11:14 AM (10 years ago)
Author:
johnbillion
Message:

Build/Test Tools: Begin eliminating unnecessary randomness in tests.

Although unlikely, clashes in randomly generated strings could cause unexpected failures. In addition, most randomness is entirely unnecessary, is bad practice, and increases test time (however small it may be).

See #37371

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/tests/phpunit/tests/filters.php

    r38571 r38762  
    1010        function test_simple_filter() {
    1111                $a = new MockAction();
    12                 $tag = rand_str();
    13                 $val = rand_str();
     12                $tag = __FUNCTION__;
     13                $val = __FUNCTION__ . '_val';
    1414
    1515                add_filter($tag, array($a, 'filter'));
    … …  
    2828        function test_remove_filter() {
    2929                $a = new MockAction();
    30                 $tag = rand_str();
    31                 $val = rand_str();
     30                $tag = __FUNCTION__;
     31                $val = __FUNCTION__ . '_val';
    3232
    3333                add_filter($tag, array($a, 'filter'));
    … …  
    4747
    4848        function test_has_filter() {
    49                         $tag = rand_str();
    50                         $func = rand_str();
     49                        $tag  = __FUNCTION__;
     50                        $func = __FUNCTION__ . '_func';
    5151
    5252                        $this->assertFalse( has_filter($tag, $func) );
    … …  
    6464                $a1 = new MockAction();
    6565                $a2 = new MockAction();
    66                 $tag = rand_str();
    67                 $val = rand_str();
     66                $tag = __FUNCTION__;
     67                $val = __FUNCTION__ . '_val';
    6868
    6969                // add both filters to the hook
    … …  
    8080        function test_filter_args_1() {
    8181                $a = new MockAction();
    82                 $tag = rand_str();
    83                 $val = rand_str();
    84                 $arg1 = rand_str();
     82                $tag  = __FUNCTION__;
     83                $val  = __FUNCTION__ . '_val';
     84                $arg1 = __FUNCTION__ . '_arg1';
    8585
    8686                add_filter($tag, array($a, 'filter'), 10, 2);
    … …  
    9696                $a1 = new MockAction();
    9797                $a2 = new MockAction();
    98                 $tag = rand_str();
    99                 $val = rand_str();
    100                 $arg1 = rand_str();
    101                 $arg2 = rand_str();
     98                $tag  = __FUNCTION__;
     99                $val  = __FUNCTION__ . '_val';
     100                $arg1 = __FUNCTION__ . '_arg1';
     101                $arg2 = __FUNCTION__ . '_arg2';
    102102
    103103                // a1 accepts two arguments, a2 doesn't
    … …  
    120120        function test_filter_priority() {
    121121                $a = new MockAction();
    122                 $tag = rand_str();
    123                 $val = rand_str();
     122                $tag = __FUNCTION__;
     123                $val = __FUNCTION__ . '_val';
    124124
    125125                // make two filters with different priorities
    … …  
    151151        function test_all_filter() {
    152152                $a = new MockAction();
    153                 $tag1 = rand_str();
    154                 $tag2 = rand_str();
    155                 $val = rand_str();
     153                $tag1 = __FUNCTION__ . '_1';
     154                $tag2 = __FUNCTION__ . '_2';
     155                $val  = __FUNCTION__ . '_val';
    156156
    157157                // add an 'all' filter
    … …  
    175175        function test_remove_all_filter() {
    176176                $a = new MockAction();
    177                 $tag = rand_str();
    178                 $val = rand_str();
     177                $tag = __FUNCTION__;
     178                $val = __FUNCTION__ . '_val';
    179179
    180180                add_filter('all', array($a, 'filterall'));
    … …  
    202202        function test_remove_all_filters_should_respect_the_priority_argument() {
    203203                $a = new MockAction();
    204                 $tag = rand_str();
    205                 $val = rand_str();
     204                $tag = __FUNCTION__;
     205                $val = __FUNCTION__ . '_val';
    206206
    207207                add_filter( $tag, array( $a, 'filter' ), 12 );
    … …  
    222222                $obj = new stdClass();
    223223                $a = new MockAction();
    224                 $tag = rand_str();
     224                $tag = __FUNCTION__;
    225225
    226226                add_action($tag, array($a, 'filter'));
    … …  
    242242                $a = new MockAction();
    243243                $b = new MockAction();
    244                 $tag = rand_str();
     244                $tag = __FUNCTION__;
    245245
    246246                add_action($tag, array($a, 'filter_append'), 10, 2);
    … …  
    275275        function test_has_filter_after_remove_all_filters() {
    276276                $a = new MockAction();
    277                 $tag = rand_str();
    278                 $val = rand_str();
     277                $tag = __FUNCTION__;
     278                $val = __FUNCTION__ . '_val';
    279279
    280280                // No priority
Note: See TracChangeset for help on using the changeset viewer.