reactrb 0.8.8 → 0.9.0

data/spec/react/test/rspec_spec.rb

@@ -0,0 +1,62 @@
+require 'spec_helper'
+
+if opal?
+  RSpec.describe 'react/test/rspec', type: :component do
+    before do
+      stub_const 'Greeter', Class.new
+      Greeter.class_eval do
+        include React::Component
+        params do
+          optional :message
+          optional :from
+        end
+
+        def render
+          span { "Hello #{params.message}" }
+        end
+      end
+    end
+
+    it 'should include react/test in rspec' do
+      comp = mount(Greeter)
+      expect(component.instance).to eq(comp)
+    end
+
+    it 'includes rspec matchers' do
+      expect(Greeter).to render(
+        '<span>Hello world</span>'
+      ).with_params(message: 'world')
+    end
+
+    describe 'resetting the session' do
+      it 'creates an instance of the mounted component in one example' do
+        mount(Greeter)
+      end
+
+      it '...then is not availalbe in the next' do
+        expect { component.instance }.to raise_error
+      end
+    end
+  end
+
+  RSpec.describe 'react/test/rspec', type: :other do
+    before do
+      stub_const 'Greeter', Class.new
+      Greeter.class_eval do
+        include React::Component
+        params do
+          optional :message
+          optional :from
+        end
+
+        def render
+          span { "Hello #{params.message}" }
+        end
+      end
+    end
+
+    it 'should not include react/test in rspec' do
+      expect { mount(Greeter) }.to raise_error
+    end
+  end
+end

data/spec/react/test/session_spec.rb

@@ -0,0 +1,100 @@
+require 'spec_helper'
+
+if opal?
+  RSpec.describe React::Test::Session do
+    subject { described_class.new }
+    before do
+      stub_const 'Greeter', Class.new
+      Greeter.class_eval do
+        include React::Component
+
+        params do
+          optional :message
+          optional :from
+        end
+
+        def render
+          span { "Hello #{params.message}" }
+        end
+      end
+    end
+
+    describe '#mount' do
+      it 'returns an instance of the mounted component' do
+        expect(subject.mount(Greeter)).to be_a(Greeter)
+      end
+
+      it 'actualy mounts the component' do
+        expect(subject.mount(Greeter)).to be_mounted
+      end
+
+      it 'optionaly passes params to the component' do
+        instance = subject.mount(Greeter, message: 'world')
+        expect(instance.params.message).to eq('world')
+      end
+    end
+
+    describe '#instance' do
+      it 'returns the instance of the mounted component' do
+        instance = subject.mount(Greeter)
+        expect(subject.instance).to eq(instance)
+      end
+    end
+
+    describe '#element' do
+      it 'returns the React::Element for the mounted component' do
+        subject.mount(Greeter)
+        expect(subject.element).to be_a(React::Element)
+      end
+    end
+
+    describe '#native' do
+      it 'returns the React native instance of the component' do
+        instance = subject.mount(Greeter)
+        native = instance.instance_variable_get('@native')
+        expect(subject.native).to eq(native)
+      end
+    end
+
+    describe '#html' do
+      it 'returns the component rendered to static html' do
+        subject.mount(Greeter, message: 'world')
+        expect(subject.html).to eq('<span>Hello world</span>')
+      end
+
+      it 'returns the updated static html' do
+        subject.mount(Greeter)
+        subject.update_params(message: 'moon')
+        expect(subject.html).to eq('<span>Hello moon</span>')
+      end
+    end
+
+    describe '#update_params' do
+      it 'sends new params to the component' do
+        instance = subject.mount(Greeter, message: 'world')
+        subject.update_params(message: 'moon')
+        expect(instance.params.message).to eq('moon')
+      end
+
+      it 'leaves unspecified params in tact' do
+        instance = subject.mount(Greeter, message: 'world', from: 'outerspace')
+        subject.update_params(message: 'moon')
+        expect(instance.params.from).to eq('outerspace')
+      end
+
+      it 'causes the component to render' do
+        instance = subject.mount(Greeter, message: 'world')
+        expect(instance).to receive(:render)
+        subject.update_params(message: 'moon')
+      end
+    end
+
+    describe '#force_update' do
+      it 'causes the component to render' do
+        instance = subject.mount(Greeter)
+        expect(instance).to receive(:render)
+        subject.force_update!
+      end
+    end
+  end
+end

data/spec/react/test/utils_spec.rb

