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

[gui-tests] Add oauth2 login test scenario #10511

Merged
merged 3 commits into from
Feb 27, 2023
Merged

Conversation

saw-jan
Copy link
Member

@saw-jan saw-jan commented Feb 16, 2023

Added a test scenario to test the successful oauth2 login (only for oC10 server)

Related Issues

@ownclouders
Copy link
Contributor

ownclouders commented Feb 16, 2023

@saw-jan saw-jan marked this pull request as ready for review February 17, 2023 03:46
@saw-jan saw-jan marked this pull request as draft February 17, 2023 03:46
@saw-jan saw-jan marked this pull request as ready for review February 17, 2023 08:25
@saw-jan saw-jan marked this pull request as draft February 17, 2023 08:25
@saw-jan saw-jan marked this pull request as ready for review February 22, 2023 06:50
@saw-jan saw-jan changed the base branch from 3.2 to master February 23, 2023 03:40
Comment on lines +44 to +52
// login
await page.fill("#user", config.username);
await page.fill("#password", config.password);
await page.click("button[type=submit]");
// authorize
await page.click("button >> text=Authorize");
// confirm successful login
await expect(page.locator("span.error")).toContainText(
"The application was authorized successfully"
Copy link
Contributor

Choose a reason for hiding this comment

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

do we have to add such normal comments also?

Copy link
Member Author

Choose a reason for hiding this comment

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

just to group the actions.

@saw-jan saw-jan force-pushed the tests/oauth-enabled branch 2 times, most recently from ed238b4 to f1540f7 Compare February 27, 2023 04:14
do oauth login from squish tests

dry webui test code
add install pnpm pipeline step
@sonarcloud
Copy link

sonarcloud bot commented Feb 27, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@saw-jan saw-jan merged commit fa40523 into master Feb 27, 2023
@delete-merged-branch delete-merged-branch bot deleted the tests/oauth-enabled branch February 27, 2023 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants