Linting

marcusgreen · May 20, 2024 · 97c2879 · 97c2879
1 parent bb02499
commit 97c2879
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 13 deletions.
diff --git a/classes/ai/ai.php b/classes/ai/ai.php
@@ -64,7 +64,7 @@ class ai {
      * @param string $model
      */
     public function __construct($model = null) {
-        $this->model = $model ?? trim(explode(',',get_config('tool_aiconnect','model'))[0]);
+        $this->model = $model ?? trim(explode(',', get_config('tool_aiconnect', 'model'))[0]);
         $this->openaiapikey = get_config('tool_aiconnect', 'apikey');
         $this->temperature = get_config('tool_aiconnect', 'temperature');
         $this->endpoint = trim(get_config('tool_aiconnect', 'endpoint'));
@@ -87,9 +87,9 @@ private function make_request($data, $apikey, $multipart = null) {
                 'Empty API Key.');
         }
         $headers = $multipart ? [
-            "Content-Type: multipart/form-data"
+            "Content-Type: multipart/form-data",
         ] : [
-            "Content-Type: application/json;charset=utf-8"
+            "Content-Type: application/json;charset=utf-8",
         ];
 
         $headers[] = "Authorization: Bearer $apikey";
@@ -100,11 +100,10 @@ private function make_request($data, $apikey, $multipart = null) {
         ];
         $start = microtime(true);
         $jsonresponse = $curl->post($this->endpoint, json_encode($data), $options);
-        $response = json_decode($jsonresponse,true);
+        $response = json_decode($jsonresponse, true);
         if ($response == null) {
             return ['curl_error' => $curl->get_info()->http_code, 'execution_time' => $executiontime];
         }
-        xdebug_break();
 
         if(isset($response['error'])){
              throw new moodle_exception('endpointerror', 'tool_aiconnect', '', null,
@@ -149,7 +148,7 @@ public function prompt_completion($prompttext) {
      * @param string $prompttext The prompt text.
      * @return array The prompt data.
      */
-    private function get_prompt_data($prompttext) : array {
+    private function get_prompt_data($prompttext): array {
             $data = [
                 'model' => $this->model,
                 'temperature' => $this->temperature,
@@ -159,7 +158,7 @@ private function get_prompt_data($prompttext) : array {
             ];
             return $data;
     }
-    public function get_models() : \stdClass {
+    public function get_models(): \stdClass {
         $url = new \moodle_url($this->endpoint);
         $ollama = true;
         // Ollama url.

diff --git a/tests/test_aiconnect.php b/tests/test_aiconnect.php
@@ -57,7 +57,7 @@ public function setUp(): void {
      *
      * @return void
      */
-    public function test_get_prompt_data() :void {
+    public function test_get_prompt_data(): void {
          $this->assertTrue(true);
          $mockai = $this->getMockBuilder(ai\ai::class)->getMock();
          $getpromptdata = new \ReflectionMethod(
@@ -71,13 +71,13 @@ public function test_get_prompt_data() :void {
              ['myprompt']
          );
          $this->assertStringContainsString("You: myprompt", $result['messages'][0]['content']);
-     }
+    }
 
      /**
-     * This doesn't do anything especially useful.
-     * @return void
-     */
-    public function test_prompt_completion() :void {
+      * This doesn't do anything especially useful.
+      * @return void
+      */
+    public function test_prompt_completion(): void {
         $result = $this->ai->prompt_completion('query');
         $this->assertIsArray($result);
     }