WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
11/04/2021 03:22:47 PM (3 years ago)
Author:
hellofromTonya
Message:

Coding Standards: Add visibility to methods in tests/phpunit/tests/.

Adds a public visibility to test fixtures, tests, data providers, and callbacks methods.

Adds a private visibility to helper methods within test classes.

Renames callbacks and helpers that previously started with a _ prefix. Why? For consistency and to leverage using the method visibility. Further naming standardizations is beyond the scope of this commit.

Props costdev, jrf, hellofromTonya.
Fixes #54177.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/tests/phpunit/tests/formatting/wpTexturize.php

    r51623 r52010  
    55 */
    66class Tests_Formatting_wpTexturize extends WP_UnitTestCase {
    7     function test_dashes() {
     7    public function test_dashes() {
    88        $this->assertSame( 'Hey — boo?', wptexturize( 'Hey -- boo?' ) );
    99        $this->assertSame( '<a href="http://xx--xx">Hey &#8212; boo?</a>', wptexturize( '<a href="http://xx--xx">Hey -- boo?</a>' ) );
    1010    }
    1111
    12     function test_disable() {
     12    public function test_disable() {
    1313        $this->assertSame( '<pre>---&</pre>', wptexturize( '<pre>---&</pre>' ) );
    1414        $this->assertSame( '<pre><code></code>--&</pre>', wptexturize( '<pre><code></code>--&</pre>' ) );
    … …  
    3636     * @ticket 1418
    3737     */
    38     function test_bracketed_quotes_1418() {
     38    public function test_bracketed_quotes_1418() {
    3939        $this->assertSame( '(&#8220;test&#8221;)', wptexturize( '("test")' ) );
    4040        $this->assertSame( '(&#8216;test&#8217;)', wptexturize( "('test')" ) );
    … …  
    4545     * @ticket 3810
    4646     */
    47     function test_bracketed_quotes_3810() {
     47    public function test_bracketed_quotes_3810() {
    4848        $this->assertSame( 'A dog (&#8220;Hubertus&#8221;) was sent out.', wptexturize( 'A dog ("Hubertus") was sent out.' ) );
    4949    }
    … …  
    5252     * @ticket 4539
    5353     */
    54     function test_basic_quotes() {
     54    public function test_basic_quotes() {
    5555        $this->assertSame( 'test&#8217;s', wptexturize( 'test\'s' ) );
    5656
    … …  
    7575     * @ticket 15241
    7676     */
    77     function test_full_sentences_with_unmatched_single_quotes() {
     77    public function test_full_sentences_with_unmatched_single_quotes() {
    7878        $this->assertSame(
    7979            'That means every moment you&#8217;re working on something without it being in the public it&#8217;s actually dying.',
    … …  
    8585     * @ticket 4539
    8686     */
    87     function test_quotes() {
     87    public function test_quotes() {
    8888        $this->assertSame( '&#8220;Quoted String&#8221;', wptexturize( '"Quoted String"' ) );
    8989        // $this->assertSame( 'Here is &#8220;<a href="http://example.com">a test with a link</a>&#8221;', wptexturize( 'Here is "<a href="http://example.com">a test with a link</a>"' ) );
    … …  
    104104     * @ticket 4539
    105105     */
    106     function test_quotes_before_s() {
     106    public function test_quotes_before_s() {
    107107        $this->assertSame( 'test&#8217;s', wptexturize( "test's" ) );
    108108        $this->assertSame( '&#8216;test&#8217;s', wptexturize( "'test's" ) );
    … …  
    115115     * @ticket 4539
    116116     */
    117     function test_quotes_before_numbers() {
     117    public function test_quotes_before_numbers() {
    118118        $this->assertSame( 'Class of &#8217;99', wptexturize( "Class of '99" ) );
    119119        $this->assertSame( 'Class of &#8217;99&#8217;s', wptexturize( "Class of '99's" ) );
    … …  
    142142    }
    143143
    144     function test_quotes_after_numbers() {
     144    public function test_quotes_after_numbers() {
    145145        $this->assertSame( 'Class of &#8217;99', wptexturize( "Class of '99" ) );
    146146    }
    … …  
    150150     * @ticket 15241
    151151     */
    152     function test_other_html() {
     152    public function test_other_html() {
    153153        $this->assertSame( '&#8216;<strong>', wptexturize( "'<strong>" ) );
    154154        // $this->assertSame( '&#8216;<strong>Quoted Text</strong>&#8217;,', wptexturize( "'<strong>Quoted Text</strong>'," ) );
    … …  
    156156    }
    157157
    158     function test_x() {
     158    public function test_x() {
    159159        $this->assertSame( '14&#215;24', wptexturize( '14x24' ) );
    160160    }
    161161
    162     function test_minutes_seconds() {
     162    public function test_minutes_seconds() {
    163163        $this->assertSame( '9&#8242;', wptexturize( '9\'' ) );
    164164        $this->assertSame( '9&#8243;', wptexturize( '9"' ) );
    … …  
    174174     * @ticket 8775
    175175     */
    176     function test_wptexturize_quotes_around_numbers() {
     176    public function test_wptexturize_quotes_around_numbers() {
    177177        $this->assertSame( '&#8220;12345&#8221;', wptexturize( '"12345"' ) );
    178178        $this->assertSame( '&#8216;12345&#8217;', wptexturize( '\'12345\'' ) );
    … …  
    184184     * @ticket 8912
    185185     */
    186     function test_wptexturize_html_comments() {
     186    public function test_wptexturize_html_comments() {
    187187        $this->assertSame( '<!--[if !IE]>--><!--<![endif]-->', wptexturize( '<!--[if !IE]>--><!--<![endif]-->' ) );
    188188        $this->assertSame( '<!--[if !IE]>"a 9\' plus a \'9\', maybe a 9\' \'9\' "<![endif]-->', wptexturize( '<!--[if !IE]>"a 9\' plus a \'9\', maybe a 9\' \'9\' "<![endif]-->' ) );
    … …  
    194194     * @ticket 15241
    195195     */
    196     function test_entity_quote_cuddling() {
     196    public function test_entity_quote_cuddling() {
    197197        $this->assertSame( '&nbsp;&#8220;Testing&#8221;', wptexturize( '&nbsp;"Testing"' ) );
    198198        // $this->assertSame( '&#38;&#8220;Testing&#8221;', wptexturize( '&#38;"Testing"' ) );
    … …  
    202202     * @ticket 22823
    203203     */
    204     function test_apostrophes_before_primes() {
     204    public function test_apostrophes_before_primes() {
    205205        $this->assertSame( 'WordPress 3.5&#8217;s release date', wptexturize( "WordPress 3.5's release date" ) );
    206206    }
    … …  
    209209     * @ticket 23185
    210210     */
    211     function test_spaces_around_hyphens() {
     211    public function test_spaces_around_hyphens() {
    212212        $nbsp = "\xC2\xA0";
    213213
    … …  
    232232     * @ticket 31030
    233233     */
    234     function test_hyphens_at_start_and_end() {
     234    public function test_hyphens_at_start_and_end() {
    235235        $this->assertSame( '&#8211; ', wptexturize( '- ' ) );
    236236        $this->assertSame( '&#8211; &#8211;', wptexturize( '- -' ) );
    … …  
    249249     * @ticket 22692
    250250     */
    251     function test_spaces_around_quotes_never() {
     251    public function test_spaces_around_quotes_never() {
    252252        $nbsp = "\xC2\xA0";
    253253
    … …  
    266266     * @dataProvider data_spaces_around_quotes
    267267     */
    268     function test_spaces_around_quotes( $input, $output ) {
     268    public function test_spaces_around_quotes( $input, $output ) {
    269269        return $this->assertSame( $output, wptexturize( $input ) );
    270270    }
    271271
    272     function data_spaces_around_quotes() {
     272    public function data_spaces_around_quotes() {
    273273        $nbsp = "\xC2\xA0";
    274274        $pi   = "\xCE\xA0";
    … …  
    322322     * @dataProvider data_apos_before_digits
    323323     */
    324     function test_apos_before_digits( $input, $output ) {
     324    public function test_apos_before_digits( $input, $output ) {
    325325        return $this->assertSame( $output, wptexturize( $input ) );
    326326    }
    327327
    328     function data_apos_before_digits() {
     328    public function data_apos_before_digits() {
    329329        return array(
    330330            array(
    … …  
    363363     * @dataProvider data_opening_single_quote
    364364     */
    365     function test_opening_single_quote( $input, $output ) {
     365    public function test_opening_single_quote( $input, $output ) {
    366366        return $this->assertSame( $output, wptexturize( $input ) );
    367367    }
    368368
    369     function data_opening_single_quote() {
     369    public function data_opening_single_quote() {
    370370        return array(
    371371            array(
    … …  
    492492     * @dataProvider data_double_prime
    493493     */
    494     function test_double_prime( $input, $output ) {
     494    public function test_double_prime( $input, $output ) {
    495495        return $this->assertSame( $output, wptexturize( $input ) );
    496496    }
    497497
    498     function data_double_prime() {
     498    public function data_double_prime() {
    499499        return array(
    500500            array(
    … …  
    525525     * @dataProvider data_single_prime
    526526     */
    527     function test_single_prime( $input, $output ) {
     527    public function test_single_prime( $input, $output ) {
    528528        return $this->assertSame( $output, wptexturize( $input ) );
    529529    }
    530530
    531     function data_single_prime() {
     531    public function data_single_prime() {
    532532        return array(
    533533            array(
    … …  
    558558     * @dataProvider data_contractions
    559559     */
    560     function test_contractions( $input, $output ) {
     560    public function test_contractions( $input, $output ) {
    561561        return $this->assertSame( $output, wptexturize( $input ) );
    562562    }
    563563
    564     function data_contractions() {
     564    public function data_contractions() {
    565565        return array(
    566566            array(
    … …  
    599599     * @dataProvider data_opening_quote
    600600     */
    601     function test_opening_quote( $input, $output ) {
     601    public function test_opening_quote( $input, $output ) {
    602602        return $this->assertSame( $output, wptexturize( $input ) );
    603603    }
    604604
    605     function data_opening_quote() {
     605    public function data_opening_quote() {
    606606        return array(
    607607            array(
    … …  
    676676     * @dataProvider data_closing_quote
    677677     */
    678     function test_closing_quote( $input, $output ) {
     678    public function test_closing_quote( $input, $output ) {
    679679        return $this->assertSame( $output, wptexturize( $input ) );
    680680    }
    681681
    682     function data_closing_quote() {
     682    public function data_closing_quote() {
    683683        return array(
    684684            array(
    … …  
    765765     * @dataProvider data_closing_single_quote
    766766     */
    767     function test_closing_single_quote( $input, $output ) {
     767    public function test_closing_single_quote( $input, $output ) {
    768768        return $this->assertSame( $output, wptexturize( $input ) );
    769769    }
    770770
    771     function data_closing_single_quote() {
     771    public function data_closing_single_quote() {
    772772        return array(
    773773            array(
    … …  
    855855     * @dataProvider data_multiplication
    856856     */
    857     function test_multiplication( $input, $output ) {
     857    public function test_multiplication( $input, $output ) {
    858858        return $this->assertSame( $output, wptexturize( $input ) );
    859859    }
    860860
    861     function data_multiplication() {
     861    public function data_multiplication() {
    862862        return array(
    863863            array(
    … …  
    905905     * @dataProvider data_ampersand
    906906     */
    907     function test_ampersand( $input, $output ) {
     907    public function test_ampersand( $input, $output ) {
    908908        return $this->assertSame( $output, wptexturize( $input ) );
    909909    }
    910910
    911     function data_ampersand() {
     911    public function data_ampersand() {
    912912        return array(
    913913            array(
    … …  
    970970     * @dataProvider data_cockney
    971971     */
    972     function test_cockney( $input, $output ) {
     972    public function test_cockney( $input, $output ) {
    973973        return $this->assertSame( $output, wptexturize( $input ) );
    974974    }
    975975
    976     function data_cockney() {
     976    public function data_cockney() {
    977977        return array(
    978978            array(
    … …  
    10311031     * @dataProvider data_smart_dashes
    10321032     */
    1033     function test_smart_dashes( $input, $output ) {
     1033    public function test_smart_dashes( $input, $output ) {
    10341034        return $this->assertSame( $output, wptexturize( $input ) );
    10351035    }
    10361036
    1037     function data_smart_dashes() {
     1037    public function data_smart_dashes() {
    10381038        return array(
    10391039            array(
    … …  
    10841084     * @dataProvider data_misc_static_replacements
    10851085     */
    1086     function test_misc_static_replacements( $input, $output ) {
     1086    public function test_misc_static_replacements( $input, $output ) {
    10871087        return $this->assertSame( $output, wptexturize( $input ) );
    10881088    }
    10891089
    1090     function data_misc_static_replacements() {
     1090    public function data_misc_static_replacements() {
    10911091        return array(
    10921092            array(
    … …  
    11391139     * @dataProvider data_quoted_numbers
    11401140     */
    1141     function test_quoted_numbers( $input, $output ) {
     1141    public function test_quoted_numbers( $input, $output ) {
    11421142        return $this->assertSame( $output, wptexturize( $input ) );
    11431143    }
    11441144
    1145     function data_quoted_numbers() {
     1145    public function data_quoted_numbers() {
    11461146        return array(
    11471147            array(
    … …  
    11901190     * @dataProvider data_quotes_and_dashes
    11911191     */
    1192     function test_quotes_and_dashes( $input, $output ) {
     1192    public function test_quotes_and_dashes( $input, $output ) {
    11931193        return $this->assertSame( $output, wptexturize( $input ) );
    11941194    }
    11951195
    1196     function data_quotes_and_dashes() {
     1196    public function data_quotes_and_dashes() {
    11971197        return array(
    11981198            array(
    … …  
    12531253     * @dataProvider data_tag_avoidance
    12541254     */
    1255     function test_tag_avoidance( $input, $output ) {
     1255    public function test_tag_avoidance( $input, $output ) {
    12561256        return $this->assertSame( $output, wptexturize( $input ) );
    12571257    }
    12581258
    1259     function data_tag_avoidance() {
     1259    public function data_tag_avoidance() {
    12601260        return array(
    12611261            array(
    … …  
    14761476     * @dataProvider data_year_abbr
    14771477     */
    1478     function test_year_abbr( $input, $output ) {
     1478    public function test_year_abbr( $input, $output ) {
    14791479        return $this->assertSame( $output, wptexturize( $input ) );
    14801480    }
    14811481
    1482     function data_year_abbr() {
     1482    public function data_year_abbr() {
    14831483        return array(
    14841484            array(
    … …  
    15651565     * @dataProvider data_translate
    15661566     */
    1567     function test_translate( $input, $output ) {
     1567    public function test_translate( $input, $output ) {
    15681568        add_filter( 'gettext_with_context', array( $this, 'filter_translate' ), 10, 4 );
    15691569
    … …  
    15761576    }
    15771577
    1578     function filter_translate( $translations, $text, $context, $domain ) {
     1578    public function filter_translate( $translations, $text, $context, $domain ) {
    15791579        switch ( $text ) {
    15801580            case '&#8211;':
    … …  
    16051605    }
    16061606
    1607     function data_translate() {
     1607    public function data_translate() {
    16081608        return array(
    16091609            array(
    … …  
    17921792     * @dataProvider data_element_stack
    17931793     */
    1794     function test_element_stack( $input, $output ) {
     1794    public function test_element_stack( $input, $output ) {
    17951795        return $this->assertSame( $output, wptexturize( $input ) );
    17961796    }
    17971797
    1798     function data_element_stack() {
     1798    public function data_element_stack() {
    17991799        return array(
    18001800            array(
    … …  
    18431843     * @dataProvider data_unregistered_shortcodes
    18441844     */
    1845     function test_unregistered_shortcodes( $input, $output ) {
     1845    public function test_unregistered_shortcodes( $input, $output ) {
    18461846        add_filter( 'no_texturize_shortcodes', array( $this, 'filter_shortcodes' ), 10, 1 );
    18471847
    … …  
    18521852    }
    18531853
    1854     function filter_shortcodes( $disabled ) {
     1854    public function filter_shortcodes( $disabled ) {
    18551855        $disabled[] = 'audio';
    18561856        return $disabled;
    18571857    }
    18581858
    1859     function data_unregistered_shortcodes() {
     1859    public function data_unregistered_shortcodes() {
    18601860        return array(
    18611861            array(
    … …  
    19281928     * @dataProvider data_primes_vs_quotes
    19291929     */
    1930     function test_primes_vs_quotes( $input, $output ) {
     1930    public function test_primes_vs_quotes( $input, $output ) {
    19311931        return $this->assertSame( $output, wptexturize( $input ) );
    19321932    }
    19331933
    1934     function data_primes_vs_quotes() {
     1934    public function data_primes_vs_quotes() {
    19351935        return array(
    19361936            array(
    … …  
    19901990     * @dataProvider data_primes_quotes_translation
    19911991     */
    1992     function test_primes_quotes_translation( $input, $output ) {
     1992    public function test_primes_quotes_translation( $input, $output ) {
    19931993        add_filter( 'gettext_with_context', array( $this, 'filter_translate2' ), 10, 4 );
    19941994
    … …  
    20012001    }
    20022002
    2003     function filter_translate2( $translations, $text, $context, $domain ) {
     2003    public function filter_translate2( $translations, $text, $context, $domain ) {
    20042004        switch ( $text ) {
    20052005            case '&#8211;':
    … …  
    20282028    }
    20292029
    2030     function data_primes_quotes_translation() {
     2030    public function data_primes_quotes_translation() {
    20312031        return array(
    20322032            array(
    … …  
    20832083     * @dataProvider data_whole_posts
    20842084     */
    2085     function test_pcre_performance( $input ) {
     2085    public function test_pcre_performance( $input ) {
    20862086        global $shortcode_tags;
    20872087
    … …  
    21032103     * @ticket 35864
    21042104     */
    2105     function test_trailing_less_than() {
     2105    public function test_trailing_less_than() {
    21062106        $this->assertSame( 'F&#8211;oo<', wptexturize( 'F--oo<', true ) );
    21072107    }
    21082108
    2109     function data_whole_posts() {
     2109    public function data_whole_posts() {
    21102110        require_once DIR_TESTDATA . '/formatting/whole-posts.php';
    21112111        return data_whole_posts();
Note: See TracChangeset for help on using the changeset viewer.