fix: test for simple_set and simple_set

paoloricciuti · paoloricciuti · commit 35e2afe2bb45 · 2025-01-24T10:24:57.000+01:00
diff --git a/packages/svelte/src/internal/client/reactivity/sources.js b/packages/svelte/src/internal/client/reactivity/sources.js
@@ -225,6 +225,8 @@ export function simple_set(source, value, should_proxy = false, needs_previous =
 
 	source.v = new_value;
 
+	source.o?.onchange?.();
+
 	return new_value;
 }
 
diff --git a/packages/svelte/tests/runtime-runes/samples/state-onchange/main.svelte b/packages/svelte/tests/runtime-runes/samples/state-onchange/main.svelte
@@ -37,7 +37,7 @@
 
 
 		constructor(){
-			this.#in_constructor++;
+			this.#in_constructor = 42;
 			this.#in_constructor_proxy.count++;
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -225,6 +225,8 @@ export function simple_set(source, value, should_proxy = false, needs_previous =`
`225`	`225`
`226`	`226`	`source.v = new_value;`
`227`	`227`
	`228`	`+ source.o?.onchange?.();`
	`229`	`+`
`228`	`230`	`return new_value;`
`229`	`231`	`}`
`230`	`232`
Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@`
`37`	`37`
`38`	`38`
`39`	`39`	`constructor(){`
`40`		`- this.#in_constructor++;`
	`40`	`+ this.#in_constructor = 42;`
`41`	`41`	`this.#in_constructor_proxy.count++;`
`42`	`42`	`}`
`43`	`43`	`}`