#6299 - Throw a compile error when passing an empty directive name

[qurafi]

Fixes #6299.

• Removed Expect a compile error when class binding name is empty #5858 code for checking for empty class directive names.
• Added more generalized compile error when passing empty directive names.
• Added test for it.
• No failed tests.

Example of the error message:

<!-- ParseError: Action name cannot be empty -->
<element use: />

<!-- ParseError: Class name cannot be empty -->
<element class: />
<element class:="className" />
<element class:={class} />

<!-- ParseError: Transition name cannot be empty -->
<element in: />

[j3rem1e]

Does this break autocompletion in vscode when writing "class:" ?

[dummdidumm]

Good point. Probably yes. In general it would be good to be able to pass a flag to the parser to be more forgiving, which the language-tools could use then.

[qurafi]

Does this break autocompletion in vscode when writing "class:" ?

I tried it before and after the change. I didn't notice any issues or changes with autocomplete.

[dummdidumm]

The VS Code extension uses a built-in version of Svelte do you can't test it that way. You would need to build from source.

[qurafi]

I reload my vscode on every change and it seems it uses my local svelte version because of npm link