Include test cases.

Created on 20 June 2024, 11 months ago

Problem/Motivation

Module should have test cases to know the functionality running properly.
Maybe there some deprecated could which could be reported in the phpstan and phpunit but due to no test cases those kind of deprecations are gonna be missed.
And Going forward with D11, it should be taken care of.

Remaining tasks

Functional/Unit tests.

📌 Task
Status

Active

Version

1.1

Component

Code

Created by

🇮🇳India ankitv18

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @ankitv18
  • Status changed to Needs work 11 months ago
  • 🇪🇸Spain psf_ Huelva

    Hi,

    The library that we use to connect to OpenAI have compatiblity with tests: https://github.com/openai-php/client?tab=readme-ov-file#testing

    But I don't have time to implement it, can anyone create tests?

  • First commit to issue fork.
  • 🇵🇱Poland dlevchik Poznan, Poland

    Created test cases for ai_conversation node and image generation. Needs a review

    I didn't create test cases for assistants because I don't understand how they work and how to test them properly. This is because there is a bunch of code that does getClient from a wrapper service, and just uses it directly. This is not very testable because the only thing you will receive is an error from a OpenAI\Testing\ClientFake 'No fake responses left.'

    Also, maybe we need a list of unexpected behaviors, like API key not exist or something like that, to test errors, IDK

Production build 0.71.5 2024