From d727a0d6e530dbf71003dea1bf1d9fbcb19bf6af Mon Sep 17 00:00:00 2001
From: Yuichiro Yamashita <xydybaseball@gmail.com>
Date: Sun, 22 May 2022 15:09:30 +0900
Subject: [PATCH] add more tests

---
 .../component-not-constructor-dev/Sub.svelte  |  1 +
 .../component-not-constructor-dev/_config.js  | 11 +++++++++++
 .../component-not-constructor-dev/main.svelte |  9 +++++++++
 .../component-not-constructor/_config.js      |  5 +----
 .../component-not-constructor2-dev/Sub.svelte |  1 +
 .../component-not-constructor2-dev/_config.js | 19 +++++++++++++++++++
 .../main.svelte                               | 14 ++++++++++++++
 .../component-not-constructor2/_config.js     | 14 +++++++-------
 .../component-not-constructor2/main.svelte    | 11 ++++++++---
 9 files changed, 71 insertions(+), 14 deletions(-)
 create mode 100644 test/runtime/samples/component-not-constructor-dev/Sub.svelte
 create mode 100644 test/runtime/samples/component-not-constructor-dev/_config.js
 create mode 100644 test/runtime/samples/component-not-constructor-dev/main.svelte
 create mode 100644 test/runtime/samples/component-not-constructor2-dev/Sub.svelte
 create mode 100644 test/runtime/samples/component-not-constructor2-dev/_config.js
 create mode 100644 test/runtime/samples/component-not-constructor2-dev/main.svelte

diff --git a/test/runtime/samples/component-not-constructor-dev/Sub.svelte b/test/runtime/samples/component-not-constructor-dev/Sub.svelte
new file mode 100644
index 0000000000..dd6d0bb1a9
--- /dev/null
+++ b/test/runtime/samples/component-not-constructor-dev/Sub.svelte
@@ -0,0 +1 @@
+<div>Sub</div>
\ No newline at end of file
diff --git a/test/runtime/samples/component-not-constructor-dev/_config.js b/test/runtime/samples/component-not-constructor-dev/_config.js
new file mode 100644
index 0000000000..facdcccd25
--- /dev/null
+++ b/test/runtime/samples/component-not-constructor-dev/_config.js
@@ -0,0 +1,11 @@
+export default {
+	skip_if_ssr: true,
+	skip_if_hydrate_from_ssr: true,
+	compileOptions: {
+		dev: true
+	},
+	props: {
+		selected: false
+	},
+	error: 'this={...} of <svelte:component> should specify a Svelte component.'
+};
diff --git a/test/runtime/samples/component-not-constructor-dev/main.svelte b/test/runtime/samples/component-not-constructor-dev/main.svelte
new file mode 100644
index 0000000000..dec7e2af79
--- /dev/null
+++ b/test/runtime/samples/component-not-constructor-dev/main.svelte
@@ -0,0 +1,9 @@
+<script>
+	import Sub from './Sub.svelte';
+	export let selected;
+	let banana = {};
+	let component = banana;
+	$: selected ? component = Sub : component = banana;
+</script>
+
+<svelte:component this={component} />
diff --git a/test/runtime/samples/component-not-constructor/_config.js b/test/runtime/samples/component-not-constructor/_config.js
index facdcccd25..8c0dcced41 100644
--- a/test/runtime/samples/component-not-constructor/_config.js
+++ b/test/runtime/samples/component-not-constructor/_config.js
@@ -1,11 +1,8 @@
 export default {
 	skip_if_ssr: true,
 	skip_if_hydrate_from_ssr: true,
-	compileOptions: {
-		dev: true
-	},
 	props: {
 		selected: false
 	},
-	error: 'this={...} of <svelte:component> should specify a Svelte component.'
+	error: 'component is not a constructor'
 };
