Changeset 42343 for trunk/tests/phpunit/tests/formatting/JSEscape.php

Timestamp:

11/30/2017 11:09:33 PM (8 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:

• trunk/tests/phpunit/tests/formatting/JSEscape.php (modified) (1 diff)

trunk/tests/phpunit/tests/formatting/JSEscape.php

@@ -6 +6 @@
 class Tests_Formatting_JSEscape extends WP_UnitTestCase {
     function test_js_escape_simple() {
-        $out = esc_js('foo bar baz();');
-        $this->assertEquals('foo bar baz();', $out);
+        $out = esc_js( 'foo bar baz();' );
+        $this->assertEquals( 'foo bar baz();', $out );
     }
 
     function test_js_escape_quotes() {
-        $out = esc_js('foo "bar" \'baz\'');
+        $out = esc_js( 'foo "bar" \'baz\'' );
         // does it make any sense to change " into "?  Why not \"?
-        $this->assertEquals("foo "bar" \'baz\'", $out);
+        $this->assertEquals( "foo "bar" \'baz\'", $out );
     }
 
     function test_js_escape_backslash() {
-        $bs = '\\';
-        $out = esc_js('foo '.$bs.'t bar '.$bs.$bs.' baz');
+        $bs  = '\\';
+        $out = esc_js( 'foo ' . $bs . 't bar ' . $bs . $bs . ' baz' );
         // \t becomes t - bug?
-        $this->assertEquals('foo t bar '.$bs.$bs.' baz', $out);
+        $this->assertEquals( 'foo t bar ' . $bs . $bs . ' baz', $out );
     }
 
     function test_js_escape_amp() {
-        $out = esc_js('foo & bar &baz;  ');
-        $this->assertEquals("foo & bar &baz;  ", $out);
+        $out = esc_js( 'foo & bar &baz;  ' );
+        $this->assertEquals( 'foo & bar &baz;  ', $out );
     }
 
     function test_js_escape_quote_entity() {
-        $out = esc_js('foo ' bar ' baz &');
-        $this->assertEquals("foo \\' bar \\' baz &", $out);
+        $out = esc_js( 'foo ' bar ' baz &' );
+        $this->assertEquals( "foo \\' bar \\' baz &", $out );
     }
 
     function test_js_no_carriage_return() {
-        $out = esc_js("foo\rbar\nbaz\r");
+        $out = esc_js( "foo\rbar\nbaz\r" );
         // \r is stripped
-        $this->assertequals("foobar\\nbaz", $out);
+        $this->assertequals( "foobar\\nbaz", $out );
     }
 
     function test_js_escape_rn() {
-        $out = esc_js("foo\r\nbar\nbaz\r\n");
+        $out = esc_js( "foo\r\nbar\nbaz\r\n" );
         // \r is stripped
-        $this->assertequals("foo\\nbar\\nbaz\\n", $out);
+        $this->assertequals( "foo\\nbar\\nbaz\\n", $out );
     }
 }