@@ -0,0 +1,45 @@
+require 'spec_helper'
+
+if opal?
+  module React
+    module Test
+      class Utils
+        def self.simulate(event, element)
+          Simulate.new.click(element)
+        end
+
+        class Simulate
+          include Native
+          def initialize
+            super(`React.addons.TestUtils.Simulate`)
+          end
+
+          def click(component_instance)
+            `#{@native}['click']`.call(component_instance.dom_node, {})
+          end
+        end
+      end
+    end
+  end
+  RSpec.describe React::Test::Utils do
+    it 'simulates' do
+      stub_const 'Foo', Class.new
+      Foo.class_eval do
+        include React::Component
+
+        def hello
+          @hello
+        end
+
+        def render
+          @hello = 'hello'
+          div { 'Click Me' }.on(:click) { |e| click(e) }
+        end
+      end
+
+      instance = renderToDocument(Foo)
+      expect_any_instance_of(Foo).to receive(:click)
+      described_class.simulate(:click, instance)
+    end
+  end
+end

data/spec/react/top_level_component_spec.rb

@@ -1,9 +1,7 @@
 require 'spec_helper'
 
 if opal?
-
 module Components
-
   module Controller
     class Component1
       include React::Component
@@ -40,10 +38,9 @@ class Component1
   end
 end
 
-
 def render_top_level(controller, component_name)
-  React.render_to_static_markup(React.create_element(
-    React::TopLevelRailsComponent, {controller: controller, component_name: component_name, render_params: {}}))
+  render_to_html(React::TopLevelRailsComponent, controller: controller,
+                 component_name: component_name, render_params: {})
 end
 
 describe React::TopLevelRailsComponent do
@@ -64,5 +61,36 @@ describe React::TopLevelRailsComponent do
     expect(render_top_level("Controller", "::Components::Component1")).to eq('<span>Components::Component1</span>')
   end
 
+  describe '.html_tag?' do
+    it 'is truthy for valid html tags' do
+      expect(React.html_tag?('a')).to be_truthy
+      expect(React.html_tag?('div')).to be_truthy
+    end
+
+    it 'is truthy for valid svg tags' do
+      expect(React.html_tag?('svg')).to be_truthy
+      expect(React.html_tag?('circle')).to be_truthy
+    end
+
+    it 'is falsey for invalid tags' do
+      expect(React.html_tag?('tagizzle')).to be_falsey
+    end
+  end
+
+  describe '.html_attr?' do
+    it 'is truthy for valid html attributes' do
+      expect(React.html_attr?('id')).to be_truthy
+      expect(React.html_attr?('data')).to be_truthy
+    end
+
+    it 'is truthy for valid svg attributes' do
+      expect(React.html_attr?('cx')).to be_truthy
+      expect(React.html_attr?('strokeWidth')).to be_truthy
+    end
+
+    it 'is falsey for invalid attributes' do
+      expect(React.html_tag?('attrizzle')).to be_falsey
+    end
+  end
 end
 end

data/spec/react/tutorial/tutorial_spec.rb

@@ -8,9 +8,9 @@ class HelloMessage
   end
 end
 
-describe 'An Example from the react.rb doc' do
+describe 'An Example from the react.rb doc', type: :component do
   it 'produces the correct result' do
-    expect(React.render_to_static_markup(React.create_element(HelloMessage))).to eq('<div>Hello World!</div>')
+    expect(HelloMessage).to render('<div>Hello World!</div>')
   end
 end
 
@@ -18,13 +18,13 @@ class HelloMessage2
   include React::Component
   define_state(:user_name) { '@catmando' }
   def render
-    div { "Hello #{user_name}" }
+    div { "Hello #{state.user_name}" }
   end
 end
 
-describe 'Adding state to a component (second tutorial example)' do
+describe 'Adding state to a component (second tutorial example)', type: :component do
   it "produces the correct result" do
-    expect(React.render_to_static_markup(React.create_element(HelloMessage2))).to eq('<div>Hello @catmando</div>')
+    expect(HelloMessage2).to render('<div>Hello @catmando</div>')
   end
 
   it 'renders to the document' do

data/spec/react/validator_spec.rb

@@ -2,13 +2,10 @@ require "spec_helper"
 
 if opal?
 describe React::Validator do
-  before do
-    stub_const 'Foo', Class.new(React::Component::Base)
-  end
   describe '#validate' do
     describe "Presence validation" do
       it "should check if required props provided" do
-        validator = React::Validator.new(Foo).build do
+        validator = React::Validator.new.build do
           requires :foo
           requires :bar
         end
@@ -18,7 +15,7 @@ describe React::Validator do
       end
 
       it "should check if passed non specified prop" do
-        validator = React::Validator.new(Foo).build do
+        validator = React::Validator.new.build do
           optional :foo
         end
 
@@ -29,7 +26,7 @@ describe React::Validator do
 
     describe "Type validation" do
       it "should check if passed value with wrong type" do
-        validator = React::Validator.new(Foo).build do
+        validator = React::Validator.new.build do
           requires :foo, type: String
         end
 
@@ -39,7 +36,7 @@ describe React::Validator do
 
       it "should check if passed value with wrong custom type" do
         stub_const 'Bar', Class.new
