fix for issue #696

long was converted to int

Author: plamen5kov

runtime/src/main/jni/JsArgConverter.cpp

@@ -371,7 +371,7 @@ bool JsArgConverter::ConvertJavaScriptArray(const Local<Array>& jsArr, int index
     case 'J':
         arr = m_env.NewLongArray(arrLength);
         for (jsize i = 0; i < arrLength; i++) {
-            jlong value = jsArr->Get(i)->Int32Value();
+            jlong value = jsArr->Get(i)->NumberValue();
             m_env.SetLongArrayRegion((jlongArray) arr, i, 1, &value);
         }
         break;

test-app/app/src/main/assets/app/MyActivity.js

@@ -55,9 +55,6 @@ var MyActivity = (function (_super) {
     		onClick:  function() {
     			button.setBackgroundColor(colors[taps % colors.length]);
     			taps++;
-                taps++;
-                taps++;
-                taps++;
     		}}));
     };
     MyActivity = __decorate([

test-app/app/src/main/assets/app/tests/numericConversionTests.js

@@ -700,4 +700,15 @@ describe("Tests numeric conversions and constructor/method resolutions", functio
 				expect(exceptionThrown).toBe(true);
 				expect(exceptionCaught).toBe(true);
 			});
+
+            it("TestLongArrayIsConvertedCorrectly", function() {
+
+                __log("TEST: TestLongArrayIsConvertedCorrectly");
+
+                var t = new com.tns.tests.LongArrayTest();
+                var result = t.test([ 1485609581321, 1485609581321 ]);
+                var expectedResult = "[1485609581321, 1485609581321]"
+
+                expect(result).toBe(expectedResult);
+            });
 });

test-app/app/src/main/assets/metadata/treeNodeStream.dat

@@ -0,0 +1,9 @@
+package com.tns.tests;
+
+public class LongArrayTest {
+
+    public String test(long[] arr)
+    {
+        return java.util.Arrays.toString(arr);
+    }
+}