search

dundalek / closh

Bash-like shell based on Clojure
Eclipse Public License 1.0
1.62k stars 67 forks source link

Slow tests #11

Closed dundalek closed 6 years ago

dundalek commented 7 years ago

The whole test suite takes minutes to run, which is too slow. The reason for this is that it spawns child process to evaluate each closh case.

Possible solutions: 1) Make the test runner run on Lumo, that way we could evaluate the test cases within the test runner process. 2) Start one evaluator process, then use some sort of IPC to send code to evaluate and results back to test runner process.

mnewt commented 7 years ago

@dundalek What's the reason the tests are running under doo? I'm trying to remove it and make a thin lumo wrapper to run from there. So far it looks workable, I'm just having to re-plumb a few things. Not sure if I'll run into a roadblock but nothing yet.

dundalek commented 7 years ago

@mnewt Doo is the default runner in the template I used. If we can switch to lumo directly that would be great.

mnewt commented 6 years ago

Please see #25