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 twins API #2098

Merged
merged 1 commit into from
Apr 16, 2024

Conversation

rodneyosodo
Copy link
Member

What type of PR is this?

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

What does this do?

Adds schemathesis to twins service

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

No issue

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 https://github.com/absmach/magistrala/pull/2096

@rodneyosodo rodneyosodo force-pushed the twins-schemathesis branch 4 times, most recently from cbd3ca2 to 5359e47 Compare April 11, 2024 08:11
@rodneyosodo rodneyosodo force-pushed the twins-schemathesis branch 4 times, most recently from 088af4b to b1e1fe3 Compare April 12, 2024 15:42
@rodneyosodo rodneyosodo marked this pull request as ready for review April 12, 2024 17:57
@dborovcanin dborovcanin merged commit f334ee2 into absmach:main Apr 16, 2024
3 of 6 checks passed
JeffMboya added a commit to JeffMboya/supermq that referenced this pull request Apr 16, 2024
Signed-off-by: JeffMboya <[email protected]>

NOISSUE - Add property based testing to twins API (absmach#2098)

Signed-off-by: Rodney Osodo <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>
JeffMboya added a commit to JeffMboya/supermq that referenced this pull request Apr 16, 2024
Signed-off-by: JeffMboya <[email protected]>

NOISSUE - Add property based testing to twins API (absmach#2098)

Signed-off-by: Rodney Osodo <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>
JeffMboya added a commit to JeffMboya/supermq that referenced this pull request Apr 16, 2024
Signed-off-by: JeffMboya <[email protected]>

Add mockery directive to CI

Signed-off-by: JeffMboya <[email protected]>

Revert changes

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Refactor endpoints_test.go to use generated mocks

Signed-off-by: JeffMboya <[email protected]>

Remove unused fmt.Printf statement in listMessagesEndpoint function

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

NOISSUE - Add property based testing to twins API (absmach#2098)

Signed-off-by: Rodney Osodo <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>
JeffMboya added a commit to JeffMboya/supermq that referenced this pull request Apr 16, 2024
Signed-off-by: JeffMboya <[email protected]>

Add mockery directive to CI

Signed-off-by: JeffMboya <[email protected]>

Revert changes

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Refactor endpoints_test.go to use generated mocks

Signed-off-by: JeffMboya <[email protected]>

Remove unused fmt.Printf statement in listMessagesEndpoint function

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

NOISSUE - Add property based testing to twins API (absmach#2098)

Signed-off-by: Rodney Osodo <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>
JeffMboya added a commit to JeffMboya/supermq that referenced this pull request Apr 16, 2024
Signed-off-by: JeffMboya <[email protected]>

Add mockery directive to CI

Signed-off-by: JeffMboya <[email protected]>

Revert changes

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Refactor endpoints_test.go to use generated mocks

Signed-off-by: JeffMboya <[email protected]>

Remove unused fmt.Printf statement in listMessagesEndpoint function

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

NOISSUE - Add property based testing to twins API (absmach#2098)

Signed-off-by: Rodney Osodo <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>
JeffMboya added a commit to JeffMboya/supermq that referenced this pull request Apr 16, 2024
Signed-off-by: JeffMboya <[email protected]>

Add mockery directive to CI

Signed-off-by: JeffMboya <[email protected]>

Revert changes

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Refactor endpoints_test.go to use generated mocks

Signed-off-by: JeffMboya <[email protected]>

Remove unused fmt.Printf statement in listMessagesEndpoint function

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

NOISSUE - Add property based testing to twins API (absmach#2098)

Signed-off-by: Rodney Osodo <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>
dborovcanin pushed a commit to JeffMboya/supermq that referenced this pull request Apr 16, 2024
Signed-off-by: JeffMboya <[email protected]>

Add mockery directive to CI

Signed-off-by: JeffMboya <[email protected]>

Revert changes

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Refactor endpoints_test.go to use generated mocks

Signed-off-by: JeffMboya <[email protected]>

Remove unused fmt.Printf statement in listMessagesEndpoint function

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

NOISSUE - Add property based testing to twins API (absmach#2098)

Signed-off-by: Rodney Osodo <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>
JeffMboya added a commit to JeffMboya/supermq that referenced this pull request Apr 16, 2024
Signed-off-by: JeffMboya <[email protected]>

Add mockery directive to CI

Signed-off-by: JeffMboya <[email protected]>

Revert changes

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Refactor endpoints_test.go to use generated mocks

Signed-off-by: JeffMboya <[email protected]>

Remove unused fmt.Printf statement in listMessagesEndpoint function

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

NOISSUE - Add property based testing to twins API (absmach#2098)

Signed-off-by: Rodney Osodo <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>

Generate mocks with mockery for readers

Signed-off-by: JeffMboya <[email protected]>
JeffMboya pushed a commit to JeffMboya/supermq that referenced this pull request May 15, 2024
JeffMboya pushed a commit to JeffMboya/supermq that referenced this pull request May 15, 2024
andychao217 pushed a commit to andychao217/magistrala that referenced this pull request Sep 23, 2024
@rodneyosodo rodneyosodo deleted the twins-schemathesis branch October 22, 2024 08:14
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