Migrate Harbor UI E2E tests from Robot + Selenium to Playwright#268
Migrate Harbor UI E2E tests from Robot + Selenium to Playwright#268bupd wants to merge 7 commits intogoharbor:mainfrom
Conversation
7534034 to
10ae484
Compare
|
Hi @bupd , We may need more thoroughly mindful investigation on the harbor whole robot-case code base, especially on the UI part.
|
I don't get this. Why do we need a cross environment - what cross environment mean here ?? Please explain.
This makes no sense. The goal is clearly mentioned here https://github.com/goharbor/community/pull/268/files#diff-e9c149ab1ab59f390feca2254d8ccd8c76643c1016b5e6651c7dcd712341ada2R26 We are not removing Robot from areas where it might be the right tool. I am sure that for UI tests robot is a inadequate tool. We are eliminating XPath-dependent UI flakiness by shifting only those tests to Playwright. |
|
Hi @bupd , Generally, One of our key acceptance standards is do not decrease the current quality which would reflects in these criteria that you need to take it consideration:
Could you please help to demonstrate in your proposal how to achieve above goals or solutions for the potential issues? Best, |
|
I believe there're still disagreements about this proposal. Miner has expressed a lot of concerns in last week's community meeting. We should make sure the comments are resolved before this proposal can be merged. |
There was a problem hiding this comment.
Hi @bupd
Please help to update the proposal per this comment and the discussion we had last month.
#268 (comment)
Best,
Miner
@reasonerjt I would like to clarify that the concerns have been addressed in the previous community meeting. I was in OSS summit for past week. currently getting back on this. Only thing pending on this is to update the proposal with screenshots of reports and add more info on the running this containerised, environments. as we discussed earlier I am currently recovering will update the proposal shortly. Thanks for understanding |
|
@MinerYang @stonezdj @reasonerjt All concerns have been addressed. |
88041b7 to
f3d3a2f
Compare
Signed-off-by: bupd <bupdprasanth@gmail.com>
Expand on the benefits and features of using Playwright for UI testing, including portability, containerization, and improved debugging capabilities. Outline goals and non-goals for the migration process from Robot tests to Playwright. Signed-off-by: Prasanth Baskar <bupdprasanth@gmail.com>
Signed-off-by: bupd <bupdprasanth@gmail.com>
Add Technical Considerations section covering: - Process orchestration and parallelism handling - Credential safety in CI environments - SSH operations support - Reporting comparison with Robot Framework Clarify proposal scope to UI E2E tests only, with infrastructure tests remaining in Robot Framework. Remove bold formatting per style guide. Signed-off-by: bupd <bupdprasanth@gmail.com>
Update Dockerfile to Ubuntu 20.04-based image matching Robot Framework structure. Add comprehensive Q&A section answering: - How running commands look like (Docker and direct execution) - Keyword-driven execution support with real examples Include references to PRs #22462 and #22591 showing actual implementation. Signed-off-by: bupd <bupdprasanth@gmail.com>
…hots Add three screenshots demonstrating Playwright reporting capabilities: - HTML test report showing all tests passing - Video recording feature in test reports - Interactive debug mode with Playwright Inspector Signed-off-by: bupd <bupdprasanth@gmail.com>
f3d3a2f to
0374737
Compare
|
@MinerYang @stonezdj @reasonerjt also added relevant screenshots of test reports, debugging, networking capabilities. please take a look. Thanks |
Tracking Issue: goharbor/harbor#22134