Skip to content

Label Components - #5540

Merged
greenbonebot merged 6 commits into
mainfrom
label-components
Aug 19, 2026
Merged

Label Components#5540
greenbonebot merged 6 commits into
mainfrom
label-components

Conversation

@bjoernricks

Copy link
Copy Markdown
Contributor

What

Improve Label components

  • Add tests for all components
  • Use consistent naming for components and modules

Why

Have tests for all components in web/components 🥳

Checklist

  • Tests

@bjoernricks
bjoernricks requested a review from a team as a code owner August 18, 2026 13:24
@greenbonebot
greenbonebot enabled auto-merge (rebase) August 18, 2026 13:24
@github-actions

github-actions Bot commented Aug 18, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 5343cff.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@codecov

codecov Bot commented Aug 18, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 76.92308% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.11%. Comparing base (975c7b2) to head (5343cff).
⚠️ Report is 6 commits behind head on main.

Files with missing lines Patch % Lines
src/web/testing/custom-matchers.ts 68.42% 6 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5540   +/-   ##
=======================================
  Coverage   84.11%   84.11%           
=======================================
  Files        1294     1294           
  Lines       33775    33780    +5     
  Branches    10062    10060    -2     
=======================================
+ Hits        28409    28414    +5     
  Misses       4816     4816           
  Partials      550      550           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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

@bjoernricks
bjoernricks force-pushed the label-components branch 2 times, most recently from 4f898ce to a37cbbe Compare August 19, 2026 09:20
Use same name as the object of the default export. Also improve its
tests.
Ensure the function behaves as expected.
Make comparing colors more prone against the actual implementation by
converting the css color to the computed style color.
Use same name as the object exported as the default export. Also add
tests for the module.
Improve the module and add tests too.
Use PascalCase naming for components and modules.
@greenbonebot
greenbonebot merged commit 1165c20 into main Aug 19, 2026
23 checks passed
@greenbonebot
greenbonebot deleted the label-components branch August 19, 2026 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants