replace skip_if_ssr and skip_if_hydrate with modes (#10956)

Author: Rich-Harris

packages/svelte/tests/runtime-browser/assert.js

@@ -111,9 +111,9 @@ function normalize_children(node) {
  * @template Props
  * @param {{
  *	skip?: boolean;
- *	skip_if_ssr?: boolean | 'permanent';
- *	skip_if_hydrate?: boolean | 'permanent';
  *	solo?: boolean;
+ *  mode?: Array<'server' | 'client' | 'hydrate'>;
+ *  skip_mode?: Array<'server' | 'client' | 'hydrate'>;
  *	html?: string;
  *	ssrHtml?: string;
  *	props?: Props;

packages/svelte/tests/runtime-browser/samples/dynamic-element-custom-element/_config.js

@@ -1,8 +1,8 @@
 import { test } from '../../assert';
 
 export default test({
-	skip_if_ssr: 'permanent',
-	skip_if_hydrate: 'permanent',
+	mode: ['client'],
+
 	props: {
 		tag: /** @type {string | null} */ ('my-custom-element'),
 		name: /** @type {string | null | undefined} */ (null)

packages/svelte/tests/runtime-browser/samples/html-tag-script/_config.js

@@ -4,6 +4,5 @@ export default test({
 	// Test that @html does not execute scripts when instantiated in the client.
 	// Needs to be in this test suite because JSDOM does not quite get this right.
 	html: `<div></div><script>document.body.innerHTML = 'this should not be executed'</script>`,
-	skip_if_ssr: 'permanent',
-	skip_if_hydrate: 'permanent'
+	mode: ['client']
 });

packages/svelte/tests/runtime-browser/test-ssr.ts

@@ -46,7 +46,8 @@ export async function run_ssr_test(
 }
 
 const { run } = suite<ReturnType<typeof import('./assert').test>>(async (config, test_dir) => {
-	if (config.skip_if_ssr) return;
+	if (config.mode && !config.mode.includes('server')) return;
+	if (config.skip_mode?.includes('server')) return;
 	await run_ssr_test(config, test_dir);
 });
 

packages/svelte/tests/runtime-browser/test.ts

@@ -28,7 +28,11 @@ const { run: run_browser_tests } = suite_with_variants<
 >(
 	['dom', 'hydrate'],
 	(variant, config) => {
-		if (variant === 'hydrate' && config.skip_if_hydrate) return true;
+		if (variant === 'hydrate') {
+			if (config.mode && !config.mode.includes('hydrate')) return 'no-test';
+			if (config.skip_mode?.includes('hydrate')) return true;
+		}
+
 		return false;
 	},
 	() => {},

packages/svelte/tests/runtime-legacy/samples/action-custom-event-handler-node-context/_config.js

@@ -1,7 +1,7 @@
 import { ok, test } from '../../test';
 
 export default test({
-	skip_if_ssr: 'permanent', // unnecessary to test this in ssr mode
+	mode: ['client', 'hydrate'],
 
 	html: '<button>10</button>',
 

packages/svelte/tests/runtime-legacy/samples/after-render-prevents-loop/_config.js

@@ -1,7 +1,7 @@
 import { test } from '../../test';
 
 export default test({
-	skip_if_ssr: true,
+	skip_mode: ['server'],
 
 	get props() {
 		return { value: 'hello!' };

packages/svelte/tests/runtime-legacy/samples/after-render-triggers-update/_config.js

@@ -1,7 +1,7 @@
 import { test } from '../../test';
 
 export default test({
-	skip_if_ssr: true,
+	skip_mode: ['server'],
 
 	get props() {
 		return { value: 'hello!' };

packages/svelte/tests/runtime-legacy/samples/attribute-casing-custom-element/_config.js

@@ -1,8 +1,7 @@
 import { test } from '../../test';
 
 export default test({
-	skip_if_ssr: 'permanent',
-	skip_if_hydrate: 'permanent',
+	mode: ['client'],
 	html: `
 		<my-custom-element>Hello World!</my-custom-element>
 	`

packages/svelte/tests/runtime-legacy/samples/attribute-casing-foreign-namespace-compiler-option/_config.js

@@ -9,11 +9,11 @@ export default test({
 			<text wordWrap="true"></text>
 		</page>
 	`,
-	skip_if_hydrate: true,
 
 	compileOptions: {
 		namespace: 'foreign'
 	},
+
 	test({ assert, target }) {
 		// @ts-ignore
 		const attr = (/** @type {string} */ sel) => target.querySelector(sel).attributes[0].name;