Move manageConfigs helper into separate file

Author: ZIMkaRU

src/helpers/index.js

@@ -19,6 +19,7 @@ const parseEnvValToBool = require('./parse-env-val-to-bool')
 const isBfxApiStaging = require('./is-bfx-api-staging')
 const waitPort = require('./wait-port')
 const getUIFontsAsCSSString = require('./get-ui-fonts-as-css-string')
+const manageConfigs = require('./manage-configs')
 
 module.exports = {
   getFreePort,
@@ -33,5 +34,6 @@ module.exports = {
   parseEnvValToBool,
   isBfxApiStaging,
   waitPort,
-  getUIFontsAsCSSString
+  getUIFontsAsCSSString,
+  manageConfigs
 }

src/helpers/manage-configs.js

@@ -0,0 +1,60 @@
+'use strict'
+
+const path = require('node:path')
+
+const isMac = process.platform === 'darwin'
+const { REPORT_FILES_PATH_VERSION } = require('../const')
+
+const {
+  configsKeeperFactory
+} = require('../configs-keeper')
+
+const _resetReportFilesPath = async (
+  configsKeeper,
+  opts = {}
+) => {
+  const {
+    pathToUserReportFiles
+  } = opts
+
+  // Need to use a new report folder path for export
+  const reportFilesPathVersion = configsKeeper
+    .getConfigByName('reportFilesPathVersion')
+
+  if (reportFilesPathVersion === REPORT_FILES_PATH_VERSION) {
+    return
+  }
+
+  await configsKeeper.saveConfigs({
+    reportFilesPathVersion: REPORT_FILES_PATH_VERSION,
+    pathToUserReportFiles
+  })
+}
+
+module.exports = (params) => {
+  const {
+    pathToUserData,
+    pathToUserDocuments,
+    pathToUserDownloads
+  } = params ?? {}
+
+  const pathToUserReportFiles = isMac
+    ? pathToUserDownloads
+    : path.join(
+      pathToUserDocuments,
+      'bitfinex/reports'
+    )
+
+  const configsKeeper = configsKeeperFactory(
+    {
+      pathToUserData,
+      configsByDefault: { pathToUserReportFiles }
+    }
+  )
+  _resetReportFilesPath(
+    configsKeeper,
+    { pathToUserReportFiles }
+  )
+
+  return configsKeeper
+}

src/initialize-app.js

@@ -1,12 +1,8 @@
 'use strict'
 
 const { app } = require('electron')
-const path = require('path')
 const i18next = require('i18next')
 
-const isMac = process.platform === 'darwin'
-const { REPORT_FILES_PATH_VERSION } = require('./const')
-
 const TranslationIpcChannelHandlers = require(
   './window-creators/main-renderer-ipc-bridge/translation-ipc-channel-handlers'
 )
@@ -37,17 +33,15 @@ const {
 } = require('./window-creators/change-loading-win-visibility-state')
 const WINDOW_NAMES = require('./window-creators/window.names')
 const makeOrReadSecretKey = require('./make-or-read-secret-key')
-const {
-  configsKeeperFactory
-} = require('./configs-keeper')
 const {
   IpcMessageError,
   AppInitializationError
 } = require('./errors')
 const {
   deserializeError,
   getFreePort,
-  initIpcChannelHandlers
+  initIpcChannelHandlers,
+  manageConfigs
 } = require('./helpers')
 const getUserDataPath = require('./helpers/get-user-data-path')
 const {
@@ -61,28 +55,6 @@ const manageWorkerMessages = require(
 )
 const printToPDF = require('./print-to-pdf')
 
-const _resetReportFilesPath = async (
-  configsKeeper,
-  opts = {}
-) => {
-  const {
-    pathToUserReportFiles
-  } = opts
-
-  // Need to use a new report folder path for export
-  const reportFilesPathVersion = configsKeeper
-    .getConfigByName('reportFilesPathVersion')
-
-  if (reportFilesPathVersion === REPORT_FILES_PATH_VERSION) {
-    return
-  }
-
-  await configsKeeper.saveConfigs({
-    reportFilesPathVersion: REPORT_FILES_PATH_VERSION,
-    pathToUserReportFiles
-  })
-}
-
 const _ipcMessToPromise = (ipc) => {
   return new Promise((resolve, reject) => {
     try {
@@ -131,34 +103,6 @@ const _ipcMessToPromise = (ipc) => {
   })
 }
 
-const _manageConfigs = (params) => {
-  const {
-    pathToUserData,
-    pathToUserDocuments,
-    pathToUserDownloads
-  } = params ?? {}
-
-  const pathToUserReportFiles = isMac
-    ? pathToUserDownloads
-    : path.join(
-      pathToUserDocuments,
-      'bitfinex/reports'
-    )
-
-  const configsKeeper = configsKeeperFactory(
-    {
-      pathToUserData,
-      configsByDefault: { pathToUserReportFiles }
-    }
-  )
-  _resetReportFilesPath(
-    configsKeeper,
-    { pathToUserReportFiles }
-  )
-
-  return configsKeeper
-}
-
 module.exports = async () => {
   try {
     initIpcChannelHandlers(
@@ -187,7 +131,7 @@ module.exports = async () => {
     const pathToUserDocuments = app.getPath('documents')
     const pathToUserDownloads = app.getPath('downloads')
 
-    const configsKeeper = _manageConfigs({
+    const configsKeeper = manageConfigs({
       pathToUserData,
       pathToUserDocuments,
       pathToUserDownloads