Completed
Push — master ( 89a848...b1dea0 )
by frank
01:40
created
tests/test-ao.php 1 patch
Spacing   +414 added lines, -414 removed lines patch added patch discarded remove patch
@@ -10,17 +10,17 @@  discard block
 block discarded – undo
10 10
     protected $ao;
11 11
 
12 12
     protected static $flexible_url_parts_js = [
13
-        'default'          => 'wp-content/cache/autoptimize/js/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
14
-        'custom'           => 'wp-content/c/ao/js/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
15
-        'multisite'        => 'wp-content/cache/autoptimize/1/js/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
16
-        'multisite_custom' => 'wp-content/c/ao/1/js/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
13
+        'default'          => 'wp-content/cache/autoptimize/js/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
14
+        'custom'           => 'wp-content/c/ao/js/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
15
+        'multisite'        => 'wp-content/cache/autoptimize/1/js/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
16
+        'multisite_custom' => 'wp-content/c/ao/1/js/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
17 17
     ];
18 18
 
19 19
     protected static $flexible_url_parts_css = [
20
-        'default'          => 'wp-content/cache/autoptimize/css/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
21
-        'custom'           => 'wp-content/c/ao/css/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
22
-        'multisite'        => 'wp-content/cache/autoptimize/1/css/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
23
-        'multisite_custom' => 'wp-content/c/ao/1/css/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
20
+        'default'          => 'wp-content/cache/autoptimize/css/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
21
+        'custom'           => 'wp-content/c/ao/css/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
22
+        'multisite'        => 'wp-content/cache/autoptimize/1/css/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
23
+        'multisite_custom' => 'wp-content/c/ao/1/css/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
24 24
     ];
25 25
 
26 26
     protected function getAoStylesDefaultOptions()
@@ -28,17 +28,17 @@  discard block
 block discarded – undo
28 28
         $conf = autoptimizeConfig::instance();
29 29
 
30 30
         return [
31
-            'aggregate'       => $conf->get( 'autoptimize_css_aggregate' ),
32
-            'justhead'        => $conf->get( 'autoptimize_css_justhead' ),
33
-            'datauris'        => $conf->get( 'autoptimize_css_datauris' ),
34
-            'defer'           => $conf->get( 'autoptimize_css_defer' ),
35
-            'defer_inline'    => $conf->get( 'autoptimize_css_defer_inline' ),
36
-            'inline'          => $conf->get( 'autoptimize_css_inline' ),
37
-            'css_exclude'     => $conf->get( 'autoptimize_css_exclude' ),
38
-            'cdn_url'         => $conf->get( 'autoptimize_cdn_url' ),
39
-            'include_inline'  => $conf->get( 'autoptimize_css_include_inline' ),
40
-            'nogooglefont'    => $conf->get( 'autoptimize_css_nogooglefont' ),
41
-            'minify_excluded' => $conf->get( 'autoptimize_minify_excluded' ),
31
+            'aggregate'       => $conf->get('autoptimize_css_aggregate'),
32
+            'justhead'        => $conf->get('autoptimize_css_justhead'),
33
+            'datauris'        => $conf->get('autoptimize_css_datauris'),
34
+            'defer'           => $conf->get('autoptimize_css_defer'),
35
+            'defer_inline'    => $conf->get('autoptimize_css_defer_inline'),
36
+            'inline'          => $conf->get('autoptimize_css_inline'),
37
+            'css_exclude'     => $conf->get('autoptimize_css_exclude'),
38
+            'cdn_url'         => $conf->get('autoptimize_cdn_url'),
39
+            'include_inline'  => $conf->get('autoptimize_css_include_inline'),
40
+            'nogooglefont'    => $conf->get('autoptimize_css_nogooglefont'),
41
+            'minify_excluded' => $conf->get('autoptimize_minify_excluded'),
42 42
         ];
43 43
     }
44 44
 
@@ -47,14 +47,14 @@  discard block
 block discarded – undo
47 47
         $conf = autoptimizeConfig::instance();
48 48
 
49 49
         return [
50
-            'aggregate'       => $conf->get( 'autoptimize_js_aggregate' ),
51
-            'justhead'        => $conf->get( 'autoptimize_js_justhead' ),
52
-            'forcehead'       => $conf->get( 'autoptimize_js_forcehead' ),
53
-            'trycatch'        => $conf->get( 'autoptimize_js_trycatch' ),
54
-            'js_exclude'      => $conf->get( 'autoptimize_js_exclude' ),
55
-            'cdn_url'         => $conf->get( 'autoptimize_cdn_url' ),
56
-            'include_inline'  => $conf->get( 'autoptimize_js_include_inline' ),
57
-            'minify_excluded' => $conf->get( 'autoptimize_minify_excluded' ),
50
+            'aggregate'       => $conf->get('autoptimize_js_aggregate'),
51
+            'justhead'        => $conf->get('autoptimize_js_justhead'),
52
+            'forcehead'       => $conf->get('autoptimize_js_forcehead'),
53
+            'trycatch'        => $conf->get('autoptimize_js_trycatch'),
54
+            'js_exclude'      => $conf->get('autoptimize_js_exclude'),
55
+            'cdn_url'         => $conf->get('autoptimize_cdn_url'),
56
+            'include_inline'  => $conf->get('autoptimize_js_include_inline'),
57
+            'minify_excluded' => $conf->get('autoptimize_minify_excluded'),
58 58
         ];
59 59
     }
60 60
 
@@ -64,35 +64,35 @@  discard block
 block discarded – undo
64 64
     protected function get_urls()
65 65
     {
66 66
         static $site_url = null;
67
-        if ( null === $site_url ) {
67
+        if (null === $site_url) {
68 68
             $site_url = site_url();
69 69
         }
70 70
 
71 71
         static $cdn_url = null;
72
-        if ( null === $cdn_url ) {
73
-            $cdn_url = get_option( 'autoptimize_cdn_url' );
72
+        if (null === $cdn_url) {
73
+            $cdn_url = get_option('autoptimize_cdn_url');
74 74
         }
75 75
 
76 76
         static $imgopt_host = null;
77
-        if ( null === $imgopt_host ) {
77
+        if (null === $imgopt_host) {
78 78
             $optimizer   = new autoptimizeImages();
79
-            $imgopt_host = rtrim( $optimizer->get_imgopt_host(), '/' );
79
+            $imgopt_host = rtrim($optimizer->get_imgopt_host(), '/');
80 80
         }
81 81
 
82 82
         static $urls = [];
83 83
 
84
-        if ( empty( $urls ) ) {
84
+        if (empty($urls)) {
85 85
             $parts = autoptimizeUtils::get_ao_wp_site_url_parts();
86 86
             $urls  = [
87 87
                 'siteurl'    => $site_url,
88
-                'prsiteurl'  => '//' . str_replace( array( 'http://', 'https://' ), '', $site_url ),
89
-                'wwwsiteurl' => $parts['scheme'] . '://www.' . str_replace( 'www.', '', $parts['host'] ),
88
+                'prsiteurl'  => '//'.str_replace(array('http://', 'https://'), '', $site_url),
89
+                'wwwsiteurl' => $parts['scheme'].'://www.'.str_replace('www.', '', $parts['host']),
90 90
                 'cdnurl'     => $cdn_url,
91 91
                 'imgopthost' => $imgopt_host,
92 92
                 'subfolder'  => '',
93 93
             ];
94 94
 
95
-            if ( AO_TEST_SUBFOLDER_INSTALL ) {
95
+            if (AO_TEST_SUBFOLDER_INSTALL) {
96 96
                 $urls['subfolder'] = 'wordpress/';
97 97
             }
98 98
         }
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
105 105
      */
106 106
     public function setUp()
107 107
     {
108
-        $this->ao = new autoptimizeMain( AUTOPTIMIZE_PLUGIN_VERSION, AUTOPTIMIZE_PLUGIN_FILE );
108
+        $this->ao = new autoptimizeMain(AUTOPTIMIZE_PLUGIN_VERSION, AUTOPTIMIZE_PLUGIN_FILE);
109 109
 
110 110
         parent::setUp();
111 111
     }
@@ -124,8 +124,8 @@  discard block
 block discarded – undo
124 124
             'autoptimize_filter_css_inlinesize',
125 125
             'autoptimize_filter_css_fonts_cdn',
126 126
         );
127
-        foreach ( $filter_tags as $filter_tag ) {
128
-            remove_all_filters( $filter_tag );
127
+        foreach ($filter_tags as $filter_tag) {
128
+            remove_all_filters($filter_tag);
129 129
         }
130 130
 
131 131
         parent::tearDown();
@@ -249,15 +249,15 @@  discard block
 block discarded – undo
249 249
     protected function get_test_markup_output()
250 250
     {
251 251
         $key = 'default';
252
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
252
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
253 253
             $key = 'custom';
254 254
         }
255 255
 
256
-        $csspart = self::$flexible_url_parts_css[ $key ];
257
-        $jspart  = self::$flexible_url_parts_js[ $key ];
256
+        $csspart = self::$flexible_url_parts_css[$key];
257
+        $jspart  = self::$flexible_url_parts_js[$key];
258 258
 
259 259
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
260
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
260
+        if (AO_TEST_SUBFOLDER_INSTALL) {
261 261
             $csshash = '56398de576d59887e88e3011715250e0';
262 262
         }
263 263
 
@@ -316,15 +316,15 @@  discard block
 block discarded – undo
316 316
     protected function get_test_markup_output_ms()
317 317
     {
318 318
         $key = 'multisite';
319
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
319
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
320 320
             $key = 'multisite_custom';
321 321
         }
322 322
 
323
-        $csspart = self::$flexible_url_parts_css[ $key ];
324
-        $jspart  = self::$flexible_url_parts_js[ $key ];
323
+        $csspart = self::$flexible_url_parts_css[$key];
324
+        $jspart  = self::$flexible_url_parts_js[$key];
325 325
 
326 326
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
327
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
327
+        if (AO_TEST_SUBFOLDER_INSTALL) {
328 328
             $csshash = '56398de576d59887e88e3011715250e0';
329 329
         }
330 330
 
@@ -383,15 +383,15 @@  discard block
 block discarded – undo
383 383
     protected function get_test_markup_output_inline_defer()
384 384
     {
385 385
         $key = 'default';
386
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
386
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
387 387
             $key = 'custom';
388 388
         }
389 389
 
390
-        $csspart = self::$flexible_url_parts_css[ $key ];
391
-        $jspart  = self::$flexible_url_parts_js[ $key ];
390
+        $csspart = self::$flexible_url_parts_css[$key];
391
+        $jspart  = self::$flexible_url_parts_js[$key];
392 392
 
393 393
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
394
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
394
+        if (AO_TEST_SUBFOLDER_INSTALL) {
395 395
             $csshash = '56398de576d59887e88e3011715250e0';
396 396
         }
397 397
 
@@ -450,15 +450,15 @@  discard block
 block discarded – undo
450 450
     protected function get_test_markup_output_inline_defer_ms()
451 451
     {
452 452
         $key = 'multisite';
453
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
453
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
454 454
             $key = 'multisite_custom';
455 455
         }
456 456
 
457
-        $csspart = self::$flexible_url_parts_css[ $key ];
458
-        $jspart  = self::$flexible_url_parts_js[ $key ];
457
+        $csspart = self::$flexible_url_parts_css[$key];
458
+        $jspart  = self::$flexible_url_parts_js[$key];
459 459
 
460 460
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
461
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
461
+        if (AO_TEST_SUBFOLDER_INSTALL) {
462 462
             $csshash = '56398de576d59887e88e3011715250e0';
463 463
         }
464 464
 
@@ -517,11 +517,11 @@  discard block
 block discarded – undo
517 517
     /**
518 518
      * @dataProvider provider_test_rewrite_markup_with_cdn
519 519
      */
520
-    function test_rewrite_markup_with_cdn( $input, $expected )
520
+    function test_rewrite_markup_with_cdn($input, $expected)
521 521
     {
522
-        $actual = $this->ao->end_buffering( $input );
522
+        $actual = $this->ao->end_buffering($input);
523 523
 
524
-        $this->assertEquals( $expected, $actual );
524
+        $this->assertEquals($expected, $actual);
525 525
     }
526 526
 
527 527
     public function provider_test_rewrite_markup_with_cdn()
@@ -533,7 +533,7 @@  discard block
 block discarded – undo
533 533
                 $this->get_test_markup(),
534 534
                 // Expected output.
535 535
                 // TODO/FIXME: This seemed like the fastest way to get MS crude test to pass...
536
-                ( is_multisite() ? $this->get_test_markup_output_ms() : $this->get_test_markup_output() ),
536
+                (is_multisite() ? $this->get_test_markup_output_ms() : $this->get_test_markup_output()),
537 537
             ),
538 538
 
539 539
         );
@@ -593,12 +593,12 @@  discard block
 block discarded – undo
593 593
 }
594 594
 CSS;
595 595
 
596
-        $instance = new autoptimizeStyles( $css_in );
597
-        $instance->setOption( 'cdn_url', $cdnurl );
596
+        $instance = new autoptimizeStyles($css_in);
597
+        $instance->setOption('cdn_url', $cdnurl);
598 598
 
599
-        $css_actual = $instance->rewrite_assets( $css_in );
599
+        $css_actual = $instance->rewrite_assets($css_in);
600 600
 
601
-        $this->assertEquals( $css_expected, $css_actual );
601
+        $this->assertEquals($css_expected, $css_actual);
602 602
     }
603 603
 
604 604
     public function test_default_cssmin_minifier()
@@ -632,21 +632,21 @@  discard block
 block discarded – undo
632 632
 .bg{background:url('img/something.svg')}.bg-no-quote{background:url(img/something.svg)}.bg-double-quotes{background:url("img/something.svg")}.whitespaces{background:url ("../../somewhere-else/svg.svg")}.host-relative{background:url("/img/something.svg")}.protocol-relative{background:url("//something/somewhere/example.png")}@font-face{font-family:'Roboto';font-style:normal;font-weight:100;src:url(../fonts/roboto-v15-latin-ext_latin-100.eot);src:local('Roboto Thin'),local('Roboto-Thin'),url(../fonts/roboto-v15-latin-ext_latin-100.eot?#iefix) format('embedded-opentype'),url(../fonts/roboto-v15-latin-ext_latin-100.woff2) format('woff2'),url(../fonts/roboto-v15-latin-ext_latin-100.woff) format('woff'),url(../fonts/roboto-v15-latin-ext_latin-100.ttf) format('truetype'),url(../fonts/roboto-v15-latin-ext_latin-100.svg#Roboto) format('svg')}
633 633
 CSS;
634 634
 
635
-        $instance = new autoptimizeStyles( $css );
636
-        $minified = $instance->run_minifier_on( $css );
635
+        $instance = new autoptimizeStyles($css);
636
+        $minified = $instance->run_minifier_on($css);
637 637
 
638
-        $this->assertEquals( $expected, $minified );
638
+        $this->assertEquals($expected, $minified);
639 639
     }
640 640
 
641 641
     /**
642 642
      * @dataProvider provider_test_should_aggregate_script_types
643 643
      */
644
-    public function test_should_aggregate_script_types( $input, $expected )
644
+    public function test_should_aggregate_script_types($input, $expected)
645 645
     {
646
-        $instance = new autoptimizeScripts( '' );
647
-        $actual   = $instance->should_aggregate( $input );
646
+        $instance = new autoptimizeScripts('');
647
+        $actual   = $instance->should_aggregate($input);
648 648
 
649
-        $this->assertEquals( $expected, $actual );
649
+        $this->assertEquals($expected, $actual);
650 650
     }
651 651
 
652 652
     public function provider_test_should_aggregate_script_types()
@@ -708,11 +708,11 @@  discard block
 block discarded – undo
708 708
     /**
709 709
      * @dataProvider provider_is_valid_buffer
710 710
      */
711
-    public function test_valid_buffers( $input, $expected )
711
+    public function test_valid_buffers($input, $expected)
712 712
     {
713
-        $actual = $this->ao->is_valid_buffer( $input );
713
+        $actual = $this->ao->is_valid_buffer($input);
714 714
 
715
-        $this->assertEquals( $expected, $actual );
715
+        $this->assertEquals($expected, $actual);
716 716
     }
717 717
 
718 718
     public function provider_is_valid_buffer()
@@ -774,11 +774,11 @@  discard block
 block discarded – undo
774 774
     /**
775 775
      * @dataProvider provider_is_amp_markup
776 776
      */
777
-    public function test_autoptimize_is_amp_markup( $input, $expected )
777
+    public function test_autoptimize_is_amp_markup($input, $expected)
778 778
     {
779
-        $actual = autoptimizeMain::is_amp_markup( $input );
779
+        $actual = autoptimizeMain::is_amp_markup($input);
780 780
 
781
-        $this->assertEquals( $expected, $actual );
781
+        $this->assertEquals($expected, $actual);
782 782
     }
783 783
 
784 784
     public function provider_is_amp_markup()
@@ -809,31 +809,31 @@  discard block
 block discarded – undo
809 809
     public function test_skips_buffering_when_ao_noptimize_filter_is_true()
810 810
     {
811 811
         // True => disable autoptimize.
812
-        add_filter( 'autoptimize_filter_noptimize', '__return_true' );
812
+        add_filter('autoptimize_filter_noptimize', '__return_true');
813 813
 
814 814
         // Buffering should not run due to the above filter.
815 815
         $expected = false;
816
-        $actual   = $this->ao->should_buffer( $doing_tests = true );
816
+        $actual   = $this->ao->should_buffer($doing_tests = true);
817 817
 
818
-        $this->assertEquals( $expected, $actual );
818
+        $this->assertEquals($expected, $actual);
819 819
     }
820 820
 
821 821
     public function test_does_buffering_when_ao_noptimize_filter_is_false()
822 822
     {
823 823
         // False => disable noptimize, aka, run normally (weird, yes...).
824
-        add_filter( 'autoptimize_filter_noptimize', '__return_false' );
824
+        add_filter('autoptimize_filter_noptimize', '__return_false');
825 825
 
826 826
         // Buffering should run because of above.
827 827
         $expected = true;
828
-        $actual   = $this->ao->should_buffer( $doing_tests = true );
828
+        $actual   = $this->ao->should_buffer($doing_tests = true);
829 829
 
830
-        $this->assertEquals( $expected, $actual );
830
+        $this->assertEquals($expected, $actual);
831 831
     }
832 832
 
833 833
     public function test_ignores_ao_noptimize_qs_when_instructed()
834 834
     {
835 835
         // Should skip checking for the qs completely due to filter.
836
-        add_filter( 'autoptimize_filter_honor_qs_noptimize', '__return_false' );
836
+        add_filter('autoptimize_filter_honor_qs_noptimize', '__return_false');
837 837
 
838 838
         /**
839 839
          * The above should then result in the "current" value being `false`
@@ -844,23 +844,23 @@  discard block
 block discarded – undo
844 844
          */
845 845
 
846 846
         $that = $this; // Makes it work on php 5.3!
847
-        add_filter( 'autoptimize_filter_noptimize', function ( $current_value ) use ( $that ) {
847
+        add_filter('autoptimize_filter_noptimize', function($current_value) use ($that) {
848 848
             $expected = false;
849
-            if ( defined( 'DONOTMINIFY' ) && DONOTMINIFY ) {
849
+            if (defined('DONOTMINIFY') && DONOTMINIFY) {
850 850
                 $expected = true;
851 851
             }
852 852
 
853
-            $that->assertEquals( $expected, $current_value );
853
+            $that->assertEquals($expected, $current_value);
854 854
         });
855 855
 
856
-        $this->ao->should_buffer( $doing_tests = true );
856
+        $this->ao->should_buffer($doing_tests = true);
857 857
     }
858 858
 
859 859
     public function test_wpengine_cache_flush()
860 860
     {
861 861
         // Creating a mock so that we can get past class_exists() and method_exists() checks present
862 862
         // in `autoptimizeCache::flushPageCache()`...
863
-        $stub = $this->getMockBuilder( 'WpeCommon' )->disableAutoload()
863
+        $stub = $this->getMockBuilder('WpeCommon')->disableAutoload()
864 864
                 ->disableOriginalConstructor()->setMethods(
865 865
                     array(
866 866
                         'purge_varnish_cache',
@@ -869,9 +869,9 @@  discard block
 block discarded – undo
869 869
                 ->getMock();
870 870
 
871 871
         $that = $this;
872
-        add_filter( 'autoptimize_flush_wpengine_methods', function( $methods ) use ( $that ) {
873
-            $expected_methods = array( 'purge_varnish_cache' );
874
-            $that->assertEquals( $methods, $expected_methods );
872
+        add_filter('autoptimize_flush_wpengine_methods', function($methods) use ($that) {
873
+            $expected_methods = array('purge_varnish_cache');
874
+            $that->assertEquals($methods, $expected_methods);
875 875
 
876 876
             return $methods;
877 877
         });
@@ -885,7 +885,7 @@  discard block
 block discarded – undo
885 885
     public function test_wpengine_cache_flush_agressive()
886 886
     {
887 887
         // Creating a mock so that we can get past class_exists() and method_exists() checks `autoptimize_flush_pagecache()`...
888
-        $stub = $this->getMockBuilder( 'WpeCommon' )->disableAutoload()
888
+        $stub = $this->getMockBuilder('WpeCommon')->disableAutoload()
889 889
                 ->disableOriginalConstructor()->setMethods(
890 890
                     array(
891 891
                         'purge_varnish_cache',
@@ -895,19 +895,19 @@  discard block
 block discarded – undo
895 895
                 )
896 896
                 ->getMock();
897 897
 
898
-        add_filter( 'autoptimize_flush_wpengine_aggressive', function() {
898
+        add_filter('autoptimize_flush_wpengine_aggressive', function() {
899 899
             return true;
900 900
         });
901 901
 
902 902
         $that = $this;
903
-        add_filter( 'autoptimize_flush_wpengine_methods', function( $methods ) use ( $that ) {
903
+        add_filter('autoptimize_flush_wpengine_methods', function($methods) use ($that) {
904 904
             $expected_methods = array(
905 905
                 'purge_varnish_cache',
906 906
                 'purge_memcached',
907 907
                 'clear_maxcdn_cache',
908 908
             );
909 909
 
910
-            $that->assertEquals( $methods, $expected_methods );
910
+            $that->assertEquals($methods, $expected_methods);
911 911
 
912 912
             return $methods;
913 913
         });
@@ -918,15 +918,15 @@  discard block
 block discarded – undo
918 918
     /**
919 919
      * @dataProvider provider_test_url_replace_cdn
920 920
      */
921
-    public function test_url_replace_cdn( $cdn_url, $input, $expected )
921
+    public function test_url_replace_cdn($cdn_url, $input, $expected)
922 922
     {
923
-        $mock = $this->getMockBuilder( 'autoptimizeBase' )
923
+        $mock = $this->getMockBuilder('autoptimizeBase')
924 924
                 ->disableOriginalConstructor()
925 925
                 ->getMockForAbstractClass();
926 926
 
927 927
         $mock->cdn_url = $cdn_url;
928
-        $actual        = $mock->url_replace_cdn( $input );
929
-        $this->assertEquals( $expected, $actual );
928
+        $actual        = $mock->url_replace_cdn($input);
929
+        $this->assertEquals($expected, $actual);
930 930
     }
931 931
 
932 932
     public function provider_test_url_replace_cdn()
@@ -943,13 +943,13 @@  discard block
 block discarded – undo
943 943
                 // CDN base url, url, expected result...
944 944
                 'http://cdn-test.example.org',
945 945
                 '/a.jpg',
946
-                'http://cdn-test.example.org/' . $subfolder . 'a.jpg',
946
+                'http://cdn-test.example.org/'.$subfolder.'a.jpg',
947 947
             ),
948 948
             // Full link with a matching AUTOPTIMIZE_WP_SITE_URL gets properly replaced...
949 949
             array(
950 950
                 'http://cdn-test.example.org',
951
-                $siteurl . '/wp-content/themes/something/example.svg',
952
-                'http://cdn-test.example.org/' . $subfolder . 'wp-content/themes/something/example.svg',
951
+                $siteurl.'/wp-content/themes/something/example.svg',
952
+                'http://cdn-test.example.org/'.$subfolder.'wp-content/themes/something/example.svg',
953 953
             ),
954 954
             // Protocol-relative url with a "local" hostname that doesn't match example.org (AUTOPTIMIZE_WP_SITE_URL)...
955 955
             array(
@@ -960,59 +960,59 @@  discard block
 block discarded – undo
960 960
             // www.example.org does not match example.org (AUTOPTIMIZE_WP_SITE_URL) so it's left alone...
961 961
             array(
962 962
                 'http://cdn-test.example.org',
963
-                $wwwsiteurl . '/wp-content/themes/something/example.svg',
964
-                $wwwsiteurl . '/wp-content/themes/something/example.svg',
963
+                $wwwsiteurl.'/wp-content/themes/something/example.svg',
964
+                $wwwsiteurl.'/wp-content/themes/something/example.svg',
965 965
             ),
966 966
             // SSL cdn url + host-relative link...
967 967
             array(
968 968
                 'https://cdn.example.org',
969 969
                 '/a.jpg',
970
-                'https://cdn.example.org/' . $subfolder . 'a.jpg',
970
+                'https://cdn.example.org/'.$subfolder.'a.jpg',
971 971
             ),
972 972
             // SSL cdn url + http site url that matches AUTOPTIMIZE_WP_SITE_URL is properly replaced...
973 973
             array(
974 974
                 'https://cdn.example.org',
975
-                $siteurl . '/wp-content/themes/something/example.svg',
976
-                'https://cdn.example.org/' . $subfolder . 'wp-content/themes/something/example.svg',
975
+                $siteurl.'/wp-content/themes/something/example.svg',
976
+                'https://cdn.example.org/'.$subfolder.'wp-content/themes/something/example.svg',
977 977
             ),
978 978
             // Protocol-relative cdn url given with protocol relative link that matches AUTOPTIMIZE_WP_SITE_URL host...
979 979
             array(
980 980
                 '//cdn.example.org',
981
-                $protorelativesiteurl . '/something.jpg',
982
-                '//cdn.example.org/' . $subfolder . 'something.jpg',
981
+                $protorelativesiteurl.'/something.jpg',
982
+                '//cdn.example.org/'.$subfolder.'something.jpg',
983 983
             ),
984 984
             // Protocol-relative cdn url given a http link that matches AUTOPTIMIZE_WP_SITE_URL host...
985 985
             array(
986 986
                 '//cdn.example.org',
987
-                $siteurl . '/something.png',
988
-                '//cdn.example.org/' . $subfolder . 'something.png',
987
+                $siteurl.'/something.png',
988
+                '//cdn.example.org/'.$subfolder.'something.png',
989 989
             ),
990 990
             // Protocol-relative cdn url with a host-relative link...
991 991
             array(
992 992
                 '//cdn.example.org',
993 993
                 '/a.jpg',
994
-                '//cdn.example.org/' . $subfolder . 'a.jpg',
994
+                '//cdn.example.org/'.$subfolder.'a.jpg',
995 995
             ),
996 996
             // Testing cdn urls with an explicit port number...
997 997
             array(
998 998
                 'http://cdn.com:8080',
999 999
                 '/a.jpg',
1000
-                'http://cdn.com:8080/' . $subfolder . 'a.jpg',
1000
+                'http://cdn.com:8080/'.$subfolder.'a.jpg',
1001 1001
             ),
1002 1002
             array(
1003 1003
                 '//cdn.com:4433',
1004 1004
                 '/a.jpg',
1005
-                '//cdn.com:4433/' . $subfolder . 'a.jpg',
1005
+                '//cdn.com:4433/'.$subfolder.'a.jpg',
1006 1006
             ),
1007 1007
             array(
1008 1008
                 '//cdn.com:4433',
1009
-                $siteurl . '/something.jpg',
1010
-                '//cdn.com:4433/' . $subfolder . 'something.jpg',
1009
+                $siteurl.'/something.jpg',
1010
+                '//cdn.com:4433/'.$subfolder.'something.jpg',
1011 1011
             ),
1012 1012
             array(
1013 1013
                 '//cdn.com:1234',
1014
-                $protorelativesiteurl . '/something.jpg',
1015
-                '//cdn.com:1234/' . $subfolder . 'something.jpg',
1014
+                $protorelativesiteurl.'/something.jpg',
1015
+                '//cdn.com:1234/'.$subfolder.'something.jpg',
1016 1016
             ),
1017 1017
             // Relative links should not be touched by url_replace_cdn()...
1018 1018
             array(
@@ -1037,8 +1037,8 @@  discard block
 block discarded – undo
1037 1037
             // even though they are really strange...
1038 1038
             array(
1039 1039
                 $siteurl, // example.org or http://localhost or http://localhost/wordpress
1040
-                $siteurl . '/something.jpg',
1041
-                $siteurl . '/something.jpg',
1040
+                $siteurl.'/something.jpg',
1041
+                $siteurl.'/something.jpg',
1042 1042
             ),
1043 1043
             // These shouldn't really be changed, or even if replacements do
1044 1044
             // happen, they shouldn't be destructive...
@@ -1077,31 +1077,31 @@  discard block
 block discarded – undo
1077 1077
         $test_link = '/a.jpg';
1078 1078
         $cdn_url   = '//cdn.example.org';
1079 1079
 
1080
-        $with_ssl = function( $cdn ) {
1080
+        $with_ssl = function($cdn) {
1081 1081
             return '';
1082 1082
         };
1083 1083
         $expected_with_ssl = '/a.jpg';
1084 1084
 
1085
-        $without_ssl = function( $cdn ) {
1085
+        $without_ssl = function($cdn) {
1086 1086
             return $cdn;
1087 1087
         };
1088
-        $expected_without_ssl = '//cdn.example.org/' . $sub . 'a.jpg';
1088
+        $expected_without_ssl = '//cdn.example.org/'.$sub.'a.jpg';
1089 1089
 
1090 1090
         // With a filter that returns something considered "empty", cdn replacement shouldn't occur...
1091
-        add_filter( 'autoptimize_filter_base_cdnurl', $with_ssl );
1092
-        $mock = $this->getMockBuilder( 'autoptimizeBase' )
1091
+        add_filter('autoptimize_filter_base_cdnurl', $with_ssl);
1092
+        $mock = $this->getMockBuilder('autoptimizeBase')
1093 1093
                 ->disableOriginalConstructor()
1094 1094
                 ->getMockForAbstractClass();
1095 1095
 
1096 1096
         $mock->cdn_url   = $cdn_url;
1097
-        $actual_with_ssl = $mock->url_replace_cdn( $test_link );
1098
-        $this->assertEquals( $expected_with_ssl, $actual_with_ssl );
1099
-        remove_filter( 'autoptimize_filter_base_cdnurl', $with_ssl );
1097
+        $actual_with_ssl = $mock->url_replace_cdn($test_link);
1098
+        $this->assertEquals($expected_with_ssl, $actual_with_ssl);
1099
+        remove_filter('autoptimize_filter_base_cdnurl', $with_ssl);
1100 1100
 
1101 1101
         // With a filter that returns an actual cdn url, cdn replacement should occur.
1102
-        add_filter( 'autoptimize_filter_base_cdnurl', $without_ssl );
1103
-        $actual_without_ssl = $mock->url_replace_cdn( $test_link );
1104
-        $this->assertEquals( $expected_without_ssl, $actual_without_ssl );
1102
+        add_filter('autoptimize_filter_base_cdnurl', $without_ssl);
1103
+        $actual_without_ssl = $mock->url_replace_cdn($test_link);
1104
+        $this->assertEquals($expected_without_ssl, $actual_without_ssl);
1105 1105
     }
1106 1106
 
1107 1107
     public function provider_cssmin_issues()
@@ -1140,11 +1140,11 @@  discard block
 block discarded – undo
1140 1140
     /**
1141 1141
      * @dataProvider provider_cssmin_issues
1142 1142
      */
1143
-    public function test_cssmin_issues( $input, $expected )
1143
+    public function test_cssmin_issues($input, $expected)
1144 1144
     {
1145
-        $minifier = new autoptimizeCSSmin( false ); // No need to raise limits for now.
1146
-        $actual   = $minifier->run( $input );
1147
-        $this->assertEquals( $expected, $actual );
1145
+        $minifier = new autoptimizeCSSmin(false); // No need to raise limits for now.
1146
+        $actual   = $minifier->run($input);
1147
+        $this->assertEquals($expected, $actual);
1148 1148
     }
1149 1149
 
1150 1150
     public function provider_getpath()
@@ -1169,7 +1169,7 @@  discard block
 block discarded – undo
1169 1169
             // This file comes with core, so should exist...
1170 1170
             array(
1171 1171
                 '/wp-includes/js/jquery/jquery.js',
1172
-                WP_ROOT_DIR . '/wp-includes/js/jquery/jquery.js',
1172
+                WP_ROOT_DIR.'/wp-includes/js/jquery/jquery.js',
1173 1173
             ),
1174 1174
             // Empty $url should return false.
1175 1175
             array(
@@ -1194,12 +1194,12 @@  discard block
 block discarded – undo
1194 1194
     /**
1195 1195
      * @dataProvider provider_getpath
1196 1196
      */
1197
-    public function test_getpath( $input, $expected )
1197
+    public function test_getpath($input, $expected)
1198 1198
     {
1199
-        $mock = $this->getMockBuilder( 'autoptimizeBase' )->disableOriginalConstructor()->getMockForAbstractClass();
1199
+        $mock = $this->getMockBuilder('autoptimizeBase')->disableOriginalConstructor()->getMockForAbstractClass();
1200 1200
 
1201
-        $actual = $mock->getpath( $input );
1202
-        $this->assertEquals( $expected, $actual );
1201
+        $actual = $mock->getpath($input);
1202
+        $this->assertEquals($expected, $actual);
1203 1203
     }
1204 1204
 
1205 1205
     /**
@@ -1213,7 +1213,7 @@  discard block
 block discarded – undo
1213 1213
 
1214 1214
         $protorelativesiteurl = $urls['prsiteurl'];
1215 1215
 
1216
-        $css_orig     = <<<CSS
1216
+        $css_orig = <<<CSS
1217 1217
 header{background-image:url("data:image/svg+xml,%3Csvg xmlns='http://www.w3.org/2000/svg' width='90px' height='110px' viewBox='0 0 90 110'%3E%3Cstyle%3E.a%7Bstop-color:%23FFF;%7D.b%7Bstop-color:%23B2D235;%7D.c%7Bstop-color:%23BEE7FA;%7D.d%7Bfill:%23590C15;%7D%3C/style%3E%3ClinearGradient id='c' y2='135.4' gradientUnits='userSpaceOnUse' x2='209.1' gradientTransform='rotate(-1.467 -4082.888 7786.794)' y1='205.8' x1='262'%3E%3Cstop class='b' offset='0'/%3E%3Cstop class='b' offset='.48'/%3E%3Cstop stop-color='%23829D25' offset='1'/%3E%3C/linearGradient%3E%3Cpath stroke-width='.3' d='M77.3 45.4c-3-3.5-7.1-6.5-11.6-7.8-5.1-1.5-10-.1-14.9 1.5C52 35.4 54.3 29 60 24l-4.8-5.5c-3.4 3-5.8 6.3-7.5 9.4-1.7-4.3-4.1-8.4-7.5-12C33.4 8.6 24.3 4.7 15.1 4.2c-.2 9.3 3.1 18.6 9.9 25.9 5.2 5.6 11.8 9.2 18.7 10.8-2.5.2-4.9-.1-7.7-.9-5.2-1.4-10.5-2.8-15.8-1C10.6 42.3 4.5 51.9 4 61.7c-.5 11.6 3.8 23.8 9.9 33.5 3.9 6.3 9.6 13.7 17.7 13.4 3.8-.1 7-2.1 10.7-2.7 5.2-.8 9.1 1.2 14.1 1.8 16.4 2 24.4-23.6 26.4-35.9 1.2-9.1.8-19.1-5.5-26.4z' stroke='%233E6D1F' fill='url(%23c)'/%3E%3C/svg%3E")}
1218 1218
 section.clipped.clippedTop {clip-path:url("#clipPolygonTop")}
1219 1219
 section.clipped.clippedBottom {clip-path:url("#clipPolygonBottom")}
@@ -1226,13 +1226,13 @@  discard block
 block discarded – undo
1226 1226
 .myimg {background-image: url($protorelativesiteurl/wp-content/themes/my-theme/images/under-left-leaf.png), url($protorelativesiteurl/wp-content/themes/my-theme/images/over-blue-bird.png), url($protorelativesiteurl/wp-content/themes/my-theme/images/under-top.png), url($protorelativesiteurl/wp-content/themes/my-theme/images/bg-top-grunge.png);}
1227 1227
 CSS;
1228 1228
 
1229
-        $fixurls_result = autoptimizeStyles::fixurls( ABSPATH . 'wp-content/themes/my-theme/style.css', $css_orig );
1230
-        $this->assertEquals( $css_expected, $fixurls_result );
1229
+        $fixurls_result = autoptimizeStyles::fixurls(ABSPATH.'wp-content/themes/my-theme/style.css', $css_orig);
1230
+        $this->assertEquals($css_expected, $fixurls_result);
1231 1231
     }
1232 1232
 
1233 1233
     public function test_background_datauri_sprites_with_fixurls()
1234 1234
     {
1235
-        $css_orig     = <<<CSS
1235
+        $css_orig = <<<CSS
1236 1236
 .shadow { background:url(img/1x1.png) top center; }
1237 1237
 .shadow1 { background-image:url(img/1x1.png) 0 -767px repeat-x; }
1238 1238
 .shadow2 {background:url(img/1x1.png) top center}
@@ -1284,27 +1284,27 @@  discard block
 block discarded – undo
1284 1284
 CSS;
1285 1285
 
1286 1286
         // For test purposes, ALL images in the css are being inline with a 1x1 trans png string/datauri.
1287
-        add_filter( 'autoptimize_filter_css_is_datauri_candidate', function( $is_candidate, $path ) {
1287
+        add_filter('autoptimize_filter_css_is_datauri_candidate', function($is_candidate, $path) {
1288 1288
             return true;
1289
-        }, 10, 2 );
1289
+        }, 10, 2);
1290 1290
 
1291 1291
         // For test purposes, ALL images in the css are being inline with a 1x1 trans png string/datauri.
1292
-        add_filter( 'autoptimize_filter_css_datauri_image', function( $base64array, $path ) {
1292
+        add_filter('autoptimize_filter_css_datauri_image', function($base64array, $path) {
1293 1293
             $head = 'data:image/png;base64,';
1294 1294
             $data = 'iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAQAAAC1HAwCAAAAC0lEQVR42mNkYAAAAAYAAjCB0C8AAAAASUVORK5CYII=';
1295 1295
 
1296
-            $result['full']       = $head . $data;
1296
+            $result['full']       = $head.$data;
1297 1297
             $result['base64data'] = $data;
1298 1298
             return $result;
1299
-        }, 10, 2 );
1299
+        }, 10, 2);
1300 1300
 
1301
-        $instance = new autoptimizeStyles( $css_orig );
1302
-        $instance->setOption( 'datauris', true );
1301
+        $instance = new autoptimizeStyles($css_orig);
1302
+        $instance->setOption('datauris', true);
1303 1303
 
1304
-        $fixurls_result = autoptimizeStyles::fixurls( ABSPATH . 'wp-content/themes/my-theme/style.css', $css_orig );
1305
-        $css_actual     = $instance->rewrite_assets( $fixurls_result );
1304
+        $fixurls_result = autoptimizeStyles::fixurls(ABSPATH.'wp-content/themes/my-theme/style.css', $css_orig);
1305
+        $css_actual     = $instance->rewrite_assets($fixurls_result);
1306 1306
 
1307
-        $this->assertEquals( $css_expected, $css_actual );
1307
+        $this->assertEquals($css_expected, $css_actual);
1308 1308
     }
1309 1309
 
1310 1310
     /**
@@ -1324,7 +1324,7 @@  discard block
 block discarded – undo
1324 1324
      */
1325 1325
     public function test_background_datauri_sprites_without_fixurls()
1326 1326
     {
1327
-        $css_orig     = <<<CSS
1327
+        $css_orig = <<<CSS
1328 1328
 .shadow { background:url(img/1x1.png) top center; }
1329 1329
 .shadow1 { background-image:url(img/1x1.png) 0 -767px repeat-x; }
1330 1330
 .shadow2 {background:url(img/1x1.png) top center}
@@ -1368,24 +1368,24 @@  discard block
 block discarded – undo
1368 1368
 CSS;
1369 1369
 
1370 1370
         // For test purposes, ALL images in the css are being inlined with a 1x1 trans png string/datauri.
1371
-        add_filter( 'autoptimize_filter_css_is_datauri_candidate', function( $is_candidate, $path ) {
1371
+        add_filter('autoptimize_filter_css_is_datauri_candidate', function($is_candidate, $path) {
1372 1372
             return true;
1373
-        }, 10, 2 );
1373
+        }, 10, 2);
1374 1374
 
1375 1375
         // For test purposes, ALL images in the css are being inlined with a 1x1 trans png string/datauri.
1376
-        add_filter( 'autoptimize_filter_css_datauri_image', function( $base64array, $path ) {
1376
+        add_filter('autoptimize_filter_css_datauri_image', function($base64array, $path) {
1377 1377
             $head = 'data:image/png;base64,';
1378 1378
             $data = 'iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAQAAAC1HAwCAAAAC0lEQVR42mNkYAAAAAYAAjCB0C8AAAAASUVORK5CYII=';
1379 1379
 
1380
-            $result['full']       = $head . $data;
1380
+            $result['full']       = $head.$data;
1381 1381
             $result['base64data'] = $data;
1382 1382
             return $result;
1383
-        }, 10, 2 );
1383
+        }, 10, 2);
1384 1384
 
1385
-        $instance = new autoptimizeStyles( $css_orig );
1386
-        $instance->setOption( 'datauris', true );
1387
-        $css_actual = $instance->rewrite_assets( $css_orig );
1388
-        $this->assertEquals( $css_expected, $css_actual );
1385
+        $instance = new autoptimizeStyles($css_orig);
1386
+        $instance->setOption('datauris', true);
1387
+        $css_actual = $instance->rewrite_assets($css_orig);
1388
+        $this->assertEquals($css_expected, $css_actual);
1389 1389
     }
1390 1390
 
1391 1391
     /**
@@ -1505,12 +1505,12 @@  discard block
 block discarded – undo
1505 1505
 CSS;
1506 1506
 
1507 1507
         // Test with fonts pointed to the CDN + cdn option is set.
1508
-        add_filter( 'autoptimize_filter_css_fonts_cdn', '__return_true' );
1509
-        $instance = new autoptimizeStyles( $css_in );
1510
-        $instance->setOption( 'cdn_url', $cdnurl );
1511
-        $css_actual_fonts_cdn = $instance->rewrite_assets( $css_in );
1508
+        add_filter('autoptimize_filter_css_fonts_cdn', '__return_true');
1509
+        $instance = new autoptimizeStyles($css_in);
1510
+        $instance->setOption('cdn_url', $cdnurl);
1511
+        $css_actual_fonts_cdn = $instance->rewrite_assets($css_in);
1512 1512
 
1513
-        $this->assertEquals( $css_expected_fonts_cdn, $css_actual_fonts_cdn );
1513
+        $this->assertEquals($css_expected_fonts_cdn, $css_actual_fonts_cdn);
1514 1514
     }
1515 1515
 
1516 1516
     /**
@@ -1626,10 +1626,10 @@  discard block
 block discarded – undo
1626 1626
 }
1627 1627
 CSS;
1628 1628
         // Test without moving fonts to CDN, but cdn option is set.
1629
-        $instance = new autoptimizeStyles( $css_in );
1630
-        $instance->setOption( 'cdn_url', $urls['cdnurl'] );
1631
-        $css_actual = $instance->rewrite_assets( $css_in );
1632
-        $this->assertEquals( $css_expected, $css_actual );
1629
+        $instance = new autoptimizeStyles($css_in);
1630
+        $instance->setOption('cdn_url', $urls['cdnurl']);
1631
+        $css_actual = $instance->rewrite_assets($css_in);
1632
+        $this->assertEquals($css_expected, $css_actual);
1633 1633
     }
1634 1634
 
1635 1635
     public function test_assets_regex_replaces_multi_bg_images()
@@ -1650,16 +1650,16 @@  discard block
 block discarded – undo
1650 1650
 }
1651 1651
 CSS;
1652 1652
 
1653
-        $instance = new autoptimizeStyles( $in );
1654
-        $instance->setOption( 'cdn_url', $cdnurl );
1655
-        $actual = $instance->rewrite_assets( $in );
1653
+        $instance = new autoptimizeStyles($in);
1654
+        $instance->setOption('cdn_url', $cdnurl);
1655
+        $actual = $instance->rewrite_assets($in);
1656 1656
 
1657
-        $this->assertEquals( $expected, $actual );
1657
+        $this->assertEquals($expected, $actual);
1658 1658
     }
1659 1659
 
1660 1660
     public function test_at_supports_spacing_issue_110()
1661 1661
     {
1662
-        $in       = <<<CSS
1662
+        $in = <<<CSS
1663 1663
 @supports (-webkit-filter: blur(3px)) or (filter: blur(3px)) {
1664 1664
     .blur {
1665 1665
         filter:blur(3px);
@@ -1673,10 +1673,10 @@  discard block
 block discarded – undo
1673 1673
 @supports (-webkit-filter:blur(3px)) or (filter:blur(3px)){.blur{filter:blur(3px)}}@supports((position:-webkit-sticky) or (position:sticky)){.sticky{position:sticky}}
1674 1674
 CSS;
1675 1675
 
1676
-        $instance = new autoptimizeStyles( $in );
1677
-        $actual   = $instance->run_minifier_on( $in );
1676
+        $instance = new autoptimizeStyles($in);
1677
+        $actual   = $instance->run_minifier_on($in);
1678 1678
 
1679
-        $this->assertEquals( $expected, $actual );
1679
+        $this->assertEquals($expected, $actual);
1680 1680
     }
1681 1681
 
1682 1682
     public function test_css_import_semicolon_url_issue_122()
@@ -1692,18 +1692,18 @@  discard block
 block discarded – undo
1692 1692
 </style>
1693 1693
 HTML;
1694 1694
 
1695
-        $expected = '<style type="text/css" media="all">@import url(' . $cdnurl . '/' . $sub . 'foo.css?a&#038;b);@import url(' . $cdnurl . '/' . $sub . 'bar.css);</style><!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `title` missing --><!--/noptimize-->';
1695
+        $expected = '<style type="text/css" media="all">@import url('.$cdnurl.'/'.$sub.'foo.css?a&#038;b);@import url('.$cdnurl.'/'.$sub.'bar.css);</style><!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `title` missing --><!--/noptimize-->';
1696 1696
 
1697 1697
         $options = [
1698 1698
             'autoptimizeStyles' => $this->getAoStylesDefaultOptions(),
1699 1699
         ];
1700 1700
 
1701
-        $instance = new autoptimizeStyles( $in );
1702
-        $instance->read( $options['autoptimizeStyles'] );
1701
+        $instance = new autoptimizeStyles($in);
1702
+        $instance->read($options['autoptimizeStyles']);
1703 1703
         $instance->minify();
1704 1704
         $instance->cache();
1705 1705
         $actual = $instance->getcontent();
1706
-        $this->assertEquals( $expected, $actual );
1706
+        $this->assertEquals($expected, $actual);
1707 1707
     }
1708 1708
 
1709 1709
     public function test_fixurls_with_at_imports_and_media_queries()
@@ -1713,10 +1713,10 @@  discard block
 block discarded – undo
1713 1713
         $subfolder = $urls['subfolder'];
1714 1714
 
1715 1715
         $in  = '@import "foo.css"; @import "bar.css" (orientation:landscape);';
1716
-        $exp = '@import url(' . $prsiteurl . '/wp-content/themes/my-theme/foo.css); @import url(' . $prsiteurl . '/wp-content/themes/my-theme/bar.css) (orientation:landscape);';
1716
+        $exp = '@import url('.$prsiteurl.'/wp-content/themes/my-theme/foo.css); @import url('.$prsiteurl.'/wp-content/themes/my-theme/bar.css) (orientation:landscape);';
1717 1717
 
1718
-        $actual = autoptimizeStyles::fixurls( ABSPATH . 'wp-content/themes/my-theme/style.css', $in );
1719
-        $this->assertEquals( $exp, $actual );
1718
+        $actual = autoptimizeStyles::fixurls(ABSPATH.'wp-content/themes/my-theme/style.css', $in);
1719
+        $this->assertEquals($exp, $actual);
1720 1720
     }
1721 1721
 
1722 1722
     public function test_aostyles_at_imports_with_media_queries()
@@ -1731,43 +1731,43 @@  discard block
 block discarded – undo
1731 1731
 </style>
1732 1732
 HTML;
1733 1733
 
1734
-        $expected = '<style type="text/css" media="all">@import url(' . $cdnurl . '/' . $sub . 'foo.css);@import url(' . $cdnurl . '/' . $sub . 'bar.css) (orientation:landscape);</style><!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `title` missing --><!--/noptimize-->';
1734
+        $expected = '<style type="text/css" media="all">@import url('.$cdnurl.'/'.$sub.'foo.css);@import url('.$cdnurl.'/'.$sub.'bar.css) (orientation:landscape);</style><!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `title` missing --><!--/noptimize-->';
1735 1735
 
1736 1736
         $options = [
1737 1737
             'autoptimizeStyles' => $this->getAoStylesDefaultOptions(),
1738 1738
         ];
1739 1739
 
1740
-        $instance = new autoptimizeStyles( $in );
1741
-        $instance->read( $options['autoptimizeStyles'] );
1740
+        $instance = new autoptimizeStyles($in);
1741
+        $instance->read($options['autoptimizeStyles']);
1742 1742
         $instance->minify();
1743 1743
         $instance->cache();
1744 1744
 
1745 1745
         $actual = $instance->getcontent();
1746
-        $this->assertEquals( $expected, $actual );
1746
+        $this->assertEquals($expected, $actual);
1747 1747
     }
1748 1748
 
1749 1749
     public function test_cache_size_checker_hooked_by_default()
1750 1750
     {
1751
-        $this->assertNotEmpty( autoptimizeCacheChecker::SCHEDULE_HOOK );
1751
+        $this->assertNotEmpty(autoptimizeCacheChecker::SCHEDULE_HOOK);
1752 1752
 
1753 1753
         // No schedule, because it's only added when is_admin() is true.
1754
-        $this->assertEquals( false, wp_get_schedule( autoptimizeCacheChecker::SCHEDULE_HOOK ) );
1754
+        $this->assertEquals(false, wp_get_schedule(autoptimizeCacheChecker::SCHEDULE_HOOK));
1755 1755
 
1756 1756
         // Proving that setup() sets the schedule as needed.
1757 1757
         $checker = new autoptimizeCacheChecker();
1758 1758
         $checker->setup();
1759
-        $this->assertEquals( 'twicedaily', wp_get_schedule( autoptimizeCacheChecker::SCHEDULE_HOOK ) );
1759
+        $this->assertEquals('twicedaily', wp_get_schedule(autoptimizeCacheChecker::SCHEDULE_HOOK));
1760 1760
     }
1761 1761
 
1762 1762
     public function test_cache_size_checker_disabled_with_filter()
1763 1763
     {
1764
-        add_filter( 'autoptimize_filter_cachecheck_do', '__return_false' );
1764
+        add_filter('autoptimize_filter_cachecheck_do', '__return_false');
1765 1765
 
1766 1766
         $checker = new autoptimizeCacheChecker();
1767 1767
         $checker->setup();
1768
-        $this->assertEquals( false, wp_get_schedule( autoptimizeCacheChecker::SCHEDULE_HOOK ) );
1768
+        $this->assertEquals(false, wp_get_schedule(autoptimizeCacheChecker::SCHEDULE_HOOK));
1769 1769
 
1770
-        remove_all_filters( 'autoptimize_filter_cachecheck_do' );
1770
+        remove_all_filters('autoptimize_filter_cachecheck_do');
1771 1771
     }
1772 1772
 
1773 1773
     public function test_is_start_buffering_hooked_properly()
@@ -1780,12 +1780,12 @@  discard block
 block discarded – undo
1780 1780
          * https://www.theaveragedev.com/mocking-constants-in-tests/.
1781 1781
          */
1782 1782
 
1783
-        if ( defined( 'AUTOPTIMIZE_INIT_EARLIER' ) ) {
1783
+        if (defined('AUTOPTIMIZE_INIT_EARLIER')) {
1784 1784
             $this->assertEquals(
1785 1785
                 autoptimizeMain::INIT_EARLIER_PRIORITY,
1786
-                has_action( 'init', array( $instance, 'start_buffering' ) )
1786
+                has_action('init', array($instance, 'start_buffering'))
1787 1787
             );
1788
-            $this->assertTrue( ! defined( 'AUTOPTIMIZE_HOOK_INTO' ) );
1788
+            $this->assertTrue(!defined('AUTOPTIMIZE_HOOK_INTO'));
1789 1789
         }
1790 1790
 
1791 1791
         /*
@@ -1802,18 +1802,18 @@  discard block
 block discarded – undo
1802 1802
 
1803 1803
     public function test_inline_and_defer_markup()
1804 1804
     {
1805
-        add_filter( 'autoptimize_filter_css_defer', '__return_true' );
1806
-        add_filter( 'autoptimize_filter_css_defer_inline', '__return_true' );
1805
+        add_filter('autoptimize_filter_css_defer', '__return_true');
1806
+        add_filter('autoptimize_filter_css_defer_inline', '__return_true');
1807 1807
 
1808
-        $actual = $this->ao->end_buffering( $this->get_test_markup() );
1809
-        if ( is_multisite() ) {
1810
-            $this->assertEquals( $this->get_test_markup_output_inline_defer_ms(), $actual );
1808
+        $actual = $this->ao->end_buffering($this->get_test_markup());
1809
+        if (is_multisite()) {
1810
+            $this->assertEquals($this->get_test_markup_output_inline_defer_ms(), $actual);
1811 1811
         } else {
1812
-            $this->assertEquals( $this->get_test_markup_output_inline_defer(), $actual );
1812
+            $this->assertEquals($this->get_test_markup_output_inline_defer(), $actual);
1813 1813
         }
1814 1814
 
1815
-        remove_all_filters( 'autoptimize_filter_css_defer' );
1816
-        remove_all_filters( 'autoptimize_filter_css_defer_inline' );
1815
+        remove_all_filters('autoptimize_filter_css_defer');
1816
+        remove_all_filters('autoptimize_filter_css_defer_inline');
1817 1817
     }
1818 1818
 
1819 1819
     public function test_js_aggregation_decision_and_dontaggregate_filter()
@@ -1821,47 +1821,47 @@  discard block
 block discarded – undo
1821 1821
         $opts = $this->getAoScriptsDefaultOptions();
1822 1822
 
1823 1823
         // Aggregating: true by default.
1824
-        $scripts = new autoptimizeScripts( '' );
1825
-        $scripts->read( $opts );
1826
-        $this->assertTrue( $scripts->aggregating() );
1824
+        $scripts = new autoptimizeScripts('');
1825
+        $scripts->read($opts);
1826
+        $this->assertTrue($scripts->aggregating());
1827 1827
 
1828 1828
         // Aggregating: option=true (dontaggregate=false by default).
1829 1829
         $opts['aggregate'] = true;
1830
-        $scripts           = new autoptimizeScripts( '' );
1831
-        $scripts->read( $opts );
1832
-        $this->assertTrue( $scripts->aggregating() );
1830
+        $scripts           = new autoptimizeScripts('');
1831
+        $scripts->read($opts);
1832
+        $this->assertTrue($scripts->aggregating());
1833 1833
 
1834 1834
         // Aggregating: option=true, dontaggregate=false explicit.
1835 1835
         $opts['aggregate'] = true;
1836
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_false' );
1837
-        $scripts = new autoptimizeScripts( '' );
1838
-        $scripts->read( $opts );
1839
-        $this->assertTrue( $scripts->aggregating() );
1840
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1836
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_false');
1837
+        $scripts = new autoptimizeScripts('');
1838
+        $scripts->read($opts);
1839
+        $this->assertTrue($scripts->aggregating());
1840
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1841 1841
 
1842 1842
         // Not aggregating: option=true, dontaggregate=true.
1843 1843
         $opts['aggregate'] = true;
1844
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_true' );
1845
-        $scripts = new autoptimizeScripts( '' );
1846
-        $scripts->read( $opts );
1847
-        $this->assertFalse( $scripts->aggregating() );
1848
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1844
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_true');
1845
+        $scripts = new autoptimizeScripts('');
1846
+        $scripts->read($opts);
1847
+        $this->assertFalse($scripts->aggregating());
1848
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1849 1849
 
1850 1850
         // Not aggregating: option=false, dontaggregate=false.
1851 1851
         $opts['aggregate'] = false;
1852
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_false' );
1853
-        $scripts = new autoptimizeScripts( '' );
1854
-        $scripts->read( $opts );
1855
-        $this->assertFalse( $scripts->aggregating() );
1856
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1852
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_false');
1853
+        $scripts = new autoptimizeScripts('');
1854
+        $scripts->read($opts);
1855
+        $this->assertFalse($scripts->aggregating());
1856
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1857 1857
 
1858 1858
         // Not aggregating: option=false, dontaggregate=true.
1859 1859
         $opts['aggregate'] = false;
1860
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_true' );
1861
-        $scripts = new autoptimizeScripts( '' );
1862
-        $scripts->read( $opts );
1863
-        $this->assertFalse( $scripts->aggregating() );
1864
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1860
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_true');
1861
+        $scripts = new autoptimizeScripts('');
1862
+        $scripts->read($opts);
1863
+        $this->assertFalse($scripts->aggregating());
1864
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1865 1865
     }
1866 1866
 
1867 1867
     public function test_css_aggregation_decision_and_dontaggregate_filter()
@@ -1869,78 +1869,78 @@  discard block
 block discarded – undo
1869 1869
         $opts = $this->getAoStylesDefaultOptions();
1870 1870
 
1871 1871
         // Aggregating: true by default.
1872
-        $styles = new autoptimizeStyles( '' );
1873
-        $this->assertTrue( $styles->aggregating() );
1872
+        $styles = new autoptimizeStyles('');
1873
+        $this->assertTrue($styles->aggregating());
1874 1874
 
1875 1875
         // Aggregating: option=true (dontaggregate=false by default).
1876 1876
         $opts['aggregate'] = true;
1877
-        $styles            = new autoptimizeStyles( '' );
1878
-        $styles->read( $opts );
1879
-        $this->assertTrue( $styles->aggregating() );
1877
+        $styles            = new autoptimizeStyles('');
1878
+        $styles->read($opts);
1879
+        $this->assertTrue($styles->aggregating());
1880 1880
 
1881 1881
         // Aggregating: option=true, dontaggregate=false explicit.
1882 1882
         $opts['aggregate'] = true;
1883
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_false' );
1884
-        $styles = new autoptimizeStyles( '' );
1885
-        $styles->read( $opts );
1886
-        $this->assertTrue( $styles->aggregating() );
1887
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1883
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_false');
1884
+        $styles = new autoptimizeStyles('');
1885
+        $styles->read($opts);
1886
+        $this->assertTrue($styles->aggregating());
1887
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1888 1888
 
1889 1889
         // Not aggregating: option=true, dontaggregate=true.
1890 1890
         $opts['aggregate'] = true;
1891
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_true' );
1892
-        $styles = new autoptimizeStyles( '' );
1893
-        $styles->read( $opts );
1894
-        $this->assertFalse( $styles->aggregating() );
1895
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1891
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_true');
1892
+        $styles = new autoptimizeStyles('');
1893
+        $styles->read($opts);
1894
+        $this->assertFalse($styles->aggregating());
1895
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1896 1896
 
1897 1897
         // Not aggregating: option=false, dontaggregate=false.
1898 1898
         $opts['aggregate'] = false;
1899
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_false' );
1900
-        $styles = new autoptimizeStyles( '' );
1901
-        $styles->read( $opts );
1902
-        $this->assertFalse( $styles->aggregating() );
1903
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1899
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_false');
1900
+        $styles = new autoptimizeStyles('');
1901
+        $styles->read($opts);
1902
+        $this->assertFalse($styles->aggregating());
1903
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1904 1904
 
1905 1905
         // Not aggregating: option=false, dontaggregate=true.
1906 1906
         $opts['aggregate'] = false;
1907
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_true' );
1908
-        $styles = new autoptimizeStyles( '' );
1909
-        $styles->read( $opts );
1910
-        $this->assertFalse( $styles->aggregating() );
1911
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1907
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_true');
1908
+        $styles = new autoptimizeStyles('');
1909
+        $styles->read($opts);
1910
+        $this->assertFalse($styles->aggregating());
1911
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1912 1912
     }
1913 1913
 
1914 1914
     public function test_css_minify_single_with_cdning()
1915 1915
     {
1916
-        $pathname = dirname( __FILE__ ) . '/fixtures/minify-single.css';
1916
+        $pathname = dirname(__FILE__).'/fixtures/minify-single.css';
1917 1917
         $opts     = $this->getAoStylesDefaultOptions();
1918
-        $styles   = new autoptimizeStyles( '' );
1919
-        $styles->read( $opts );
1918
+        $styles   = new autoptimizeStyles('');
1919
+        $styles->read($opts);
1920 1920
 
1921
-        $url = $styles->minify_single( $pathname, $cache_miss = true );
1921
+        $url = $styles->minify_single($pathname, $cache_miss = true);
1922 1922
 
1923 1923
         // Minified url filename + its pointed to cdn.
1924
-        $this->assertContains( AUTOPTIMIZE_CACHE_CHILD_DIR, $url );
1925
-        $this->assertContains( '/autoptimize_single_', $url );
1926
-        $this->assertContains( $styles->cdn_url, $url );
1924
+        $this->assertContains(AUTOPTIMIZE_CACHE_CHILD_DIR, $url);
1925
+        $this->assertContains('/autoptimize_single_', $url);
1926
+        $this->assertContains($styles->cdn_url, $url);
1927 1927
 
1928 1928
         // Actual minified css contents are minified and cdn-ed.
1929
-        $path     = $styles->getpath( $url );
1930
-        $contents = file_get_contents( $path );
1931
-        $this->assertContains( $styles->cdn_url, $contents );
1932
-        $this->assertContains( '.bg{background:url(' . $styles->cdn_url, $contents );
1929
+        $path     = $styles->getpath($url);
1930
+        $contents = file_get_contents($path);
1931
+        $this->assertContains($styles->cdn_url, $contents);
1932
+        $this->assertContains('.bg{background:url('.$styles->cdn_url, $contents);
1933 1933
     }
1934 1934
 
1935 1935
     public function test_ao_partners_instantiation_without_explicit_include()
1936 1936
     {
1937 1937
         $partners = new autoptimizePartners();
1938
-        $this->assertTrue( $partners instanceof autoptimizePartners );
1938
+        $this->assertTrue($partners instanceof autoptimizePartners);
1939 1939
     }
1940 1940
 
1941 1941
     public function test_html_minify_keep_html_comments_inside_script_blocks()
1942 1942
     {
1943
-        $markup   = <<<MARKUP
1943
+        $markup = <<<MARKUP
1944 1944
 <script>
1945 1945
 <!-- End Support AJAX add to cart -->
1946 1946
 var a = "b";
@@ -1970,23 +1970,23 @@  discard block
 block discarded – undo
1970 1970
             ],
1971 1971
         ];
1972 1972
 
1973
-        $instance = new autoptimizeHTML( $markup );
1974
-        $instance->read( $options['autoptimizeHTML'] );
1973
+        $instance = new autoptimizeHTML($markup);
1974
+        $instance->read($options['autoptimizeHTML']);
1975 1975
         $instance->minify();
1976 1976
         $actual = $instance->getcontent();
1977
-        $this->assertEquals( $expected, $actual );
1977
+        $this->assertEquals($expected, $actual);
1978 1978
 
1979
-        $instance = new autoptimizeHTML( $markup2 );
1980
-        $instance->read( $options['autoptimizeHTML'] );
1979
+        $instance = new autoptimizeHTML($markup2);
1980
+        $instance->read($options['autoptimizeHTML']);
1981 1981
         $instance->minify();
1982 1982
         $actual2 = $instance->getcontent();
1983
-        $this->assertEquals( $expected2, $actual2 );
1983
+        $this->assertEquals($expected2, $actual2);
1984 1984
     }
1985 1985
 
1986 1986
     public function test_html_minify_remove_html_comments_inside_script_blocks()
1987 1987
     {
1988 1988
         // Default case, html comments removed (keepcomments = false).
1989
-        $markup1   = <<<MARKUP
1989
+        $markup1 = <<<MARKUP
1990 1990
 <script>
1991 1991
 var a = "b";
1992 1992
 <!-- End Support AJAX add to cart -->
@@ -1997,7 +1997,7 @@  discard block
 block discarded – undo
1997 1997
 <!-- End Support AJAX add to cart</script>
1998 1998
 MARKUP;
1999 1999
 
2000
-        $markup2   = <<<MARKUP
2000
+        $markup2 = <<<MARKUP
2001 2001
 <script>
2002 2002
 <!-- End Support AJAX add to cart -->
2003 2003
 var a = "b";
@@ -2014,17 +2014,17 @@  discard block
 block discarded – undo
2014 2014
             ],
2015 2015
         ];
2016 2016
 
2017
-        $instance = new autoptimizeHTML( $markup1 );
2018
-        $instance->read( $options['autoptimizeHTML'] );
2017
+        $instance = new autoptimizeHTML($markup1);
2018
+        $instance->read($options['autoptimizeHTML']);
2019 2019
         $instance->minify();
2020 2020
         $actual = $instance->getcontent();
2021
-        $this->assertEquals( $expected1, $actual );
2021
+        $this->assertEquals($expected1, $actual);
2022 2022
 
2023
-        $instance = new autoptimizeHTML( $markup2 );
2024
-        $instance->read( $options['autoptimizeHTML'] );
2023
+        $instance = new autoptimizeHTML($markup2);
2024
+        $instance->read($options['autoptimizeHTML']);
2025 2025
         $instance->minify();
2026 2026
         $actual2 = $instance->getcontent();
2027
-        $this->assertEquals( $expected2, $actual2 );
2027
+        $this->assertEquals($expected2, $actual2);
2028 2028
     }
2029 2029
 
2030 2030
     public function test_html_minify_html_comments_inside_script_blocks_old_school_pattern()
@@ -2048,11 +2048,11 @@  discard block
 block discarded – undo
2048 2048
             ],
2049 2049
         ];
2050 2050
 
2051
-        $instance = new autoptimizeHTML( $markup );
2052
-        $instance->read( $options['autoptimizeHTML'] );
2051
+        $instance = new autoptimizeHTML($markup);
2052
+        $instance->read($options['autoptimizeHTML']);
2053 2053
         $instance->minify();
2054 2054
         $actual = $instance->getcontent();
2055
-        $this->assertEquals( $expected, $actual );
2055
+        $this->assertEquals($expected, $actual);
2056 2056
     }
2057 2057
 
2058 2058
     public function test_html_minify_html_comments_inside_script_blocks_old_school_pattern_untouched()
@@ -2077,49 +2077,49 @@  discard block
 block discarded – undo
2077 2077
             ],
2078 2078
         ];
2079 2079
 
2080
-        $instance = new autoptimizeHTML( $markup );
2081
-        $instance->read( $options['autoptimizeHTML'] );
2080
+        $instance = new autoptimizeHTML($markup);
2081
+        $instance->read($options['autoptimizeHTML']);
2082 2082
         $instance->minify();
2083 2083
         $actual = $instance->getcontent();
2084
-        $this->assertEquals( $expected, $actual );
2084
+        $this->assertEquals($expected, $actual);
2085 2085
     }
2086 2086
 
2087 2087
     public function test_utils_mbstring_availabilty_overriding()
2088 2088
     {
2089 2089
         $orig     = autoptimizeUtils::mbstring_available();
2090
-        $opposite = ! $orig;
2090
+        $opposite = !$orig;
2091 2091
 
2092
-        $this->assertSame( $orig, autoptimizeUtils::mbstring_available() );
2092
+        $this->assertSame($orig, autoptimizeUtils::mbstring_available());
2093 2093
         // Override works...
2094
-        $this->assertSame( $opposite, autoptimizeUtils::mbstring_available( $opposite ) );
2094
+        $this->assertSame($opposite, autoptimizeUtils::mbstring_available($opposite));
2095 2095
         // And override remains cached as the last version.
2096
-        $this->assertSame( $opposite, autoptimizeUtils::mbstring_available() );
2096
+        $this->assertSame($opposite, autoptimizeUtils::mbstring_available());
2097 2097
     }
2098 2098
 
2099 2099
     public function test_utils_mbstring_basics()
2100 2100
     {
2101 2101
         // Turn on mbstring usage.
2102
-        autoptimizeUtils::mbstring_available( true );
2102
+        autoptimizeUtils::mbstring_available(true);
2103 2103
 
2104
-        $this->assertSame( 2, autoptimizeUtils::strlen( "\x00\xFF", 'ASCII' ) );
2105
-        $this->assertSame( 2, autoptimizeUtils::strlen( "\x00\xFF", 'CP850' ) );
2106
-        $this->assertSame( 3, autoptimizeUtils::strlen( '한국어' ) );
2104
+        $this->assertSame(2, autoptimizeUtils::strlen("\x00\xFF", 'ASCII'));
2105
+        $this->assertSame(2, autoptimizeUtils::strlen("\x00\xFF", 'CP850'));
2106
+        $this->assertSame(3, autoptimizeUtils::strlen('한국어'));
2107 2107
 
2108
-        $this->assertFalse( @autoptimizeUtils::strpos( 'abc', '' ) );
2109
-        $this->assertFalse( @autoptimizeUtils::strpos( 'abc', 'a', -1 ) );
2110
-        $this->assertFalse( autoptimizeUtils::strpos( 'abc', 'd' ) );
2111
-        $this->assertFalse( autoptimizeUtils::strpos( 'abc', 'a', 3 ) );
2112
-        $this->assertSame( 1, autoptimizeUtils::strpos( '한국어', '국' ) );
2108
+        $this->assertFalse(@autoptimizeUtils::strpos('abc', ''));
2109
+        $this->assertFalse(@autoptimizeUtils::strpos('abc', 'a', -1));
2110
+        $this->assertFalse(autoptimizeUtils::strpos('abc', 'd'));
2111
+        $this->assertFalse(autoptimizeUtils::strpos('abc', 'a', 3));
2112
+        $this->assertSame(1, autoptimizeUtils::strpos('한국어', '국'));
2113 2113
     }
2114 2114
 
2115 2115
     /**
2116 2116
      * @dataProvider provider_utils_substr_replace
2117 2117
      */
2118
-    function test_utils_substr_replace_basics_mbstring( $s, $repl, $start, $len, $expected )
2118
+    function test_utils_substr_replace_basics_mbstring($s, $repl, $start, $len, $expected)
2119 2119
     {
2120 2120
         // Force mbstring code path...
2121
-        autoptimizeUtils::mbstring_available( true );
2122
-        $this->assertEquals( $expected, autoptimizeUtils::substr_replace( $s, $repl, $start, $len ) );
2121
+        autoptimizeUtils::mbstring_available(true);
2122
+        $this->assertEquals($expected, autoptimizeUtils::substr_replace($s, $repl, $start, $len));
2123 2123
     }
2124 2124
 
2125 2125
     public function provider_utils_substr_replace()
@@ -2161,48 +2161,48 @@  discard block
 block discarded – undo
2161 2161
 
2162 2162
     function test_mb_substr_replace_with_ascii_input_string()
2163 2163
     {
2164
-        autoptimizeUtils::mbstring_available( false );
2164
+        autoptimizeUtils::mbstring_available(false);
2165 2165
 
2166 2166
         $str = 'Ascii';
2167 2167
 
2168
-        $this->assertSame( 'Añ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) );
2169
-        $this->assertSame( 'ñcii', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2170
-        $this->assertSame( 'Asñx', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2171
-        $this->assertSame( 'Asz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) );
2172
-        $this->assertSame( 'Añii', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2168
+        $this->assertSame('Añ', autoptimizeUtils::substr_replace($str, 'ñ', 1));
2169
+        $this->assertSame('ñcii', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2170
+        $this->assertSame('Asñx', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2171
+        $this->assertSame('Asz', autoptimizeUtils::substr_replace($str, 'z', 2, 10));
2172
+        $this->assertSame('Añii', autoptimizeUtils::substr_replace($str, 'ñ', 1, 2));
2173 2173
     }
2174 2174
 
2175 2175
     function test_mb_substr_replace_with_utf8_input_string()
2176 2176
     {
2177
-        autoptimizeUtils::mbstring_available( true );
2177
+        autoptimizeUtils::mbstring_available(true);
2178 2178
 
2179 2179
         $str = 'âønæë';
2180 2180
 
2181
-        $this->assertSame( 'âñ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) ); // No length.
2182
-        $this->assertSame( 'ñnæë', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2183
-        $this->assertSame( 'âøñx', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2184
-        $this->assertSame( 'âøz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) ); // Length larger than possible...
2185
-        $this->assertSame( 'âñæë', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2181
+        $this->assertSame('âñ', autoptimizeUtils::substr_replace($str, 'ñ', 1)); // No length.
2182
+        $this->assertSame('ñnæë', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2183
+        $this->assertSame('âøñx', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2184
+        $this->assertSame('âøz', autoptimizeUtils::substr_replace($str, 'z', 2, 10)); // Length larger than possible...
2185
+        $this->assertSame('âñæë', autoptimizeUtils::substr_replace($str, 'ñ', 1, 2));
2186 2186
     }
2187 2187
 
2188 2188
     function test_default_substr_replace_with_ascii_input_string()
2189 2189
     {
2190 2190
         // Disable mbstring which should fall ack to substr_replace...
2191
-        autoptimizeUtils::mbstring_available( false );
2191
+        autoptimizeUtils::mbstring_available(false);
2192 2192
 
2193 2193
         $str = 'Ascii';
2194 2194
 
2195
-        $this->assertSame( 'Añ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) );
2196
-        $this->assertSame( 'ñcii', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2197
-        $this->assertSame( 'Asñx', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2198
-        $this->assertSame( 'Asz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) );
2199
-        $this->assertSame( 'Añii', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2195
+        $this->assertSame('Añ', autoptimizeUtils::substr_replace($str, 'ñ', 1));
2196
+        $this->assertSame('ñcii', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2197
+        $this->assertSame('Asñx', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2198
+        $this->assertSame('Asz', autoptimizeUtils::substr_replace($str, 'z', 2, 10));
2199
+        $this->assertSame('Añii', autoptimizeUtils::substr_replace($str, 'ñ', 1, 2));
2200 2200
     }
2201 2201
 
2202 2202
     function test_default_substr_replace_with_utf8_input_string()
2203 2203
     {
2204 2204
         // Disabling mbstring, falling back to substr_replace...
2205
-        autoptimizeUtils::mbstring_available( false );
2205
+        autoptimizeUtils::mbstring_available(false);
2206 2206
 
2207 2207
         // This is really impossible to make work properly, since
2208 2208
         // any start/len parameters we give are working with bytes instead
@@ -2211,55 +2211,55 @@  discard block
 block discarded – undo
2211 2211
 
2212 2212
         // $this->assertSame( '�ñ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) ); // No length.
2213 2213
         // $this->assertSame( 'ñ�næë', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2214
-        $this->assertSame( 'ñønæë', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2215
-        $this->assertSame( 'âñxæë', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2216
-        $this->assertSame( 'âz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) ); // Length larger than possible...
2214
+        $this->assertSame('ñønæë', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2215
+        $this->assertSame('âñxæë', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2216
+        $this->assertSame('âz', autoptimizeUtils::substr_replace($str, 'z', 2, 10)); // Length larger than possible...
2217 2217
     }
2218 2218
 
2219 2219
     public function test_cache_fast_delete()
2220 2220
     {
2221
-        add_filter( 'autoptimize_filter_cache_clear_advanced', '__return_true' );
2221
+        add_filter('autoptimize_filter_cache_clear_advanced', '__return_true');
2222 2222
 
2223 2223
         autoptimizeCache::clearall_actionless();
2224 2224
 
2225
-        remove_all_filters( 'autoptimize_filter_cache_clear_advanced' );
2225
+        remove_all_filters('autoptimize_filter_cache_clear_advanced');
2226 2226
     }
2227 2227
 
2228 2228
     public function test_delete_advanced_cache_clear_artifacts()
2229 2229
     {
2230
-        $this->assertTrue( autoptimizeCache::delete_advanced_cache_clear_artifacts() );
2230
+        $this->assertTrue(autoptimizeCache::delete_advanced_cache_clear_artifacts());
2231 2231
     }
2232 2232
 
2233 2233
     public function provider_canonicalization()
2234 2234
     {
2235 2235
         return array(
2236
-            array( '../common', 'common' ),
2237
-            array( '../what-does-this-mean/really?/', 'what-does-this-mean/really?/' ),
2238
-            array( '../../what/where/how', 'what/where/how' ),
2239
-            array( '/../more.dots.please/', '/more.dots.please/' ),
2240
-            array( '/../../what/where/how', '/what/where/how' ),
2241
-            array( '/a/b/c/../../../d/e/file.txt', '/d/e/file.txt' ),
2242
-            array( 'a/b/../c', 'a/c' ),
2243
-            array( './../../etc/passwd', './etc/passwd' ),
2244
-            array( '/var/.////./user/./././..//.//../////../././.././test/////', '/test/' ),
2245
-            array( '/var/user/./././.././../.././././test/', '/test/' ),
2246
-            array( '/hello/0//how/../are/../you', '/hello/0/you' ),
2247
-            array( '', '' ),
2248
-            array( '.', '.' ),
2249
-            array( '..', '..' ),
2250
-            array( './..', './..' ),
2251
-            array( '../.', '.' ),
2236
+            array('../common', 'common'),
2237
+            array('../what-does-this-mean/really?/', 'what-does-this-mean/really?/'),
2238
+            array('../../what/where/how', 'what/where/how'),
2239
+            array('/../more.dots.please/', '/more.dots.please/'),
2240
+            array('/../../what/where/how', '/what/where/how'),
2241
+            array('/a/b/c/../../../d/e/file.txt', '/d/e/file.txt'),
2242
+            array('a/b/../c', 'a/c'),
2243
+            array('./../../etc/passwd', './etc/passwd'),
2244
+            array('/var/.////./user/./././..//.//../////../././.././test/////', '/test/'),
2245
+            array('/var/user/./././.././../.././././test/', '/test/'),
2246
+            array('/hello/0//how/../are/../you', '/hello/0/you'),
2247
+            array('', ''),
2248
+            array('.', '.'),
2249
+            array('..', '..'),
2250
+            array('./..', './..'),
2251
+            array('../.', '.'),
2252 2252
             // This might be debatable...
2253
-            array( '../..', '..' ),
2253
+            array('../..', '..'),
2254 2254
         );
2255 2255
     }
2256 2256
 
2257 2257
     /**
2258 2258
      * @dataProvider provider_canonicalization
2259 2259
      */
2260
-    public function test_path_canonicalize( $path, $canonicalized )
2260
+    public function test_path_canonicalize($path, $canonicalized)
2261 2261
     {
2262
-        $this->assertSame( $canonicalized, autoptimizeUtils::path_canonicalize( $path ) );
2262
+        $this->assertSame($canonicalized, autoptimizeUtils::path_canonicalize($path));
2263 2263
     }
2264 2264
 
2265 2265
     /**
@@ -2273,16 +2273,16 @@  discard block
 block discarded – undo
2273 2273
     public function test_autoptimize_filter_cdn_magic_path_check()
2274 2274
     {
2275 2275
         $cdnurl = 'http://cdn.example.org';
2276
-        add_filter( 'autoptimize_filter_cdn_magic_path_check', '__return_false', 10, 2 );
2276
+        add_filter('autoptimize_filter_cdn_magic_path_check', '__return_false', 10, 2);
2277 2277
 
2278 2278
         // Even when site is in a subfolder, the resulting cdn-rewritten url
2279 2279
         // should not magically include it, due to the above filter.
2280 2280
         // The second parameter is here to force a cache miss and re-run
2281 2281
         // the filter since we're using the same cdn url all over the place,
2282 2282
         // but want to prove different things with it.
2283
-        $this->assertSame( $cdnurl, autoptimizeUtils::tweak_cdn_url_if_needed( $cdnurl, true ) );
2283
+        $this->assertSame($cdnurl, autoptimizeUtils::tweak_cdn_url_if_needed($cdnurl, true));
2284 2284
 
2285
-        remove_all_filters( 'autoptimize_filter_cdn_magic_path_check' );
2285
+        remove_all_filters('autoptimize_filter_cdn_magic_path_check');
2286 2286
     }
2287 2287
 
2288 2288
     /**
@@ -2307,9 +2307,9 @@  discard block
 block discarded – undo
2307 2307
 <img src='$imgopthost/client/q_glossy,ret_img,w_400,h_200/$siteurl/wp-content/image.jpg' width='400' height='200' srcset="$imgopthost/client/q_glossy,ret_img,w_300/$siteurl/wp-content/image-300X150.jpg 300w, $imgopthost/client/q_glossy,ret_img,w_600/$siteurl/wp-content/image-600X300.jpg 600w" sizes="(max-width: 300px) 100vw, 300px" />
2308 2308
 MARKUP;
2309 2309
         $instance = autoptimizeImages::instance();
2310
-        $instance->set_options( $opts );
2311
-        $actual = $instance->filter_optimize_images( $markup );
2312
-        $this->assertEquals( $expected, $actual );
2310
+        $instance->set_options($opts);
2311
+        $actual = $instance->filter_optimize_images($markup);
2312
+        $this->assertEquals($expected, $actual);
2313 2313
     }
2314 2314
 
2315 2315
     /**
@@ -2335,9 +2335,9 @@  discard block
 block discarded – undo
2335 2335
 MARKUP;
2336 2336
 
2337 2337
         $instance = autoptimizeImages::instance();
2338
-        $instance->set_options( $opts );
2339
-        $actual = $instance->filter_optimize_images( $markup );
2340
-        $this->assertEquals( $expected, $actual );
2338
+        $instance->set_options($opts);
2339
+        $actual = $instance->filter_optimize_images($markup);
2340
+        $this->assertEquals($expected, $actual);
2341 2341
     }
2342 2342
 
2343 2343
     /**
@@ -2363,9 +2363,9 @@  discard block
 block discarded – undo
2363 2363
 MARKUP;
2364 2364
 
2365 2365
         $instance = autoptimizeImages::instance();
2366
-        $instance->set_options( $opts );
2367
-        $actual = $instance->filter_optimize_images( $markup );
2368
-        $this->assertEquals( $expected, $actual );
2366
+        $instance->set_options($opts);
2367
+        $actual = $instance->filter_optimize_images($markup);
2368
+        $this->assertEquals($expected, $actual);
2369 2369
     }
2370 2370
 
2371 2371
     /**
@@ -2390,9 +2390,9 @@  discard block
 block discarded – undo
2390 2390
 MARKUP;
2391 2391
 
2392 2392
         $instance = autoptimizeImages::instance();
2393
-        $instance->set_options( $opts );
2394
-        $actual = $instance->filter_lazyload_images( $markup );
2395
-        $this->assertEquals( $expected, $actual );
2393
+        $instance->set_options($opts);
2394
+        $actual = $instance->filter_lazyload_images($markup);
2395
+        $this->assertEquals($expected, $actual);
2396 2396
     }
2397 2397
 
2398 2398
     /**
@@ -2411,11 +2411,11 @@  discard block
 block discarded – undo
2411 2411
 MARKUP;
2412 2412
 
2413 2413
         $instance = autoptimizeImages::instance();
2414
-        $instance->set_options( $opts );
2415
-        $actual = $instance->filter_optimize_images( $markup );
2414
+        $instance->set_options($opts);
2415
+        $actual = $instance->filter_optimize_images($markup);
2416 2416
 
2417 2417
         // Expecting $markup since replacement should not happen.
2418
-        $this->assertEquals( $markup, $actual );
2418
+        $this->assertEquals($markup, $actual);
2419 2419
     }
2420 2420
 
2421 2421
     /**
@@ -2441,10 +2441,10 @@  discard block
 block discarded – undo
2441 2441
 MARKUP;
2442 2442
 
2443 2443
         $instance = autoptimizeImages::instance();
2444
-        $instance->set_options( $opts );
2445
-        $actual = $instance->filter_optimize_images( $markup );
2444
+        $instance->set_options($opts);
2445
+        $actual = $instance->filter_optimize_images($markup);
2446 2446
 
2447
-        $this->assertEquals( $expected, $actual );
2447
+        $this->assertEquals($expected, $actual);
2448 2448
     }
2449 2449
 
2450 2450
     /**
@@ -2468,10 +2468,10 @@  discard block
 block discarded – undo
2468 2468
 MARKUP;
2469 2469
 
2470 2470
         $instance = autoptimizeImages::instance();
2471
-        $instance->set_options( $opts );
2472
-        $actual = $instance->filter_optimize_images( $markup );
2471
+        $instance->set_options($opts);
2472
+        $actual = $instance->filter_optimize_images($markup);
2473 2473
 
2474
-        $this->assertEquals( $expected, $actual );
2474
+        $this->assertEquals($expected, $actual);
2475 2475
     }
2476 2476
 
2477 2477
     /**
@@ -2496,53 +2496,53 @@  discard block
 block discarded – undo
2496 2496
 MARKUP;
2497 2497
 
2498 2498
         $instance = autoptimizeImages::instance();
2499
-        $instance->set_options( $opts );
2500
-        $actual = $instance->filter_optimize_images( $markup );
2499
+        $instance->set_options($opts);
2500
+        $actual = $instance->filter_optimize_images($markup);
2501 2501
 
2502
-        $this->assertEquals( $expected, $actual );
2502
+        $this->assertEquals($expected, $actual);
2503 2503
     }
2504 2504
 
2505 2505
     /**
2506 2506
      * @dataProvider provider_str_is_valid_regex
2507 2507
      */
2508
-    public function test_str_is_valid_regex( $str, $expected )
2508
+    public function test_str_is_valid_regex($str, $expected)
2509 2509
     {
2510
-        $actual = autoptimizeUtils::str_is_valid_regex( $str );
2510
+        $actual = autoptimizeUtils::str_is_valid_regex($str);
2511 2511
 
2512
-        $this->assertEquals( $expected, $actual );
2512
+        $this->assertEquals($expected, $actual);
2513 2513
     }
2514 2514
 
2515 2515
     public function provider_str_is_valid_regex()
2516 2516
     {
2517 2517
         return [
2518
-            [ '<!--[if', false ],
2519
-            [ '&lt;--[if', false ],
2520
-            [ '/booya/i', true ],
2521
-            [ '~Valid(Regular)Expression~', true ],
2518
+            ['<!--[if', false],
2519
+            ['&lt;--[if', false],
2520
+            ['/booya/i', true],
2521
+            ['~Valid(Regular)Expression~', true],
2522 2522
         ];
2523 2523
     }
2524 2524
 
2525 2525
     /**
2526 2526
      * @dataProvider provider_protocol_relative_tests
2527 2527
      */
2528
-    public function test_is_protocol_relative( $str, $expected )
2528
+    public function test_is_protocol_relative($str, $expected)
2529 2529
     {
2530
-        $actual = autoptimizeUtils::is_protocol_relative( $str );
2530
+        $actual = autoptimizeUtils::is_protocol_relative($str);
2531 2531
 
2532
-        $this->assertEquals( $expected, $actual );
2532
+        $this->assertEquals($expected, $actual);
2533 2533
     }
2534 2534
 
2535 2535
     public function provider_protocol_relative_tests()
2536 2536
     {
2537 2537
         return [
2538
-            [ '//something-that-might-be-an-url-but-isnt-really', true ],
2539
-            [ '', false ],
2540
-            [ null, false ],
2541
-            [ 'booya/i', false ],
2542
-            [ '/root-relative', false ],
2543
-            [ 'http://what.ever/', false ],
2544
-            [ 'https://booya.kasha', false ],
2545
-            [ '1/', false ],
2538
+            ['//something-that-might-be-an-url-but-isnt-really', true],
2539
+            ['', false],
2540
+            [null, false],
2541
+            ['booya/i', false],
2542
+            ['/root-relative', false],
2543
+            ['http://what.ever/', false],
2544
+            ['https://booya.kasha', false],
2545
+            ['1/', false],
2546 2546
         ];
2547 2547
     }
2548 2548
 
@@ -2557,38 +2557,38 @@  discard block
 block discarded – undo
2557 2557
 
2558 2558
         $expected = 'img.centered,.aligncenter{-webkit-box-shadow:none;-moz-box-shadow:none;box-shadow:none}';
2559 2559
 
2560
-        $instance = new autoptimizeStyles( $css );
2561
-        $minified = $instance->run_minifier_on( $css );
2560
+        $instance = new autoptimizeStyles($css);
2561
+        $minified = $instance->run_minifier_on($css);
2562 2562
 
2563
-        $this->assertEquals( $expected, $minified );
2563
+        $this->assertEquals($expected, $minified);
2564 2564
     }
2565 2565
 
2566 2566
     public function test_css_font_names_mangling()
2567 2567
     {
2568 2568
         // Properly quoted font names dont get mangled.
2569 2569
         $css      = 'h2{font-family:"Archivo Black"}';
2570
-        $instance = new autoptimizeStyles( $css );
2571
-        $actual   = $instance->run_minifier_on( $css );
2572
-        $this->assertEquals( $css, $actual );
2570
+        $instance = new autoptimizeStyles($css);
2571
+        $actual   = $instance->run_minifier_on($css);
2572
+        $this->assertEquals($css, $actual);
2573 2573
 
2574 2574
         // When not quoted, 'Black' used to become '#000', but not anymore... :)
2575 2575
         $css_unquoted = 'h2{font-family:Archivo Black;}';
2576 2576
         $expected     = 'h2{font-family:Archivo Black}';
2577
-        $instance     = new autoptimizeStyles( $css_unquoted );
2578
-        $actual       = $instance->run_minifier_on( $css_unquoted );
2579
-        $this->assertEquals( $expected, $actual );
2577
+        $instance     = new autoptimizeStyles($css_unquoted);
2578
+        $actual       = $instance->run_minifier_on($css_unquoted);
2579
+        $this->assertEquals($expected, $actual);
2580 2580
 
2581 2581
         $css_unquoted = 'h1{font:italic 1.2em Fira White,serif}';
2582
-        $instance     = new autoptimizeStyles( $css_unquoted );
2583
-        $actual       = $instance->run_minifier_on( $css_unquoted );
2584
-        $this->assertEquals( $css_unquoted, $actual );
2582
+        $instance     = new autoptimizeStyles($css_unquoted);
2583
+        $actual       = $instance->run_minifier_on($css_unquoted);
2584
+        $this->assertEquals($css_unquoted, $actual);
2585 2585
     }
2586 2586
 
2587 2587
     public function test_is_plugin_active_utils_wrapper()
2588 2588
     {
2589 2589
         // Our plugin is loaded via "muplugins_loaded" filter in tests/bootstrap.php
2590
-        $this->assertFalse( autoptimizeUtils::is_plugin_active( 'autoptimize/autoptimize.php' ) );
2591
-        $this->assertFalse( autoptimizeUtils::is_plugin_active( 'async-javascript/async-javascript.php' ) );
2590
+        $this->assertFalse(autoptimizeUtils::is_plugin_active('autoptimize/autoptimize.php'));
2591
+        $this->assertFalse(autoptimizeUtils::is_plugin_active('async-javascript/async-javascript.php'));
2592 2592
     }
2593 2593
 
2594 2594
     public function test_jsminphp_string_literal_minification()
@@ -2602,7 +2602,7 @@  discard block
 block discarded – undo
2602 2602
         $expected = '`line
2603 2603
 break`+`he  llo`;foo`hel( \'\');lo`;`he\nl\`lo`;(`he${one + two}`)';
2604 2604
 
2605
-        $actual = JSMin::minify( $js );
2606
-        $this->assertEquals( $expected, $actual );
2605
+        $actual = JSMin::minify($js);
2606
+        $this->assertEquals($expected, $actual);
2607 2607
     }
2608 2608
 }
Please login to merge, or discard this patch.