add invalid argument exception

Author: polidog

src/Api/Rooms.php

@@ -13,6 +13,7 @@
 use Polidog\Chatwork\Entity\Factory\RoomFactory;
 use Polidog\Chatwork\Entity\Factory\TaskFactory;
 use Polidog\Chatwork\Entity\Room;
+use Polidog\Chatwork\Exception\InvalidArgumentException;
 
 class Rooms
 {
@@ -104,10 +105,15 @@ public function update(Room $room)
      * グループチャットを退席/削除する.
      *
      * @param Room   $room
-     * @param string $actionType
+     * @param string $actionType leave or delete
+     * @throws InvalidArgumentException
      */
     public function remove(Room $room, $actionType)
     {
+        if ($actionType !== 'leave' && $actionType !== 'delete') {
+            throw new InvalidArgumentException('ActionType is only leave or delete');
+        }
+
         $this->client->delete(
             "rooms/{$room->roomId}",
             [

src/Api/Rooms/Tasks.php

@@ -70,29 +70,20 @@ public function detail($id)
     }
 
     /**
-     * @TODO api見直し
-     *
-     * @param CollectionInterface $collection
+     * @param string $body
+     * @param array $toIds
+     * @param \DateTime|null $limit
+     * @return int[] task ids
      */
-    public function create(CollectionInterface $collection)
+    public function create(string $body, array $toIds, \DateTime $limit = null) : array
     {
-        $toIds = [];
-        foreach ($collection as $entity) {
-            $toIds[] = $entity->account->accountId;
-        }
-
-        $task = clone $collection->get(0);
-        $results = $this->client->post(
+        return $this->client->post(
             "rooms/{$this->roomId}/tasks",
             [
-                'body' => $task->body,
+                'body' => $body,
                 'to_ids' => implode(',', $toIds),
-                'limit' => $task->limitTime,
+                'limit' => $limit instanceof \DateTime ? $limit->getTimestamp() : null,
             ]
         );
-
-        foreach ($results['task_ids'] as $key => $id) {
-            $collection->get($key)->taskId = $id;
-        }
     }
 }

src/Exception/InvalidArgumentException.php

@@ -0,0 +1,9 @@
+<?php
+
+namespace Polidog\Chatwork\Exception;
+
+
+class InvalidArgumentException extends \InvalidArgumentException implements ChatworkApiException
+{
+
+}