search

juxt / mach

A remake of make (in ClojureScript)
246 stars 21 forks source link

mach test: use mach classpath so andare is added #51

Closed nberger closed 7 years ago

nberger commented 7 years ago

Running mach test was failing with errors like No such namespace: cljs.core.async. That's because andare's jar was not being added to the classpath as it's done in bin/mach. We can just take lumo's classpath that already includes the jar, because we are in the context of the lumo process started from the mach test command

As an alternative, my first approach was to simply add -c ${MACH_HOME}/lib/andare-0.7.0.jar in the test command, following what's done in bin/mach. Then I thought this approach would be more generic, but I'm not sure what's better: the downside of this approach is that it adds more stuff than needed (currently: . and ./src apart from andare's jar)

jonpither commented 7 years ago

great to have the tests working again, thankyou @nberger !