diff --git a/test/runtime/samples/component-not-constructor2-dev/Sub.svelte b/test/runtime/samples/component-not-constructor2-dev/Sub.svelte
new file mode 100644
index 0000000000..dd6d0bb1a9
--- /dev/null
+++ b/test/runtime/samples/component-not-constructor2-dev/Sub.svelte
@@ -0,0 +1 @@
+<div>Sub</div>
\ No newline at end of file
diff --git a/test/runtime/samples/component-not-constructor2-dev/_config.js b/test/runtime/samples/component-not-constructor2-dev/_config.js
new file mode 100644
index 0000000000..98e3cc942b
--- /dev/null
+++ b/test/runtime/samples/component-not-constructor2-dev/_config.js
@@ -0,0 +1,19 @@
+export default {
+	compileOptions: {
+		dev: true
+	},
+	props: {
+		componentName: 'Sub'
+	},
+	html: '<div>Sub</div>',
+	test({ assert, component, target }) {
+		component.componentName = 'Proxy';
+		assert.htmlEqual(target.innerHTML, '<div>Sub</div>');
+		try {
+			component.componentName = 'banana';
+			throw new Error('Expected an error');
+		} catch (err) {
+			assert.equal(err.message, 'this={...} of <svelte:component> should specify a Svelte component.');
+		}
+	}
+};
diff --git a/test/runtime/samples/component-not-constructor2-dev/main.svelte b/test/runtime/samples/component-not-constructor2-dev/main.svelte
new file mode 100644
index 0000000000..331fdfc385
--- /dev/null
+++ b/test/runtime/samples/component-not-constructor2-dev/main.svelte
@@ -0,0 +1,14 @@
+<script>
+	import Sub from './Sub.svelte';
+	export let componentName = 'Sub';
+	let proxy = new Proxy(Sub, {});
+	let banana = {};
+	let component;
+	$: {
+		if (componentName === 'Sub') component = Sub;
+		else if (componentName === 'Proxy') component = proxy;
+		else component = banana;
+	};
+</script>
+
+<svelte:component this={component} />
diff --git a/test/runtime/samples/component-not-constructor2/_config.js b/test/runtime/samples/component-not-constructor2/_config.js
index 13ed09cecc..8fae65afdd 100644
--- a/test/runtime/samples/component-not-constructor2/_config.js
+++ b/test/runtime/samples/component-not-constructor2/_config.js
@@ -1,16 +1,16 @@
 export default {
-	compileOptions: {
-		dev: true
-	},
 	props: {
-		selected: true
+		componentName: 'Sub'
 	},
-	test({ assert, component }) {
+	html: '<div>Sub</div>',
+	test({ assert, component, target }) {
+		component.componentName = 'Proxy';
+		assert.htmlEqual(target.innerHTML, '<div>Sub</div>');
 		try {
-			component.selected = false;
+			component.componentName = 'banana';
 			throw new Error('Expected an error');
 		} catch (err) {
-			assert.equal(err.message, 'this={...} of <svelte:component> should specify a Svelte component.');
+			assert.equal(err.message, 'component is not a constructor');
 		}
 	}
 };
diff --git a/test/runtime/samples/component-not-constructor2/main.svelte b/test/runtime/samples/component-not-constructor2/main.svelte
index dec7e2af79..331fdfc385 100644
--- a/test/runtime/samples/component-not-constructor2/main.svelte
+++ b/test/runtime/samples/component-not-constructor2/main.svelte
@@ -1,9 +1,14 @@
 <script>
 	import Sub from './Sub.svelte';
-	export let selected;
+	export let componentName = 'Sub';
+	let proxy = new Proxy(Sub, {});
 	let banana = {};
-	let component = banana;
-	$: selected ? component = Sub : component = banana;
+	let component;
+	$: {
+		if (componentName === 'Sub') component = Sub;
+		else if (componentName === 'Proxy') component = proxy;
+		else component = banana;
+	};
 </script>
 
 <svelte:component this={component} />