Revert "reorder constructor parameters, remove userAgent parameter"

This reverts commit eed2bd3.

Author: pavog

README.md

@@ -40,8 +40,6 @@ Most endpoints are public. Some endpoints (e.g. the daily stats of projects and
 require an API key, which you can create in your [account settings](https://hangar.papermc.io/auth/settings/api-keys).
 If an API key is set, it is used for all requests. The client exchanges it for a JWT automatically.
 ```php
-$hangarClient = new HangarAPIClient("your-api-key");
-// or
 $hangarClient->setApiKey("your-api-key");
 ```
 
@@ -69,15 +67,13 @@ try {
 
 ### Custom HTTP Client or Configuration
 You can pass your own [Guzzle](https://docs.guzzlephp.org/) client, e.g. to set timeouts or add middlewares,
-and your own `Aternos\HangarApi\Configuration`.
-Each client creates its own configuration if none is passed, so changes to `Configuration::getDefaultConfiguration()`
-do not affect the client. A configuration passed to the client is used as is, so set your user agent on it.
+and your own `Aternos\HangarApi\Configuration`:
 ```php
 use Aternos\HangarApi\Configuration;
 use GuzzleHttp\Client;
 
 $hangarClient = new HangarAPIClient(
-    configuration: new Configuration()->setUserAgent("my-app/1.0.0 (contact@example.com)"),
+    configuration: new Configuration(),
     httpClient: new Client(["timeout" => 10]),
 );
 // or

src/Client/HangarAPIClient.php

@@ -70,22 +70,17 @@ class HangarAPIClient
 
     protected PagesApi $pages;
 
-    /**
-     * @param string|null $apiKey API key used for authentication, see {@see setApiKey()}
-     * @param Configuration|null $configuration Configuration used for all requests. If null, a new configuration
-     * with the default user agent of this library is created, see {@see getDefaultUserAgent()}.
-     * @param ClientInterface|null $httpClient HTTP client used for all requests, see {@see setHttpClient()}
-     */
     public function __construct(
-        ?string          $apiKey = null,
         ?Configuration   $configuration = null,
+        ?string          $apiKey = null,
         ?ClientInterface $httpClient = null,
+        ?string          $userAgent = null,
     )
     {
         $this->apiKey = $apiKey;
         $this->httpClient = $httpClient;
         $this->setConfiguration($configuration ?? (new Configuration())
-            ->setUserAgent(static::getDefaultUserAgent()));
+            ->setUserAgent($userAgent ?? static::getDefaultUserAgent()));
     }
 
     /**

tests/Unit/Client/ClientTestCase.php

@@ -3,7 +3,6 @@
 namespace Aternos\HangarApi\Tests\Unit\Client;
 
 use Aternos\HangarApi\Client\HangarAPIClient;
-use Aternos\HangarApi\Configuration;
 use Aternos\HangarApi\Model\Pagination;
 use GuzzleHttp\Client;
 use GuzzleHttp\Handler\MockHandler;
@@ -36,19 +35,14 @@ public function setUp(): void
      * Create a client that answers every request with one of the given responses, in order,
      * and records all requests in {@see $requests}
      * @param Response[] $responses
-     * @param string|null $apiKey API key passed to the constructor of the client
-     * @param Configuration|null $configuration configuration passed to the constructor of the client
+     * @param string|null $userAgent user agent passed to the constructor of the client
      * @return HangarAPIClient
      */
-    protected function createClient(
-        array          $responses = [],
-        ?string        $apiKey = null,
-        ?Configuration $configuration = null,
-    ): HangarAPIClient
+    protected function createClient(array $responses = [], ?string $userAgent = null): HangarAPIClient
     {
         $stack = HandlerStack::create(new MockHandler($responses));
         $stack->push(Middleware::history($this->requests));
-        return new HangarAPIClient($apiKey, $configuration, new Client(['handler' => $stack]));
+        return new HangarAPIClient(httpClient: new Client(['handler' => $stack]), userAgent: $userAgent);
     }
 
     /**

tests/Unit/Client/HangarAPIClientTest.php

@@ -16,7 +16,6 @@
 use Aternos\HangarApi\Client\Project;
 use Aternos\HangarApi\Client\User;
 use Aternos\HangarApi\Client\Version;
-use Aternos\HangarApi\Configuration;
 use Aternos\HangarApi\Model\Category;
 use Aternos\HangarApi\Model\Platform;
 use Aternos\HangarApi\Model\ProjectMember;
@@ -58,37 +57,15 @@ public function testDefaultUserAgentIsSentWithRequests(): void
     /**
      * @throws ApiException
      */
-    public function testConfigurationFromConstructorIsUsed(): void
+    public function testUserAgentFromConstructorReplacesDefaultUserAgent(): void
     {
-        $configuration = new Configuration()
-            ->setHost("https://hangar.example.com")
-            ->setUserAgent("test-agent/1.0");
-        $client = $this->createClient([new Response(200, [], "1.0.0")], configuration: $configuration);
+        $client = $this->createClient([new Response(200, [], "1.0.0")], "test-agent/1.0");
 
         $client->getLatestReleaseVersion("mclogs");
 
-        $this->assertEquals("hangar.example.com", $this->getRequest()->getUri()->getHost());
         $this->assertEquals("test-agent/1.0", $this->getRequest()->getHeaderLine("User-Agent"));
     }
 
-    /**
-     * @throws ApiException
-     */
-    public function testApiKeyFromConstructorIsUsedForAuthentication(): void
-    {
-        $client = $this->createClient([
-            new Response(200, [], json_encode(["token" => "test-jwt", "expiresIn" => 3600])),
-            new Response(200, [], "1.0.0"),
-        ], "test-api-key");
-
-        $client->getLatestReleaseVersion("mclogs");
-
-        $this->assertEquals("POST", $this->getRequest(0)->getMethod());
-        $this->assertEquals("/api/v1/authenticate", $this->getRequest(0)->getUri()->getPath());
-        $this->assertEquals(["apiKey" => "test-api-key"], $this->getQuery(0));
-        $this->assertEquals("Bearer test-jwt", $this->getRequest(1)->getHeaderLine("Authorization"));
-    }
-
     /**
      * @throws ApiException
      */