Changeset 44478

Timestamp:

01/08/2019 07:04:51 PM (6 years ago)

Author:

desrosj

Message:

Tests: Add missing test cases for is_serialized_string().

Also, reorganize these tests into their own class.

Props pbearne.
Fixes #42870.

Location:

trunk/tests/phpunit/tests

Files:

• functions.php (modified) (1 diff)
• functions/isSerializedString.php (added)

trunk/tests/phpunit/tests/functions.php

@@ -237 +237 @@
     }
 
-    /**
-     * @dataProvider data_is_serialized_string
-     */
-    public function test_is_serialized_string( $value, $result ) {
-        $this->assertSame( is_serialized_string( $value ), $result );
-    }
-
-    public function data_is_serialized_string() {
-        return array(
-            // Not a string.
-            array( 0, false ),
-
-            // Too short when trimmed.
-            array( 's:3   ', false ),
-
-            // Too short.
-            array( 's:3', false ),
-
-            // No colon in second position.
-            array( 's!3:"foo";', false ),
-
-            // No trailing semicolon.
-            array( 's:3:"foo"', false ),
-
-            // Wrong type.
-            array( 'a:3:"foo";', false ),
-
-            // No closing quote.
-            array( 'a:3:"foo;', false ),
-
-            // Wrong number of characters is close enough for is_serialized_string().
-            array( 's:12:"foo";', true ),
-
-            // Okay.
-            array( 's:3:"foo";', true ),
-
-        );
-    }
 
     /**