From 97326308fabf65f337d09c82f063e76a784d182e Mon Sep 17 00:00:00 2001 From: Shane Kilkelly Date: Thu, 10 Sep 2015 15:40:43 +0100 Subject: [PATCH] Update the Acceptance tests to include a case where a user doesn't exist. --- .../test/acceptance/coffee/GettingUpdatesTests.coffee | 6 +++--- .../test/acceptance/coffee/helpers/MockWebApi.coffee | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/services/track-changes/test/acceptance/coffee/GettingUpdatesTests.coffee b/services/track-changes/test/acceptance/coffee/GettingUpdatesTests.coffee index 6452649fc4..7c94bcdc34 100644 --- a/services/track-changes/test/acceptance/coffee/GettingUpdatesTests.coffee +++ b/services/track-changes/test/acceptance/coffee/GettingUpdatesTests.coffee @@ -15,6 +15,7 @@ describe "Getting updates", -> @now = Date.now() @to = @now @user_id = ObjectId().toString() + @deleted_user_id = 'deleted_user' @doc_id = ObjectId().toString() @project_id = ObjectId().toString() @@ -44,6 +45,7 @@ describe "Getting updates", -> meta: { ts: @now - (9 - i) * @hours, user_id: @user_id } v: 2 * i + 2 } + @updates[0].meta.user_id = @deleted_user_id TrackChangesClient.pushRawUpdates @project_id, @doc_id, @updates, (error) => throw error if error? @@ -91,7 +93,6 @@ describe "Getting updates", -> users: [@user] }] - describe "getting updates beyond the end of the database", -> before (done) -> TrackChangesClient.getUpdates @project_id, { before: @to - 8 * @hours + 1, min_count: 30 }, (error, body) => @@ -115,6 +116,5 @@ describe "Getting updates", -> meta: start_ts: @to - 9 * @hours - 2 * @minutes end_ts: @to - 9 * @hours - users: [@user] + users: [@user, null] }] - diff --git a/services/track-changes/test/acceptance/coffee/helpers/MockWebApi.coffee b/services/track-changes/test/acceptance/coffee/helpers/MockWebApi.coffee index f7cfc9e2e9..002d944dfd 100644 --- a/services/track-changes/test/acceptance/coffee/helpers/MockWebApi.coffee +++ b/services/track-changes/test/acceptance/coffee/helpers/MockWebApi.coffee @@ -7,7 +7,7 @@ module.exports = MockWebApi = projects: {} getUser: (user_id, callback = (error) ->) -> - callback null, @users[user_id] + callback null, @users[user_id] or null getProject: (project_id, callback = (error, project) ->) -> callback null, @projects[project_id]