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

feature: Implement playwright basic setup #114

Open
7 tasks
Nishchit14 opened this issue Oct 16, 2023 · 10 comments · May be fixed by #130
Open
7 tasks

feature: Implement playwright basic setup #114

Nishchit14 opened this issue Oct 16, 2023 · 10 comments · May be fixed by #130
Assignees
Labels
features hacktoberfest Issue that are part of Hacktoberfest

Comments

@Nishchit14
Copy link
Contributor

Nishchit14 commented Oct 16, 2023

Description:

Firecamp currently does not have any playwright configuration. This feature request is to implement a basic playwright setup to allow for testing and deploying Firecamp services using Playwright.

This should include:

  • 1. Adding playwright and any required plugins as dependencies in package.json
  • 2. Creating a simple playwright config file with a test script that runs a basic test
  • 3. Setting up a playwright command to run tests
  • 4. Creating a sample test file that imports and tests a simple Firecamp module (open Firecamp and test simple GET api. maybe JSONplaceholder)
  • 5. Setting up Playwright to run on CI so tests are executed on each commit
  • 6. Updating CONTRIBUTING.md with instructions on how to run tests locally
  • 7. Updating README with a badge for playwright test status and note about the use of playwright for testing

This will lay the groundwork for an automated testing framework using Playwright. Additional tests can be added in future issues.

Resources

https://playwright.dev
https://playwright.dev/docs/intro#installing-playwright
https://playwright.dev/docs/writing-tests

@Nishchit14 Nishchit14 added hacktoberfest Issue that are part of Hacktoberfest features labels Oct 16, 2023
@aggmoulik
Copy link
Contributor

@Nishchit14 I will work on it, please assign to me

@Nishchit14
Copy link
Contributor Author

@aggmoulik are you working on it?

@nish17
Copy link

nish17 commented Oct 21, 2023

interested in contributing to this issue. let me know if i can

@aggmoulik
Copy link
Contributor

Hey @Nishchit14 yes working on it. Will create a PR today.

@aggmoulik aggmoulik linked a pull request Oct 21, 2023 that will close this issue
@shreya-gr
Copy link
Collaborator

Thank you @aggmoulik, more excited about the full PR. @nish17 let this PR get merged first, It will be followed by subsequent e2e tests. you can work on them.

@Nishchit14
Copy link
Contributor Author

@aggmoulik you seem busy, as we have other tasks that are dependent on this issue to get started. I will have to move forward from here. really appreciate your interest.

@nish17 would you like to continue with this task?

@aggmoulik
Copy link
Contributor

Hey @Nishchit14 sorry I was out of office, I suppose we can move forward as example test is only pending. I can make another PR for that if it is okay.

@nish17
Copy link

nish17 commented Oct 28, 2023

@aggmoulik - let me know if I can help out in anyway possible.

@Nishchit14
Copy link
Contributor Author

@aggmoulik sure, you can update the same PR. let me know once you're ready. i am planning to complete and merge this task by this week.

@Nishchit14
Copy link
Contributor Author

@nish17 would you be happy to take it further?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
features hacktoberfest Issue that are part of Hacktoberfest
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants