Skip to content

Add initial Playwright automation#473

Open
tanmay-srivastav4 wants to merge 5 commits intopointblank-club:stagingfrom
tanmay-srivastav4:feature/playwright-mcp-setup
Open

Add initial Playwright automation#473
tanmay-srivastav4 wants to merge 5 commits intopointblank-club:stagingfrom
tanmay-srivastav4:feature/playwright-mcp-setup

Conversation

@tanmay-srivastav4
Copy link
Copy Markdown

@tanmay-srivastav4 tanmay-srivastav4 commented Jan 6, 2026 •
edited
Loading

Summary: What does this PR do?

<!
This PR introduces an initial Playwright-based end-to-end (E2E) testing setup for the website.

It adds route-level browser tests that verify pages are reachable and render correctly in a real browser. The setup is designed to be extensible and supports future automation and test generation workflows without impacting existing development or CI behavior.

-->

Changes Made

Describe the changes you've made in this PR:

  • Feature implementation
  • Added Playwright E2E test setup
  • Enabled test artifacts (screenshots, videos, traces, HTML report)

Type of Change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Describe the tests you ran:

  • Integration tests
  • Manual tests

Please describe the test cases and expected behavior:
1.Page load tests – Routes load correctly and render without errors
2. Failure handling – Screenshots, videos, and traces are generated on failure

Screenshots

<!
Screenshot 2026-01-06 190719

-->

Dependencies

<!
Added Playwright as a dev dependency
-->

@github-actions github-actions bot added size-XL size label for pr with extra large changes Dependencies Pull requests that update a dependency file Config Changes to project configuration files and removed size-XL size label for pr with extra large changes labels Jan 6, 2026
@vaibhavgarg25
Copy link
Copy Markdown
Contributor

vaibhavgarg25 commented Jan 8, 2026

Resolve the merge conflicts

@github-actions github-actions bot added the size-XL size label for pr with extra large changes label Jan 8, 2026
@tanmay-srivastav4
Copy link
Copy Markdown
Author

tanmay-srivastav4 commented Jan 8, 2026

resolved conflicts. please merge it bhaiya.

@tanmay-srivastav4 tanmay-srivastav4 changed the title Add initial Playwright MCP setup Add initial Playwright automation Jan 10, 2026
@github-actions github-actions bot added size-XL size label for pr with extra large changes and removed size-XL size label for pr with extra large changes labels Jan 10, 2026
@github-actions github-actions bot added size-XL size label for pr with extra large changes and removed size-XL size label for pr with extra large changes labels Jan 26, 2026
@ArkVex
Copy link
Copy Markdown
Contributor

ArkVex commented Jan 26, 2026

@tanmay-srivastav4 LGTM from my side can you just run Build and Lint tests and show me the results real quick?

@ArkVex
Copy link
Copy Markdown
Contributor

ArkVex commented Jan 26, 2026

Wait a minute...where did you do tooling of this mcp server?

@tanmay-srivastav4
Copy link
Copy Markdown
Author

tanmay-srivastav4 commented Jan 27, 2026 •
edited
Loading

Wait a minute...where did you do tooling of this mcp server?

this was just a playwright tests orchestration. Building e2e mcp server for it could be done in future

@ArkVex
Copy link
Copy Markdown
Contributor

ArkVex commented Jan 27, 2026

Acha acha...show me the build and lint tests

@tanmay-srivastav4 tanmay-srivastav4 force-pushed the feature/playwright-mcp-setup branch from a54c649 to b2c5c0c Compare January 27, 2026 18:25
@github-actions github-actions bot added size-XL size label for pr with extra large changes and removed size-XL size label for pr with extra large changes labels Jan 27, 2026
ArkVex
ArkVex reviewed Feb 7, 2026
View reviewed changes
Comment thread tsconfig.mcp.json Outdated
"strict": true,
"esModuleInterop": true
},
"include": ["mcp/**/*.ts"]
Copy link
Copy Markdown
Contributor

@ArkVex ArkVex Feb 7, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I dont think this is necessary

Copy link
Copy Markdown
Author

@tanmay-srivastav4 tanmay-srivastav4 Feb 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes looked into it again. it isn't required

ArkVex
ArkVex reviewed Feb 7, 2026
View reviewed changes
Comment thread tests/specs/navbar.spec.json
Comment on lines +4 to +14
{
"name": "Navbar links are visible",
"steps": [
{ "action": "goto", "value": "/" },
{ "action": "assertVisible", "value": "nav" },
{
"action": "assertVisible",
"value": "nav >> role=link[name=\"Achievements\"]"
}
]
}
Copy link
Copy Markdown
Contributor

@ArkVex ArkVex Feb 7, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ummm...I am not sure if this is how it is done

Copy link
Copy Markdown
Author

@tanmay-srivastav4 tanmay-srivastav4 Feb 10, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as far my knowledge, the .json file is used as input for test generation. also we can add more external test if required. when we run, it is converted into a Playwright .spec.ts test and then executed by Playwright like any other test.

@ArkVex
Copy link
Copy Markdown
Contributor

ArkVex commented Feb 7, 2026

I have added comments plz fix this

@github-actions github-actions bot added size-XL size label for pr with extra large changes and removed size-XL size label for pr with extra large changes labels Feb 10, 2026
@ArkVex
Copy link
Copy Markdown
Contributor

ArkVex commented Feb 10, 2026

Ggs lgtm...let's hope it doesn't break prod

@ArkVex
Copy link
Copy Markdown
Contributor

ArkVex commented Feb 10, 2026

@slashexx merge this plz

@ArkVex
Copy link
Copy Markdown
Contributor

ArkVex commented Feb 10, 2026

Wait hold

@github-actions github-actions bot added size-XL size label for pr with extra large changes and removed size-XL size label for pr with extra large changes labels Feb 10, 2026
@ArkVex
Copy link
Copy Markdown
Contributor

ArkVex commented Feb 11, 2026

Now this looks clean

@ArkVex
Copy link
Copy Markdown
Contributor

ArkVex commented Feb 11, 2026

@slashexx merge plz

@tanmay-srivastav4 tanmay-srivastav4 force-pushed the feature/playwright-mcp-setup branch from e413e31 to 7d8e66c Compare February 15, 2026 17:05
@github-actions github-actions bot added size-XL size label for pr with extra large changes and removed size-XL size label for pr with extra large changes labels Feb 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@SkySingh04 SkySingh04 Awaiting requested review from SkySingh04 SkySingh04 is a code owner

@XploY04 XploY04 Awaiting requested review from XploY04 XploY04 is a code owner

@alfiyafatima09 alfiyafatima09 Awaiting requested review from alfiyafatima09 alfiyafatima09 is a code owner

@0Xero7 0Xero7 Awaiting requested review from 0Xero7

@mohit-nagaraj mohit-nagaraj Awaiting requested review from mohit-nagaraj mohit-nagaraj is a code owner

@WasThatRudy WasThatRudy Awaiting requested review from WasThatRudy WasThatRudy is a code owner

@slashexx slashexx Awaiting requested review from slashexx slashexx is a code owner

@ShreyashSri ShreyashSri Awaiting requested review from ShreyashSri ShreyashSri is a code owner

1 more reviewer

@ArkVex ArkVex ArkVex left review comments

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

Config Changes to project configuration files Dependencies Pull requests that update a dependency file size-XL size label for pr with extra large changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@tanmay-srivastav4 @vaibhavgarg25 @ArkVex @dhruvpuri-aspora