Skip to content

Commit 269ceff

Browse files
authored
chore: bump ui-svelte to 1.23.1 and fix tests (#479)
Signed-off-by: Philippe Martin <[email protected]>
1 parent 138faf0 commit 269ceff

File tree

5 files changed

+46
-8
lines changed

5 files changed

+46
-8
lines changed

packages/webview/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@
3030
"@fortawesome/free-brands-svg-icons": "^7.0.0",
3131
"@fortawesome/free-regular-svg-icons": "^7.0.1",
3232
"@fortawesome/free-solid-svg-icons": "^7.0.0",
33-
"@podman-desktop/ui-svelte": "1.22.1",
33+
"@podman-desktop/ui-svelte": "1.23.1",
3434
"@podman-desktop/webview-api": "1.23.1",
3535
"@podman-desktop/kubernetes-dashboard-extension-api": "workspace:*",
3636
"@sveltejs/vite-plugin-svelte": "^6",

packages/webview/src/component/connection/CurrentContextConnectionBadge.spec.ts

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ import { StatesMocks } from '/@/tests/state-mocks';
2626
import { FakeStateObject } from '/@/state/util/fake-state-object.svelte';
2727
import CurrentContextConnectionBadge from './CurrentContextConnectionBadge.svelte';
2828
import type { ContextsHealthsInfo } from '@podman-desktop/kubernetes-dashboard-extension-api';
29+
import { fireEvent } from '@testing-library/dom';
2930

3031
const statesMocks = new StatesMocks();
3132

@@ -102,6 +103,9 @@ describe('current context is reachable', () => {
102103
test('no tooltip', async () => {
103104
render(CurrentContextConnectionBadge);
104105

106+
const tooltipTrigger = screen.getByTestId('tooltip-trigger');
107+
fireEvent.mouseEnter(tooltipTrigger);
108+
105109
expect(screen.queryByLabelText('tooltip')).toBeNull();
106110
});
107111
});
@@ -146,6 +150,9 @@ describe('current context is not reachable', () => {
146150
test('tooltip', async () => {
147151
render(CurrentContextConnectionBadge);
148152

153+
const tooltipTrigger = screen.getByTestId('tooltip-trigger');
154+
fireEvent.mouseEnter(tooltipTrigger);
155+
149156
expect(screen.getByLabelText('tooltip')).toBeDefined();
150157
});
151158
});
@@ -190,6 +197,9 @@ describe('current context is offline', () => {
190197
test('expect tooltip when offline', async () => {
191198
render(CurrentContextConnectionBadge);
192199

200+
const tooltipTrigger = screen.getByTestId('tooltip-trigger');
201+
fireEvent.mouseEnter(tooltipTrigger);
202+
193203
const tooltip = screen.getByLabelText('tooltip');
194204
expect(tooltip).toHaveTextContent('connection lost, you can try to reconnect');
195205
});

packages/webview/src/component/label/Label.spec.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818

1919
import '@testing-library/jest-dom/vitest';
2020

21-
import { render, screen } from '@testing-library/svelte';
21+
import { fireEvent, render, screen } from '@testing-library/svelte';
2222
import { expect, test } from 'vitest';
2323

2424
import LabelSpec from './LabelSpec.svelte';
@@ -48,6 +48,10 @@ test('Expect tooltip', async () => {
4848
name: 'label',
4949
tip: tip,
5050
});
51+
52+
const tooltipTrigger = screen.getByTestId('tooltip-trigger');
53+
await fireEvent.mouseEnter(tooltipTrigger);
54+
5155
const label = screen.getByText(tip);
5256
expect(label).toBeInTheDocument();
5357
expect(label.parentElement?.firstChild).toBeInTheDocument();

packages/webview/src/component/port-forward/KubePort.spec.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818

1919
import '@testing-library/jest-dom/vitest';
2020

21-
import { fireEvent, render } from '@testing-library/svelte';
21+
import { fireEvent, render, screen } from '@testing-library/svelte';
2222
import { beforeEach, describe, expect, test, vi } from 'vitest';
2323

2424
import KubePort from './KubePort.svelte';
@@ -203,6 +203,9 @@ describe('port forwarding', () => {
203203
kind: WorkloadKind.POD,
204204
});
205205

206+
const tooltipTrigger = screen.getByTestId('tooltip-trigger');
207+
await fireEvent.mouseEnter(tooltipTrigger);
208+
206209
const port80 = queryByTitle('Forward port 80');
207210
expect(port80).toBeNull();
208211

pnpm-lock.yaml

Lines changed: 26 additions & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)