added inactive and reactivate project logic

services/web/test/UnitTests/coffee/Docstore/DocstoreManagerTests.coffee

@@ -13,7 +13,7 @@ describe "DocstoreManager", ->
 				apis:
 					docstore:
 						url: "docstore.sharelatex.com"
-			"logger-sharelatex": @logger = {log: sinon.stub(), error: sinon.stub()}
+			"logger-sharelatex": @logger = {log: sinon.stub(), error: sinon.stub(), err:->}
 
 		@requestDefaults.calledWith(jar: false).should.equal true
 
@@ -179,3 +179,42 @@ describe "DocstoreManager", ->
 						project_id: @project_id
 					}, "error getting all docs from docstore")
 					.should.equal true
+
+
+	describe "archiveProject", ->
+		describe "with a successful response code", ->
+			beforeEach ->
+				@request.post = sinon.stub().callsArgWith(1, null, statusCode: 204)
+				@DocstoreManager.archiveProject @project_id, @callback
+
+			it "should call the callback", ->
+				@callback.called.should.equal true
+
+		describe "with a failed response code", ->
+			beforeEach ->
+				@request.post = sinon.stub().callsArgWith(1, null, statusCode: 500)
+				@DocstoreManager.archiveProject @project_id, @callback
+
+			it "should call the callback with an error", ->
+				@callback.calledWith(new Error("docstore api responded with non-success code: 500")).should.equal true
+
+
+
+	describe "unarchiveProject", ->
+		describe "with a successful response code", ->
+			beforeEach ->
+				@request.post = sinon.stub().callsArgWith(1, null, statusCode: 204)
+				@DocstoreManager.unarchiveProject @project_id, @callback
+
+			it "should call the callback", ->
+				@callback.called.should.equal true
+
+		describe "with a failed response code", ->
+			beforeEach ->
+				@request.post = sinon.stub().callsArgWith(1, null, statusCode: 500)
+				@DocstoreManager.unarchiveProject @project_id, @callback
+
+			it "should call the callback with an error", ->
+				@callback.calledWith(new Error("docstore api responded with non-success code: 500")).should.equal true
+
+

services/web/test/UnitTests/coffee/InactiveData/InactiveProjectManagerTests.coffee

@@ -0,0 +1,85 @@
+should = require('chai').should()
+SandboxedModule = require('sandboxed-module')
+assert = require('assert')
+path = require('path')
+sinon = require('sinon')
+modulePath = path.join __dirname, "../../../../app/js/Features/InactiveData/InactiveProjectManager"
+expect = require("chai").expect
+
+describe "InactiveProjectManager", ->
+
+	beforeEach ->
+
+		@settings = {}
+		@DocstoreManager =
+			unarchiveProject:sinon.stub()
+		@ProjectUpdateHandler = 
+			markAsActive:sinon.stub()
+			markAsInactive:sinon.stub()
+		@ProjectGetter = 
+			getProject:sinon.stub()
+		@InactiveProjectManager = SandboxedModule.require modulePath, requires:
+			"settings-sharelatex":@settings
+			"logger-sharelatex": 
+				log:->
+				err:->
+			"../Docstore/DocstoreManager":@DocstoreManager
+			"../Project/ProjectUpdateHandler":@ProjectUpdateHandler
+			"../Project/ProjectGetter":@ProjectGetter
+
+		@project_id = "1234"
+
+	describe "reactivateProjectIfRequired", ->
+
+		beforeEach ->
+			@project = {inactive:true}
+			@ProjectGetter.getProject.callsArgWith(2, null, @project)
+			@ProjectUpdateHandler.markAsActive.callsArgWith(1)
+
+		it "should call unarchiveProject", (done)->
+			@DocstoreManager.unarchiveProject.callsArgWith(1)
+			@InactiveProjectManager.reactivateProjectIfRequired @project_id, (err)=>
+				@DocstoreManager.unarchiveProject.calledWith(@project_id).should.equal true
+				@ProjectUpdateHandler.markAsActive.calledWith(@project_id).should.equal true
+				done()
+
+		it "should not mark project as active if error with unarchinging", (done)->
+			@DocstoreManager.unarchiveProject.callsArgWith(1, "error")
+			@InactiveProjectManager.reactivateProjectIfRequired @project_id, (err)=>
+				err.should.equal "error"
+				@DocstoreManager.unarchiveProject.calledWith(@project_id).should.equal true
+				@ProjectUpdateHandler.markAsActive.calledWith(@project_id).should.equal false
+				done()
+
+
+		it "should not call unarchiveProject if it is not inactive", (done)->
+			delete @project.inactive
+			@DocstoreManager.unarchiveProject.callsArgWith(1)
+			@InactiveProjectManager.reactivateProjectIfRequired @project_id, (err)=>
+				@DocstoreManager.unarchiveProject.calledWith(@project_id).should.equal false
+				@ProjectUpdateHandler.markAsActive.calledWith(@project_id).should.equal false
+				done()
+
+
+	describe "deactivateProject", ->
+
+		beforeEach ->
+
+		it "should call unarchiveProject and markAsInactive", (done)->
+			@DocstoreManager.unarchiveProject.callsArgWith(1)
+			@ProjectUpdateHandler.markAsInactive.callsArgWith(1)
+
+			@InactiveProjectManager.deactivateProject @project_id, (err)->
+				@DocstoreManager.unarchiveProject.calledWith(@project_id).should.equal true
+				@ProjectUpdateHandler.markAsInactive.callsArgWith(@project_id).should.equal true
+				done()
+
+		it "should not call markAsInactive if there was a problem unarchiving", (done)->
+			@DocstoreManager.unarchiveProject.callsArgWith(1, "errorrr")
+			@ProjectUpdateHandler.markAsInactive.callsArgWith(1)
+
+			@InactiveProjectManager.deactivateProject @project_id, (err)->
+				err.should.equal "errorrr"
+				@DocstoreManager.unarchiveProject.calledWith(@project_id).should.equal true
+				@ProjectUpdateHandler.markAsInactive.callsArgWith(@project_id).should.equal false
+				done()

