Follow-up from "Add reusable TokenValidityBadge component"
The following discussion from !190538 (merged) should be addressed:
-
@dpisek started a discussion: suggestion (non-blocking/follow-up): I think we should remove this expectation.
Testing for the existence of CSS and therefore the visual representation of the component should not be done within a unit test.
There are a few reasons:
- Unit tests should focus on component behaviour and functionality rather than styling
- CSS class names may change during refactoring without affecting functionality
- Testing styling can make tests brittle and lead to unnecessary maintenance
It is a small change and since we want to get this out for %18.0 I'll open a follow-up issue where it can be handled. Feel free to assign the resulting MR directly to me
🙂