Skip to content

Commit 38f1944

Browse files
jeremylenzclaude
andcommitted
Fix test failures - remove unsupported ouiaId from Label, Progress, Tooltip
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 db30b0a commit 38f1944

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)