Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[AGE-175] Create tests for llm_apps_service #1616

Closed
mmabrouk opened this issue May 5, 2024 · 0 comments
Closed

[AGE-175] Create tests for llm_apps_service #1616

mmabrouk opened this issue May 5, 2024 · 0 comments
Assignees
Labels
3 points Created by Linear-GitHub Sync High Priority linear Created by Linear-GitHub Sync Task Created by Linear-GitHub Sync tests
Milestone

Comments

@mmabrouk
Copy link
Member

mmabrouk commented May 5, 2024

We had multiple bugs in llm_apps_service in the past. The logic for invoking the applications is quite complex. We need to have tests for the different scenarios and errors. We should use monkey patching to mock some of the results and errors. Most important thing is to make sure the error handling works well.

From SyncLinear.com | AGE-175

@mmabrouk mmabrouk added High Priority Task Created by Linear-GitHub Sync labels May 5, 2024
@dosubot dosubot bot added the tests label May 5, 2024
@mmabrouk mmabrouk added this to the v.45 milestone May 12, 2024
@mmabrouk mmabrouk added the 3 points Created by Linear-GitHub Sync label May 12, 2024
@aybruhm aybruhm added the linear Created by Linear-GitHub Sync label May 15, 2024
@mmabrouk mmabrouk modified the milestones: v.45, v.46 May 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 points Created by Linear-GitHub Sync High Priority linear Created by Linear-GitHub Sync Task Created by Linear-GitHub Sync tests
Projects
None yet
Development

No branches or pull requests

2 participants