Merge pull request #208 in LCL/wolframclientforpython from bugfix/377082-throw to master

* commit 'fc209a0d9c8acc8c339a77f576157970367a75e9':
  Use in place update.
  Minor fix in init script
  adding test also for 2 argument throw
  refactor
  adding a test
  performing BinarySerialize outside EvaluationData

Author: crazyeng

wolframclient/evaluation/kernel/initkernel.m

@@ -139,14 +139,16 @@ no matter the total size (tested with 80MB) *)
 socketEventHandler[data_] := Block[
 	{expr},
 	ClientLibrary`debug["Evaluating a new expression."];
-	expr = EvaluationData[BinarySerialize[BinaryDeserialize[data]]];
+	expr = EvaluationData[BinaryDeserialize[data]];
 	(* Produce inline InputForm string messages. *)
 	AssociateTo[
-		expr,
-		"MessagesText" -> Map[
-			fmtmsg,
-			expr["MessagesExpressions"]
-		]
+		expr, {
+			"Result" -> BinarySerialize[expr["Result"]],
+			"MessagesText" -> Map[
+				fmtmsg,
+				expr["MessagesExpressions"]
+			]
+		}
 	];
 	ClientLibrary`debug["Done evaluating."];
 	SocketWriteByteArrayFunc[
@@ -156,7 +158,6 @@ no matter the total size (tested with 80MB) *)
 	ClientLibrary`debug["Done responding."];
 ];
 
-
 SendAck[] := WriteString[$OutputSocket, "OK"]
 
 $MaxIdlePause=.001;

wolframclient/tests/configure.py

@@ -106,7 +106,7 @@ def _parse_config(config):
     try:
         with open(expended_path, "r") as fp:
             json_config = json.load(fp)
-            secured_authentication_key, user_configuration, server, kernel_path, api_owner= _parse_config(
+            secured_authentication_key, user_configuration, server, kernel_path, api_owner = _parse_config(
                 json_config
             )
     except IOError:

wolframclient/tests/evaluation/test_coroutine.py

@@ -27,6 +27,7 @@
 
 LOOP = get_event_loop()
 
+
 class TestCoroutineSession(BaseTestCase):
     @classmethod
     def setUpClass(cls):

wolframclient/tests/evaluation/test_kernel.py

@@ -287,6 +287,13 @@ def test_stop_start_restart_status(self):
         self.assertFalse(session.started)
         self.assertTrue(session.stopped)
 
+    def test_throw(self):
+
+        self.assertEqual(self.kernel_session.evaluate(wl.Throw(2)), wl.Hold(wl.Throw(2)))
+        self.assertEqual(
+            self.kernel_session.evaluate(wl.Throw(2, "foo")), wl.Hold(wl.Throw(2, "foo"))
+        )
+
 
 class TestSessionTimeout(TestCaseSettings):
     def test_evaluate_async_basic_inputform(self):