-
Notifications
You must be signed in to change notification settings - Fork 12
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
test: minikube cluster e2e tests #355
base: main
Are you sure you want to change the base?
Conversation
5756668
to
5806a5a
Compare
tests/playwright/src/model/pages/minikube-cluster-creation-page.ts
Outdated
Show resolved
Hide resolved
if (containerRuntime) { | ||
await playExpect(this.driverCombobox).toBeVisible(); | ||
const providerTypeOptions = await this.driverCombobox.locator('option').allInnerTexts(); | ||
if (providerTypeOptions.includes(containerRuntime)) { |
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.
Same as above.
test('Uninstall Minikube extension', async ({ navigationBar }) => { | ||
await navigationBar.openExtensions(); | ||
await playExpect(extensionsPage.header).toBeVisible(); | ||
const minikubeExtension = await extensionsPage.getInstalledExtension(EXTENSION_NAME, EXTENSION_LABEL); | ||
await minikubeExtension.removeExtension(); | ||
}); | ||
}); | ||
|
||
async function createMinikubeCluster(page: Page, clusterName: string = 'minikube', usedefaultOptions: boolean = true, timeout: number = 300_000, { driver, containerRuntime, baseImage, mountDefinition }: MinikubeClusterOptions = {}): Promise<void> { |
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.
maybe extract it into some local util file?
@amisskii But otherwise it looks good, tests are even passing on gha runners. |
19a56bf
to
94c7ddd
Compare
b58fd7e
to
5c4e904
Compare
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.
issue: several files are missing the EOF/newline
Signed-off-by: Anton Misskii <[email protected]>
c433a53
to
041f2f7
Compare
issue: #298