-
Notifications
You must be signed in to change notification settings - Fork 0
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
task/WMAQA-45 Shared Workspace Tests #23
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's a couple of bugs I found, but the skip and teardown are working great!
test('Cleanup shared workspaces', async ({ page, baseURL }) => { | ||
const {USERNAME: username, PASSWORD: password} = process.env; | ||
|
||
const tenant = 'https://portals.tapis.io'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is ok for now, but we're going to need to parameterize this if we want to test portals on different tenants.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I have one change to make this more secure and another short research task for myself.
|
||
} | ||
|
||
async function getSystems(page, tenant, projectPrefix, accessToken) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to delete all the shared systems, or just the one we created in these tests? Is there an easy or free way to get that system id at system creation? I'll take a look to try and save you a headache.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, it looks like it's available in the Shared Workspaces view and in the URL.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In practice, it seems like the 'Edit Shared Workspace Name and Description' test is a little flaky, but I was able to get it to pass 3 times in a row.
Overview
Added tests for shared workspaces. Also added a teardown project that deals with deleting shared workspaces directly using tapis
Changes
Testing
Notes
In the shared workspace teardown there's a new function
page.evaluate
being used when doing external http calls. More details about why this is needed and what it does can be found here