Skip to content
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

Rewrite test userstory for WTO under admin #22748

Merged
merged 8 commits into from
Jan 4, 2024
Merged

Rewrite test userstory for WTO under admin #22748

merged 8 commits into from
Jan 4, 2024

Conversation

musienko-maxim
Copy link
Contributor

What does this PR do?

  • This PR implements verification Web Terminal user story under an admin user

Screenshot/screencast of this PR

What issues does this PR fix or reference?

https://issues.redhat.com/browse/WTO-233

How to test this PR?

Prepare OpenShift infra with installed WebTerminal and run test:

MOCHA_DIRECTORY=web-terminal;
NODE_TLS_REJECT_UNAUTHORIZED=0;
OCP_INFRA=PSI;
OCP_VERSION=4.14;
RP_API_KEY=******;
RP_ENDPOINT=https://reportportal-wto.apps.ocp-c1.prod.psi.redhat.com/api/v1;
RP_IS_LOCAL_SERVER=false;
RP_PROJECT=wto;
SAVE_RP_REPORT_DATA=true;
TESTING_APPLICATION_NAME=wto;
TS_OCP_LOGIN_PAGE_PROVIDER_TITLE=htpasswd;
TS_SELENIUM_BASE_URL=https://devspaces.apps.ocp*****.com;
TS_SELENIUM_OCP_PASSWORD=****
TS_SELENIUM_OCP_USERNAME=*****;
TS_SELENIUM_VALUE_OPENSHIFT_OAUTH=true;
USERSTORY=WebTerminalUnderAdmin

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

@che-bot che-bot added target/branch Indicates that a PR will be merged into a branch other than master. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. labels Jan 2, 2024
@artaleks9
Copy link
Contributor

On my opinion, looks good

Copy link
Contributor

@dmytro-ndp dmytro-ndp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to merge

@musienko-maxim musienko-maxim merged commit 66ed29f into main Jan 4, 2024
5 checks passed
@musienko-maxim musienko-maxim deleted the WTO-220 branch January 4, 2024 12:40
@che-bot che-bot removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Jan 4, 2024
dmytro-ndp added a commit that referenced this pull request Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
target/branch Indicates that a PR will be merged into a branch other than master.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants