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
Playwright UI #6014
Draft
neo-cypher
wants to merge
55
commits into
main
Choose a base branch
from
playwright-ui
base: main
Could not load branches
Branch not found: {{ refName }}
Could not load tags
Nothing to show
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Playwright UI #6014
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This commit removes the outdated playwright tests so that they can be replaced with the most up to date version.
This commit implements the most up to date version of the ui tests.
This commit pushes the jenkinsfile that is required for a ui test to be performed in the pipeline. Any branch with an assoicated github pr that contains this file will automatically execute the jenkins job. Further hooks will be implemented in a future PR to allow UI tests on gerrit changes.
UI tests are now configured to connect to the local services instead of using the nightly build.
Gateway was calling gateway-mt executable Removed linting, auditing, and testing of web ui code. This copy over was redundant to the .verify jenkins job and should reduce overall test runtime.
Gateway was calling gateway-mt executable Removed linting, auditing, and testing of web ui code. This copy over was redundant to the .verify jenkins job and should reduce overall test runtime. Fixes typo in Jenkinsfile.ui
# Conflicts: # Jenkinsfile.ui
Tests dont set the port assignment when pulling configuration parameters from testConfig.ts. Instead of fixing all the tests I just added the port to the baseurl.
added allure reporting dependency to the node modules dependencies.
disables the limiter on the console auth endpoint
updated .gitignore to exclude html-report directory from commits
diff --git a/Jenkinsfile.ui b/Jenkinsfile.ui index b8f212a9e..973ba0c50 100644 --- a/Jenkinsfile.ui +++ b/Jenkinsfile.ui @@ -16,6 +16,7 @@ pipeline { GOTRACEBACK = 'all' COCKROACH_MEMPROF_INTERVAL=0 PLAYWRIGHT_BROWSERS_PATH= '/ms-playwright' + SLACK_WEBHOOK_URL = credentials('SLACK_WEBHOOK_URL') SLACK_BOT_USER_OAUTH_TOKEN = credentials('SLACK_BOT_USER_OAUTH_TOKEN') }
diff --git a/Jenkinsfile.ui b/Jenkinsfile.ui index b8f212a9e..973ba0c50 100644 --- a/Jenkinsfile.ui +++ b/Jenkinsfile.ui @@ -16,6 +16,7 @@ pipeline { GOTRACEBACK = 'all' COCKROACH_MEMPROF_INTERVAL=0 PLAYWRIGHT_BROWSERS_PATH= '/ms-playwright' + SLACK_WEBHOOK_URL = credentials('SLACK_WEBHOOK_URL') SLACK_BOT_USER_OAUTH_TOKEN = credentials('SLACK_BOT_USER_OAUTH_TOKEN') }
Modify the jenkinsfile to call the regular test instead of the test-debug.
this build is for debugging purposes only.
This commit adds the login test and updates the test-debug script in the package.json file.
gitignore updated to allow commiting of test.ts file.
updated package.json to use playwright 1.37.1.
Force the browser to use chromium instead of chrome. Chrome requires some packages that arent on the ci-image but generally its not needed to run the full desktop verseion.
this commit forces chromium instead of chrome.
this commit forces chromium instead of chrome.
neo-cypher
force-pushed
the
playwright-ui
branch
from
September 8, 2023 12:46
bc7a024
to
37d14da
Compare
neo-cypher
force-pushed
the
playwright-ui
branch
from
September 20, 2023 20:20
9a16aa2
to
8d4f6ed
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What: Playwright Automated UI Tests
Why: Because we need automated UI test coverage in order to limit the gaps in our current manual testing.
Please describe the tests:
Please describe the performance impact:
Code Review Checklist (to be filled out by reviewer)