Add failing test for non-changing props

packages/e2e-tests/plugins/interactive-blocks/get-server-state/render.php

@@ -14,12 +14,14 @@
 
 <div
 	data-wp-interactive="test/get-server-state"
-	data-wp-watch="callbacks.updateState"
+	data-wp-watch---update-state="callbacks.updateState"
+	data-wp-watch---non-changing="callbacks.updateNonChanging"
 >
 	<div data-testid="prop" data-wp-text="state.prop"></div>
 	<div data-testid="nested.prop" data-wp-text="state.nested.prop"></div>
 	<div data-testid="newProp" data-wp-text="state.newProp"></div>
 	<div data-testid="nested.newProp" data-wp-text="state.nested.newProp"></div>
+	<div data-testid="nonChanging" data-wp-text="state.nonChanging"></div>
 
 	<button
 		data-testid="tryToModifyServerState"
@@ -30,6 +32,12 @@
 		modify
 	</button>
 
+	<button
+		data-testid="updateNonChanging"
+		data-wp-on--click="actions.updateNonChanging"
+	>
+		update non-changing prop
+	</button>
 
 	<nav>
 		<?php

packages/e2e-tests/plugins/interactive-blocks/get-server-state/view.js

@@ -25,6 +25,9 @@ const { state } = store( 'test/get-server-state', {
 				getContext().result = 'not modified ✅';
 			}
 		},
+		updateNonChanging() {
+			state.nonChanging = 'modified from client';
+		},
 	},
 	callbacks: {
 		updateState() {
@@ -34,5 +37,12 @@ const { state } = store( 'test/get-server-state', {
 			state.nested.prop = nested.prop;
 			state.nested.newProp = nested.newProp;
 		},
+		updateNonChanging() {
+			// This property never changes in the server, but it changes in the
+			// client so every time there's a navigation, we need to overwrite
+			// it.
+			const { nonChanging } = getServerState();
+			state.nonChanging = nonChanging;
+		},
 	},
 } );

test/e2e/specs/interactivity/get-sever-state.spec.ts

@@ -16,6 +16,7 @@ test.describe( 'getServerState()', () => {
 						prop: 'link 1',
 						newProp: 'link 1',
 					},
+					nonChanging: 'modified from server',
 				},
 			},
 		} );
@@ -29,6 +30,7 @@ test.describe( 'getServerState()', () => {
 						prop: 'link 2',
 						newProp: 'link 2',
 					},
+					nonChanging: 'modified from server',
 				},
 			},
 		} );
@@ -42,6 +44,7 @@ test.describe( 'getServerState()', () => {
 					nested: {
 						prop: 'main',
 					},
+					nonChanging: 'modified from server',
 				},
 			},
 		} );
@@ -116,4 +119,34 @@ test.describe( 'getServerState()', () => {
 		await expect( prop ).toHaveText( 'main' );
 		await expect( button ).toHaveText( 'not modified ✅' );
 	} );
+
+	test( 'should overwrite non-changing props on navigation', async ( {
+		page,
+	} ) => {
+		const nonChanging = page.getByTestId( 'nonChanging' );
+		const button = page.getByTestId( 'updateNonChanging' );
+
+		await expect( page ).toHaveTitle( /main/ );
+		await expect( nonChanging ).toHaveText( 'modified from server' );
+
+		await button.click();
+
+		await expect( nonChanging ).toHaveText( 'modified from client' );
+
+		await page.getByTestId( 'link 1' ).click();
+		await expect( page ).toHaveTitle( /link 1/ );
+
+		// The prop is overwritten on navigation.
+		await expect( nonChanging ).toHaveText( 'modified from server' );
+
+		await button.click();
+
+		await expect( nonChanging ).toHaveText( 'modified from client' );
+
+		await page.goBack();
+		await expect( page ).toHaveTitle( /main/ );
+
+		// The prop is overwritten on navigation.
+		await expect( nonChanging ).toHaveText( 'modified from server' );
+	} );
 } );