CartoDB-SQL-API icon indicating copy to clipboard operation
CartoDB-SQL-API copied to clipboard

Test does not run without modifying the redis_utils.js

Open drzhouq opened this issue 9 years ago • 2 comments

tried to run npm test or make check and got error : Cannot read property 'redis_host' of undefined

added "require("../helper");" to the redis_utils solved the problem.

Please check whether this is the case.

drzhouq avatar Sep 21 '16 17:09 drzhouq

Hi @drzhouq,

Is it happening yet? Your fix makes sense, it was probably happening due suite execution order, where a suite was not including the helper to setup the environment accordingly.

rochoa avatar Oct 11 '16 07:10 rochoa

Hi @rochoa, I didn't create a pull request (blame my laziness or not exactly sure how the execution order). I will let you handle this. :-)

-John

drzhouq avatar Oct 11 '16 13:10 drzhouq