Fixed case where saveAccessToken was called inside saveRefreshToken (in lib/grant.js); Fixed test failure accessing user.id (in test/grant.js)
Merge pull request #21 from athieme/2.0-fixes

Switch out saveAccessToken model call in saveRefreshToken
Fix clientId regex checking
camelCase all the things + switch back to explicit save*Token signature

There was still a confusing mix of under_scored naming and camelCase
naming, I've now draw the line as follows:
 - Everything coming in from the wild, should be underscored (as
 per the spec)
 - Everything is camelCased internally

Switching to the save*Token(data, callback) was shorter, but made
variable names VERY important, this removes some of that overhead
by reintroducing the longer
save*Token(token, clientId, expires, userId, callback)
style signatures.
Merge remote-tracking branch 'upstream/2.0' into dynamodb2.0

* upstream/2.0:
  camelCase all the things + switch back to explicit save*Token signature
  Fix clientId regex checking
  Fixed case where saveAccessToken was called inside saveRefreshToken (in lib/grant.js); Fixed test failure accessing user.id (in test/grant.js)