Merge pull request #2613 from overleaf/spd-tags-timeout

Use a longer timeout when talking to the tags API

GitOrigin-RevId: 467f981c1818293c28dd288d7223540597a8ca3e

services/web/app/src/Features/Tags/TagsHandler.js

@@ -17,7 +17,7 @@ const request = require('request')
 const logger = require('logger-sharelatex')
 const { promisifyAll } = require('../../util/promises')
 
-const TIMEOUT = 1000
+const TIMEOUT = 10000
 const TagsHandler = {
   getAllTags(user_id, callback) {
     this._requestTags(user_id, (err, allTags) => {

services/web/test/unit/src/Tags/TagsHandlerTests.js

@@ -59,7 +59,7 @@ describe('TagsHandler', function() {
         this.request.del
           .calledWith({
             url: `${tagsUrl}/user/${user_id}/project/${project_id}`,
-            timeout: 1000
+            timeout: 10000
           })
           .should.equal(true)
         return done()
@@ -136,7 +136,7 @@ describe('TagsHandler', function() {
         const getOpts = {
           url: `${tagsUrl}/user/${user_id}/tag`,
           json: true,
-          timeout: 1000
+          timeout: 10000
         }
         this.request.get.calledWith(getOpts).should.equal(true)
         return done()
@@ -174,7 +174,7 @@ describe('TagsHandler', function() {
           json: {
             name: this.name
           },
-          timeout: 1000
+          timeout: 10000
         })
         .should.equal(true)
     })
@@ -197,7 +197,7 @@ describe('TagsHandler', function() {
         return this.request.del
           .calledWith({
             url: `${tagsUrl}/user/${user_id}/tag/${tag_id}`,
-            timeout: 1000
+            timeout: 10000
           })
           .should.equal(true)
       })
@@ -244,7 +244,7 @@ describe('TagsHandler', function() {
             json: {
               name: this.name
             },
-            timeout: 1000
+            timeout: 10000
           })
           .should.equal(true)
       })
@@ -288,7 +288,7 @@ describe('TagsHandler', function() {
         return this.request.del
           .calledWith({
             url: `${tagsUrl}/user/${user_id}/tag/${tag_id}/project/${project_id}`,
-            timeout: 1000
+            timeout: 10000
           })
           .should.equal(true)
       })
@@ -337,7 +337,7 @@ describe('TagsHandler', function() {
         return this.request.post
           .calledWith({
             url: `${tagsUrl}/user/${user_id}/tag/${tag_id}/project/${project_id}`,
-            timeout: 1000
+            timeout: 10000
           })
           .should.equal(true)
       })
@@ -389,7 +389,7 @@ describe('TagsHandler', function() {
               name: tag
             },
             url: `${tagsUrl}/user/${user_id}/tag/project/${project_id}`,
-            timeout: 1000
+            timeout: 10000
           })
           .should.equal(true)
       })
@@ -440,7 +440,7 @@ describe('TagsHandler', function() {
               user_id: 'new-user-id'
             },
             url: `${tagsUrl}/user/old-user-id/tag`,
-            timeout: 1000
+            timeout: 10000
           })
           .should.equal(true)
       })