harbor icon indicating copy to clipboard operation
harbor copied to clipboard

feat(test): Migrate Common.robot to Playwright

Open vg006 opened this issue 2 weeks ago • 2 comments

Comprehensive Summary of your change

Issue being fixed

Fixes #22134

Summary

This PR aims to migrate tests in Common.robot. As a first step, few basic testcases are migrated and to track the progress the testcases are listed below.

In addition to that, this PR includes the setup for developing Playwright tests for the project, based on the PR #22462.

Please indicate you've done the following:

  • [x] Well Written Title and Summary of the PR
  • [x] Label the PR as needed. "release-note/ignore-for-release, release-note/new-feature, release-note/update, release-note/enhancement, release-note/community, release-note/breaking-change, release-note/docs, release-note/infra, release-note/deprecation"
  • [x] Accepted the DCO. Commits without the DCO will delay acceptance.
  • [x] Made sure tests are passing and test coverage is added if needed.
  • [x] Considered the docs impact and opened a new docs issue or PR with docs changes if needed in website repository.

vg006 avatar Dec 09 '25 05:12 vg006

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 65.85%. Comparing base (c8c11b4) to head (ce73295). :warning: Report is 604 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main   #22630       +/-   ##
===========================================
+ Coverage   45.36%   65.85%   +20.48%     
===========================================
  Files         244     1073      +829     
  Lines       13333   116095   +102762     
  Branches     2719     2931      +212     
===========================================
+ Hits         6049    76451    +70402     
- Misses       6983    35407    +28424     
- Partials      301     4237     +3936     
Flag Coverage Δ
unittests 65.85% <ø> (+20.48%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more. see 987 files with indirect coverage changes

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Dec 09 '25 07:12 codecov[bot]

Well done

Vad1mo avatar Dec 09 '25 17:12 Vad1mo

Hey @bupd!

As described in this test Verify Download CA Link, I noticed that there isn't any text like Registry Root Certificate in the System Settings[1].

Also I witnessed that this test case was last updated 4 years back during v1.10.7, So the text may be updated or removed due to a major bump. What's your thought on this? How to migrate this test case?

vg006 avatar Dec 15 '25 14:12 vg006

@vg006 feel free to remove it. its just for trusting the registry's self signed certificate.

bupd avatar Dec 17 '25 02:12 bupd