feat: implement test for merge diffs action

Author: arg2009

src/Actions/MergeDiffsAction.php

@@ -10,17 +10,16 @@
 class MergeDiffsAction
 {
     /**
-     * @param Collection<JsonDiff> $diffs
+     * @param Collection<JsonDiff> $jsonDiffs
      */
-    public function execute(Collection $diffs): JsonDiff
+    public function execute(Collection $jsonDiffs): JsonDiff
     {
-        /** @var JsonDiff $mergedDiff */
-        $mergedDiff = $diffs->shift();
-        $diffs
-            ->each(function (JsonDiff $diff) use ($mergedDiff): void {
-                $mergedDiff->mergeChanges($diff);
+        $mergedJsonDiff = new JsonDiff([], []);
+        $jsonDiffs
+            ->each(function (JsonDiff $jsonDiff) use ($mergedJsonDiff): void {
+                $mergedJsonDiff->mergeChanges($jsonDiff);
             });
 
-        return $mergedDiff;
+        return $mergedJsonDiff;
     }
 }

tests/Factories/JsonDiffFactory.php

@@ -0,0 +1,91 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Jet\Tests\Factories;
+
+use Illuminate\Support\Collection;
+use Jet\JsonDiff\JsonDiff;
+
+class JsonDiffFactory
+{
+    /**
+     * @var bool
+     */
+    protected $createListArray = false;
+
+    /**
+     * @var array
+     */
+    protected $original;
+
+    /**
+     * @var array
+     */
+    protected $new;
+
+    public static function new(): self
+    {
+        return new self();
+    }
+
+    public function create(?array $original = null, ?array $new = null): JsonDiff
+    {
+        if ($this->createListArray) {
+            $arrayFactory = ListArrayFactory::new();
+        } else {
+            $arrayFactory = AssociativeArrayFactory::new();
+        }
+
+        return new JsonDiff(
+            $original ?? $this->original ?? $arrayFactory->create(),
+            $new ?? $this->new ?? $arrayFactory->create()
+        );
+    }
+
+    /**
+     * @return Collection<JsonDiff>
+     */
+    public function createMany(int $count): Collection
+    {
+        $jsonDiffs = collect([]);
+
+        for ($i = 0; $i < $count; $i++) {
+            $jsonDiffs->push($this->create());
+        }
+
+        return $jsonDiffs;
+    }
+
+    public function withOriginal(array $original): self
+    {
+        $clone = clone $this;
+        $clone->original = $original;
+
+        return $clone;
+    }
+
+    public function withNew(array $new): self
+    {
+        $clone = clone $this;
+        $clone->new = $new;
+
+        return $clone;
+    }
+
+    public function forAssociativeArray(): self
+    {
+        $clone = clone $this;
+        $this->createListArray = false;
+
+        return $clone;
+    }
+
+    public function forListArray(): self
+    {
+        $clone = clone $this;
+        $this->createListArray = true;
+
+        return $clone;
+    }
+}

tests/Unit/Actions/MergeDiffsActionTest.php

@@ -0,0 +1,34 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Jet\Tests\Unit\Actions;
+
+use Illuminate\Container\Container;
+use Jet\JsonDiff\Actions\MergeDiffsAction;
+use Jet\JsonDiff\JsonDiff;
+use Jet\Tests\Factories\JsonDiffFactory;
+use PHPUnit\Framework\TestCase;
+
+class MergeDiffsActionTest extends TestCase
+{
+    public function test_it_merges_all_json_diffs_into_one_json_diff(): void
+    {
+        $jsonDiffs = JsonDiffFactory::new()->createMany(2);
+
+        /** @var JsonDiff $firstJsonDiff */
+        $firstJsonDiff = $jsonDiffs->first();
+        /** @var JsonDiff $secondJsonDiff */
+        $secondJsonDiff = $jsonDiffs->skip(1)->first();
+
+        /** @var MergeDiffsAction $action */
+        $action = Container::getInstance()->make(MergeDiffsAction::class);
+
+        $mergedJsonDiff = $action->execute($jsonDiffs);
+
+        $this->assertSame(
+            $firstJsonDiff->getNumberOfChanges() + $secondJsonDiff->getNumberOfChanges(),
+            $mergedJsonDiff->getNumberOfChanges()
+        );
+    }
+}