Fix wrong value for useContext

Author: marvinhagemeister

src/index.js

@@ -60,28 +60,29 @@ function renderToString(vnode, context, opts, inner, isSvgMode) {
 			nodeName = getComponentName(nodeName);
 		}
 		else {
-			vnode.__c = { __v: vnode };
-			if (options.render) options.render(vnode);
 			let rendered;
+			
+			let c = vnode.__c = { __v: vnode, context, props: vnode.props };
+			if (options.render) options.render(vnode);
 
 			if (!nodeName.prototype || typeof nodeName.prototype.render!=='function') {
 				// stateless functional components
-				rendered = nodeName.apply(vnode.__c, props, context);
+				rendered = nodeName.call(vnode.__c, props, context);
 			}
 			else {
 				// class-based components
-				let c = new nodeName(props, context);
+				c = new nodeName(props, context);
 				// turn off stateful re-rendering:
 				c._dirty = c.__d = true;
 				c.props = props;
 				c.context = context;
 				if (nodeName.getDerivedStateFromProps) c.state = assign(assign({}, c.state), nodeName.getDerivedStateFromProps(c.props, c.state));
 				else if (c.componentWillMount) c.componentWillMount();
 				rendered = c.render(c.props, c.state, c.context);
-
-				if (c.getChildContext) {
-					context = assign(assign({}, context), c.getChildContext());
-				}
+			}
+			
+			if (c.getChildContext) {
+				context = assign(assign({}, context), c.getChildContext());
 			}
 
 			return renderToString(rendered, context, opts, opts.shallowHighOrder!==false);

test/render.js

@@ -1,6 +1,6 @@
 import { render, shallowRender } from '../src';
 import { h, Component, createContext } from 'preact';
-import { useState, useContext } from 'preact/hooks';
+import { useState, useContext, useEffect } from 'preact/hooks';
 import chai, { expect } from 'chai';
 import { spy, stub, match } from 'sinon';
 import sinonChai from 'sinon-chai';
@@ -621,20 +621,52 @@ describe('render', () => {
 			}).to.not.throw();
 		});
 
-		it('should work with useContext', () => {
+		it('should work with useContext and default value', () => {
+			let Ctx = createContext('foo');
+			function Foo() {
+				let v = useContext(Ctx);
+				return <div>{v}</div>;
+			}
+
+			let res = render(<Foo />);
+
+			expect(res).to.equal('<div>foo</div>');
+		});
+
+		it('should work with useContext + custom value', () => {
 			let Ctx = createContext('foo');
 			function Foo() {
 				let v = useContext(Ctx);
 				return <div>{v}</div>;
 			}
 
 			let res = render(
-				<Ctx.Provider value="foo">
+				<Ctx.Provider value="bar">
 					<Foo />
 				</Ctx.Provider>
 			);
 
-			expect(res).to.equal('<div>foo</div>');
+			expect(res).to.equal('<div>bar</div>');
+		});
+
+		it('should work with useState', () => {
+			function Foo() {
+				let [v] = useState(0);
+				return <div>{v}</div>;
+			}
+
+			expect(render(<Foo />)).to.equal('<div>0</div>');
+		});
+
+		it('should not trigger useEffect callbacks', () => {
+			let called = false;
+			function Foo() {
+				useEffect(() => called = true);
+				return <div />;
+			}
+
+			render(<Foo />);
+			expect(called).to.equal(false);
 		});
 	});
 });