Skip to content

Fixes #39487 - Replace enzyme test in SubstringWrapper with RTL#11079

Open
kfamilonidis wants to merge 1 commit into
theforeman:developfrom
kfamilonidis:pf5-substring-wrapper
Open

Fixes #39487 - Replace enzyme test in SubstringWrapper with RTL#11079
kfamilonidis wants to merge 1 commit into
theforeman:developfrom
kfamilonidis:pf5-substring-wrapper

Conversation

@kfamilonidis

@kfamilonidis kfamilonidis commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Fixes #39487: Replace enzyme test in SubstringWrapper with RTL

@github-actions github-actions Bot added the UI label Jul 2, 2026
@kfamilonidis kfamilonidis changed the title Fixes #39487: Replace enzyme test in SubscringWrapper with RTL Fixes #39487: Replace enzyme test in SubstringWrapper with RTL Jul 3, 2026
@Lukshio

Lukshio commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Summary

Clean 1:1 migration of all 5 snapshot fixtures to explicit RTL assertions (default wrap, double wrap, custom element, no-match, invalid-regex fallback). No functional concerns.

Critical (must fix before merge)

  • None

Suggestions (should fix)

  • Commit message uses Fixes #39487: ... - the documented convention (see develop history, e.g. Fixes #38653 - Fix settings show endpoint) is Fixes #XXXXX - description with a hyphen, not a colon.

Nice to have (optional)

  • None

Checklist gaps

  • Commit message hyphen convention.

@kfamilonidis
kfamilonidis force-pushed the pf5-substring-wrapper branch from 775fd17 to fb932fe Compare July 14, 2026 12:40
@kfamilonidis kfamilonidis changed the title Fixes #39487: Replace enzyme test in SubstringWrapper with RTL Fixes #39487 - Replace enzyme test in SubstringWrapper with RTL Jul 14, 2026
@kfamilonidis

Copy link
Copy Markdown
Contributor Author

@Lukshio commit title fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants