DoSomething/ds-mdata-responder — pull request #224 by jonuy (2014-10-17 15:56:36+00:00)
Title: Converting remaining console logs to logger

#### What's this PR do?
Log cleanup and converting them all to use logger. Most will fall into the `info` channel. Some that are particularly verbose and not useful unless one is debugging are slotted into the `verbose` or `debug` channels.

There's also a bug fix in here related to delivering the delayed messages for SMS games.

#### Where should the reviewer start?
* **Delayed messages fix:** SGCompetitiveStoryController.delayedOptinSingleUser - this used to convert `currentGameId` to a string and then do a comparison off of that. Problems were that `.valueOf()` still returned an ObjectId, not a string. While we could use `.toString()` to convert to a string and then check IDs that way, it just seemed to keep things cleaner to use `ObjectId.equals(ObjectId)` to check IDs against one another.
* `DonorsChooseDonationController.prototype._post` was removed because it's actually already defined.
* And then the rest of the changes should just be fairly straightforward conversions from `console.log` to `logger.info` (or `logger.log` if it's going into a different channel).

#### How should this be manually tested?
* `$ npm test`

Things I tested using Postman:
* donation flow
* campaign tip delivery
* Bully Text with two player from start to finish

#### What are the relevant tickets?
Closes #215

#### Questions
@tongxiang In particular, check out the delayedOptinSingleUser fix. Is it ok to assume that the currentGameId getting passed in while always be an object and not a string?