~go-bot/juju-core/1.18

Viewing all changes in revision 2288.

Tags: juju-1.18.3
[r=jameinel],[bug=1301353],[bug=1304770] testing: use juju-mongodb when appropriate

This should address parts of both bug #1301353 and bug #1304770.
Instead of requiring a 'mongod' to be in PATH, we'll also search for
/usr/lib/juju/bin/mongod which is from the juju-mongodb package.
For core tests, we prefer the juju-mongodb version (since jujud itself
will prefer it), and for store tests we prefer 'mongod' on PATH since
it will have the v8 engine and allow us to run all the tests.

This also auto-detects when noTestMongoJS should be set because it is
using the /usr/lib/juju/bin/mongod executable. Note that this
autodetection *fails* for people who have been hacking around support by
symlinking /usr/lib/juju/bin/mongod back into their PATH. So I didn't
remove the ENV variable, but it should no longer be necessary.

I'm targetting 1.18 because with this patch I think Juju CI will have to
do less hackery to have the test suite JustWork for them.

https://codereview.appspot.com/97240043/

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: