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

NOISSUE - Add property based testing to notifiers API #172

Closed
wants to merge 11 commits into from

Conversation

rodneyosodo
Copy link
Member

What type of PR is this?

This is a feature PR as it adds property-based testing to notifiers API. In addition it is a refactor to error handling on the API

What does this do?

Adds schemathesis to notifiers service

Which issue(s) does this PR fix/relate to?

  • Related Issue #
  • Resolves #

Have you included tests for your changes?

Yes, I have included tests for my changes.

Did you document any new/modified feature?

No

Notes

To be merged after absmach/magistrala#171

- Update API endpoints, error codes, schemas, and validation patterns
- Improve descriptions and documentation for various components
- Add operation IDs and links for operations

Signed-off-by: Rodney Osodo <[email protected]>
@rodneyosodo
Copy link
Member Author

Closed as it was already merged to main on https://github.com/absmach/magistrala

@rodneyosodo rodneyosodo closed this Jul 3, 2024
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.

None yet

1 participant