Merge pull request #4793 from frontman-git/fix/4769-forward-render-prop

fix(compat): add .render property referencing original render function

Author: marvinhagemeister

compat/src/forwardRef.js

@@ -22,7 +22,7 @@ export function forwardRef(fn) {
 	// It expects an object here with a `render` property,
 	// and prototype.render will fail. Without this
 	// mobx-react throws.
-	Forwarded.render = Forwarded;
+	Forwarded.render = fn;
 
 	Forwarded.prototype.isReactComponent = true;
 	Forwarded.displayName = 'ForwardRef(' + (fn.displayName || fn.name) + ')';

compat/test/browser/forwardRef.test.js

@@ -498,4 +498,15 @@ describe('forwardRef', () => {
 
 		expect(actual).to.equal(null);
 	});
+
+	// Issue #4769
+	it('should attach .render pointing to the original render function', () => {
+		function Foo(props, ref) {
+			return <div ref={ref} />;
+		}
+
+		const Forwarded = forwardRef(Foo);
+
+		expect(Forwarded.render).to.equal(Foo);
+  });
 });