Migrate some test to use React::Test::Utils.render_into_document

Author: zetachang

spec/react/children_spec.rb

@@ -15,7 +15,7 @@ def render
   let(:children) { described_class.new(`#{element.to_n}.props.children`) }
 
   before(:each) do
-    renderElementToDocument(element)
+    React::Test::Utils.render_into_document(element)
   end
 
   it 'is enumerable' do

spec/react/component_spec.rb

@@ -34,6 +34,8 @@ def render
   end
 
   describe 'Life Cycle' do
+    let(:element_to_render) { React.create_element(Foo) }
+
     before(:each) do
       stub_const 'Foo', Class.new
       Foo.class_eval do
@@ -54,7 +56,7 @@ def bar2; end
       expect_any_instance_of(Foo).to receive(:bar)
       expect_any_instance_of(Foo).to receive(:bar2)
 
-      renderToDocument(Foo)
+      React::Test::Utils.render_into_document(element_to_render)
     end
 
     it 'invokes `after_mount` registered methods when `componentDidMount()`' do
@@ -67,7 +69,7 @@ def bar4; end
       expect_any_instance_of(Foo).to receive(:bar3)
       expect_any_instance_of(Foo).to receive(:bar4)
 
-      renderToDocument(Foo)
+      React::Test::Utils.render_into_document(element_to_render)
     end
 
     it 'allows multiple class declared life cycle hooker' do
@@ -88,7 +90,7 @@ def render
 
       expect_any_instance_of(Foo).to receive(:bar)
 
-      renderToDocument(Foo)
+      React::Test::Utils.render_into_document(element_to_render)
     end
 
     it 'allows block for life cycle callback' do
@@ -169,6 +171,8 @@ def render
   end
 
   describe 'State setter & getter' do
+    let(:element_to_render) { React.create_element(Foo) }
+
     before(:each) do
       stub_const 'Foo', Class.new
       Foo.class_eval do
@@ -188,17 +192,17 @@ def set_up
         end
       end
 
-      element = renderToDocument(Foo)
-      expect(element.state.foo).to be('bar')
+      instance = React::Test::Utils.render_into_document(element_to_render)
+      expect(instance.state.foo).to be('bar')
     end
 
     it 'defines init state by passing a block to `define_state`' do
       Foo.class_eval do
         define_state(:foo) { 10 }
       end
 
-      element = renderToDocument(Foo)
-      expect(element.state.foo).to be(10)
+      instance = React::Test::Utils.render_into_document(element_to_render)
+      expect(instance.state.foo).to be(10)
     end
 
     it 'defines getter using `define_state`' do
@@ -210,8 +214,8 @@ def bump
         end
       end
 
-      element = renderToDocument(Foo)
-      expect(element.state.foo).to be(30)
+      instance = React::Test::Utils.render_into_document(element_to_render)
+      expect(instance.state.foo).to be(30)
     end
 
     it 'defines multiple state accessors by passing array to `define_state`' do
@@ -224,9 +228,9 @@ def set_up
         end
       end
 
-      element = renderToDocument(Foo)
-      expect(element.state.foo).to be(10)
-      expect(element.state.foo2).to be(20)
+      instance = React::Test::Utils.render_into_document(element_to_render)
+      expect(instance.state.foo).to be(10)
+      expect(instance.state.foo2).to be(20)
     end
 
     it 'invokes `define_state` multiple times to define states' do
@@ -235,29 +239,30 @@ def set_up
         define_state(:foo2) { 40 }
       end
 
-      element = renderToDocument(Foo)
-      expect(element.state.foo).to be(30)
-      expect(element.state.foo2).to be(40)
+      instance = React::Test::Utils.render_into_document(element_to_render)
+      expect(instance.state.foo).to be(30)
+      expect(instance.state.foo2).to be(40)
     end
 
     it 'can initialize multiple state variables with a block' do
       Foo.class_eval do
         define_state(:foo, :foo2) { 30 }
       end
-      element = renderToDocument(Foo)
-      expect(element.state.foo).to be(30)
-      expect(element.state.foo2).to be(30)
+
+      instance = React::Test::Utils.render_into_document(element_to_render)
+      expect(instance.state.foo).to be(30)
+      expect(instance.state.foo2).to be(30)
     end
 
     it 'can mix multiple state variables with initializers and a block' do
       Foo.class_eval do
         define_state(:x, :y, foo: 1, bar: 2) {3}
       end
-      element = renderToDocument(Foo)
-      expect(element.state.x).to be(3)
-      expect(element.state.y).to be(3)
-      expect(element.state.foo).to be(1)
-      expect(element.state.bar).to be(2)
+      instance = React::Test::Utils.render_into_document(element_to_render)
+      expect(instance.state.x).to be(3)
+      expect(instance.state.y).to be(3)
+      expect(instance.state.foo).to be(1)
+      expect(instance.state.bar).to be(2)
     end
 
     it 'gets state in render method' do
@@ -268,8 +273,8 @@ def render
         end
       end
 
-      element = renderToDocument(Foo)
-      expect(`#{element.dom_node}.textContent`).to eq('10')
+      instance = React::Test::Utils.render_into_document(element_to_render)
+      expect(`#{instance.dom_node}.textContent`).to eq('10')
     end
 
     it 'supports original `setState` as `set_state` method' do

spec/react/react_spec.rb

@@ -43,8 +43,9 @@
         element = React.create_element('div') do
           [React.create_element('span'), React.create_element('span'), React.create_element('span')]
         end
-        instance = renderElementToDocument(element)
-        expect(`#{instance.dom_node}.children.length`).to eq(3)
+        dom_node = React::Test::Utils.render_into_document(element)
+
+        expect(`#{dom_node}.children.length`).to eq(3)
       end
     end
 
@@ -117,8 +118,8 @@ def render
           end
         end
 
-        renderToDocument(Foo)
-        renderToDocument(Foo)
+        React::Test::Utils.render_into_document(React.create_element(Foo))
+        React::Test::Utils.render_into_document(React.create_element(Foo))
 
         expect(`count`).to eq(2)
       end