services/web/test/UnitTests/coffee/Project/ProjectControllerTests.coffee

@@ -42,6 +42,10 @@ describe "ProjectController", ->
 			userCanAccessProject:sinon.stub()
 		@EditorController = 
 			renameProject:sinon.stub()
+		@InactiveProjectManager =
+			reactivateProjectIfRequired:sinon.stub()
+		@ProjectUpdateHandler =
+			markOpened: sinon.stub()
 		@ProjectController = SandboxedModule.require modulePath, requires:
 			"settings-sharelatex":@settings
 			"logger-sharelatex": 
@@ -57,6 +61,8 @@ describe "ProjectController", ->
 			'../../models/Project': Project:@ProjectModel
 			"../../models/User":User:@UserModel
 			"../../managers/SecurityManager":@SecurityManager
+			"../InactiveData/InactiveProjectManager":@InactiveProjectManager
+			"./ProjectUpdateHandler":@ProjectUpdateHandler
 
 		@user = 
 			_id:"!£123213kjljkl"
@@ -282,6 +288,9 @@ describe "ProjectController", ->
 			@SubscriptionLocator.getUsersSubscription.callsArgWith(1, null, {})
 			@SecurityManager.userCanAccessProject.callsArgWith 2, true, "owner"
 			@ProjectDeleter.unmarkAsDeletedByExternalSource = sinon.stub()
+			@InactiveProjectManager.reactivateProjectIfRequired.callsArgWith(1)
+			@ProjectUpdateHandler.markOpened.callsArgWith(1)
+
 
 		it "should render the project/editor page", (done)->
 			@res.render = (pageName, opts)=>
@@ -321,3 +330,16 @@ describe "ProjectController", ->
 				resCode.should.equal 401
 				done()
 			@ProjectController.loadEditor @req, @res
+
+		it "should reactivateProjectIfRequired", (done)->
+			@res.render = (pageName, opts)=>
+				@InactiveProjectManager.reactivateProjectIfRequired.calledWith(@project_id).should.equal true
+				done()
+			@ProjectController.loadEditor @req, @res
+
+		it "should mark project as opened", (done)->
+			@res.render = (pageName, opts)=>
+				@ProjectUpdateHandler.markOpened.calledWith(@project_id).should.equal true
+				done()
+			@ProjectController.loadEditor @req, @res
+

services/web/test/UnitTests/coffee/Project/ProjectUpdateHandlerTests.coffee

@@ -3,7 +3,7 @@ chai = require('chai').should()
 modulePath = "../../../../app/js/Features/Project/ProjectUpdateHandler.js"
 SandboxedModule = require('sandboxed-module')
 
-describe 'updating a project', ->
+describe 'ProjectUpdateHandler', ->
 
 
 	beforeEach ->
@@ -22,3 +22,36 @@ describe 'updating a project', ->
 				now = Date.now()+""
 				date.substring(0,5).should.equal now.substring(0,5)
 				done()
+
+	describe "markAsOpened", ->
+
+		it 'should send an update to mongo', (done)->
+			project_id = "project_id"
+			@handler.markAsOpened project_id, (err)=>
+				args = @ProjectModel.update.args[0]
+				args[0]._id.should.equal project_id
+				date = args[1].lastOpened+""
+				now = Date.now()+""
+				date.substring(0,5).should.equal now.substring(0,5)
+				done()
+
+	describe "markAsInactive", ->
+
+		it 'should send an update to mongo', (done)->
+			project_id = "project_id"
+			@handler.markAsInactive project_id, (err)=>
+				args = @ProjectModel.update.args[0]
+				args[0]._id.should.equal project_id
+				args[1].inactive.should.equal true
+				done()
+
+	describe "markAsActive", ->
+		it 'should send an update to mongo', (done)->
+			project_id = "project_id"
+			@handler.markAsActive project_id, (err)=>
+				args = @ProjectModel.update.args[0]
+				args[0]._id.should.equal project_id
+				args[1]["$unset"].inactive.should.equal true
+				done()
+
+