firecamp icon indicating copy to clipboard operation
firecamp copied to clipboard
verified publisher icon firecamp-dev

feature: Implement playwright basic setup

Open Nishchit14 opened this issue 2 years ago • 10 comments

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 avatar Oct 16 '23 12:10 Nishchit14

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

aggmoulik avatar Oct 16 '23 15:10 aggmoulik

@aggmoulik are you working on it?

Nishchit14 avatar Oct 20 '23 12:10 Nishchit14

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

nish17 avatar Oct 21 '23 04:10 nish17

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

aggmoulik avatar Oct 21 '23 05:10 aggmoulik

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.

shreya-gr avatar Oct 22 '23 19:10 shreya-gr

@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?

Nishchit14 avatar Oct 28 '23 07:10 Nishchit14

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.

aggmoulik avatar Oct 28 '23 08:10 aggmoulik

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

nish17 avatar Oct 28 '23 08:10 nish17

@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 avatar Oct 31 '23 11:10 Nishchit14

@nish17 would you be happy to take it further?

Nishchit14 avatar Nov 02 '23 06:11 Nishchit14