Add tags into listing response

[MAILPOET-4441]

mailpoet/lib/API/JSON/ResponseBuilders/SubscribersResponseBuilder.php

@@ -66,6 +66,7 @@ class SubscribersResponseBuilder {
       'is_woocommerce_user' => $subscriber->getIsWoocommerceUser(),
       'created_at' => ($createdAt = $subscriber->getCreatedAt()) ? $createdAt->format(self::DATE_FORMAT) : null,
       'engagement_score' => $subscriber->getEngagementScore(),
+      'tags' => $this->buildTags($subscriber),
     ];
   }
 

mailpoet/tests/integration/API/JSON/ResponseBuilders/SubscribersResponseBuilderTest.php

@@ -60,6 +60,7 @@ class SubscribersResponseBuilderTest extends \MailPoetTest {
       ->withIsWooCommerceUser()
       ->withWpUserId(2)
       ->withSegments([$this->segment])
+      ->withTags([$this->tag])
       ->create();
   }
 
@@ -90,23 +91,10 @@ class SubscribersResponseBuilderTest extends \MailPoetTest {
     $this->assertCount(0, $response['unsubscribes']);
     $this->checkSubscription($response, $subscriber);
     // check tags
-    $tag = reset($response['tags']);
+    $this->checkTag($response, $subscriber);
-    $this->assertCount(1, $response['tags']);
-    $this->assertEquals($subscriber->getId(), $tag['subscriber_id']);
-    $this->assertEquals($this->tag->getId(), $tag['tag_id']);
-    $this->assertEquals($this->tag->getName(), $tag['name']);
-    $this->assertArrayHasKey('created_at', $tag);
-    $this->assertArrayHasKey('updated_at', $tag);
   }
 
   public function testItBuildsListingResponse(): void {
-//    'email' => $subscriber->getEmail(),
-//    'first_name' => $subscriber->getFirstName(),
-//    'last_name' => $subscriber->getLastName(),
-//    'subscriptions' => $this->buildSubscriptions($subscriber),
-//    'status' => $subscriber->getStatus(),
-//    'wp_user_id' => $subscriber->getWpUserId(),
-//    'is_woocommerce_user' => $subscriber->getIsWoocommerceUser(),
     $subscribers = [
       $this->subscriber1,
       $this->subscriber2,
@@ -128,6 +116,8 @@ class SubscribersResponseBuilderTest extends \MailPoetTest {
       $this->assertArrayHasKey('engagement_score', $item);
       // check subscriptions
       $this->checkSubscription($item, $subscriber);
+      // check tags
+      $this->checkTag($item, $subscriber);
     }
   }
 
@@ -141,6 +131,16 @@ class SubscribersResponseBuilderTest extends \MailPoetTest {
     $this->assertArrayHasKey('updated_at', $subscription);
   }
 
+  private function checkTag(array $responseItem, SubscriberEntity $subscriber): void {
+    $this->assertCount(1, $responseItem['tags']);
+    $tag = reset($responseItem['tags']);
+    $this->assertEquals($subscriber->getId(), $tag['subscriber_id']);
+    $this->assertEquals($this->tag->getId(), $tag['tag_id']);
+    $this->assertEquals($this->tag->getName(), $tag['name']);
+    $this->assertArrayHasKey('created_at', $tag);
+    $this->assertArrayHasKey('updated_at', $tag);
+  }
+
   protected function _after() {
     $this->truncateEntity(SegmentEntity::class);
     $this->truncateEntity(SubscriberEntity::class);
@@ -148,6 +148,4 @@ class SubscribersResponseBuilderTest extends \MailPoetTest {
     $this->truncateEntity(TagEntity::class);
     $this->truncateEntity(SubscriberTagEntity::class);
   }
-
-
 }