fix some more tests

src/compile/Component.ts

@@ -168,7 +168,7 @@ export default class Component {
 		const helpers = new Set();
 
 		// TODO use same regex for both
-		result = result.replace(options.generate === 'ssr' ? /(@+|#+|%+)(\w*(?:-\w*)?)/g : /(%+|@+)(\w*(?:-\w*)?)/g, (match: string, sigil: string, name: string) => {
+		result = result.replace(options.generate === 'ssr' ? /(@+|#+)(\w*(?:-\w*)?)/g : /(@+)(\w*(?:-\w*)?)/g, (match: string, sigil: string, name: string) => {
 			if (sigil === '@') {
 				if (name in internal) {
 					if (options.dev && `${name}Dev` in internal) name = `${name}Dev`;
@@ -178,10 +178,6 @@ export default class Component {
 				return this.alias(name);
 			}
 
-			if (sigil === '%') {
-				return this.templateVars.get(name);
-			}
-
 			return sigil.slice(1) + name;
 		});
 

src/compile/nodes/shared/Expression.ts

@@ -8,6 +8,7 @@ import addToSet from '../../../utils/addToSet';
 import globalWhitelist from '../../../utils/globalWhitelist';
 import deindent from '../../../utils/deindent';
 import Wrapper from '../../render-dom/wrappers/shared/Wrapper';
+import sanitize from '../../../utils/sanitize';
 
 const binaryOperators: Record<string, number> = {
 	'**': 15,
@@ -194,7 +195,9 @@ export default class Expression {
 						// the return value doesn't matter
 					}
 
-					const name = component.getUniqueName(get_function_name(node, owner));
+					const name = component.getUniqueName(
+						sanitize(get_function_name(node, owner))
+					);
 
 					const args = contextual_dependencies.size > 0
 						? [`{ ${[...contextual_dependencies].join(', ')} }`]

src/compile/render-dom/wrappers/Slot.ts

@@ -5,10 +5,7 @@ import Slot from '../../nodes/Slot';
 import { quotePropIfNecessary } from '../../../utils/quoteIfNecessary';
 import FragmentWrapper from './Fragment';
 import deindent from '../../../utils/deindent';
-
+import sanitize from '../../../utils/sanitize';
-function sanitize(name) {
-	return name.replace(/[^a-zA-Z]+/g, '_').replace(/^_/, '').replace(/_$/, '');
-}
 
 export default class SlotWrapper extends Wrapper {
 	node: Slot;

src/utils/sanitize.ts

@@ -0,0 +1,3 @@
+export default function sanitize(name) {
+	return name.replace(/[^a-zA-Z]+/g, '_').replace(/^_/, '').replace(/_$/, '');
+}

src/utils/stringify.ts

@@ -3,7 +3,7 @@ export function stringify(data: string, options = {}) {
 }
 
 export function escape(data: string, { onlyEscapeAtSymbol = false } = {}) {
-	return data.replace(onlyEscapeAtSymbol ? /(%+|@+)/g : /(%+|@+|#+)/g, (match: string) => {
+	return data.replace(onlyEscapeAtSymbol ? /@+/g : /(@+|#+)/g, (match: string) => {
 		return match + match[0];
 	});
 }

test/runtime/samples/dynamic-component-nulled-out/_config.js

@@ -6,13 +6,13 @@ export default {
 	nestedTransitions: true,
 
 	test(assert, component, target) {
-		const Foo = component.Foo;
+		const Bar = component.Bar;
 
-		component.Foo = null;
+		component.Bar = null;
 
 		assert.htmlEqual(target.innerHTML, ``);
 
-		component.Foo = Foo;
+		component.Bar = Bar;
 
 		assert.htmlEqual(target.innerHTML, `
 			<p>Foo</p>

test/runtime/samples/dynamic-component-nulled-out/main.html

@@ -1,5 +1,7 @@
 <script>
 	import Foo from './Foo.html';
+
+	export let Bar = Foo;
 </script>
 
-<svelte:component this={Foo}/>
+<svelte:component this={Bar}/>

test/runtime/samples/event-handler-shorthand-dynamic-component/_config.js

@@ -9,7 +9,7 @@ export default {
 
 		let answer;
 		component.$on('foo', event => {
-			answer = event.answer;
+			answer = event.detail.answer;
 		});
 
 		button.dispatchEvent(event);