nealrichardson commented on PR #14365:
URL: https://github.com/apache/arrow/pull/14365#issuecomment-1273661319

   > Ahh, good to know. How involved is it to enable those tests? I don't have 
cycles to learn how to write integration tests right now, unfortunately.
   
   I'm not sure. If it were me, I'd start by removing the 
`.skip_category('JS')` lines I linked above, at least for the types you think 
should work, and see if the integration tests pass on CI. 
   
   The JS test runner is 
https://github.com/apache/arrow/blob/master/dev/archery/archery/integration/tester_js.py,
 which points at the files in 
https://github.com/apache/arrow/tree/master/js/bin. It's possible the tests 
will just work (if the implementation is correct), or it's possible that there 
is some JSON writer/reader specific to the integration tests that needs to be 
updated. 
   
   @zeroshade has been in here recently, enabling more Go tests, so he may be 
able to advise too.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to