From 0329c759dc99f836e511e9c38c4a1e1410385e22 Mon Sep 17 00:00:00 2001
From: Simon Detheridge <s@sd.ai>
Date: Sat, 4 Jan 2020 17:27:12 +0000
Subject: [PATCH] Remove wrapper function in PersistorManager

---
 services/filestore/app/js/PersistorManager.js | 46 +++++++++----------
 1 file changed, 22 insertions(+), 24 deletions(-)

diff --git a/services/filestore/app/js/PersistorManager.js b/services/filestore/app/js/PersistorManager.js
index f8ca7b9d2c..8124d66101 100644
--- a/services/filestore/app/js/PersistorManager.js
+++ b/services/filestore/app/js/PersistorManager.js
@@ -1,28 +1,26 @@
 const settings = require('settings-sharelatex')
 const logger = require('logger-sharelatex')
 
-module.exports = (function() {
-  logger.log(
-    {
-      backend: settings.filestore.backend
-    },
-    'Loading backend'
-  )
+logger.log(
+  {
+    backend: settings.filestore.backend
+  },
+  'Loading backend'
+)
+if (!settings.filestore.backend) {
+  throw new Error('no backend specified - config incomplete')
+}
 
-  if (!settings.filestore.backend) {
-    throw new Error('no backend specified - config incomplete')
-  }
-
-  switch (settings.filestore.backend) {
-    case 'aws-sdk':
-      return require('./AWSSDKPersistorManager')
-    case 's3':
-      return require('./S3PersistorManager')
-    case 'fs':
-      return require('./FSPersistorManager')
-    default:
-      throw new Error(
-        `unknown filestore backend: ${settings.filestore.backend}`
-      )
-  }
-})()
+switch (settings.filestore.backend) {
+  case 'aws-sdk':
+    module.exports = require('./AWSSDKPersistorManager')
+    break
+  case 's3':
+    module.exports = require('./S3PersistorManager')
+    break
+  case 'fs':
+    module.exports = require('./FSPersistorManager')
+    break
+  default:
+    throw new Error(`unknown filestore backend: ${settings.filestore.backend}`)
+}