add newsletter field to data model

mozilla · Dec 12, 2012 · 5839bc3 · 5839bc3
1 parent f1e7103
commit 5839bc3
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 4 deletions.
diff --git a/lib/api/v1/account.js b/lib/api/v1/account.js
@@ -14,8 +14,9 @@ module.exports = {
     },
     description: 'Stage a new account',
     schema: {
-      email: B(),
+      email: S(),
-      pass: S()
+      pass: S(),
+      newsletter: B()
     },
     response: {
       success: B()

diff --git a/lib/db/couchbase.js b/lib/db/couchbase.js
@@ -17,6 +17,7 @@ module.exports = {
     var account = {
       pass: data.pass,
       email: data.email,
+      newsletter: !!data.newsletter,
       staged: true
     };
     db.set(data.email, account, function (err, meta) {

diff --git a/lib/db/json.js b/lib/db/json.js
@@ -13,8 +13,8 @@ module.exports = {
     var account = {
       pass: data.pass,
       email: data.email,
+      newsletter: !!data.newsletter,
       staged: false
-      //staged: true
     };
     setTimeout(function() {
       db[data.email] = account;

diff --git a/test/api.v1.account.js b/test/api.v1.account.js
@@ -22,7 +22,8 @@ describe('/api/v1/account', function() {
   it('staging should return success', function(done) {
     client.account({
       email: 'foo',
-      pass: 'bar'
+      pass: 'bar',
+      newsletter: false
     }, function(err, r) {
       should.not.exist(err);
       should.exist(r);