adding threading test

jbarnette · Apr 24, 2008 · c8e3eac · c8e3eac
1 parent 5732d81
commit c8e3eac
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 2 deletions.
diff --git a/ext/spidermonkey/conversions.c b/ext/spidermonkey/conversions.c
@@ -103,8 +103,8 @@ JSBool convert_to_js(OurContext* context, VALUE ruby, jsval* retval)
       if (ruby_value_is_proxy(ruby))
         return unwrap_ruby_land_proxy(context, ruby, retval);
 
-      if (rb_cProc == rb_class_of(ruby) || rb_cMethod == rb_class_of(ruby))
-        return make_js_land_proxy(context, ruby, retval);
+      // If we can't identify the object, just wrap it
+      return make_js_land_proxy(context, ruby, retval);
 
     default:
       Johnson_Error_raise("unknown ruby type in switch");

diff --git a/test/johnson/conversions/thread_test.rb b/test/johnson/conversions/thread_test.rb
@@ -0,0 +1,17 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "/../../helper"))
+
+module Johnson
+  module Conversions
+    class ThreadTest < Johnson::TestCase
+      def setup
+        @context = Johnson::Context.new
+      end
+
+      def test_manipulate_thread
+        thread = Thread.new { }
+        @context['thread'] = thread
+        assert_js_equal(false, "thread.send('alive?')")
+      end
+    end
+  end
+end