added test for numeric values

Author: rssdev10

src/main/java/org/julia/scripting/JuliaEngine.java

@@ -36,9 +36,9 @@ public Object eval(String script, ScriptContext context) throws ScriptException
                 String formatStr;
                 // TODO: add direct write of values without text transformations
                 if (entry.getValue() instanceof Number) {
-                    formatStr = "%s = %s";
+                    formatStr = "%s = %s;\n";
                 } else {
-                    formatStr = "%s = \"%s\"";
+                    formatStr = "%s = \"%s\";\n";
                 }
                 builder.append(String.format(formatStr, entry.getKey(), entry.getValue()));
             });

src/test/java/Julia4JScriptingTest.java

@@ -12,8 +12,8 @@
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
 /**
- *  Test of jsr223 Julia binding
- *
+ * Test of jsr223 Julia binding
+ * <p>
  * Created by rss on 25/08/2018
  */
 @SuppressWarnings("WeakerAccess")
@@ -56,23 +56,31 @@ public void testBasic() throws ScriptException {
 
     @Test
     public void testReturnvalue() throws ScriptException {
-            ScriptEngineManager manager = new ScriptEngineManager();
-            ScriptEngine engine = manager.getEngineByName("julia");
+        ScriptEngineManager manager = new ScriptEngineManager();
+        ScriptEngine engine = manager.getEngineByName("julia");
+
+        engine.put("x", "hello");
+        // print global variable "x"
+        engine.eval("println(x);");
+        // the above line prints "hello"
+
+        // Now, pass a different script context
+        ScriptContext newContext = new SimpleScriptContext();
+        Bindings engineScope = newContext.getBindings(ScriptContext.ENGINE_SCOPE);
 
-            engine.put("x", "hello");
-            // print global variable "x"
-            engine.eval("println(x);");
-            // the above line prints "hello"
+        // add new variable "x" to the new engineScope
+        engineScope.put("x", "world");
 
-            // Now, pass a different script context
-            ScriptContext newContext = new SimpleScriptContext();
-            Bindings engineScope = newContext.getBindings(ScriptContext.ENGINE_SCOPE);
+        // execute the same script - but this time pass a different script context
+        engine.eval("println(x);", newContext);
+        // the above line prints "world"
 
-            // add new variable "x" to the new engineScope
-            engineScope.put("x", "world");
+        // Add some numeric values
+        engineScope.put("a", 2);
+        engineScope.put("b", 10);
+        engine.eval("for i in 1:5 println(\"Value: $(a * b * i)\"); end", newContext);
 
-            // execute the same script - but this time pass a different script context
-            engine.eval("println(x);", newContext);
-            // the above line prints "world"
+        Object result = engine.eval("a + b", newContext);
+        assertEquals(12, Julia4J.jl_unbox_int64((SWIGTYPE_p_jl_value_t) result));
     }
 }