-        validator = React::Validator.new(Foo).build do
+        validator = React::Validator.new.build do
           requires :foo, type: Bar
         end
 
@@ -48,7 +45,7 @@ describe React::Validator do
       end
 
       it 'coerces native JS prop types to opal objects' do
-        validator = React::Validator.new(Foo).build do
+        validator = React::Validator.new.build do
           requires :foo, type: `{ x: 1 }`
         end
 
@@ -57,7 +54,7 @@ describe React::Validator do
       end
 
       it 'coerces native JS values to opal objects' do
-        validator = React::Validator.new(Foo).build do
+        validator = React::Validator.new.build do
           requires :foo, type: Array[Fixnum]
         end
         message = "Provided prop `foo`[0] could not be converted to #{Fixnum.name}"
@@ -65,7 +62,7 @@ describe React::Validator do
       end
 
       it "should support Array[Class] validation" do
-        validator = React::Validator.new(Foo).build do
+        validator = React::Validator.new.build do
           requires :foo, type: Array[Hash]
         end
 
@@ -82,7 +79,7 @@ describe React::Validator do
 
     describe "Limited values" do
       it "should check if passed value is not one of the specified values" do
-        validator = React::Validator.new(Foo).build do
+        validator = React::Validator.new.build do
           requires :foo, values: [4,5,6]
         end
 
@@ -95,7 +92,7 @@ describe React::Validator do
   describe '#undefined_props' do
     let(:props) { { foo: 'foo', bar: 'bar', biz: 'biz', baz: 'baz' } }
     let(:validator) do
-      React::Validator.new(Foo).build do
+      React::Validator.new.build do
         requires :foo
         optional :bar
       end
@@ -114,7 +111,7 @@ describe React::Validator do
 
   describe "default_props" do
     it "should return specified default values" do
-      validator = React::Validator.new(Foo).build do
+      validator = React::Validator.new.build do
         requires :foo, default: 10
         requires :bar
         optional :lorem, default: 20

data/spec/reactive-ruby/component_loader_spec.rb

@@ -6,6 +6,9 @@ RSpec.describe ReactiveRuby::ComponentLoader do
     var global = global || this;
     var self = self || this;
     var window = window || this;
+    var console = {
+      warn: function(s) {  }
+    };
   JS
 
   let(:js) { ::Rails.application.assets['components'].to_s }

data/spec/reactive-ruby/rails/asset_pipeline_spec.rb

@@ -1,9 +1,10 @@
 require 'spec_helper'
 
 if ruby?
-  RSpec.describe 'test_app generator' do
-    it "does not interfer with asset precompilation" do
-      expect(system("cd spec/test_app; bundle exec rake assets:precompile")).to be_truthy
-    end
+RSpec.describe 'test_app generator' do
+  it "does not interfer with asset precompilation" do
+    cmd = "cd spec/test_app; BUNDLE_GEMFILE=#{ENV['REAL_BUNDLE_GEMFILE']} bundle exec rake assets:precompile"
+    expect(system(cmd)).to be_truthy
   end
 end
+end

data/spec/spec_helper.rb

@@ -15,8 +15,10 @@ end
 
 if RUBY_ENGINE == 'opal'
   require File.expand_path('../vendor/jquery-2.2.4.min', __FILE__)
-  require 'react-v14'
+  require 'react/react-source'
   require 'reactrb'
+  require 'react/test/rspec'
+
   require File.expand_path('../support/react/spec_helpers', __FILE__)
 
   module Opal
@@ -68,10 +70,11 @@ if RUBY_ENGINE == 'opal'
 
 
   RSpec.configure do |config|
-    config.run_all_when_everything_filtered = true
-    #config.filter_run_including only: true
     config.include React::SpecHelpers
     config.filter_run_excluding :ruby
+    if `(React.version.search(/^0\.13/) === -1)`
+      config.filter_run_excluding :v13_only
+    end
   end
 end
 

data/spec/support/react/spec_helpers.rb

@@ -1,16 +1,22 @@
+if opal?
 module React
   module SpecHelpers
     `var ReactTestUtils = React.addons.TestUtils`
 
+    def render_to_html(type, options = {})
+      element = React.create_element(type, options)
+      React.render_to_static_markup(element)
+    end
+
     def renderToDocument(type, options = {})
       element = React.create_element(type, options)
-      return renderElementToDocument(element)
+      renderElementToDocument(element)
     end
 
     def renderElementToDocument(element)
       instance = Native(`ReactTestUtils.renderIntoDocument(#{element.to_n})`)
       instance.class.include(React::Component::API)
-      return instance
+      instance
     end
 
     def simulateEvent(event, element, params = {})
@@ -55,3 +61,4 @@ module React
     end
   end
 end
+end