Simplify test

Author: Maks3w

test/FastPriorityQueueTest.php

@@ -128,17 +128,7 @@ public function testCanRetrieveQueueAsArray()
 
     public function testIteratorFunctions()
     {
-        $this->queue->rewind();
-
-        $i = 0;
-        while ($this->queue->valid()) {
-            $key   = $this->queue->key();
-            $value = $this->queue->current();
-            $this->assertEquals($this->expected[$i], $value);
-            $this->queue->next();
-            ++$i;
-        }
-        $this->assertFalse($this->queue->valid());
+        $this->assertEquals($this->expected, iterator_to_array($this->queue));
     }
 
     public function testRewindOperation()

test/PriorityListTest.php

@@ -101,11 +101,7 @@ public function testLIFOOnly()
         $this->list->insert('foobar', new \stdClass());
         $this->list->insert('barbaz', new \stdClass());
 
-        $orders = [];
-
-        foreach ($this->list as $key => $value) {
-            $orders[] = $key;
-        }
+        $orders = array_keys(iterator_to_array($this->list));
 
         $this->assertEquals(['barbaz', 'foobar', 'baz', 'bar', 'foo'], $orders);
     }
@@ -116,11 +112,7 @@ public function testPriorityOnly()
         $this->list->insert('bar', new \stdClass(), 0);
         $this->list->insert('baz', new \stdClass(), 2);
 
-        $orders = [];
-
-        foreach ($this->list as $key => $value) {
-            $orders[] = $key;
-        }
+        $orders = array_keys(iterator_to_array($this->list));
 
         $this->assertEquals(['baz', 'foo', 'bar'], $orders);
     }
@@ -131,11 +123,7 @@ public function testLIFOWithPriority()
         $this->list->insert('bar', new \stdClass(), 0);
         $this->list->insert('baz', new \stdClass(), 1);
 
-        $orders = [];
-
-        foreach ($this->list as $key => $value) {
-            $orders[] = $key;
-        }
+        $orders = array_keys(iterator_to_array($this->list));
 
         $this->assertEquals(['baz', 'bar', 'foo'], $orders);
     }
@@ -147,11 +135,7 @@ public function testFIFOWithPriority()
         $this->list->insert('bar', new \stdClass(), 0);
         $this->list->insert('baz', new \stdClass(), 1);
 
-        $orders = [];
-
-        foreach ($this->list as $key => $value) {
-            $orders[] = $key;
-        }
+        $orders = array_keys(iterator_to_array($this->list));
 
         $this->assertEquals(['baz', 'foo', 'bar'], $orders);
     }
@@ -165,11 +149,7 @@ public function testFIFOOnly()
         $this->list->insert('foobar', new \stdClass());
         $this->list->insert('barbaz', new \stdClass());
 
-        $orders = [];
-
-        foreach ($this->list as $key => $value) {
-            $orders[] = $key;
-        }
+        $orders = array_keys(iterator_to_array($this->list));
 
         $this->assertEquals(['foo', 'bar', 'baz', 'foobar', 'barbaz'], $orders);
     }
@@ -180,11 +160,7 @@ public function testPriorityWithNegativesAndNull()
         $this->list->insert('bar', new \stdClass(), 1);
         $this->list->insert('baz', new \stdClass(), -1);
 
-        $orders = [];
-
-        foreach ($this->list as $key => $value) {
-            $orders[] = $key;
-        }
+        $orders = array_keys(iterator_to_array($this->list));
 
         $this->assertEquals(['bar', 'foo', 'baz'], $orders);
     }

test/PriorityQueueTest.php

@@ -37,14 +37,8 @@ public function testSerializationAndDeserializationShouldMaintainState()
         $count = count($this->queue);
         $this->assertSame($count, count($unserialized), 'Expected count ' . $count . '; received ' . count($unserialized));
 
-        $expected = [];
-        foreach ($this->queue as $item) {
-            $expected[] = $item;
-        }
-        $test = [];
-        foreach ($unserialized as $item) {
-            $test[] = $item;
-        }
+        $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));
     }
 
@@ -101,10 +95,7 @@ public function testCanRemoveItemFromQueue()
     {
         $this->queue->remove('baz');
         $expected = ['bar', 'foo', 'bat'];
-        $test = [];
-        foreach ($this->queue as $item) {
-            $test[] = $item;
-        }
+        $test = array_values(iterator_to_array($this->queue));
         $this->assertEquals($expected, $test);
     }
 

test/SplPriorityQueueTest.php

@@ -39,10 +39,7 @@ public function testMaintainsInsertOrderForDataOfEqualPriority()
         $queue->insert('bat', 1000);
 
         $expected = ['foo', 'bar', 'baz', 'bat'];
-        $test     = [];
-        foreach ($queue as $datum) {
-            $test[] = $datum;
-        }
+        $test = array_values(iterator_to_array($queue));
         $this->assertEquals($expected, $test);
     }
 
@@ -53,14 +50,8 @@ public function testSerializationAndDeserializationShouldMaintainState()
         $count = count($this->queue);
         $this->assertSame($count, count($unserialized), 'Expected count ' . $count . '; received ' . count($unserialized));
 
-        $expected = [];
-        foreach ($this->queue as $item) {
-            $expected[] = $item;
-        }
-        $test = [];
-        foreach ($unserialized as $item) {
-            $test[] = $item;
-        }
+        $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));
     }
 

test/SplQueueTest.php

@@ -36,14 +36,8 @@ public function testSerializationAndDeserializationShouldMaintainState()
         $count = count($this->queue);
         $this->assertSame($count, count($unserialized));
 
-        $expected = [];
-        foreach ($this->queue as $item) {
-            $expected[] = $item;
-        }
-        $test = [];
-        foreach ($unserialized as $item) {
-            $test[] = $item;
-        }
+        $expected = iterator_to_array($this->queue);
+        $test = iterator_to_array($unserialized);
         $this->assertSame($expected, $test);
     }
 

test/SplStackTest.php

@@ -37,14 +37,8 @@ public function testSerializationAndDeserializationShouldMaintainState()
         $count = count($this->stack);
         $this->assertSame($count, count($unserialized));
 
-        $expected = [];
-        foreach ($this->stack as $item) {
-            $expected[] = $item;
-        }
-        $test = [];
-        foreach ($unserialized as $item) {
-            $test[] = $item;
-        }
+        $expected = iterator_to_array($this->stack);
+        $test = iterator_to_array($unserialized);
         $this->assertSame($expected, $test);
     }