Create group rebased

.gitignore

@@ -4,3 +4,4 @@ composer.lock
 .php-cs-fixer.dist.php
 .idea/
 .build
+.php-version

Example/GroupsExample.php

@@ -41,6 +41,11 @@ public function groupsExample()
         $createGroupResponse = $group->create('Test');
         $newGroupKey = $createGroupResponse->getGroupKey();
 
+        // Obtain list of all groups
+        $listGroupsResponse = $group->list();
+        /** @var array $groups  ['name'=>'key','name2'=>'key2'] */
+        $groups = $listGroupsResponse->getGroups();
+
         // Retrieve information about the group from the API and populate the object with it.
         /** @var RetrieveGroupResponse $retrieveGroupResponse */
         $retrieveGroupResponse = $group->retrieveGroupInformation();

README.md

@@ -26,6 +26,7 @@ Light, simple and fast, yet comprehensive wrapper for the [Pushover](https://pus
   - Cancel emergency priority retry
 - Groups API ([Example](Example/GroupsExample.php))
   - Create a group
+  - List groups
   - Retrieve information about the group
   - Add / Remove users
   - Enable / Disable users

src/Api/Groups/Group.php

@@ -19,6 +19,7 @@
 use Serhiy\Pushover\Client\Request\Request;
 use Serhiy\Pushover\Client\Response\AddUserToGroupResponse;
 use Serhiy\Pushover\Client\Response\CreateGroupResponse;
+use Serhiy\Pushover\Client\Response\ListGroupsResponse;
 use Serhiy\Pushover\Client\Response\DisableUserInGroupResponse;
 use Serhiy\Pushover\Client\Response\EnableUserInGroupResponse;
 use Serhiy\Pushover\Client\Response\RemoveUserFromGroupResponse;
@@ -30,10 +31,13 @@
 /**
  * Pushover Delivery Group.
  *
- * Delivery Groups allow broadcasting the same Pushover message to a number of different users at once with just a single group token,
- * used in place of a user token (or in place of specifying multiple user keys in every request). For situations where subscriptions are not appropriate,
- * or where automated manipulation of group members is required, such as changing an on-call notification group,
- * or syncing with an external directory system, our Groups API is available.
+ * Delivery Groups allow broadcasting the same Pushover message to a number of 
+ * different users at once with just a single group token, used in place of a 
+ * user token (or in place of specifying multiple user keys in every request). 
+ * For situations where subscriptions are not appropriate, or where automated 
+ * manipulation of group members is required, such as changing an on-call 
+ * notification group, or syncing with an external directory system, our Groups
+ * API is available.
  *
  * @author Serhiy Lunak
  */
@@ -136,6 +140,23 @@ public function create(string $name): CreateGroupResponse
         return $response;
     }
 
