WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (7 years ago)
Author:
pento
Message:

Code is Poetry.
WordPress' code just... wasn't.
This is now dealt with.

Props jrf, pento, netweb, GaryJ, jdgrimes, westonruter, Greg Sherwood from PHPCS, and everyone who's ever contributed to WPCS and PHPCS.
Fixes #41057.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/tests/phpunit/tests/dependencies.php

    r25711 r42343  
    88        $dep = new WP_Dependencies;
    99
    10         $this->assertTrue($dep->add( 'one', '' ));
    11         $this->assertTrue($dep->add( 'two', '' ));
     10        $this->assertTrue( $dep->add( 'one', '' ) );
     11        $this->assertTrue( $dep->add( 'two', '' ) );
    1212
    13         $this->assertInstanceOf('_WP_Dependency', $dep->query( 'one' ));
    14         $this->assertInstanceOf('_WP_Dependency', $dep->query( 'two' ));
     13        $this->assertInstanceOf( '_WP_Dependency', $dep->query( 'one' ) );
     14        $this->assertInstanceOf( '_WP_Dependency', $dep->query( 'two' ) );
    1515
    1616        //Cannot reuse names
    17         $this->assertFalse($dep->add( 'one', '' ));
     17        $this->assertFalse( $dep->add( 'one', '' ) );
    1818    }
    1919
    … …  
    2121        $dep = new WP_Dependencies;
    2222
    23         $this->assertTrue($dep->add( 'one', '' ));
    24         $this->assertTrue($dep->add( 'two', '' ));
     23        $this->assertTrue( $dep->add( 'one', '' ) );
     24        $this->assertTrue( $dep->add( 'two', '' ) );
    2525
    2626        $dep->remove( 'one' );
    2727
    28         $this->assertFalse($dep->query( 'one'));
    29         $this->assertInstanceOf('_WP_Dependency', $dep->query( 'two' ));
     28        $this->assertFalse( $dep->query( 'one' ) );
     29        $this->assertInstanceOf( '_WP_Dependency', $dep->query( 'two' ) );
    3030
    3131    }
    … …  
    3434        $dep = new WP_Dependencies;
    3535
    36         $this->assertTrue($dep->add( 'one', '' ));
    37         $this->assertTrue($dep->add( 'two', '' ));
     36        $this->assertTrue( $dep->add( 'one', '' ) );
     37        $this->assertTrue( $dep->add( 'two', '' ) );
    3838
    39         $this->assertFalse($dep->query( 'one', 'queue' ));
    40         $dep->enqueue('one');
    41         $this->assertTrue($dep->query( 'one', 'queue' ));
    42         $this->assertFalse($dep->query( 'two', 'queue' ));
     39        $this->assertFalse( $dep->query( 'one', 'queue' ) );
     40        $dep->enqueue( 'one' );
     41        $this->assertTrue( $dep->query( 'one', 'queue' ) );
     42        $this->assertFalse( $dep->query( 'two', 'queue' ) );
    4343
    44         $dep->enqueue('two');
    45         $this->assertTrue($dep->query( 'one', 'queue' ));
    46         $this->assertTrue($dep->query( 'two', 'queue' ));
     44        $dep->enqueue( 'two' );
     45        $this->assertTrue( $dep->query( 'one', 'queue' ) );
     46        $this->assertTrue( $dep->query( 'two', 'queue' ) );
    4747    }
    4848
    … …  
    5050        $dep = new WP_Dependencies;
    5151
    52         $this->assertTrue($dep->add( 'one', '' ));
    53         $this->assertTrue($dep->add( 'two', '' ));
     52        $this->assertTrue( $dep->add( 'one', '' ) );
     53        $this->assertTrue( $dep->add( 'two', '' ) );
    5454
    55         $dep->enqueue('one');
    56         $dep->enqueue('two');
    57         $this->assertTrue($dep->query( 'one', 'queue' ));
    58         $this->assertTrue($dep->query( 'two', 'queue' ));
     55        $dep->enqueue( 'one' );
     56        $dep->enqueue( 'two' );
     57        $this->assertTrue( $dep->query( 'one', 'queue' ) );
     58        $this->assertTrue( $dep->query( 'two', 'queue' ) );
    5959
    60         $dep->dequeue('one');
    61         $this->assertFalse($dep->query( 'one', 'queue' ));
    62         $this->assertTrue($dep->query( 'two', 'queue' ));
     60        $dep->dequeue( 'one' );
     61        $this->assertFalse( $dep->query( 'one', 'queue' ) );
     62        $this->assertTrue( $dep->query( 'two', 'queue' ) );
    6363
    64         $dep->dequeue('two');
    65         $this->assertFalse($dep->query( 'one', 'queue' ));
    66         $this->assertFalse($dep->query( 'two', 'queue' ));
     64        $dep->dequeue( 'two' );
     65        $this->assertFalse( $dep->query( 'one', 'queue' ) );
     66        $this->assertFalse( $dep->query( 'two', 'queue' ) );
    6767    }
    6868
    … …  
    7070        $dep = new WP_Dependencies;
    7171
    72         $this->assertTrue($dep->add( 'one', '' ));
    73         $this->assertTrue($dep->add( 'two', '' ));
     72        $this->assertTrue( $dep->add( 'one', '' ) );
     73        $this->assertTrue( $dep->add( 'two', '' ) );
    7474
    75         $this->assertFalse($dep->query( 'one', 'queue' ));
    76         $dep->enqueue('one?arg');
    77         $this->assertTrue($dep->query( 'one', 'queue' ));
    78         $this->assertFalse($dep->query( 'two', 'queue' ));
    79         $this->assertEquals('arg', $dep->args['one']);
     75        $this->assertFalse( $dep->query( 'one', 'queue' ) );
     76        $dep->enqueue( 'one?arg' );
     77        $this->assertTrue( $dep->query( 'one', 'queue' ) );
     78        $this->assertFalse( $dep->query( 'two', 'queue' ) );
     79        $this->assertEquals( 'arg', $dep->args['one'] );
    8080
    81         $dep->enqueue('two?arg');
    82         $this->assertTrue($dep->query( 'one', 'queue' ));
    83         $this->assertTrue($dep->query( 'two', 'queue' ));
    84         $this->assertEquals('arg', $dep->args['two']);
     81        $dep->enqueue( 'two?arg' );
     82        $this->assertTrue( $dep->query( 'one', 'queue' ) );
     83        $this->assertTrue( $dep->query( 'two', 'queue' ) );
     84        $this->assertEquals( 'arg', $dep->args['two'] );
    8585    }
    8686
    … …  
    8888        $dep = new WP_Dependencies;
    8989
    90         $this->assertTrue($dep->add( 'one', '' ));
    91         $this->assertTrue($dep->add( 'two', '' ));
     90        $this->assertTrue( $dep->add( 'one', '' ) );
     91        $this->assertTrue( $dep->add( 'two', '' ) );
    9292
    93         $dep->enqueue('one?arg');
    94         $dep->enqueue('two?arg');
    95         $this->assertTrue($dep->query( 'one', 'queue' ));
    96         $this->assertTrue($dep->query( 'two', 'queue' ));
    97         $this->assertEquals('arg', $dep->args['one']);
    98         $this->assertEquals('arg', $dep->args['two']);
     93        $dep->enqueue( 'one?arg' );
     94        $dep->enqueue( 'two?arg' );
     95        $this->assertTrue( $dep->query( 'one', 'queue' ) );
     96        $this->assertTrue( $dep->query( 'two', 'queue' ) );
     97        $this->assertEquals( 'arg', $dep->args['one'] );
     98        $this->assertEquals( 'arg', $dep->args['two'] );
    9999
    100         $dep->dequeue('one');
    101         $this->assertFalse($dep->query( 'one', 'queue' ));
    102         $this->assertTrue($dep->query( 'two', 'queue' ));
    103         $this->assertFalse(isset($dep->args['one']));
     100        $dep->dequeue( 'one' );
     101        $this->assertFalse( $dep->query( 'one', 'queue' ) );
     102        $this->assertTrue( $dep->query( 'two', 'queue' ) );
     103        $this->assertFalse( isset( $dep->args['one'] ) );
    104104
    105         $dep->dequeue('two');
    106         $this->assertFalse($dep->query( 'one', 'queue' ));
    107         $this->assertFalse($dep->query( 'two', 'queue' ));
    108         $this->assertFalse(isset($dep->args['two']));
     105        $dep->dequeue( 'two' );
     106        $this->assertFalse( $dep->query( 'one', 'queue' ) );
     107        $this->assertFalse( $dep->query( 'two', 'queue' ) );
     108        $this->assertFalse( isset( $dep->args['two'] ) );
    109109    }
    110110
Note: See TracChangeset for help on using the changeset viewer.