Fixed the destructor test.

I hadn't made the js_ctor call account for the change to py2js_object changes
that were required to make the function tests work properly.

spidermonkey/pyobject.c

@@ -195,8 +195,7 @@ js_finalize(JSContext* jscx, JSObject* jsobj)
 {
     Context* pycx = (Context*) JS_GetContextPrivate(jscx);
     PyObject* pyobj = NULL;
-
-
+
     if(pycx == NULL)
     {
         // Not much else we can do but yell.
@@ -337,7 +336,6 @@ js_ctor(JSContext* jscx, JSObject* jsobj, uintN argc, jsval* argv, jsval* rval)
         JS_ReportError(jscx, "Failed to construct object.");
         goto error;
     }
-    Py_INCREF(ret);
 
     *rval = py2js(pycx, ret);
     if(*rval == JSVAL_VOID)
@@ -470,6 +468,7 @@ py2js_object(Context* cx, PyObject* pyobj)
     goto success;
 
 error:
+    Py_XDECREF(attached);
 success:
     return ret;
 }

tests/test-utf16.py

@@ -11,3 +11,7 @@ def test_empty_string_script(cx):
 @t.cx()
 def test_unicode_string(cx):
     t.eq(cx.execute(u"5"), 5)
+
+@t.cx()
+def test_non_unicode_string(cx):
+    t.eq(cx.execute("5"), 5)