WordPress.org

Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (3 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.