Fix test.

Cannot rely on the order that the directory is iterated. On my development machine, in fact, these are returned in a different order.
horde · Feb 11, 2019 · 8b618e9 · 8b618e9
1 parent 1380532
commit 8b618e9
Show file tree

Hide file tree

Showing 2 changed files with 45 additions and 43 deletions.
diff --git a/test/Horde/Compress/TarTest.php b/test/Horde/Compress/TarTest.php
@@ -112,14 +112,24 @@ public function testTarDirectory()
 
         $list = $compress->decompress($tar_data);
         $this->assertCount(3, $list);
-        $this->assertEquals('one.txt', $list[0]['name']);
-        $this->assertEquals(4, $list[0]['size']);
-        $this->assertEquals("One\n", $list[0]['data']);
-        $this->assertEquals('sub/three.txt', $list[1]['name']);
-        $this->assertEquals(6, $list[1]['size']);
-        $this->assertEquals("Three\n", $list[1]['data']);
-        $this->assertEquals('two.bin', $list[2]['name']);
-        $this->assertEquals(2, $list[2]['size']);
-        $this->assertEquals("\x02\x0a", $list[2]['data']);
+
+        $fixtures = array(
+            'one.txt' => array(4, "One\n"),
+            'sub/three.txt' => array(6, "Three\n"),
+            'two.bin' => array(2, "\x02\x0a")
+        );
+        foreach ($fixtures as $key => $testValues) {
+            $found = false;
+            foreach ($list as $file) {
+                if ($file['name'] == $key) {
+                    $found = true;
+                    $this->assertEquals($testValues[0], $file['size']);
+                    $this->assertEquals($testValues[1], $file['data']);
+                }
+            }
+            if (!$found) {
+                $this->fail($key . ' not found.');
+            }
+        }
     }
 }
diff --git a/test/Horde/Compress/ZipTest.php b/test/Horde/Compress/ZipTest.php
@@ -143,41 +143,33 @@ public function testZipDirectory()
             $zip_data, array('action' => Horde_Compress_Zip::ZIP_LIST)
         );
         $this->assertCount(3, $list);
-        $this->assertEquals('one.txt', $list[0]['name']);
-        $this->assertEquals(4, $list[0]['size']);
-        $this->assertEquals('sub/three.txt', $list[1]['name']);
-        $this->assertEquals(6, $list[1]['size']);
-        $this->assertEquals('two.bin', $list[2]['name']);
-        $this->assertEquals(2, $list[2]['size']);
 
-        $data = $compress->decompress(
-            $zip_data,
-            array(
-                'action' => Horde_Compress_Zip::ZIP_DATA,
-                'info' => $list,
-                'key' => 0
-            )
-        );
-        $this->assertEquals("One\n", $data);
-
-        $data = $compress->decompress(
-            $zip_data,
-            array(
-                'action' => Horde_Compress_Zip::ZIP_DATA,
-                'info' => $list,
-                'key' => 1
-            )
-        );
-        $this->assertEquals("Three\n", $data);
-
-        $data = $compress->decompress(
-            $zip_data,
-            array(
-                'action' => Horde_Compress_Zip::ZIP_DATA,
-                'info' => $list,
-                'key' => 2
-            )
+        $fixtures = array(
+            'one.txt' => array(4, "One\n"),
+            'sub/three.txt' => array(6, "Three\n"),
+            'two.bin' => array(2, "\x02\x0a")
         );
-        $this->assertEquals("\x02\x0a", $data);
+        foreach ($fixtures as $key => $testValues) {
+            $found = false;
+            for ($i = 0; $i < 3; $i++) {
+                $file = $list[$i];
+                if ($file['name'] == $key) {
+                    $found = true;
+                    $this->assertEquals($testValues[0], $file['size']);
+                    $data = $compress->decompress(
+                        $zip_data,
+                        array(
+                            'action' => Horde_Compress_Zip::ZIP_DATA,
+                            'info' => $list,
+                            'key' => $i
+                        )
+                    );
+                    $this->assertEquals($testValues[1], $data);
+                }
+            }
+            if (!$found) {
+                $this->fail($key . ' not found.');
+            }
+        }
     }
 }