Changeset 32789 for trunk/tests/phpunit/tests/formatting/WPTexturize.php

Timestamp:

06/16/2015 12:49:45 AM (10 years ago)

Author:

wonderboymusic

Message:

Improve the consistency of punctuation matching in wptexturize().

Adds unit tests.

Props miqrogroove.
Fixes #31886.

File:

• trunk/tests/phpunit/tests/formatting/WPTexturize.php (modified) (4 diffs)

trunk/tests/phpunit/tests/formatting/WPTexturize.php

@@ -113 +113 @@
         $this->assertEquals('Class of ’99’s', wptexturize("Class of '99's"));
         $this->assertEquals('‘Class of ’99’', wptexturize("'Class of '99'"));
+        $this->assertEquals('‘Class of ’99’ ', wptexturize("'Class of '99' "));
+        $this->assertEquals('‘Class of ’99’.', wptexturize("'Class of '99'."));
+        $this->assertEquals('‘Class of ’99’, she said', wptexturize("'Class of '99', she said"));
+        $this->assertEquals('‘Class of ’99’:', wptexturize("'Class of '99':"));
+        $this->assertEquals('‘Class of ’99’;', wptexturize("'Class of '99';"));
+        $this->assertEquals('‘Class of ’99’!', wptexturize("'Class of '99'!"));
+        $this->assertEquals('‘Class of ’99’?', wptexturize("'Class of '99'?"));
         $this->assertEquals('‘Class of ’99’s’', wptexturize("'Class of '99's'"));
         $this->assertEquals('‘Class of ’99’s’', wptexturize("'Class of '99’s'"));
@@ -119 +126 @@
         $this->assertEquals('{“Class of ’99”}', wptexturize("{\"Class of '99\"}"));
         $this->assertEquals(' “Class of ’99” ', wptexturize(" \"Class of '99\" "));
+        $this->assertEquals(' “Class of ’99”.', wptexturize(" \"Class of '99\"."));
+        $this->assertEquals(' “Class of ’99”, she said', wptexturize(" \"Class of '99\", she said"));
+        $this->assertEquals(' “Class of ’99”:', wptexturize(" \"Class of '99\":"));
+        $this->assertEquals(' “Class of ’99”;', wptexturize(" \"Class of '99\";"));
+        $this->assertEquals(' “Class of ’99”!', wptexturize(" \"Class of '99\"!"));
+        $this->assertEquals(' “Class of ’99”?', wptexturize(" \"Class of '99\"?"));
         $this->assertEquals('}”Class of ’99″{', wptexturize("}\"Class of '99\"{")); // Not a quotation, may be between two other quotations.
     }
@@ -703 +716 @@
             ),
             array(
+                'test sentence",',
+                'test sentence”,',
+            ),
+            array(
+                'test sentence":',
+                'test sentence”:',
+            ),
+            array(
+                'test sentence";',
+                'test sentence”;',
+            ),
+            array(
+                'test sentence"!',
+                'test sentence”!',
+            ),
+            array(
+                'test sentence"?',
+                'test sentence”?',
+            ),
+            array(
                 'test sentence."',
                 'test sentence.”',
@@ -746 +779 @@
                 "word word', she said",
                 "word word’, she said",
+            ),
+            array(
+                "word word': word",
+                "word word’: word",
+            ),
+            array(
+                "word word'; word",
+                "word word’; word",
+            ),
+            array(
+                "word word'! word",
+                "word word’! word",
+            ),
+            array(
+                "word word'? word",
+                "word word’? word",
             ),
             array(