Remove unnecessary setAccessible(true) calls

In PHP 8.1+, ReflectionProperty::setAccessible() and ReflectionMethod::setAccessible() became no-ops and have no effect. Since this project requires PHP 8.2+, these calls are unnecessary and can be removed.

Author: OskarStark

src/platform/tests/Bridge/HuggingFace/ModelClientTest.php

@@ -37,7 +37,6 @@ public function testGetUrlForDifferentInputsAndTasks(?string $task, string $expe
     {
         $reflection = new \ReflectionClass(ModelClient::class);
         $getUrlMethod = $reflection->getMethod('getUrl');
-        $getUrlMethod->setAccessible(true);
 
         $model = new Model('test-model');
         $httpClient = new MockHttpClient();
@@ -87,7 +86,6 @@ public function testGetPayloadForDifferentInputsAndTasks(object|array|string $in
 
         $reflection = new \ReflectionClass(ModelClient::class);
         $getPayloadMethod = $reflection->getMethod('getPayload');
-        $getPayloadMethod->setAccessible(true);
 
         $httpClient = new MockHttpClient();
         $modelClient = new ModelClient($httpClient, 'test-provider', 'test-api-key');

src/platform/tests/Bridge/LmStudio/Completions/ModelClientTest.php

@@ -92,7 +92,6 @@ public function testItUsesEventSourceHttpClient()
         $client = new ModelClient($httpClient, 'http://localhost:1234');
 
         $reflection = new \ReflectionProperty($client, 'httpClient');
-        $reflection->setAccessible(true);
 
         $this->assertInstanceOf(EventSourceHttpClient::class, $reflection->getValue($client));
     }
@@ -103,7 +102,6 @@ public function testItKeepsExistingEventSourceHttpClient()
         $client = new ModelClient($eventSourceHttpClient, 'http://localhost:1234');
 
         $reflection = new \ReflectionProperty($client, 'httpClient');
-        $reflection->setAccessible(true);
 
         $this->assertSame($eventSourceHttpClient, $reflection->getValue($client));
     }