bugfix: Fix missing ids in NoSubscriptionsFound's serialized criteria output (#4)

Co-authored-by: David Schönhofen <d.schoenhofen.dev@gmail.com>

Author: d-scho

src/Subscription/SubscriptionIds.php

@@ -11,7 +11,7 @@
 /**
  * @implements \IteratorAggregate<SubscriptionId>
  */
-final readonly class SubscriptionIds implements IteratorAggregate, Countable
+final readonly class SubscriptionIds implements IteratorAggregate, Countable, JsonSerializable
 {
     /**
      * @param array<string, SubscriptionId> $subscriptionIdsById
@@ -69,4 +69,12 @@ public function toStringArray(): array
     {
         return array_values(array_map(static fn (SubscriptionId $id) => $id->value, $this->subscriptionIdsById));
     }
+
+    /**
+     * @return list<string>
+     */
+    public function jsonSerialize(): array
+    {
+        return $this->toStringArray();
+    }
 }

tests/PHPUnit/Engine/SubscriptionEngineTest.php

@@ -352,7 +352,7 @@ public function test_reset_does_not_update_subscriptions_if_none_match(): void
         );
         $this->assertEmittedEngineEvents(
             'ResetStarted: Start to setup',
-            'NoSubscriptionsFound: No subscriptions found for criteria: {"ids":{},"status":["NEW","BOOTING","ACTIVE","DETACHED","ERROR"]}'
+            'NoSubscriptionsFound: No subscriptions found for criteria: {"ids":["s3"],"status":["NEW","BOOTING","ACTIVE","DETACHED","ERROR"]}'
         );
         self::assertTrue($result->successful());
         self::assertFalse($this->subscriptionStore->_hasPendingChanges(), 'Subscription store has pending changes');