Adding semver coercion before comparing versions

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

src/main/helm/__mocks__/helm-chart-manager.ts

@@ -88,14 +88,14 @@ export class HelmChartManager {
             {
               apiVersion: "3.0.0",
               name: "pretzel",
-              version: "1.0.0-beta2",
+              version: "1.0",
               repo: "bitnami",
               digest: "test",
             },
             {
               apiVersion: "3.0.0",
               name: "pretzel",
-              version: "1.0.0",
+              version: "1.0.1",
               repo: "bitnami",
               digest: "test"
             }

src/main/helm/__tests__/helm-service.test.ts

@@ -86,14 +86,14 @@ describe("Helm Service tests", () => {
           {
             apiVersion: "3.0.0",
             name: "pretzel",
-            version: "1.0.0",
+            version: "1.0.1",
             repo: "bitnami",
             digest: "test",
           },
           {
             apiVersion: "3.0.0",
             name: "pretzel",
-            version: "1.0.0-beta2",
+            version: "1.0",
             repo: "bitnami",
             digest: "test"
           }

src/main/helm/helm-service.ts

@@ -109,7 +109,10 @@ class HelmService {
   private sortChartsByVersion(chartGroups: RepoHelmChartList) {
     for (const key in chartGroups) {
       chartGroups[key] = chartGroups[key].sort((first, second) => {
-        return semver.compare(second.version, first.version);
+        const firstVersion = semver.coerce(first.version || 0);
+        const secondVersion = semver.coerce(second.version || 0);
+
+        return semver.compare(secondVersion, firstVersion);
       });
     }