tests for #365 and #369

Author: Rich-Harris

test/generator/index.js

@@ -96,6 +96,11 @@ describe( 'generate', () => {
 					// Put the constructor on window for testing
 					window.SvelteComponent = SvelteComponent;
 
+					const warnings = [];
+					window.console.warn = warning => {
+						warnings.push( warning );
+					};
+
 					const target = window.document.querySelector( 'main' );
 
 					const component = new SvelteComponent({
@@ -108,6 +113,13 @@ describe( 'generate', () => {
 						throw new Error( 'Expected a runtime error' );
 					}
 
+					if ( config.warnings ) {
+						assert.deepEqual( warnings, config.warnings );
+					} else if ( warnings.length ) {
+						unintendedError = true;
+						throw new Error( 'Received unexpected warnings' );
+					}
+
 					if ( config.html ) {
 						assert.htmlEqual( target.innerHTML, config.html );
 					}

test/generator/samples/dev-warning-bad-observe-arguments/_config.js

@@ -0,0 +1,7 @@
+export default {
+	dev: true,
+
+	error ( assert, err ) {
+		assert.equal( err.message, `The fisrt argument to component.observe(...) must be the name of a top-level property, i.e. 'nested' rather than 'nested.data'` );
+	}
+};

test/generator/samples/dev-warning-bad-observe-arguments/main.html

@@ -0,0 +1,9 @@
+<script>
+	export default {
+		oncreate () {
+			this.observe( 'nested.data', data => {
+				console.log( 'nope' );
+			});
+		}
+	};
+</script>

test/generator/samples/dev-warning-destroy-not-teardown/_config.js

@@ -0,0 +1,7 @@
+export default {
+	dev: true,
+
+	warnings: [
+		`Use component.on('destroy', ...) instead of component.on('teardown', ...) which has been deprecated`
+	]
+};

test/generator/samples/dev-warning-destroy-not-teardown/main.html

@@ -0,0 +1,9 @@
+<script>
+	export default {
+		oncreate () {
+			this.on( 'teardown', () => {
+				this.destroyed = true;
+			});
+		}
+	};
+</script>