[TECH] Add more logs and error handling for MW Mod support

jest.config.js

@@ -5,7 +5,8 @@ module.exports = {
   globals: {
     'ts-jest': {}
   },
-
+  testTimeout: 35000,
+  setupFilesAfterEnv: ['./jest.setup.ts'],
   collectCoverageFrom: ['**/*.{js,jsx,ts,tsx}', '!**/*.config.js'],
   coverageDirectory: '<rootDir>/coverage',
   coveragePathIgnorePatterns: [

jest.setup.ts

@@ -0,0 +1 @@
+jest.setTimeout(35000)

package.json

@@ -324,7 +324,7 @@
     "@types/graceful-fs": "^4.1.9",
     "@types/i18next-fs-backend": "^1.1.5",
     "@types/ini": "^1.3.34",
-    "@types/jest": "^29.5.12",
+    "@types/jest": "^29.5.14",
     "@types/jsdom": "^20.0.1",
     "@types/mime": "^3.0.2",
     "@types/node": "^20.16.2",
@@ -334,6 +334,7 @@
     "@types/react-blockies": "^1.4.1",
     "@types/react-dom": "^18.0.8",
     "@types/react-router-dom": "^5.3.3",
+    "@types/rimraf": "^4.0.5",
     "@types/tmp": "^0.2.6",
     "@types/ws": "^8.5.12",
     "@typescript-eslint/eslint-plugin": "^6.21.0",
@@ -357,9 +358,10 @@
     "prettier": "^2.8.8",
     "pretty-quick": "^4.0.0",
     "react-markdown": "^9.0.1",
+    "rimraf": "^6.0.1",
     "sass": "^1.55.0",
     "tmp": "^0.2.3",
-    "ts-jest": "^29.1.1",
+    "ts-jest": "^29.2.5",
     "type-fest": "^3.2.0",
     "typescript": "5.3.3",
     "vite": "^5.4.0",

src/backend/__tests__/utils.test.ts

@@ -1,5 +1,11 @@
 import * as utils from '../utils'
-import { getExecutableAndArgs } from '../utils'
+import { getExecutableAndArgs, copyRecursiveAsync } from '../utils'
+import { mkdir, writeFile, symlink } from 'fs/promises'
+import { join } from 'path'
+import { existsSync } from 'fs'
+import { rimraf } from 'rimraf'
+import os from 'os'
+import * as fs from 'fs'
 
 jest.mock('electron')
 jest.mock('../logger/logger')
@@ -306,4 +312,83 @@ describe('backend/utils.ts', () => {
       expect(getExecutableAndArgs(input)).toEqual(expected)
     })
   })
+
+  describe('copyRecursiveAsync', () => {
+    const testDir = join(os.tmpdir(), `test-copy-${Date.now()}`)
+    const sourceDir = join(testDir, 'source')
+    const destDir = join(testDir, 'dest')
+
+    beforeEach(async () => {
+      jest.useFakeTimers({ advanceTimers: true })
+      await mkdir(sourceDir, { recursive: true })
+      await mkdir(destDir, { recursive: true })
+    })
+
+    afterEach(async () => {
+      jest.clearAllTimers() // Clear pending timers
+      jest.useRealTimers() // Restore real timers
+      await rimraf(testDir)
+    })
+
+    it('should copy a single file', async () => {
+      const testFile = join(sourceDir, 'test.txt')
+      const destFile = join(destDir, 'test.txt')
+      await writeFile(testFile, 'test content')
+
+      await copyRecursiveAsync(testFile, destFile)
+
+      expect(existsSync(destFile)).toBe(true)
+    })
+
+    it('should copy a directory recursively', async () => {
+      const subDir = join(sourceDir, 'subdir')
+      const testFile = join(subDir, 'test.txt')
+      await mkdir(subDir, { recursive: true })
+      await writeFile(testFile, 'test content')
+
+      await copyRecursiveAsync(sourceDir, join(destDir, 'source'))
+
+      expect(existsSync(join(destDir, 'source/subdir/test.txt'))).toBe(true)
+    })
+
+    it('should skip symbolic links', async () => {
+      const testFile = join(sourceDir, 'test.txt')
+      const linkFile = join(sourceDir, 'link.txt')
+      await writeFile(testFile, 'test content')
+      await symlink(testFile, linkFile)
+
+      await copyRecursiveAsync(linkFile, join(destDir, 'link.txt'))
+
+      expect(existsSync(join(destDir, 'link.txt'))).toBe(false)
+    })
+
+    it('should throw on timeout', async () => {
+      const COPY_TIMEOUT_MS = 30000
+      const testFile = join(sourceDir, 'test.txt')
+      await writeFile(testFile, 'test content')
+
+      // Mock the copyFile function to simulate a slow operation
+      const mockCopyFile = jest
+        .spyOn(fs.promises, 'copyFile')
+        .mockImplementation(async () => {
+          return new Promise((resolve) => {
+            setTimeout(resolve, COPY_TIMEOUT_MS + 1000)
+          })
+        })
+
+      const destFile = join(destDir, 'test.txt')
+
+      // Start the copy operation but don't await it yet
+      const copyPromise = copyRecursiveAsync(testFile, destFile)
+
+      // Advance timers to trigger timeout
+      jest.advanceTimersByTime(COPY_TIMEOUT_MS + 100)
+
+      // Now check if it throws
+      await expect(copyPromise).rejects.toThrow('Timeout')
+
+      // Restore original implementation
+      mockCopyFile.mockRestore()
+    })
+  })
 })

src/backend/ipcHandlers/mods.ts

@@ -1,3 +1,4 @@
+import { captureException } from '@sentry/electron'
 import { notify, showDialogBoxModalAuto } from 'backend/dialog/dialog'
 import { cancelQueueExtraction } from 'backend/downloadmanager/downloadqueue'
 import { LogPrefix, logDebug, logError, logInfo } from 'backend/logger/logger'
@@ -233,7 +234,15 @@ export async function prepareBaseGameForModding({
         readdirSync(extractedFolderFullPath).forEach(async (file) => {
           const srcPath = path.join(extractedFolderFullPath, file)
           const destPath = path.join(dirPath, file)
-          await copyRecursiveAsync(srcPath, destPath)
+          try {
+            await copyRecursiveAsync(srcPath, destPath)
+          } catch (error) {
+            const errorMessage = `Error copying ${srcPath} to ${destPath} ${error}`
+            logError(errorMessage, LogPrefix.HyperPlay)
+            extractService.emit('error', new Error(errorMessage))
+            captureException(error)
+            throw new Error(errorMessage)
+          }
         })
 
         // remove the extracted folder

src/backend/storeManagers/hyperplay/games.ts

@@ -810,6 +810,7 @@ export async function install(
     const gameInfo = getGameInfo(appName)
     const { title, account_name } = gameInfo
     const isMarketWars = account_name === 'marketwars'
+
     if (isMarketWars && modOptions?.zipFilePath) {
       try {
         await prepareBaseGameForModding({
@@ -818,6 +819,7 @@ export async function install(
           installPath: dirpath
         })
       } catch (error) {
+        callAbortController(appName)
         return { status: 'error' }
       }
     }