Skip to content
This repository was archived by the owner on Jul 13, 2025. It is now read-only.

add tests for (some of) API endpoints#116

Merged
Nikronic merged 6 commits intomasterfrom
api_tests
Nov 21, 2023
Merged

add tests for (some of) API endpoints#116
Nikronic merged 6 commits intomasterfrom
api_tests

Conversation

@Nikronic
Copy link
Copy Markdown
Owner

Intro

As all things need test, APIs needed that too. But we didn't have them so far. Topics discussed in #110, #111, #112, #113, etc urged to make sure these changes does not break everything. For that, we first create tests for the current state of the code, then any changes should be reflected.

Summary

  • Added tests for predict endpoint
  • Added tests for potential endpoint
  • Added tests for grouped_xai endpoint

Remarks

The following endpoints do not have any tests, even though they might need one!

  • grouped_xai_expanded: this is not needed that much as there is not much calculation on this endpoint, it is just a renaming operation for features to payload variables. XAI values are printed in a raw format.
  • xai: we don't use this endpoint at all, even though it has shared functions in grouped_xai and grouped_xai_expanded

@Nikronic Nikronic merged commit 69062e8 into master Nov 21, 2023
@Nikronic Nikronic deleted the api_tests branch November 21, 2023 07:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant