From f5e8a1834187745a50b9b5d88d21c8adb1d6baaf Mon Sep 17 00:00:00 2001 From: Chris Boden Date: Wed, 25 May 2016 08:44:41 -0400 Subject: [PATCH] Added unit tests for serialize, formatting --- src/Ratchet/Session/Serialize/PhpHandler.php | 6 +++--- tests/unit/Session/Serialize/PhpHandlerTest.php | 7 +++++++ 2 files changed, 10 insertions(+), 3 deletions(-) diff --git a/src/Ratchet/Session/Serialize/PhpHandler.php b/src/Ratchet/Session/Serialize/PhpHandler.php index 3420094..b1df356 100644 --- a/src/Ratchet/Session/Serialize/PhpHandler.php +++ b/src/Ratchet/Session/Serialize/PhpHandler.php @@ -9,14 +9,14 @@ class PhpHandler implements HandlerInterface { function serialize(array $data) { $preSerialized = array(); $serialized = ''; - + if (count($data)) { foreach ($data as $bucket => $bucketData) { $preSerialized[] = $bucket . '|' . serialize($bucketData); } - $serialized = implode('',$preSerialized); + $serialized = implode('', $preSerialized); } - + return $serialized; } diff --git a/tests/unit/Session/Serialize/PhpHandlerTest.php b/tests/unit/Session/Serialize/PhpHandlerTest.php index 4dddee9..4acf5bc 100644 --- a/tests/unit/Session/Serialize/PhpHandlerTest.php +++ b/tests/unit/Session/Serialize/PhpHandlerTest.php @@ -33,4 +33,11 @@ class PhpHandlerTest extends \PHPUnit_Framework_TestCase { public function testUnserialize($in, $expected) { $this->assertEquals($expected, $this->_handler->unserialize($in)); } + + /** + * @dataProvider serializedProvider + */ + public function testSerialize($serialized, $original) { + $this->assertEquals($serialized, $this->_handler->serialize($original)); + } }