chore(api): [GSK-4308] Rename any Conversation mention to ChatTestCase

[Inokinoki]

Main points:

• add ChatTestCasesResource and ChatTestCase data class
• mark conversation usage as deprecated with warnings
• get chat_test_case or conversation from the evaluation results to ensure compatibility

[linear]

GSK-4308 Rename any `Conversation` mention to `ChatTestCase` / `TestCase`

[henchaves]

hey @Inokinoki, it seems very good :)

could you just also add some tests like:

• creating a ChatTestCase and a Conversation and checking if their dict content are the same
• listing ChatTestCase from evaluations and datasets and checking if everything is fine

[Inokinoki]

hey @Inokinoki, it seems very good :)

could you just also add some tests like:

• creating a ChatTestCase and a Conversation and checking if their dict content are the same
• listing ChatTestCase from evaluations and datasets and checking if everything is fine

Hey, during adding the tests, I found that the checks don't seem to have the right type after _format_checks_to_cli: https://github.com/Giskard-AI/giskard-hub/pull/52/files#diff-cc6693ba3e3ac6afcac5f7de2111269e4b3248bf3bb2c188db53ddee49c05f8eR77

It's still dict while we are expecting TestCaseCheckConfig as the arg of _format_checks_to_cli and CheckConfig as the return value, as well in the checks field.

Is this something wrong? How should we process them if so, will anything break if we also fix it here?

[Inokinoki]

hey @Inokinoki, it seems very good :)
could you just also add some tests like:

• creating a ChatTestCase and a Conversation and checking if their dict content are the same
• listing ChatTestCase from evaluations and datasets and checking if everything is fine

Hey, during adding the tests, I found that the checks don't seem to have the right type after _format_checks_to_cli: https://github.com/Giskard-AI/giskard-hub/pull/52/files#diff-cc6693ba3e3ac6afcac5f7de2111269e4b3248bf3bb2c188db53ddee49c05f8eR77

It's still dict while we are expecting TestCaseCheckConfig as the arg of _format_checks_to_cli and CheckConfig as the return value, as well in the checks field.

Is this something wrong? How should we process them if so, will anything break if we also fix it here?

To be fixed in https://github.com/Giskard-AI/giskard-hub/compare/feature/gsk-4163-add-missing-resources

[Inokinoki]

Thanks @henchaves for discovering the URLs that we have already changed

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[henchaves]

Seems good to me! Thanks for all the hard work, @Inokinoki