-
Notifications
You must be signed in to change notification settings - Fork 26
Pytest for various functions added #29
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
base: main
Are you sure you want to change the base?
Conversation
|
@staru09 can you please share test results, along with the coverage you have provided. also, a few tests are being skipped, can you please check that too? |
|
Update:- The tests were skipped because I think they weren't configured properly so working on this, will be adding it to the PR. |
|
@Aaditya-G can you tell me more about what this application expects as input of these fields, specially the ones before openai key field |
|
|
@Aaditya-G I have added tests for the core functions, please check if this works |
Aaditya-G
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM

Related to #22