Skip to content
This repository has been archived by the owner on Oct 14, 2024. It is now read-only.

NOISSUE - Fix tests #82

Merged
merged 3 commits into from
Nov 29, 2023
Merged

Conversation

felixgateru
Copy link
Contributor

@felixgateru felixgateru commented Nov 28, 2023

What does this do?

Fix failing service tests in:

  • bootstrap
  • certs
  • things
  • users

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

absmach/magistrala#31

List any changes that modify/break current functionality

None

Have you included tests for your changes?

Yes

Did you document any new/modified functionality?

No

Notes

@@ -38,7 +38,7 @@ var (
ErrUniqueID = errors.New("failed to generate unique identifier")

// ErrInvalidStatus indicates an invalid status.
ErrInvalidStatus = errors.New("Invalid status")
ErrInvalidStatus = errors.New("invalid user account status")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is for both user and thing hence you can leave it as it was

Signed-off-by: felix.gateru <[email protected]>
Signed-off-by: felix.gateru <[email protected]>
Signed-off-by: felix.gateru <[email protected]>
@felixgateru felixgateru marked this pull request as ready for review November 29, 2023 14:43
@dborovcanin dborovcanin merged commit 0197244 into absmach:main Nov 29, 2023
10 of 15 checks passed
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.

3 participants