Skip to content

Commit 8f586d3

Browse files
jeremylenzclaude
andcommitted
Refs #38901 - Fix test failures by removing unsupported ouiaId props
PatternFly's Label, Progress, and Tooltip components don't support the ouiaId prop, which was causing React warnings and test failures in CI. Removed ouiaId from these components while keeping it on components that do support it (Table, Button, etc). 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent f41b77b commit 8f586d3

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

webpack/scenes/SyncStatus/components/SyncProgressCell.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ const SyncProgressCell = ({ repo, onCancelSync }) => {
2424
value={progressValue}
2525
title={__('Syncing')}
2626
size="sm"
27-
ouiaId={`progress-${id}`}
2827
/>
2928
</FlexItem>
3029
<FlexItem>

webpack/scenes/SyncStatus/components/SyncResultCell.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ const SyncResultCell = ({ repo }) => {
4646
const taskUrl = syncId ? foremanUrl(`/foreman_tasks/tasks/${syncId}`) : null;
4747

4848
const labelContent = (
49-
<Label color={color} icon={icon} ouiaId="sync-result-label">
49+
<Label color={color} icon={icon}>
5050
{taskUrl ? (
5151
<a href={taskUrl} target="_blank" rel="noopener noreferrer">
5252
{label}
@@ -65,7 +65,7 @@ const SyncResultCell = ({ repo }) => {
6565

6666
if (errorText && errorText.length > 0) {
6767
return (
68-
<Tooltip content={errorText} ouiaId="sync-error-tooltip">
68+
<Tooltip content={errorText}>
6969
{labelContent}
7070
</Tooltip>
7171
);

0 commit comments

Comments
 (0)