Skip to content

Commit 5373fd7

Browse files
committed
Refactor code structure for improved readability and maintainability
1 parent 3af6bf9 commit 5373fd7

File tree

4 files changed

+618
-757
lines changed

4 files changed

+618
-757
lines changed

packages/sveltekit-embed/package.json

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -78,15 +78,15 @@
7878
"svelte": "^4.0.0 || ^5.0.0"
7979
},
8080
"devDependencies": {
81-
"@eslint/compat": "^1.2.9",
81+
"@eslint/compat": "^2.0.0",
8282
"@eslint/js": "^9.27.0",
8383
"@playwright/test": "^1.52.0",
84-
"@sveltejs/adapter-auto": "^6.0.1",
85-
"@sveltejs/kit": "^2.21.1",
84+
"@sveltejs/adapter-auto": "^7.0.0",
85+
"@sveltejs/kit": "^2.49.1",
8686
"@sveltejs/package": "^2.3.11",
87-
"@sveltejs/vite-plugin-svelte": "^5.0.3",
88-
"@vitest/browser": "^3.1.4",
89-
"@vitest/coverage-v8": "3.1.4",
87+
"@sveltejs/vite-plugin-svelte": "^6.2.1",
88+
"@vitest/browser": "^4.0.15",
89+
"@vitest/coverage-v8": "4.0.15",
9090
"eslint": "^9.27.0",
9191
"eslint-config-prettier": "^10.1.5",
9292
"eslint-plugin-svelte": "^3.9.0",
@@ -102,9 +102,9 @@
102102
"svelte-check": "^4.2.1",
103103
"typescript": "^5.8.3",
104104
"typescript-eslint": "^8.32.1",
105-
"vite": "^6.3.5",
106-
"vitest": "^3.1.4",
107-
"vitest-browser-svelte": "^0.1.0"
105+
"vite": "^7.2.6",
106+
"vitest": "^4.0.15",
107+
"vitest-browser-svelte": "^2.0.1"
108108
},
109109
"svelte": "./dist/index.js",
110110
"types": "./dist/index.d.ts",

packages/sveltekit-embed/src/lib/components/deezer.svelte.test.ts

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -69,25 +69,25 @@ describe('Deezer', () => {
6969

7070
it('should handle special characters in frameSrc', async () => {
7171
const specialFrameSrc = 'track/123-test_track';
72-
const { getByTitle } = render(Deezer, {
72+
render(Deezer, {
7373
theme,
7474
frameSrc: specialFrameSrc,
7575
disable_observer: true,
7676
});
77-
const iframe = getByTitle('deezer-widget');
77+
const iframe = page.getByTitle('deezer-widget');
7878
const element = iframe.element() as HTMLIFrameElement;
7979

8080
expect(element.src).toContain(specialFrameSrc);
8181
});
8282

8383
it('should handle very long frameSrc values', async () => {
8484
const longFrameSrc = 'track/' + 'a'.repeat(1000);
85-
const { getByTitle } = render(Deezer, {
85+
render(Deezer, {
8686
theme,
8787
frameSrc: longFrameSrc,
8888
disable_observer: true,
8989
});
90-
const iframe = getByTitle('deezer-widget');
90+
const iframe = page.getByTitle('deezer-widget');
9191
const element = iframe.element() as HTMLIFrameElement;
9292

9393
expect(element.src).toContain(longFrameSrc);
@@ -96,23 +96,23 @@ describe('Deezer', () => {
9696

9797
describe('Default Props', () => {
9898
it('should apply default theme when not provided', async () => {
99-
const { getByTitle } = render(Deezer, {
99+
render(Deezer, {
100100
frameSrc,
101101
disable_observer: true,
102102
});
103-
const iframe = getByTitle('deezer-widget');
103+
const iframe = page.getByTitle('deezer-widget');
104104
const element = iframe.element() as HTMLIFrameElement;
105105

106106
expect(element.src).toContain('widget/auto/');
107107
});
108108

109109
it('should apply default border-radius styling', async () => {
110-
const { getByTitle } = render(Deezer, {
110+
render(Deezer, {
111111
theme,
112112
frameSrc,
113113
disable_observer: true,
114114
});
115-
const iframe = getByTitle('deezer-widget');
115+
const iframe = page.getByTitle('deezer-widget');
116116
const element = iframe.element() as HTMLIFrameElement;
117117

118118
expect(element.style.borderRadius).toBe('0.6rem');

packages/sveltekit-embed/vite.config.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,20 @@
11
import { sveltekit } from '@sveltejs/kit/vite';
2+
import { playwright } from '@vitest/browser/playwright';
23
import { defineConfig } from 'vite';
34
import { coverageConfigDefaults } from 'vitest/config';
45

56
export default defineConfig({
67
plugins: [sveltekit()],
78
test: {
8-
workspace: [
9+
projects: [
910
{
1011
extends: './vite.config.ts',
1112
test: {
1213
name: 'client',
1314
environment: 'browser',
1415
browser: {
1516
enabled: true,
16-
provider: 'playwright',
17+
provider: playwright(),
1718
instances: [
1819
{
1920
browser: 'chromium',
@@ -37,7 +38,7 @@ export default defineConfig({
3738
},
3839
],
3940
coverage: {
40-
all: true,
41+
include: ['src/**/*.{ts,svelte}'],
4142
reporter: ['text-summary', 'html'],
4243
exclude: [
4344
...coverageConfigDefaults.exclude,

0 commit comments

Comments
 (0)