Skip to content

Conversation

@staru09
Copy link

@staru09 staru09 commented Oct 13, 2025

Related to #22

@Aaditya-G Aaditya-G self-requested a review October 17, 2025 02:29
@Aaditya-G
Copy link
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
Copy link
Author

staru09 commented Oct 27, 2025

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
Copy link
Author

staru09 commented Oct 29, 2025

@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
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
Copy link
Author

staru09 commented Oct 30, 2025

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

Copy link
Collaborator

@Aaditya-G Aaditya-G left a comment

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