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
Create POMs for installation process #16
Conversation
rakoenig
commented
Aug 24, 2023
•
edited
edited
- Related ticket: https://progress.opensuse.org/issues/132890
- Verification run: https://openqa.opensuse.org/tests/3537791
- Added InstallActor to all tests: Verification runs
2483263
to
70467d6
Compare
3183a12
to
bb3eaee
Compare
3dfb91b
to
568ed2e
Compare
pages/installing-page.ts
Outdated
@@ -0,0 +1,16 @@ | |||
import { expect, type Locator, type Page } from '@playwright/test'; | |||
|
|||
export class InstallationProgressPage { |
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.
should be called InstallingPage as mentioned in other comment, file name should match with class name.
tests/full-disk-encryption.spec.ts
Outdated
@@ -7,6 +7,10 @@ import { EncryptionPasswordPopup } from '../pages/encryption-password-popup'; | |||
import { UsersPage } from '../pages/users-page'; | |||
import { DefineUserPage } from '../pages/define-user-page'; | |||
import { ConfigureRootPasswordPage } from '../pages/configure-root-password-page'; | |||
import { ConfirmInstallationPage } from '../pages/confirm-installation-page'; |
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.
besides this test, there are others that needs this improvement.
568ed2e
to
43f9055
Compare
43f9055
to
67428d6
Compare
LGTM, could we implement the install-actor for other test cases for installation part such as lvm, encrypted_lvm, etc? |
67428d6
to
3c51bef
Compare
3c51bef
to
d4dfab4
Compare