misc: Fix new eslint warnings

Author: bilelmoussaoui

src/extension.ts

@@ -155,7 +155,7 @@ class Extension {
     }
 
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    private registerCommand(name: string, callback: (...args: any) => any | Promise<void>) {
+    private registerCommand(name: string, callback: (...args: any) => Promise<void>) {
         this.extCtx.subscriptions.push(
             commands.registerCommand(`${EXTENSION_ID}.${name}`, async (args) => {
                 try {

src/manifest.ts

@@ -1,7 +1,6 @@
 import * as vscode from 'vscode'
 import { BuildOptionsPathKeys, ManifestSchema, Module, SdkExtension } from './flatpak.types'
 import * as path from 'path'
-import { getuid } from 'process'
 import { cpus } from 'os'
 import * as fs from 'fs/promises'
 import { Command } from './command'
@@ -576,7 +575,7 @@ export class Manifest {
     }
 
     async runInRepo(shellCommand: string, mountExtensions: boolean, additionalEnvVars?: Map<string, string>): Promise<Command> {
-        const uid = getuid()
+        const uid = process.geteuid ? process.geteuid() : 1000
         const appId = this.id()
         if (this.fontsArgs.length === 0) {
             this.fontsArgs = await getFontsArgs()

src/test/suite/index.ts

@@ -12,26 +12,22 @@ export function run(): Promise<void> {
     const testsRoot = path.resolve(__dirname, '..')
 
     return new Promise((c, e) => {
-        glob('**/**.test.js', { cwd: testsRoot }, (err, files) => {
-            if (err) {
-                return e(err)
-            }
+        try {
+            const files = glob.globSync('**/**.test.js', { cwd: testsRoot })
 
             // Add files to the test suite
             files.forEach((f) => mocha.addFile(path.resolve(testsRoot, f)))
 
-            try {
-                // Run the mocha test
-                mocha.run((failures) => {
-                    if (failures > 0) {
-                        e(new Error(`${failures} tests failed.`))
-                    } else {
-                        c()
-                    }
-                })
-            } catch (err) {
-                e(err)
-            }
-        })
+            // Run the mocha test
+            mocha.run((failures) => {
+                if (failures > 0) {
+                    e(new Error(`${failures} tests failed.`))
+                } else {
+                    c()
+                }
+            })
+        } catch (err) {
+            e(err)
+        }
     })
 }