+    /**
+     * List groups.
+     */
+    public function list(): ListGroupsResponse
+    {
+        $client = new GroupsClient($this, GroupsClient::ACTION_LIST_GROUPS);
+        $request = new Request($client->buildApiUrl(), Request::GET);
+
+        $curlResponse = Curl::do($request);
+
+        $response = new ListGroupsResponse($curlResponse);
+        $response->setRequest($request);
+
+        return $response;
+    }
+
+
     /**
      * Adds an existing Pushover user to your Delivery Group.
      *

src/Client/AssignLicenseClient.php

@@ -24,6 +24,9 @@ class AssignLicenseClient extends Client implements ClientInterface
 {
     public const API_PATH = 'licenses/assign.json';
 
+    /**
+     * {@inheritDoc}
+     */
     public function buildApiUrl(): string
     {
         return Curl::API_BASE_URL.'/'.Curl::API_VERSION.'/'.self::API_PATH;

src/Client/GroupsClient.php

@@ -30,6 +30,8 @@ class GroupsClient extends Client implements ClientInterface
     public const ACTION_ENABLE_USER = 'enable_user';
     public const ACTION_RENAME_GROUP = 'rename';
     public const ACTION_CREATE_GROUP = 'create';
+    public const ACTION_LIST_GROUPS = 'list';
+
     private Group $group;
 
     /**
@@ -53,6 +55,10 @@ public function buildApiUrl(): string
             return Curl::API_BASE_URL.'/'.Curl::API_VERSION.'/groups.json';
         }
 
+        if ($this->action == self::ACTION_LIST_GROUPS) {
+            return Curl::API_BASE_URL.'/'.Curl::API_VERSION.'/groups.json?token='.$this->group->getApplication()->getToken();
+        }
+
         if ($this->action === self::ACTION_RETRIEVE_GROUP) {
             return Curl::API_BASE_URL.'/'.Curl::API_VERSION.'/groups/'.$this->group->getKey().'.json?token='.$this->group->getApplication()->getToken();
         }

src/Client/Response/ListGroupsResponse.php

@@ -0,0 +1,57 @@
+<?php
+
+/*
+ * This file is part of the Pushover package.
+ *
+ * (c) Serhiy Lunak <https://github.com/slunak>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Serhiy\Pushover\Client\Response;
+
+use Serhiy\Pushover\Client\Response\Base\Response;
+
+/**
+ * @author Vítězslav Dvořák
+ */
+class ListGroupsResponse extends Response
+{
+    /**
+     * @var array
+     */
+    public $groups = [];
+
+    /**
+     * @param mixed $curlResponse
+     */
+    public function __construct($curlResponse)
+    {
+        $this->processCurlResponse($curlResponse);
+    }
+
+    /**
+     * @param mixed $curlResponse
+     */
+    private function processCurlResponse($curlResponse): void
+    {
+        $this->groups = [];
+        $decodedCurlResponse = $this->processInitialCurlResponse($curlResponse);
+        if (property_exists($decodedCurlResponse, 'groups')) {
+            foreach ($decodedCurlResponse->groups as $grp) {
+                $this->groups[$grp->name] = $grp->group;
+            }
+        }
+    }
+
+    /**
+     * List of groups
+     *
+     * @return array<string, string> group names with keys eg.['name'=>'key',..]
+     */
+    public function getGroups(): array
+    {
+        return $this->groups;
+    }
+}

tests/Api/Groups/GroupTest.php

@@ -18,6 +18,7 @@
 use Serhiy\Pushover\Application;
 use Serhiy\Pushover\Client\Response\CreateGroupResponse;
 use Serhiy\Pushover\Client\Response\RetrieveGroupResponse;
+use Serhiy\Pushover\Client\Response\ListGroupsResponse;
 
 /**
  * @author Serhiy Lunak
@@ -76,4 +77,18 @@ public function testCreate(): void
 
         $this->assertInstanceOf(CreateGroupResponse::class, $response);
     }
+
+    /**
+     * @group Integration
+     */
+    public function testList()
+    {
+        $application = new Application('cccc3333CCCC3333dddd4444DDDD44'); // using dummy token
+        $group = new Group('eeee5555EEEE5555ffff6666FFFF66', $application);
+
+        $response = $group->list();
+
+        $this->assertInstanceOf(ListGroupsResponse::class, $response);
+    }    
+
 }

tests/Client/GroupsClientTest.php

@@ -46,6 +46,8 @@ public function testBuildApiUrl(): void
 
         $client = new GroupsClient($group, GroupsClient::ACTION_ADD_USER);
         $this->assertEquals('https://api.pushover.net/1/groups/eeee5555EEEE5555ffff6666FFFF66/add_user.json?token=cccc3333CCCC3333dddd4444DDDD44', $client->buildApiUrl());
+      	$client = new GroupsClient($group, GroupsClient::ACTION_LIST_GROUPS);
+        $this->assertSame('https://api.pushover.net/1/groups.json?token=cccc3333CCCC3333dddd4444DDDD44', $client->buildApiUrl());
     }
 
     public function testBuildCurlPostFields(): void

tests/Client/Response/ListGroupsResponseTest.php

@@ -0,0 +1,44 @@
+<?php
+
+/*
+ * This file is part of the Pushover package.
+ *
+ * (c) Serhiy Lunak <https://github.com/slunak>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Client\Response;
+
+use Serhiy\Pushover\Client\Response\ListGroupsResponse;
+use PHPUnit\Framework\TestCase;
+use Serhiy\Pushover\Recipient;
+
+class ListGroupsResponseTest extends TestCase
+{
+    /**
+     * prepare ListGroupsResponse Test
+     */
+    public function testCanBeCreated(): ListGroupsResponse
+    {
+        $successfulCurlResponse = '{"groups":[{"group":"111111111111111111111111111111","name":"Group1"},{"group":"222222222222222222222222222222","name":"group2"},{"group":"333333333333333333333333333333","name":"Group 3"}],"status":1,"request":"aaaaaaaa-1111-bbbb-2222-cccccccccccc"}';
+        $response = new ListGroupsResponse($successfulCurlResponse);
+
+        $this->assertInstanceOf(ListGroupsResponse::class, $response);
+        $this->assertTrue($response->isSuccessful());
+        $this->assertEquals("aaaaaaaa-1111-bbbb-2222-cccccccccccc", $response->getRequestToken());
+        return $response;
+    }
+
+
+    /**
+     * @depends testCanBeCreated
+     * @param ListGroupsResponse $response
+     */
+    public function testGetGroups(ListGroupsResponse $response)
+    {
+        $groups = $response->getGroups();
+        $this->assertEquals($groups, ['Group1'=>'111111111111111111111111111111','group2'=>'222222222222222222222222222222', 'Group 3'=>'333333333333333333333333333333']);
+    }
+}