Merge pull request #92 from azjezz/patch-1

Fix SplPriorityQueue::$serial not being reset after serialization

Conflicts:
	CHANGELOG.md

Author: weierophinney

CHANGELOG.md

@@ -22,6 +22,9 @@ All notable changes to this project will be documented in this file, in reverse
 
 ### Fixed
 
+- [#92](https://github.com/zendframework/zend-stdlib/pull/92) fixes serialization of `SplPriorityQueue` by ensuring its `$serial`
+  property is also serialized.
+
 - [#91](https://github.com/zendframework/zend-stdlib/pull/91) fixes behavior in the `ArrayObject` implementation that was not
   compatible with PHP 7.3.
 

src/SplPriorityQueue.php

@@ -84,7 +84,9 @@ public function serialize()
      */
     public function unserialize($data)
     {
+        $this->serial = PHP_INT_MAX;
         foreach (unserialize($data) as $item) {
+            $this->serial--;
             $this->insert($item['data'], $item['priority']);
         }
     }

test/SplPriorityQueueTest.php

@@ -48,20 +48,15 @@ public function testSerializationAndDeserializationShouldMaintainState()
     {
         $s = serialize($this->queue);
         $unserialized = unserialize($s);
-        $count = count($this->queue);
-        $this->assertSame(
-            $count,
-            count($unserialized),
-            'Expected count ' . $count . '; received ' . count($unserialized)
-        );
 
-        $expected = iterator_to_array($this->queue);
-        $test = iterator_to_array($unserialized);
-        $this->assertSame(
-            $expected,
-            $test,
-            'Expected: ' . var_export($expected, 1) . "\nReceived:" . var_export($test, 1)
-        );
+        // assert same size
+        $this->assertSameSize($this->queue, $unserialized);
+
+        // assert same values
+        $this->assertSame(iterator_to_array($this->queue), iterator_to_array($unserialized));
+
+        // assert equal
+        $this->assertEquals($this->queue, $unserialized);
     }
 
     public function testCanRetrieveQueueAsArray()