Skip to content

Pytest for various functions added#29

Open
staru09 wants to merge 6 commits into
Repello-AI:mainfrom
staru09:pytest
Open

Pytest for various functions added#29
staru09 wants to merge 6 commits into
Repello-AI:mainfrom
staru09:pytest

Conversation

@staru09

@staru09 staru09 commented Oct 13, 2025

Copy link
Copy Markdown

Related to #22

@Aaditya-G Aaditya-G self-requested a review October 17, 2025 02:29
@Aaditya-G

Copy link
Copy Markdown
Collaborator

@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?

@staru09

staru09 commented Oct 27, 2025

Copy link
Copy Markdown
Author

Update:- The tests were skipped because I think they weren't configured properly so working on this, will be adding it to the PR.

@staru09

staru09 commented Oct 29, 2025

Copy link
Copy Markdown
Author

@Aaditya-G can you tell me more about what this application expects as input of these fields, specially the ones before openai key field
image

@Aaditya-G

Copy link
Copy Markdown
Collaborator

@staru09

  • api url : a string, the url will be used
  • api key : this will be a header sent to the code to verify that the request came from whistleblower, it is a string
  • both request body input field and response body output field can either be either a key-value pair or a json.

@staru09

staru09 commented Oct 30, 2025

Copy link
Copy Markdown
Author

@Aaditya-G I have added tests for the core functions, please check if this works

Comment thread requirements.txt Outdated

@Aaditya-G Aaditya-G left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants