This repository has been archived by the owner on Oct 14, 2024. It is now read-only.
NOISSUE - Fix Users Tests #42
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
rodneyosodo
force-pushed
the
11-14-Fix_users_tests
branch
5 times, most recently
from
a5fe742 to
5159ea6
Compare
Fix tests in users service to ensure there are no failing tests
Added test cases to cover the authn and authz functionality of the user service with domains integration. The tests involve mocking function calls, setting up expectations, and checking for expected results. The code includes conditional statements and different scenarios to ensure the proper behavior of the functions. This commit also fixes updating client secret Signed-off-by: Rodney Osodo <[email protected]>
Fix bug in loading auth config and also refactor postgres connect
rodneyosodo
force-pushed
the
11-14-Fix_users_tests
branch
from
05b71e1 to
f73fffb
Compare
dborovcanin
approved these changes
Nov 22, 2023
dborovcanin
pushed a commit
that referenced
this pull request
Nov 27, 2023
* NOISSUE - Fix Users Tests (#42) * Fix users tests Fix tests in users service to ensure there are no failing tests * test: update tests with domains Added test cases to cover the authn and authz functionality of the user service with domains integration. The tests involve mocking function calls, setting up expectations, and checking for expected results. The code includes conditional statements and different scenarios to ensure the proper behavior of the functions. This commit also fixes updating client secret Signed-off-by: Rodney Osodo <[email protected]> * NOISSUE - Fix loading Auth config (#64) Fix bug in loading auth config and also refactor postgres connect --------- Signed-off-by: Rodney Osodo <[email protected]> * Fix failing tests in SDK Fix failing tests in `sdk` package. * fix(tests): update sdk tests with domains Add domain integration and fix failing tests on SDK package Signed-off-by: Rodney Osodo <[email protected]> --------- Signed-off-by: Rodney Osodo <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this do?
Fix tests in users service to ensure there are no failing tests
Which issue(s) does this PR fix/relate to?
Related to absmach/magistrala#2014
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
N/A