Merge branch 'master' into refactor

src/generators/dom/index.js

@@ -193,7 +193,7 @@ export default function dom ( parsed, source, options ) {
 		constructorBlock.addBlock( generator.builders.metaBindings );
 	}
 
-	if ( templateProperties.computed ) {
+	if ( computations.length ) {
 		constructorBlock.addLine(
 			`${generator.alias( 'recompute' )}( this._state, this._state, {}, true );`
 		);

src/shared/methods.js

@@ -12,7 +12,7 @@ export function fire ( eventName, data ) {
 }
 
 export function observe ( key, callback, options ) {
-	var group = ( options && options.defer ) ? this._observers.pre : this._observers.post;
+	var group = ( options && options.defer ) ? this._observers.post : this._observers.pre;
 
 	( group[ key ] || ( group[ key ] = [] ) ).push( callback );
 

test/generator/samples/computed-empty/_config.js

@@ -0,0 +1,8 @@
+export default {
+	html: '<div>empty</div>',
+	test ( assert, component, target ) {
+		assert.equal( component.get( 'created' ), true );
+		assert.equal( target.innerHTML, '<div>empty</div>' );
+		component.destroy();
+	}
+};

test/generator/samples/computed-empty/main.html

@@ -0,0 +1,14 @@
+<div>empty</div>
+<script>
+	export default {
+		data () {
+			return {};
+		},
+
+		computed: {},
+
+		oncreate () {
+			this.set({ created: true });
+		}
+	};
+</script>

test/generator/samples/observe-deferred/_config.js

@@ -0,0 +1,22 @@
+export default {
+	'skip-ssr': true,
+
+	data: {
+		value: 'hello!'
+	},
+
+	html: `
+		<p>hello!</p>
+		<p>hello!</p>
+	`,
+
+	test ( assert, component, target ) {
+		component.set({ value: 'goodbye!' });
+		assert.htmlEqual( target.innerHTML, `
+			<p>goodbye!</p>
+			<p>goodbye!</p>
+		` );
+
+		component.destroy();
+	}
+};

test/generator/samples/observe-deferred/main.html

@@ -0,0 +1,14 @@
+<p ref:a>{{value}}</p>
+<p ref:b></p>
+
+<script>
+	export default {
+		oncreate () {
+			this.observe( 'value', () => {
+				this.refs.b.textContent = this.refs.a.textContent;
+			}, {
+				defer: true
+			});
+		}
+	};
+</script>

test/runtime/index.js

@@ -138,6 +138,8 @@ describe( 'runtime', () => {
 						assert.htmlEqual( target.innerHTML, config.html );
 					}
 
+					Object.assign = Object_assign;
+
 					if ( config.test ) {
 						config.test( assert, component, target, window );
 					} else {