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

Added tests for the Rest class of rest-interface. #493

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

muaz-32
Copy link

@muaz-32 muaz-32 commented Apr 16, 2024

Summary

Added tests for the Rest class of rest-interface as there were no tests for the class. With a deeper understanding of the system, more tests can be written and refactored.

Fixes: #492

Changes

Changes were made in the RequestParser.Filters class to make the testing possible.

Features

Tests

Can test almost all methods of the Rest class. Handles some corner cases.

ToDo

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

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.

1 participant