refactor(client): Remove unused methods from DotCMSClient

Clean up the DotCMSClient class by removing the following methods:
- getConfig()
- getHttpClient()
- getPageService()

Update README and tests accordingly to reflect these changes.

Author: fmontes

README.md

@@ -189,9 +189,6 @@ The main client for interacting with the dotCMS API.
 - `getPage(PageRequest $request)`: Fetch a page synchronously
 - `getPageAsync(PageRequest $request)`: Fetch a page asynchronously
 - `createPageRequest(string $pagePath, string $format = 'json')`: Create a new page request
-- `getConfig()`: Get the client configuration
-- `getHttpClient()`: Get the HTTP client
-- `getPageService()`: Get the page service
 
 ### PageRequest
 

src/DotCMSClient.php

@@ -31,36 +31,6 @@ public function __construct(private readonly Config $config)
         $this->pageService = new PageService($this->httpClient);
     }
 
-    /**
-     * Get the configuration
-     *
-     * @return Config The client configuration
-     */
-    public function getConfig(): Config
-    {
-        return $this->config;
-    }
-
-    /**
-     * Get the HTTP client
-     *
-     * @return HttpClient The HTTP client
-     */
-    public function getHttpClient(): HttpClient
-    {
-        return $this->httpClient;
-    }
-
-    /**
-     * Get the page service
-     *
-     * @return PageService The page service
-     */
-    public function getPageService(): PageService
-    {
-        return $this->pageService;
-    }
-
     /**
      * Fetch a page from dotCMS
      *

tests/DotCMSClientTest.php

@@ -6,7 +6,6 @@
 
 use Dotcms\PhpSdk\Config\Config;
 use Dotcms\PhpSdk\DotCMSClient;
-use Dotcms\PhpSdk\Http\HttpClient;
 use Dotcms\PhpSdk\Model\PageAsset;
 use Dotcms\PhpSdk\Request\PageRequest;
 use Dotcms\PhpSdk\Service\PageService;
@@ -29,21 +28,6 @@ protected function setUp(): void
         $this->client = new DotCMSClient($this->config);
     }
 
-    public function testGetConfig(): void
-    {
-        $this->assertSame($this->config, $this->client->getConfig());
-    }
-
-    public function testGetHttpClient(): void
-    {
-        $this->assertInstanceOf(HttpClient::class, $this->client->getHttpClient());
-    }
-
-    public function testGetPageService(): void
-    {
-        $this->assertInstanceOf(PageService::class, $this->client->getPageService());
-    }
-
     public function testCreatePageRequest(): void
     {
         $pagePath = '/about-us/index';
@@ -64,18 +48,22 @@ public function testGetPage(): void
         // Create a mock PageRequest
         $pageRequestMock = $this->createMock(PageRequest::class);
 
-        // Create a test double for DotCMSClient
-        $clientMock = $this->getMockBuilder(DotCMSClient::class)
-            ->setConstructorArgs([$this->config])
-            ->onlyMethods(['getPage'])
-            ->getMock();
-
-        // Configure the mock to return our mock page asset
-        $clientMock->expects($this->once())
+        // Create a mock PageService that will return our mock PageAsset
+        $pageServiceMock = $this->createMock(PageService::class);
+        $pageServiceMock->expects($this->once())
             ->method('getPage')
             ->with($pageRequestMock)
             ->willReturn($pageAssetMock);
 
+        // Create a partial mock of DotCMSClient with the pageService property replaced
+        $clientMock = $this->createPartialMock(DotCMSClient::class, []);
+
+        // Set the mocked pageService using reflection
+        $reflection = new \ReflectionClass($clientMock);
+        $property = $reflection->getProperty('pageService');
+        $property->setAccessible(true);
+        $property->setValue($clientMock, $pageServiceMock);
+
         // Call the method
         $result = $clientMock->getPage($pageRequestMock);
 
@@ -94,18 +82,22 @@ public function testGetPageAsync(): void
         // Create a mock Promise
         $promiseMock = new FulfilledPromise($pageAssetMock);
 
-        // Create a test double for DotCMSClient
-        $clientMock = $this->getMockBuilder(DotCMSClient::class)
-            ->setConstructorArgs([$this->config])
-            ->onlyMethods(['getPageAsync'])
-            ->getMock();
-
-        // Configure the mock to return our mock Promise
-        $clientMock->expects($this->once())
+        // Create a mock PageService that will return our mock Promise
+        $pageServiceMock = $this->createMock(PageService::class);
+        $pageServiceMock->expects($this->once())
             ->method('getPageAsync')
             ->with($pageRequestMock)
             ->willReturn($promiseMock);
 
+        // Create a partial mock of DotCMSClient with the pageService property replaced
+        $clientMock = $this->createPartialMock(DotCMSClient::class, []);
+
+        // Set the mocked pageService using reflection
+        $reflection = new \ReflectionClass($clientMock);
+        $property = $reflection->getProperty('pageService');
+        $property->setAccessible(true);
+        $property->setValue($clientMock, $pageServiceMock);
+
         // Call the method
         $result = $clientMock->getPageAsync($pageRequestMock);