Refactoring

Signed-off-by: Lauri Nevala <lauri.nevala@gmail.com>

src/common/utils/app-version.ts

@@ -1,3 +1,4 @@
+import requestPromise from "request-promise-native";
 import packageInfo from "../../../package.json";
 
 export function getAppVersion(): string {
@@ -11,3 +12,13 @@ export function getBundledKubectlVersion(): string {
 export function getBundledExtensions(): string[] {
   return packageInfo.lens?.extensions || [];
 }
+
+export async function getAppVersionFromProxyServer(proxyPort: number): Promise<string> {
+  const response = await requestPromise({
+    method: "GET",
+    uri: `http://localhost:${proxyPort}/version`,
+    resolveWithFullResponse: true
+  });
+
+  return JSON.parse(response.body).version;
+}

src/main/developer-tools.ts

@@ -1,9 +1,12 @@
+import logger from "./logger";
+
 /**
  * Installs Electron developer tools in the development build.
  * The dependency is not bundled to the production build.
  */
 export const installDeveloperTools = async () => {
   if (process.env.NODE_ENV === "development") {
+    logger.info("🤓 Installing developer tools");
     const { default: devToolsInstaller, REACT_DEVELOPER_TOOLS } = await import("electron-devtools-installer");
 
     return devToolsInstaller([REACT_DEVELOPER_TOOLS]);

src/main/index.ts

@@ -26,8 +26,7 @@ import { InstalledExtension, extensionDiscovery } from "../extensions/extension-
 import type { LensExtensionId } from "../extensions/lens-extension";
 import { installDeveloperTools } from "./developer-tools";
 import { filesystemProvisionerStore } from "./extension-filesystem";
-import requestPromise from "request-promise-native";
-import { getAppVersion } from "../common/utils";
+import { getAppVersion, getAppVersionFromProxyServer } from "../common/utils";
 
 const workingDir = path.join(app.getPath("appData"), appName);
 let proxyPort: number;
@@ -77,7 +76,6 @@ app.on("ready", async () => {
 
   registerFileProtocol("static", __static);
 
-  logger.info("🤓 Installing developer tools");
   await installDeveloperTools();
 
   logger.info("💾 Loading stores");
@@ -117,16 +115,10 @@ app.on("ready", async () => {
   // test proxy connection
   try {
     logger.info("🔎 Testing LensProxy connection ...");
-    const response = await requestPromise({
-      method: "GET",
-      uri: `http://localhost:${proxyPort}/version`,
-      resolveWithFullResponse: true
-    });
+    const versionFromProxy = await getAppVersionFromProxyServer(proxyPort);
 
-    const appVersion = JSON.parse(response.body).version;
-
-    if (getAppVersion() != appVersion) {
-      logger.error(`Proxy server responded with invalid response: ${response.body}`);
+    if (getAppVersion() !== versionFromProxy) {
+      logger.error(`Proxy server responded with invalid response`);
     }
     logger.info("⚡ LensProxy connection OK");
   } catch (error) {