Failing test for memory leak issue #954

renderComponentToString registers events. If the calling code
does not unmount the component or unregister the listeners
this will leak memory. Passing this test should fix this.

src/environment/__tests__/ReactServerRendering-test.js

@@ -27,7 +27,8 @@ require('mock-modules')
   .dontMock('ReactMount')
   .dontMock('ReactServerRendering')
   .dontMock('ReactTestUtils')
-  .dontMock('ReactMarkupChecksum');
+  .dontMock('ReactMarkupChecksum')
+  .dontMock('EventPluginHub');
 
 var mocks = require('mocks');
 
@@ -37,6 +38,7 @@ var ReactTestUtils;
 var ReactServerRendering;
 var ReactMarkupChecksum;
 var ExecutionEnvironment;
+var EventPluginHub;
 
 var ID_ATTRIBUTE_NAME;
 
@@ -50,6 +52,7 @@ describe('ReactServerRendering', function() {
     ExecutionEnvironment.canUseDOM = false;
     ReactServerRendering = require('ReactServerRendering');
     ReactMarkupChecksum = require('ReactMarkupChecksum');
+    EventPluginHub = require('EventPluginHub');
 
     var DOMProperty = require('DOMProperty');
     ID_ATTRIBUTE_NAME = DOMProperty.ID_ATTRIBUTE_NAME;
@@ -234,6 +237,20 @@ describe('ReactServerRendering', function() {
     expect(numClicks).toEqual(1);
   });
 
+  it('should not register listeners', function() {
+    var Link = React.createClass({
+      handleClick: function() {},
+      render: function() {
+        return <a onClick={this.handleClick}>link</a>;
+      }
+    });
+    var link = <Link />;
+    ReactServerRendering.renderComponentToString(link, function(str) {
+      var listener = EventPluginHub.getListener(link._rootNodeID, 'onClick');
+      expect(listener).not.toEqual(link.handleClick);
+    });
+  });
+
   it('should throw with silly args', function() {
     expect(
       ReactServerRendering.renderComponentToString.bind(