Ticket #4029: 4029.testcases.diff

test_includes_functions.php

@@ -9 +9 @@
 
         function tearDown() {
                 parent::tearDown();
+        }
+
+        function test_maybe_serialize_array_should_be_serialized()
+        {
+                $data = array();
+
+                $actual = maybe_serialize($data);
+
+                $this->assertTrue( is_string($data) );
+                $this->assertEquals($actual, serialize($data) );
+        }
+
+        function test_maybe_serialize_string_should_not_be_serialized()
+        {
+                $data = 'thisisatest';
+
+                $actual = maybe_serialize($data);
+
+                $this->assertEquals($actual, $data );
+        }
+
+        function test_maybe_serialized_string_should_be_serialized()
+        {
+                $data = serialize(array());
+                
+                $actual = maybe_serialize($data);
+                
+                $this->assertEquals($actual, serialize($data) );
+        }
+
+        function test_maybe_serialize_integer_should_not_be_serialized()
+        {
+                $data = 10;
+                
+                $actual = maybe_serialize($data);
+                
+                $this->assertEquals($actual, $data );
         }
 
         function test_wp_parse_args_object() {
@@ -19 +56 @@
                 $this->assertEquals(array('_baba' => 5, 'yZ' => 'baba', 'a' => array(5, 111, 'x')), wp_parse_args($x));
                 $y = new MockClass;
                 $this->assertEquals(array(), wp_parse_args($y));
-        }
+        }
+
         function test_wp_parse_args_array()  {
                 // arrays
                 $a = array();
                 $this->assertEquals(array(), wp_parse_args($x));
                 $b = array('_baba' => 5, 'yZ' => 'baba', 'a' => array(5, 111, 'x'));
                 $this->assertEquals(array('_baba' => 5, 'yZ' => 'baba', 'a' => array(5, 111, 'x')), wp_parse_args($b));
-        }
+        }
+
         function test_wp_parse_args_defaults() {
                 $x = new MockClass;
                 $x->_baba = 5;
@@ -36 +75 @@
                 $this->assertEquals(array('pu' => 'bu', '_baba' => 5, 'yZ' => 'baba', 'a' => array(5, 111, 'x')), wp_parse_args($x, $d));
                 $e = array('_baba' => 6);
                 $this->assertEquals(array('_baba' => 5, 'yZ' => 'baba', 'a' => array(5, 111, 'x')), wp_parse_args($x, $e));
-        }
+        }
+
         function test_wp_parse_args_other() {
                 $b = true;
                 wp_parse_str($b, $s);
@@ -44 +84 @@
                 $q = 'x=5&_baba=dudu&';
                 wp_parse_str($q, $ss);
                 $this->assertEquals($ss, wp_parse_args($q));
-        }
+        }
+
         function test_size_format() {
                 $this->knownWPBug(5246);
                 $kb = 1024;
@@ -65 +106 @@
                 $this->assertFalse(size_format('baba'));
                 $this->assertFalse(size_format(array()));
         }
-        
+
         function test_path_is_absolute() {
                 if ( !is_callable('path_is_absolute') )
                         $this->markTestSkipped();
@@ -106 +147 @@
                         $this->assertFalse( path_is_absolute($path), "path_is_absolute('$path') should return false" );
         }
 
-
         function test_wp_unique_filename() {
                 $this->knownWPBug(6294);