From dbd6daeca013c65eb3185ff3e74bc867c7c303c1 Mon Sep 17 00:00:00 2001 From: Keith Holliday Date: Mon, 16 May 2016 13:12:46 -0500 Subject: [PATCH] Updated task model to allow setting streak (#7306) --- test/api/v3/integration/tasks/POST-tasks_user.test.js | 4 +--- website/server/models/task.js | 2 +- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/test/api/v3/integration/tasks/POST-tasks_user.test.js b/test/api/v3/integration/tasks/POST-tasks_user.test.js index b39e640480..623d9bb4a4 100644 --- a/test/api/v3/integration/tasks/POST-tasks_user.test.js +++ b/test/api/v3/integration/tasks/POST-tasks_user.test.js @@ -108,7 +108,7 @@ describe('POST /tasks/user', () => { }); it(`ignores setting userId, history, createdAt, - updatedAt, challenge, completed, streak, + updatedAt, challenge, completed, dateCompleted fields`, async () => { let task = await user.post('/tasks/user', { text: 'test daily', @@ -119,7 +119,6 @@ describe('POST /tasks/user', () => { updatedAt: 'tomorrow', challenge: 'no', completed: true, - streak: 25, dateCompleted: 'never', value: 324, // ignored because not a reward }); @@ -130,7 +129,6 @@ describe('POST /tasks/user', () => { expect(task.updatedAt).not.to.equal('tomorrow'); expect(task.challenge).not.to.equal('no'); expect(task.completed).to.equal(false); - expect(task.streak).to.equal(0); expect(task.streak).not.to.equal('never'); expect(task.value).not.to.equal(324); }); diff --git a/website/server/models/task.js b/website/server/models/task.js index 0459a74580..27f356efa3 100644 --- a/website/server/models/task.js +++ b/website/server/models/task.js @@ -56,7 +56,7 @@ export let TaskSchema = new Schema({ }, discriminatorOptions)); TaskSchema.plugin(baseModel, { - noSet: ['challenge', 'userId', 'completed', 'history', 'streak', 'dateCompleted', 'completed'], + noSet: ['challenge', 'userId', 'completed', 'history', 'dateCompleted', 'completed'], sanitizeTransform (taskObj) { if (taskObj.type && taskObj.type !== 'reward') { // value should be settable directly only for rewards delete taskObj.value;