Skip to content

Commit bbb5db1

Browse files
committed
chore: fix tests
1 parent 662ee1e commit bbb5db1

File tree

36 files changed

+128
-17
lines changed

36 files changed

+128
-17
lines changed

packages/svelte/tests/runtime-runes/samples/props-id-everywhere/_config.js

Lines changed: 46 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -3,25 +3,57 @@ import { test } from '../../test';
33

44
export default test({
55
test({ assert, target, variant }) {
6-
const ps = [...target.querySelectorAll('p')].map((p) => p.innerHTML);
7-
const unique = new Set(ps);
8-
assert.equal(ps.length, unique.size);
9-
10-
if (variant === 'hydrate') {
11-
const start = ps.map((p) => p.substring(0, 1));
12-
assert.deepEqual(start, ['s', 's', 's', 's']);
6+
if (variant === 'dom') {
7+
assert.htmlEqual(
8+
target.innerHTML,
9+
`
10+
<button>toggle</button>
11+
<h1>c1</h1>
12+
<p>c2</p>
13+
<p>c3</p>
14+
<p>c4</p>
15+
`
16+
);
17+
} else {
18+
assert.htmlEqual(
19+
target.innerHTML,
20+
`
21+
<button>toggle</button>
22+
<h1>s1</h1>
23+
<p>s2</p>
24+
<p>s3</p>
25+
<p>s4</p>
26+
`
27+
);
1328
}
1429

1530
let button = target.querySelector('button');
1631
flushSync(() => button?.click());
1732

18-
const ps_after = [...target.querySelectorAll('p')].map((p) => p.innerHTML);
19-
const unique_after = new Set(ps_after);
20-
assert.equal(ps_after.length, unique_after.size);
21-
22-
if (variant === 'hydrate') {
23-
const start = ps_after.map((p) => p.substring(0, 1));
24-
assert.deepEqual(start, ['s', 's', 's', 's', 'c']);
33+
if (variant === 'dom') {
34+
assert.htmlEqual(
35+
target.innerHTML,
36+
`
37+
<button>toggle</button>
38+
<h1>c1</h1>
39+
<p>c2</p>
40+
<p>c3</p>
41+
<p>c4</p>
42+
<p>c5</p>
43+
`
44+
);
45+
} else {
46+
assert.htmlEqual(
47+
target.innerHTML,
48+
`
49+
<button>toggle</button>
50+
<h1>s1</h1>
51+
<p>s2</p>
52+
<p>s3</p>
53+
<p>s4</p>
54+
<p>c1</p>
55+
`
56+
);
2557
}
2658
}
2759
});

packages/svelte/tests/runtime-runes/samples/props-id-everywhere/main.svelte

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99

1010
<button onclick={() => show = !show}>toggle</button>
1111

12-
<p>{id}</p>
12+
<h1>{id}</h1>
1313

1414
<Child />
1515
<Child />

packages/svelte/tests/snapshot/samples/await-block-scope/_expected/client/index.svelte.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ function increment(_, counter) {
88
var root = $.template(`<button> </button> <!> `, 1);
99

1010
export default function Await_block_scope($$anchor) {
11+
const $$cleanup = $.setup();
1112
let counter = $.proxy({ count: 0 });
1213
const promise = $.derived(() => Promise.resolve(counter));
1314
var fragment = root();
@@ -31,6 +32,7 @@ export default function Await_block_scope($$anchor) {
3132
});
3233

3334
$.append($$anchor, fragment);
35+
$$cleanup();
3436
}
3537

3638
$.delegate(['click']);

packages/svelte/tests/snapshot/samples/await-block-scope/_expected/server/index.svelte.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import * as $ from 'svelte/internal/server';
22

33
export default function Await_block_scope($$payload) {
4+
const $$cleanup = $.setup($$payload);
45
let counter = { count: 0 };
56
const promise = Promise.resolve(counter);
67

@@ -11,4 +12,5 @@ export default function Await_block_scope($$payload) {
1112
$$payload.out += `<button>clicks: ${$.escape(counter.count)}</button> <!---->`;
1213
$.await(promise, () => {}, (counter) => {}, () => {});
1314
$$payload.out += `<!----> ${$.escape(counter.count)}`;
15+
$$cleanup($$payload);
1416
}

packages/svelte/tests/snapshot/samples/bind-component-snippet/_expected/client/index.svelte.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ const snippet = ($$anchor) => {
1313
var root = $.template(`<!> `, 1);
1414

1515
export default function Bind_component_snippet($$anchor) {
16+
const $$cleanup = $.setup();
1617
let value = $.state('');
1718
const _snippet = snippet;
1819
var fragment = root();
@@ -31,4 +32,5 @@ export default function Bind_component_snippet($$anchor) {
3132

3233
$.template_effect(() => $.set_text(text_1, ` value: ${$.get(value) ?? ''}`));
3334
$.append($$anchor, fragment);
35+
$$cleanup();
3436
}

packages/svelte/tests/snapshot/samples/bind-component-snippet/_expected/server/index.svelte.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ function snippet($$payload) {
66
}
77

88
export default function Bind_component_snippet($$payload) {
9+
const $$cleanup = $.setup($$payload);
910
let value = '';
1011
const _snippet = snippet;
1112
let $$settled = true;
@@ -32,4 +33,5 @@ export default function Bind_component_snippet($$payload) {
3233
} while (!$$settled);
3334

3435
$.assign_payload($$payload, $$inner_payload);
36+
$$cleanup($$payload);
3537
}

packages/svelte/tests/snapshot/samples/bind-this/_expected/client/index.svelte.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,5 +3,8 @@ import 'svelte/internal/flags/legacy';
33
import * as $ from 'svelte/internal/client';
44

55
export default function Bind_this($$anchor) {
6+
const $$cleanup = $.setup();
7+
68
$.bind_this(Foo($$anchor, { $$legacy: true }), ($$value) => foo = $$value, () => foo);
9+
$$cleanup();
710
}
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
11
import * as $ from 'svelte/internal/server';
22

33
export default function Bind_this($$payload) {
4+
const $$cleanup = $.setup($$payload);
5+
46
Foo($$payload, {});
7+
$$cleanup($$payload);
58
}

packages/svelte/tests/snapshot/samples/class-state-field-constructor-assignment/_expected/client/index.svelte.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@ import 'svelte/internal/disclose-version';
22
import * as $ from 'svelte/internal/client';
33

44
export default function Class_state_field_constructor_assignment($$anchor, $$props) {
5+
const $$cleanup = $.setup();
6+
57
$.push($$props, true);
68

79
class Foo {
@@ -24,4 +26,5 @@ export default function Class_state_field_constructor_assignment($$anchor, $$pro
2426
}
2527

2628
$.pop();
29+
$$cleanup();
2730
}

packages/svelte/tests/snapshot/samples/class-state-field-constructor-assignment/_expected/server/index.svelte.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
import * as $ from 'svelte/internal/server';
22

33
export default function Class_state_field_constructor_assignment($$payload, $$props) {
4+
const $$cleanup = $.setup($$payload);
5+
46
$.push();
57

68
class Foo {
@@ -14,4 +16,5 @@ export default function Class_state_field_constructor_assignment($$payload, $$pr
1416
}
1517

1618
$.pop();
19+
$$cleanup($$payload);
1720
}

0 commit comments

Comments
 (0)