catch errors in async handlers (#506)

ExpressApp.js

@@ -0,0 +1,4 @@
+const express = require('express');
+require('express-async-errors');
+
+module.exports = express

express.js

@@ -1,5 +1,5 @@
 const fafApp = require('./fafApp')
-const express = require('express')
+const express = require('./ExpressApp')
 const app = express()
 
 fafApp.setup(app)

fafApp.js

@@ -1,5 +1,5 @@
 const appConfig = require('./config/app')
-const express = require('express')
+const express = require('./ExpressApp')
 const bodyParser = require('body-parser')
 const session = require('express-session')
 const FileStore = require('session-file-store')(session)

package.json

@@ -9,6 +9,7 @@
     "connect-flash": "^0.1.1",
     "dotenv": "16.3.1",
     "express": "^4.18.1",
+    "express-async-errors": "^3.1.1",
     "express-session": "^1.17.3",
     "express-validator": "7.0.1",
     "moment": "^2.29.4",

routes/views/accountRouter.js

@@ -1,4 +1,4 @@
-const express = require('express')
+const express = require('../../ExpressApp')
 const router = express.Router();
 const middlewares = require('../middleware')
 

routes/views/auth.js

@@ -1,6 +1,6 @@
 const appConfig = require('../../config/app')
 const passport = require('passport')
-const express = require("express")
+const express = require('../../ExpressApp')
 const router = express.Router()
 
 router.get('/login', passport.authenticate(appConfig.oauth.strategy));

routes/views/clanRouter.js

@@ -1,4 +1,4 @@
-const express = require('express');
+const express = require('../../ExpressApp')
 const router = express.Router();
 
 // This will be replaced soon, therefor I did not spend time on it

routes/views/dataRouter.js

@@ -1,4 +1,4 @@
-const express = require('express');
+const express = require('../../ExpressApp')
 const router = express.Router();
 const {AcquireTimeoutError} = require('../../lib/MutexService');
 

routes/views/defaultRouter.js

@@ -1,4 +1,4 @@
-const express = require('express');
+const express = require('../../ExpressApp')
 const router = express.Router();
 
 router.get('/', (req, res) => res.render('index'))

routes/views/leaderboardRouter.js

@@ -1,4 +1,4 @@
-const express = require('express');
+const express = require('../../ExpressApp')
 const router = express.Router();
 const {AcquireTimeoutError} = require('../../lib/MutexService');
 const middlewares = require('../middleware')

routes/views/news.js

@@ -1,4 +1,4 @@
-const express = require('express');
+const express = require('../../ExpressApp')
 const router = express.Router();
 
 function getNewsArticleBySlug(articles, slug) {

routes/views/staticMarkdownRouter.js

@@ -1,4 +1,4 @@
-const express = require('express')
+const express = require('../../ExpressApp')
 const showdown = require('showdown')
 const fs = require('fs')
 const router = express.Router()

tests/integration/IsAuthenticatedMiddleware.test.js

@@ -1,4 +1,4 @@
-const Express = require('express')
+const Express = require('../../ExpressApp')
 const middlewares = require('../../routes/middleware')
 const supertestSession = require('supertest-session')
 const fafApp = require('../../fafApp')

tests/integration/NewsRouter.test.js

@@ -1,4 +1,4 @@
-const Express = require('express')
+const Express = require('../../ExpressApp')
 const supertestSession = require('supertest-session')
 const fafApp = require('../../fafApp')
 

tests/integration/accountRouter.test.js

@@ -1,4 +1,4 @@
-const Express = require('express')
+const Express = require('../../ExpressApp')
 const supertestSession = require('supertest-session')
 const fafApp = require('../../fafApp')
 

tests/integration/asyncErrorHandler.test.js

@@ -0,0 +1,28 @@
+const Express = require('../../ExpressApp')
+const supertestSession = require('supertest-session')
+
+let testApp = null
+let testSession = null
+
+beforeEach(() => {
+    const app = new Express()
+    testSession = supertestSession(app)
+    testApp = app
+})
+
+describe('Async Error Handler', function () {
+    test('route is protected and redirects to "/login"', async () => {
+        testApp.get('/', async () => {
+            throw new Error('test failed and node crashed')
+        })
+        testApp.use((err, req, res, next) => {
+            res.status(500).send('not crashed')
+            next(err)
+        })
+
+        const res = await testSession.get('/')
+
+        expect(res.status).toBe(500)
+        expect(res.text).toBe('not crashed')
+    })
+})

tests/integration/clanRouter.test.js

@@ -1,4 +1,4 @@
-const Express = require('express')
+const Express = require('../../ExpressApp')
 const supertestSession = require('supertest-session')
 const fafApp = require('../../fafApp')
 

tests/integration/defaultRouter.test.js

@@ -1,4 +1,4 @@
-const Express = require('express')
+const Express = require('../../ExpressApp')
 const supertestSession = require('supertest-session')
 const fafApp = require('../../fafApp')
 

tests/integration/leaderboardRouter.test.js

@@ -1,4 +1,4 @@
-const Express = require('express')
+const Express = require('../../ExpressApp')
 const supertestSession = require('supertest-session')
 const fafApp = require('../../fafApp')
 const passportMock = require('../helpers/PassportMock')

tests/integration/markdownRouter.test.js

@@ -1,4 +1,4 @@
-const Express = require('express')
+const Express = require('../../ExpressApp')
 const supertestSession = require('supertest-session')
 const fafApp = require('../../fafApp')
 

tests/integration/servicesMiddleware.test.js

@@ -1,4 +1,4 @@
-const Express = require('express')
+const Express = require('../../ExpressApp')
 const WordpressService = require('../../lib/WordpressService')
 const UserService = require('../../lib/LoggedInUserService')
 const LeaderboardService = require('../../lib/LeaderboardService')

yarn.lock

@@ -3220,6 +3220,11 @@ expect@^29.7.0:
     jest-message-util "^29.7.0"
     jest-util "^29.7.0"
 
+express-async-errors@^3.1.1:
+  version "3.1.1"
+  resolved "https://registry.yarnpkg.com/express-async-errors/-/express-async-errors-3.1.1.tgz#6053236d61d21ddef4892d6bd1d736889fc9da41"
+  integrity sha512-h6aK1da4tpqWSbyCa3FxB/V6Ehd4EEB15zyQq9qe75OZBp0krinNKuH4rAY+S/U/2I36vdLAUFSjQJ+TFmODng==
+
 express-session@^1.17.3:
   version "1.17.3"
   resolved "https://registry.yarnpkg.com/express-session/-/express-session-1.17.3.tgz#14b997a15ed43e5949cb1d073725675dd2777f36"