-
Notifications
You must be signed in to change notification settings - Fork 32
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
23: Global setup for Playwright, and some basic Playwright UI tests #254
base: main
Are you sure you want to change the base?
Conversation
@adigidh Exited to see this PR. Thanks for pushing this. Let me know if this PR is ready for review. |
@vishnoianil : Sent you a DM last week. Maybe this PR is good for first pass? |
@adigidh PR looks good to me. Can you please add instructions for running the test cases in the development doc in docs directory. Once you update the docs, PR it good for merge, so please mark it ready for review. |
c5289ef
to
39efb25
Compare
Signed-off-by: Aditya Gidh <[email protected]>
Signed-off-by: Aditya Gidh <[email protected]>
Signed-off-by: Aditya Gidh <[email protected]>
Signed-off-by: Aditya Gidh <[email protected]>
Signed-off-by: Aditya Gidh <[email protected]>
Signed-off-by: Aditya Gidh <[email protected]>
Signed-off-by: Aditya Gidh <[email protected]>
39efb25
to
f6e892a
Compare
Signed-off-by: Aditya Gidh <[email protected]>
re: #254 (comment) Thanks for the feedback @vishnoianil. Made some updates, and opened it up for review |
fixes #23
Summary of changes:
Example report after running the tests locally: