Issue 753: Updating MongoDB

[steaward]

Hey,

This PR is in respect to issue 753
I've updated the code to fit with MongoDB v3.
Only two things that I noticed that needed changing was the fact the DB is now passed back to the client during the initial connection. I don't know what Trustroots DB name is, so it's left blank by default. The collection 'agendaJobs' is then grabbed.

Ran all tests as well as the scripts to populate the DB with fake data and it's failing on the profile picture tests. However, the master branch from my fork fails on this too?

[simison]

Thanks!

Looks like package.json and package-lock.json are still out of sync and thus npm ci fails tests in Travis.

Running npm install should update the lock file and you can push changes to it here.

[simison]

Thanks for working on this!

I know this might've been a confusingly small task but I like to leave these out there for the community during the Hactoberfest.

Only two things that I noticed that needed changing was the fact the DB is now passed back to the client during the initial connection.

Yap, only the background job worker script uses native driver directly for Agenda's connection (job runner library) — the server app itself uses Mongoose ODM at all levels.

Agenda supports re-using Mongoose connection so we should harmonise the setup as a future iteration.

I don't know what Trustroots DB name is, so it's left blank by default. The collection 'agendaJobs' is then grabbed.

Trustroots DB name depends on environment config and is different for prod, dev and test envs. It's the last bit in these URIs:

trustroots/config/env/development.js

Line 15 in 3281c5c

uri: 'mongodb://' + (process.env.DB_1_PORT_27017_TCP_ADDR || 'localhost') + '/trustroots-dev',

trustroots/config/env/production.js

Line 15 in 3281c5c

uri: 'mongodb://' + (process.env.DB_1_PORT_27017_TCP_ADDR || 'localhost') + '/trustroots',

trustroots/config/env/test.js

Line 15 in 3281c5c

uri: 'mongodb://' + (process.env.DB_1_PORT_27017_TCP_ADDR || 'localhost') + '/trustroots-test',

Worker script gets this via config.db.uri so we're all good. 👌

[simison]

Ran all tests as well as the scripts to populate the DB with fake data and it's failing on the profile picture tests.

This could be due missing imagemagic in your computer?

They don't seem to fail on Travis CI currently (this branch shows up green as well).