more tests and tweaks

Author: trueadm

packages/svelte/src/compiler/phases/3-transform/client/visitors/SvelteBoundary.js

@@ -27,17 +27,25 @@ export function SvelteBoundary(node, context) {
 
 	for (const attribute of node.attributes) {
 		if (attribute.type === 'SpreadAttribute') {
+			const value = /** @type {Expression} */ (context.visit(attribute.expression, context.state));
 			has_spread = true;
-			props_and_spreads.push(attribute.expression);
+
+			if (attribute.metadata.expression.has_state) {
+				props_and_spreads.push(b.thunk(value));
+			} else {
+				props_and_spreads.push(value);
+			}
 			continue;
 		}
+
 		if (
 			attribute.type !== 'Attribute' ||
 			attribute.value === true ||
 			Array.isArray(attribute.value)
 		) {
 			continue;
 		}
+
 		if (attribute.name === 'onerror' || attribute.name === 'failed') {
 			const value = /** @type {Expression} */ (
 				context.visit(attribute.value.expression, context.state)

packages/svelte/tests/runtime-runes/samples/error-boundary-10/_config.js

@@ -0,0 +1,14 @@
+import { flushSync } from 'svelte';
+import { test } from '../../test';
+
+export default test({
+	test({ assert, target, logs }) {
+		const [btn1, btn2] = target.querySelectorAll('button');
+
+		btn2?.click();
+		btn1?.click();
+		flushSync();
+
+		assert.deepEqual(logs, ['error caught!!!']);
+	}
+});

packages/svelte/tests/runtime-runes/samples/error-boundary-10/main.svelte

@@ -0,0 +1,17 @@
+<script>
+	function throw_error() {
+		throw new Error('test')
+	}
+
+	let count = $state(0);
+	let onerror = $state((e) => console.log('error caught'));
+</script>
+
+<svelte:boundary {onerror}>
+	{count > 0 ? throw_error() : null}
+</svelte:boundary>
+
+<button onclick={() => count++}>+</button>
+<button onclick={() => onerror = () => console.log('error caught!!!')}>change error message</button>
+
+

packages/svelte/tests/runtime-runes/samples/error-boundary-11/_config.js

@@ -0,0 +1,14 @@
+import { flushSync } from 'svelte';
+import { test } from '../../test';
+
+export default test({
+	test({ assert, target, logs }) {
+		const [btn1, btn2] = target.querySelectorAll('button');
+
+		btn2?.click();
+		btn1?.click();
+		flushSync();
+
+		assert.deepEqual(logs, ['error caught!!!']);
+	}
+});

packages/svelte/tests/runtime-runes/samples/error-boundary-11/main.svelte

@@ -0,0 +1,17 @@
+<script>
+	function throw_error() {
+		throw new Error('test')
+	}
+
+	let count = $state(0);
+	let props = $state({ onerror: (e) => console.log('error caught') });
+</script>
+
+<svelte:boundary {...props}>
+	{count > 0 ? throw_error() : null}
+</svelte:boundary>
+
+<button onclick={() => count++}>+</button>
+<button onclick={() => props = { onerror: () => console.log('error caught!!!') }}>change error message</button>
+
+