You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Dec 1, 2017. It is now read-only.
With the new redis patch, tests/lib/util.js relies on a redis server as it starts a server every time it is required in. However, there are several methods inside util.js such as ensureFileSystem, etc. that do not need the redis server running to be used.
It is probably a good idea to rip this code and those alike from util.js and put them in a separate file so that we don't always need redis to run tests that don't really need them. Then we can just require these new files into these kind of tests and into util.js to be used as needed.
The text was updated successfully, but these errors were encountered:
With the new redis patch,
tests/lib/util.js
relies on a redis server as it starts a server every time it is required in. However, there are several methods inside util.js such as ensureFileSystem, etc. that do not need the redis server running to be used.It is probably a good idea to rip this code and those alike from util.js and put them in a separate file so that we don't always need redis to run tests that don't really need them. Then we can just require these new files into these kind of tests and into util.js to be used as needed.
The text was updated successfully, but these errors were encountered: