Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Feat/added playwright testing framework #55

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

Mubashirshariq
Copy link

/claim #54

@richiemcilroy I raised this PR today without including tests initially to demonstrate how I have integrated Playwright as the testing framework into our monorepo structure.

Copy link

vercel bot commented Jul 19, 2024

Someone is attempting to deploy a commit to the Cap Software Inc Team on Vercel.

A member of the Team first needs to authorize it.

@richiemcilroy
Copy link
Collaborator

hey @Mubashirshariq - thanks for this! how can I test it? maybe add a pnpm test command at the root level which runs all tests?

@Mubashirshariq
Copy link
Author

Mubashirshariq commented Jul 20, 2024

@richiemcilroy I will add one test for each,then we can test it

@Mubashirshariq
Copy link
Author

@richiemcilroy i have added tests as well you can run them by running pnpm test in the root of the directory

@Mubashirshariq
Copy link
Author

@richiemcilroy Can you please review this PR

@Mubashirshariq
Copy link
Author

@richiemcilroy Any updates on when it might be reviewed or merged? Thanks!

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants