Skip to content

Commit 88031ca

Browse files
committed
fix: update status snapshots for PatternFly v5 tokens on 4.18
Cherry-picked tests from main used PF6 CSS variables; release-4.18 still renders PF5 tokens. Signed-off-by: Atif Ali <atali@redhat.com>
1 parent c40ffd6 commit 88031ca

2 files changed

Lines changed: 7 additions & 7 deletions

File tree

‎src/gitops/Statuses/ApplicationSetStatus.test.tsx‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,25 +5,25 @@ import ApplicationSetStatus from './ApplicationSetStatus';
55
describe('ApplicationSetStatus', () => {
66
it('renders Healthy', () => {
77
expect(renderToStaticMarkup(<ApplicationSetStatus status="Healthy" />)).toMatchInlineSnapshot(
8-
`"<span><svg data-icon="HeartIcon" style="color:var(--pf-t--global--icon--color--status--success--default)"></svg> Healthy</span>"`,
8+
`"<span><svg data-icon="HeartIcon" style="color:var(--pf-v5-global--success-color--100)"></svg> Healthy</span>"`,
99
);
1010
});
1111

1212
it('renders Error', () => {
1313
expect(renderToStaticMarkup(<ApplicationSetStatus status="Error" />)).toMatchInlineSnapshot(
14-
`"<span><svg data-icon="HeartBrokenIcon" style="color:var(--pf-t--global--icon--color--status--danger--default)"></svg> Error</span>"`,
14+
`"<span><svg data-icon="HeartBrokenIcon" style="color:var(--pf-v5-global--danger-color--100)"></svg> Error</span>"`,
1515
);
1616
});
1717

1818
it('renders Unknown for unrecognised status', () => {
1919
expect(renderToStaticMarkup(<ApplicationSetStatus status="Unknown" />)).toMatchInlineSnapshot(
20-
`"<span><svg data-icon="UnknownIcon" style="color:var(--pf-t--global--icon--color--disabled)"></svg> Unknown</span>"`,
20+
`"<span><svg data-icon="UnknownIcon" style="color:var(--pf-v5-global--disabled-color--100)"></svg> Unknown</span>"`,
2121
);
2222
});
2323

2424
it('renders Unknown icon for empty status', () => {
2525
expect(renderToStaticMarkup(<ApplicationSetStatus status="" />)).toMatchInlineSnapshot(
26-
`"<span><svg data-icon="UnknownIcon" style="color:var(--pf-t--global--icon--color--disabled)"></svg> </span>"`,
26+
`"<span><svg data-icon="UnknownIcon" style="color:var(--pf-v5-global--disabled-color--100)"></svg> </span>"`,
2727
);
2828
});
2929
});

‎src/gitops/Statuses/HealthStatus.test.tsx‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,13 +22,13 @@ describe('HealthStatus', () => {
2222

2323
it('renders Suspended', () => {
2424
expect(renderToStaticMarkup(<HealthStatus status="Suspended" />)).toMatchInlineSnapshot(
25-
`"<div><div><svg data-icon="OutlinedPauseCircleIcon" style="color:var(--pf-t--global--icon--color--disabled)"></svg> Suspended</div></div>"`,
25+
`"<div><div><svg data-icon="OutlinedPauseCircleIcon" style="color:var(--pf-v5-global--disabled-color--100)"></svg> Suspended</div></div>"`,
2626
);
2727
});
2828

2929
it('renders Missing', () => {
3030
expect(renderToStaticMarkup(<HealthStatus status="Missing" />)).toMatchInlineSnapshot(
31-
`"<div><div><svg data-icon="GhostIcon" style="color:var(--pf-t--global--icon--color--status--warning--default)"></svg> Missing</div></div>"`,
31+
`"<div><div><svg data-icon="GhostIcon" style="color:var(--pf-v5-global--warning-color--100)"></svg> Missing</div></div>"`,
3232
);
3333
});
3434

@@ -40,7 +40,7 @@ describe('HealthStatus', () => {
4040

4141
it('renders Unknown for undefined status', () => {
4242
expect(renderToStaticMarkup(<HealthStatus status={undefined as any} />)).toMatchInlineSnapshot(
43-
`"<div><div><svg data-icon="UnknownIcon" style="color:var(--pf-t--global--icon--color--disabled)"></svg> </div></div>"`,
43+
`"<div><div><svg data-icon="UnknownIcon" style="color:var(--pf-v5-global--disabled-color--100)"></svg> </div></div>"`,
4444
);
4545
});
4646

0 commit comments

Comments
 (0)