test get_path

Author: brenoroosevelt

src/DotNotation.php

@@ -24,27 +24,6 @@ function set_path(array &$haystack, string $path, $value, string $separator = '.
     }
 
     $temp = $value;
-//
-//    $key = trim($path, $separator);
-//    $keys = explode($separator, $key);
-//    if (empty($keys)) {
-//        return;
-//    }
-//
-//    $target = &$haystack;
-//    foreach ($keys as $innerKey) {
-//        if (! is_array($target)) {
-//            break;
-//        }
-//
-//        if (! array_key_exists($innerKey, $target)) {
-//            $target[$innerKey] = [];
-//        }
-//
-//        $target = &$target[$innerKey];
-//    }
-//
-//    $target = $value;
 }
 
 /**
@@ -59,35 +38,10 @@ function get_path(array $haystack, string $path, $default = null, string $separa
     return
         array_reduce(
             explode($separator, $path),
-            fn ($arr, $p) => is_array($arr) && array_key_exists($p, $arr) ? $arr[$p] : $default,
+            fn ($arr, $key) =>
+            $default!== $arr && is_array($arr) && array_key_exists($key, $arr) ? $arr[$key] : $default,
             $haystack
         );
-
-//    $path = explode($separator, $path); //if needed
-//    $temp =& $haystack;
-//
-//    foreach($path as $key) {
-//        $temp =& $temp[$key];
-//    }
-//
-//    return $temp;
-//
-//    $key = trim($path, $separator);
-//    $keys = explode($separator, $key);
-//    if (empty($keys)) {
-//        return $default;
-//    }
-//
-//    $target = $haystack;
-//    foreach ($keys as $innerKey) {
-//        if (! is_array($target) || ! array_key_exists($innerKey, $target)) {
-//            return $default;
-//        }
-//
-//        $target = $target[$innerKey];
-//    }
-//
-//    return $target;
 }
 
 /**

tests/DotNotationTest.php

@@ -4,6 +4,7 @@
 namespace BrenoRoosevelt\Tests;
 
 use PHPUnit\Framework\TestCase;
+use function BrenoRoosevelt\get_path;
 use function BrenoRoosevelt\set_path;
 
 class DotNotationTest extends TestCase
@@ -63,4 +64,66 @@ public function testSetPath(array $haystack, string $path, $value, string $separ
         set_path($haystack, $path, $value, $separator);
         $this->assertEquals($expected, $haystack);
     }
+
+    public function getPathProvider(): array
+    {
+        return [
+            'case_1' => [
+                [],
+                '',
+                1,
+                1
+            ],
+            'case_2' => [
+                [4],
+                '0',
+                null,
+                4
+            ],
+            'case_3' => [
+                [[[[4, 5]]]],
+                '0.0.0.1',
+                null,
+                5
+            ],
+            'case_4' => [
+                [['a' => ['b' => ['c']]]],
+                '0.a.b',
+                null,
+                ['c']
+            ],
+            'case_5' => [
+                [['a' => ['b' => ['c']]]],
+                '0.a.b.c',
+                null,
+                null
+            ],
+            'case_6' => [
+                [['a' => ['b' => ['c', 'd' => 9, 3]]], 9, 8, 7],
+                '0.a.b.d',
+                null,
+                9
+            ],
+            'case_7' => [
+                ['a' => ['b' => []]],
+                'a.b.c.d',
+                ['c' => ['d' => 7]],
+                ['c' => ['d' => 7]]
+            ]
+        ];
+    }
+
+    /**
+     * @param array $items
+     * @param string $path
+     * @param $default
+     * @param $expected
+     * @return void
+     * @dataProvider getPathProvider
+     */
+    public function testGetPath(array $items, string $path, $default, $expected): void
+    {
+        $actual = get_path($items, $path, $default);
+        $this->assertEquals($expected, $actual);
+    }
 }