[minor] Add Sanity tests for Manage #211
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
MAXMF-551- Add Sanity Validation
moved tests to sanity test
Added 3 modules in requirements.txt. Rest of them are part of python module
whitfiea
requested changes
Oct 23, 2024
added 21 commits
October 23, 2024 19:51
Fix variable assignment and usage
fix minor error
removed fixture marker
Fixes
minor fix
fix
fix
fix
fix
fix variables
fix error
Add logger info
fix errors
grant permission
fix error
removed unused resource call
fix secret
fix errors
fix secret
fix error
fix secret
added 15 commits
October 25, 2024 23:24
access
fix
access
fix
add cluster role
debug
debug
debug
debug
debug
access
refactor
redesigned tests
refactor
refactor
whitfiea
requested changes
Oct 28, 2024
There was a problem hiding this comment.
The tests here currently create a user in MAS and sync it to Manage. This is a good start but we need to do more tests on Manage itself to ensure that basic functionality of Manage works. We can add a comment to say that demodata is needed and then we can run various API tests on some core Manage function to make sure it is working.
instance-applications/510-550-ibm-mas-suite-app-config/templates/04-postsync-manage-sanity.yaml
Outdated
Show resolved
Hide resolved
Comment on lines
93
to
95
| - secrets | ||
| - routes | ||
| - suites |
Comment on lines
122
to
123
| # MVI Verification tests need to be able to get/list nodes | ||
| # to check for ready GPU nodes |
instance-applications/510-550-ibm-mas-suite-app-config/templates/04-postsync-manage-sanity.yaml
Outdated
Show resolved
Hide resolved
instance-applications/510-550-ibm-mas-suite-app-config/templates/04-postsync-manage-sanity.yaml
Outdated
Show resolved
Hide resolved
updated as per feedback
Fix deleteuser and try decoded certs
try with disable warnings
added requirement
manage url for cluster
Added Manage tests for apikey and asset object
set cluster url
whitfiea
approved these changes
Nov 8, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
MAXMF-551- Add Sanity Validation.
v1_secrets
v1_routes
manage_route
manage_host
getManageComponents
getClusterInfo
test_create_super_user_token
test_v3_create_user
test_v3_user_link_to_idp
test_v3_user_add_user_to_workspace
test_v3_user_application_permission
test_check_user_sync
test_generate_api_key_for_new_added_user
test_get_api_key_for_admin
test_add_new_user_to_admin_group
test_asset_crud_operation_using_new_added_user