Merge pull request #13110 from overleaf/jk-investigate-fake-timers-problem

[web] Fix failing unit test involving `fake-timers`

GitOrigin-RevId: d166cc10079b9c4fa6d07525a9e6140e86b05fc9

services/web/test/unit/src/Project/ProjectUpdateHandlerTests.js

@@ -22,6 +22,10 @@ describe('ProjectUpdateHandler', function () {
     this.clock = sinon.useFakeTimers(this.fakeTime.getTime())
   })
 
+  afterEach(function () {
+    this.clock.restore()
+  })
+
   beforeEach(function () {
     let Project
     this.ProjectModel = Project = class Project {}

services/web/test/unit/src/ThirdPartyDataStore/TpdsUpdateHandlerTests.js

@@ -502,8 +502,7 @@ function expectProjectCreated() {
     ).to.have.been.calledWith(this.userId, this.projectName)
   })
 
-  // eslint-disable-next-line mocha/no-skipped-tests
-  it.skip('sets the root doc', function () {
+  it('sets the root doc', function () {
     // Fire pending timers
     this.clock.runAll()
     expect(
@@ -518,8 +517,7 @@ function expectProjectNotCreated() {
       .been.called
   })
 
-  // eslint-disable-next-line mocha/no-skipped-tests
-  it.skip('does not set the root doc', function () {
+  it('does not set the root doc', function () {
     // Fire pending timers
     this.clock.runAll()
     expect(this.RootDocManager.promises.setRootDocAutomatically).not.to.have