test all

Author: brenoroosevelt

composer.json

@@ -30,6 +30,7 @@
             "src/CallbackArgs.php",
             "src/IndexOf.php",
             "src/Contains.php",
+            "src/All.php",
             "src/Add.php",
             "src/Set.php",
             "src/Pipe.php",

src/All.php

@@ -0,0 +1,40 @@
+<?php
+declare(strict_types=1);
+
+namespace BrenoRoosevelt;
+
+/**
+ * Evaluates whether ALL elements match the specification
+ *
+ * @param iterable $items The collection
+ * @param callable $callback Callable must return a boolean
+ * @param bool $empty_is_valid When the collection is empty, indicates whether the function should return true/false
+ * @param int $mode [optional] <p>
+ * Flag determining what arguments are sent to <i>callback</i>:
+ * </p><ul>
+ * <li>
+ * <b>CALLBACK_USE_VALUE</b> - <b>default</b> pass value as the only argument
+ * </li>
+ * <li>
+ * <b>CALLBACK_USE_KEY</b> - pass key as the only argument
+ * to <i>callback</i> instead of the value</span>
+ * </li>
+ * <li>
+ * <b>CALLBACK_USE_BOTH</b> - pass both value and key as
+ * arguments to <i>callback</i></span>
+ * </li>
+ * </ul>
+ * @return bool
+ */
+function all(iterable $items, callable $callback, bool $empty_is_valid = true, int $mode = CALLBACK_USE_VALUE): bool
+{
+    $count = 0;
+    foreach ($items as $key => $value) {
+        $count++;
+        if (true !== call_user_func_array($callback, __args($mode, $key, $value))) {
+            return false;
+        }
+    }
+
+    return $empty_is_valid || $count > 0;
+}

src/array_functions.php

@@ -3,24 +3,6 @@
 
 namespace BrenoRoosevelt;
 
-function reindex(array &$items): void
-{
-    $items = array_values($items);
-}
-
-function all(iterable $items, callable $callback, bool $empty_is_valid = true, int $mode = CALLBACK_USE_VALUE): bool
-{
-    $count = 0;
-    foreach ($items as $key => $value) {
-        $count++;
-        if (true !== call_user_func_array($callback, __args($mode, $key, $value))) {
-            return false;
-        }
-    }
-
-    return $empty_is_valid || $count > 0;
-}
-
 function some(iterable $items, callable $callback, int $mode = CALLBACK_USE_VALUE): bool
 {
     return at_least(1, $items, $callback, $mode);

tests/AllTest.php

@@ -0,0 +1,83 @@
+<?php
+declare(strict_types=1);
+
+namespace BrenoRoosevelt\Tests;
+
+use function BrenoRoosevelt\all;
+use PHPUnit\Framework\TestCase;
+use const BrenoRoosevelt\CALLBACK_USE_BOTH;
+use const BrenoRoosevelt\CALLBACK_USE_KEY;
+use const BrenoRoosevelt\CALLBACK_USE_VALUE;
+
+class AllTest extends TestCase
+{
+    public function allProvider(): array
+    {
+        return [
+            'case_1' => [
+                ['a', 'b', 'c'], // collection
+                fn($el) => false, // callback
+                true, // empty_is_valid
+                CALLBACK_USE_VALUE, // mode
+                false, // expected
+            ],
+            'case_2' => [
+                ['a', 'b', 'c'],
+                fn($el) => true,
+                true,
+                CALLBACK_USE_VALUE,
+                true,
+            ],
+            'case_3' => [
+                [],
+                fn($el) => false,
+                true,
+                CALLBACK_USE_VALUE,
+                true,
+            ],
+            'case_4' => [
+                [],
+                fn($el) => true,
+                true,
+                CALLBACK_USE_VALUE,
+                true,
+            ],
+            'case_5' => [
+                [1, 2, 3],
+                fn($el) => $el % 2 === 0,
+                true,
+                CALLBACK_USE_VALUE,
+                false,
+            ],
+            'case_6' => [
+                ['a' => 1, 'b' => 2, 3],
+                fn($key) => is_integer($key),
+                true,
+                CALLBACK_USE_KEY,
+                false,
+            ],
+            'case_7' => [
+                ['a' => 'x', 'b' => 'y'],
+                fn($el, $key) => is_string($key) && is_string($el),
+                true,
+                CALLBACK_USE_BOTH,
+                true,
+            ]
+        ];
+    }
+
+    /**
+     * @param iterable $items
+     * @param callable $callback
+     * @param bool $emptyIsValid
+     * @param int $mode
+     * @param bool $expected
+     * @return void
+     * @dataProvider allProvider
+     */
+    public function testAdd(iterable $items, callable $callback, bool $emptyIsValid, int $mode, bool $expected): void
+    {
+        $actual = all($items, $callback, $emptyIsValid, $mode);
+        $this->assertEquals($expected, $actual);
+    }
+}