Merge e9c620f200 into 8b9380fb24

src/generators/dom/index.ts

@@ -182,9 +182,7 @@ export default function dom(
 	// TODO deprecate component.teardown()
 	builder.addBlock(deindent`
 		function ${name} ( options ) {
-			options = options || {};
-			${options.dev &&
-				`if ( !options.target && !options._root ) throw new Error( "'target' is a required option" );`}
+			if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 			${generator.usesRefs && `this.refs = {};`}
 			this._state = ${templateProperties.data
 				? `@assign( @template.data(), options.data )`

test/js/samples/collapses-text-around-comments/expected-bundle.js

@@ -201,7 +201,7 @@ function create_main_fragment ( state, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = assign( template.data(), options.data );
 
 	this._observers = {

test/js/samples/collapses-text-around-comments/expected.js

@@ -53,7 +53,7 @@ function create_main_fragment ( state, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = assign( template.data(), options.data );
 
 	this._observers = {

test/js/samples/computed-collapsed-if/expected-bundle.js

@@ -154,7 +154,7 @@ function create_main_fragment ( state, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 	recompute( this._state, this._state, {}, true );
 

test/js/samples/computed-collapsed-if/expected.js

@@ -30,7 +30,7 @@ function create_main_fragment ( state, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 	recompute( this._state, this._state, {}, true );
 

test/js/samples/css-media-query/expected-bundle.js

@@ -181,7 +181,7 @@ function create_main_fragment ( state, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 
 	this._observers = {

test/js/samples/css-media-query/expected.js

@@ -37,7 +37,7 @@ function create_main_fragment ( state, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 
 	this._observers = {

test/js/samples/each-block-changed-check/expected-bundle.js

@@ -301,7 +301,7 @@ function create_each_block ( state, each_block_value, comment, i, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 
 	this._observers = {

test/js/samples/each-block-changed-check/expected.js

@@ -144,7 +144,7 @@ function create_each_block ( state, each_block_value, comment, i, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 
 	this._observers = {

test/js/samples/event-handlers-custom/expected-bundle.js

@@ -192,7 +192,7 @@ function create_main_fragment ( state, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 
 	this._observers = {

test/js/samples/event-handlers-custom/expected.js

@@ -48,7 +48,7 @@ function create_main_fragment ( state, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 
 	this._observers = {

test/js/samples/if-block-no-update/expected-bundle.js

@@ -236,7 +236,7 @@ function select_block_type ( state ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 
 	this._observers = {

test/js/samples/if-block-no-update/expected.js

@@ -88,7 +88,7 @@ function select_block_type ( state ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 
 	this._observers = {

test/js/samples/if-block-simple/expected-bundle.js

@@ -212,7 +212,7 @@ function create_if_block ( state, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 
 	this._observers = {

test/js/samples/if-block-simple/expected.js

@@ -64,7 +64,7 @@ function create_if_block ( state, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 
 	this._observers = {

test/js/samples/non-imported-component/expected-bundle.js

@@ -184,7 +184,7 @@ function create_main_fragment ( state, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 
 	this._observers = {

test/js/samples/non-imported-component/expected.js

@@ -48,7 +48,7 @@ function create_main_fragment ( state, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 
 	this._observers = {

test/js/samples/onrender-onteardown-rewritten/expected-bundle.js

@@ -146,7 +146,7 @@ function create_main_fragment ( state, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 
 	this._observers = {

test/js/samples/onrender-onteardown-rewritten/expected.js

@@ -22,7 +22,7 @@ function create_main_fragment ( state, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 
 	this._observers = {

test/js/samples/setup-method/expected-bundle.js

@@ -157,7 +157,7 @@ function create_main_fragment ( state, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 
 	this._observers = {

test/js/samples/setup-method/expected.js

@@ -33,7 +33,7 @@ function create_main_fragment ( state, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 
 	this._observers = {

test/js/samples/use-elements-as-anchors/expected-bundle.js

@@ -396,7 +396,7 @@ function create_if_block_4 ( state, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 
 	this._observers = {

test/js/samples/use-elements-as-anchors/expected.js

@@ -248,7 +248,7 @@ function create_if_block_4 ( state, component ) {
 }
 
 function SvelteComponent ( options ) {
-	options = options || {};
+	if ( !options || ( !options.target && !options._root ) ) throw new Error( "'target' is a required option" );
 	this._state = options.data || {};
 
 	this._observers = {

test/runtime/index.js

@@ -215,8 +215,7 @@ describe("runtime", () => {
 	it("fails if options.target is missing in dev mode", () => {
 		const { code } = svelte.compile(`<div></div>`, {
 			format: "iife",
-			name: "SvelteComponent",
-			dev: true
+			name: "SvelteComponent"
 		});
 
 		const SvelteComponent = eval(

test/runtime/samples/pass-no-options/_config.js

@@ -1,8 +0,0 @@
-export default {
-	html: '<h1>Just some static HTML</h1>',
-
-	test ( assert, component, target, window ) {
-		const newComp = new window.SvelteComponent();
-		assert.equal(newComp instanceof window.SvelteComponent, true);
-	}
-};

test/runtime/samples/pass-no-options/main.html

@@ -1 +0,0 @@
-<h1>Just some static HTML</h1>