[CP-3185] Minor refactor of device-flash

libs/msc-flash/msc-flash-harmony/src/lib/selectors/index.ts

@@ -3,7 +3,7 @@
  * For licensing, see https://github.com/mudita/mudita-center/blob/master/LICENSE.md
  */
 
-export * from "./select-flashing-state.selector"
+export * from "./select-flashing-state"
 export * from "./select-flashing-abort-controller"
 export * from "./select-flashing-process-state"
 export * from "./select-is-flashing-in-active-phases"

libs/msc-flash/msc-flash-harmony/src/lib/selectors/select-flashing-abort-controller.ts

@@ -4,7 +4,7 @@
  */
 
 import { createSelector } from "@reduxjs/toolkit"
-import { flashingState } from "./select-flashing-state.selector"
+import { flashingState } from "./select-flashing-state"
 
 export const selectFlashingAbortController = createSelector(
   flashingState,

libs/msc-flash/msc-flash-harmony/src/lib/selectors/select-flashing-process-state.ts

@@ -4,7 +4,7 @@
  */
 
 import { createSelector } from "@reduxjs/toolkit"
-import { flashingState } from "./select-flashing-state.selector"
+import { flashingState } from "./select-flashing-state"
 
 export const selectFlashingProcessState = createSelector(
   flashingState,

libs/msc-flash/msc-flash-harmony/src/lib/services/device-flash/linux/linux-device-flash.service.ts

@@ -3,8 +3,11 @@
  * For licensing, see https://github.com/mudita/mudita-center/blob/master/LICENSE.md
  */
 
-import path from "path"
-import { execPromise, execCommandWithSudo } from "shared/utils"
+import {
+  execPromise,
+  execCommandWithSudo,
+  splitPathToDirNameAndBaseName,
+} from "shared/utils"
 import IDeviceFlash from "../device-flash.interface"
 import LinuxPartitionParser from "./linux-partition-parser"
 
@@ -36,7 +39,7 @@ class LinuxDeviceFlashService implements IDeviceFlash {
     const ejectDeviceCommand = await this.getEjectDeviceCommand(device)
 
     const command = `${unmountDeviceCommand} && ${flashImageToDeviceCommand} && ${ejectDeviceCommand}`
-    await execCommandWithSudo(command, { name: "Mudita Auto Flash" })
+    await execCommandWithSudo(command)
 
     console.log("Flash process completed successfully")
   }
@@ -55,9 +58,8 @@ class LinuxDeviceFlashService implements IDeviceFlash {
     imagePath: string,
     scriptPath: string
   ): Promise<string> {
-    const [path, scriptBasename] =
-      this.splitPathToDirnameAndBasename(scriptPath)
-    const [, imageBasename] = this.splitPathToDirnameAndBasename(imagePath)
+    const [path, scriptBasename] = splitPathToDirNameAndBaseName(scriptPath)
+    const [, imageBasename] = splitPathToDirNameAndBaseName(imagePath)
     return `chmod +x ${scriptPath} && cd ${path} && ./${scriptBasename} ${imageBasename} /dev/${device}`
   }
 
@@ -77,12 +79,6 @@ class LinuxDeviceFlashService implements IDeviceFlash {
     )
     return LinuxPartitionParser.parsePartitions(partitions ?? "")
   }
-
-  private splitPathToDirnameAndBasename(currentPath: string) {
-    const dirname = path.dirname(currentPath)
-    const basename = path.basename(currentPath)
-    return [dirname, basename]
-  }
 }
 
 export default LinuxDeviceFlashService

libs/msc-flash/msc-flash-harmony/src/lib/services/flash-msc-device.service.ts

@@ -84,9 +84,9 @@ const getFlashingImageDetails = async (
   await dispatch(
     getMscFlashingFilesDetails({
       signal,
+      platform,
       product: Product.MscHarmony,
       environment: RELEASE_SPACE,
-      platform: platform,
     })
   )
 }

libs/msc-flash/msc-flash-harmony/src/lib/ui/restarting-device-modal/restarting-device-modal.component.tsx

@@ -35,12 +35,16 @@ export const RestartingDeviceModal: FunctionComponent<
   const dispatch = useDispatch<Dispatch>()
 
   useEffect(() => {
-    const timer = setTimeout(() => {
-      dispatch(setFlashingProcessState(FlashingProcessState.Failed))
-    }, 2 * 60 * 1000)
+    let timeoutId: NodeJS.Timeout
 
-    return () => clearTimeout(timer)
-  }, [dispatch])
+    if (open) {
+      timeoutId = setTimeout(() => {
+        dispatch(setFlashingProcessState(FlashingProcessState.Failed))
+      }, 2 * 60 * 1000)
+    }
+
+    return () => clearTimeout(timeoutId)
+  }, [dispatch, open])
 
   return (
     <LoaderModal

libs/shared/utils/src/index.ts

@@ -13,6 +13,7 @@ export * from "./lib/prevent-default-shortcuts"
 export * from "./lib/register-shortcuts"
 export * from "./lib/routes-history.context"
 export * from "./lib/settings-service.interface"
+export * from "./lib/split-path-to-dir-name-and-base-name"
 export * from "./lib/use-debounced-events-handler"
 export * from "./lib/exec-command"
 export * from "./lib/exec-command-with-sudo"

libs/shared/utils/src/lib/exec-command-with-sudo.ts

@@ -7,7 +7,7 @@ import sudoPrompt from "@vscode/sudo-prompt"
 
 export const execCommandWithSudo = (
   command: string,
-  options: { name?: string; icns?: string; env?: { [key: string]: string } }
+  options?: { name?: string; icns?: string; env?: { [key: string]: string } }
 ): Promise<string | void> => {
   return new Promise((resolve, reject) => {
     sudoPrompt.exec(command, options, (error, stdout, stderr) => {

libs/shared/utils/src/lib/split-path-to-dir-name-and-base-name.ts

@@ -0,0 +1,14 @@
+/**
+ * Copyright (c) Mudita sp. z o.o. All rights reserved.
+ * For licensing, see https://github.com/mudita/mudita-center/blob/master/LICENSE.md
+ */
+
+import path from "path"
+
+export const splitPathToDirNameAndBaseName = (
+  currentPath: string
+): [string, string] => {
+  const dirName = path.dirname(currentPath)
+  const baseName = path.basename(currentPath)
+  return [dirName, baseName]
+}