Fix #175

Author: zetachang

lib/react/state.rb

@@ -15,15 +15,16 @@ def []=(state, new_value)
 
     def method_missing(method, *args)
       if match = method.match(/^(.+)\!$/)
+        key_name = $1
         if args.count > 0
           current_value = State.get_state(@from, match[1])
-          State.set_state(@from, $1, args[0])
+          State.set_state(@from, key_name, args[0])
           current_value
         else
           current_state = State.get_state(@from, match[1])
-          State.set_state(@from, $1, current_state)
+          State.set_state(@from, key_name, current_state)
           Observable.new(current_state) do |update|
-            State.set_state(@from, $1, update)
+            State.set_state(@from, key_name, update)
           end
         end
       else

spec/react/observable_spec.rb

@@ -1,7 +1,42 @@
 require 'spec_helper'
 
 if opal?
+
 describe 'React::Observable' do
-  # tbd
+  it "allows to set value on Observable" do
+    stub_const 'Zoo', Class.new {
+      include React::Component
+      param :foo, type: React::Observable
+      before_mount do
+        params.foo! 4
+      end
+
+      def render
+        nil
+      end
+    }
+
+    stub_const 'Foo', Class.new
+    Foo.class_eval do
+      include React::Component
+
+      def render
+        div do
+          Zoo(foo: state.foo! )
+          span { state.foo.to_s }
+        end
+      end
+    end
+
+    instance = React::Test::Utils.render_into_document(React.create_element(Foo))
+    html = `#{instance.dom_node}.innerHTML`
+    # data-reactid appear in earlier versions of reactjs
+    %x{
+        var REGEX_REMOVE_IDS = /\s?data-reactid="[^"]+"/g;
+        html = html.replace(REGEX_REMOVE_IDS, '');
+    }
+    expect(html).to eq('<span></span><span>4</span>')
+  end
 end
+
 end