Project

General

Profile

Actions

action #130045

closed

coordination #121855: [epic] Agama web interactive installation

Create POM to define user and root authentication

Added by JERiveraMoya 11 months ago. Updated 9 months ago.

Status:
Resolved
Priority:
High
Target version:
Start date:
2023-05-29
Due date:
% Done:

0%

Estimated time:

Description

Motivation

Define an user and root authentication are required actions for default installation and will be included in different tests.
In this ticket we will create POM for user page with Agama.
Navigation to User page, clicking on Install or anything else should be done just using playwright test generator, without the need to create POM there for simplification.

Scope

Installations of ALP Micro, ALP Bedrock and Tumbleweed using Agama for all architectures available in openQA (s390x and ppc64le excluded for now).

Acceptance criteria

AC1: Get familiar with Playwright and use the playwright test generator to get first version of your code.
AC2: Locate only the auto-generated code related with user definition and user creation and convert it to Page Object Model.
AC3: Add some dummy test (because it will not be a test for itself in the future, will be part of other tests) test to our QE repo e2e-agama-playwright using that POM.

Additional information

Playwright documentation: https://playwright.dev/docs/intro
Playwright documentation - Page Object Model: https://playwright.dev/docs/pom
How to use playwright with Agama: https://github.com/openSUSE/agama/blob/master/playwright/README.md
Notice that for development in your system, you will need to install playwright and run npx playwright test, due to in the live iso the command to run is different as described in doc above, playwright test.

Notice that some part might done for the CI in https://github.com/openSUSE/agama/tree/master/playwright/tests regarding the user and regarding the install buttom at the end of https://github.com/openSUSE/agama/blob/master/playwright/tests/take_screenshots.spec.ts but what we need is to put that logic as POMs.
Existing test for CI shouldn't be touched for now, but those file might give us some inspiration in case with playwright test generator would not get the best locators.

IMPORTANT: While #130000 is not yet resolve, please add a call to yupdate in your test so live-cd can pick the new test/code you implement from a branch


Files

Users.png (28.4 KB) Users.png JERiveraMoya, 2023-05-30 08:53
Actions #1

Updated by JERiveraMoya 11 months ago

  • Copied from action #130003: Add test for Full-disk encryption installation added
Actions #2

Updated by JERiveraMoya 11 months ago

  • Copied from deleted (action #130003: Add test for Full-disk encryption installation)
Actions #3

Updated by JERiveraMoya 11 months ago

  • File deleted (Encryption_settings.png)
Actions #4

Updated by JERiveraMoya 11 months ago

  • Description updated (diff)
  • Status changed from New to In Progress
  • Assignee set to syrianidou_sofia
  • Target version set to Current
Actions #5

Updated by JERiveraMoya 11 months ago

  • Description updated (diff)
Actions #6

Updated by JERiveraMoya 11 months ago

  • Description updated (diff)
Actions #7

Updated by JERiveraMoya 11 months ago

Actions #8

Updated by JERiveraMoya 11 months ago

  • Description updated (diff)
Actions #9

Updated by JERiveraMoya 10 months ago

now we can integrate here #130000

Actions #10

Updated by JERiveraMoya 10 months ago

Now you can integrate you work in existing code what was just merged (not need for extra test suite only to test user case, perhaps in the future with some more advance configuration):
https://github.com/jknphy/e2e-agama-playwright/pull/2
https://github.com/jknphy/e2e-agama-playwright/pull/3

Actions #11

Updated by syrianidou_sofia 10 months ago

PR for playwright code here: https://github.com/jknphy/e2e-agama-playwright/pull/6
Temporary modifications on os-autoinst-distri-opensuse for VRs to run on branch: https://github.com/sofiasyria/os-autoinst-distri-opensuse/tree/users_page_agama

Actions #12

Updated by JERiveraMoya 10 months ago

syrianidou_sofia wrote:

PR for playwright code here: https://github.com/jknphy/e2e-agama-playwright/pull/6
Temporary modifications on os-autoinst-distri-opensuse for VRs to run on branch: https://github.com/sofiasyria/os-autoinst-distri-opensuse/tree/users_page_agama

could you clarify what is that about VRs to run on branch? you can point yupdate to your plawright branch and no code is needed afik.

Actions #13

Updated by syrianidou_sofia 9 months ago

JERiveraMoya wrote:

syrianidou_sofia wrote:

PR for playwright code here: https://github.com/jknphy/e2e-agama-playwright/pull/6
Temporary modifications on os-autoinst-distri-opensuse for VRs to run on branch: https://github.com/sofiasyria/os-autoinst-distri-opensuse/tree/users_page_agama

could you clarify what is that about VRs to run on branch? you can point yupdate to your plawright branch and no code is needed afik.

There is no temporary code needed now that I used the production jobs for VRs. The only problem is that validate_lvm fails because the root password is not the expected one, I still don't know why "nots3cr3t" as root password fails. I tried to set the system password to 'linux' during installation but then I get 'linux' used as username, which is quite weird... I guess there must be some bug in these post-run pre-run hooks in the agama library. I will open a ticket for investigation

validate_lvm : username ->linux, password ->linux: https://openqa.opensuse.org/tests/3433327#step/validate_lvm/1
validate_encrypt: username ->linux and password ->nots3cr3t: https://openqa.opensuse.org/tests/3433326#step/validate_encrypt/1

Actions #14

Updated by syrianidou_sofia 9 months ago

  • Status changed from In Progress to Resolved

ticket for problems with $password inheritance : https://progress.opensuse.org/issues/132821

Actions

Also available in: Atom PDF