fix: copilot review 3

Author: artemmufazalov

src/utils/versions/__test__/sortVersions.test.ts

@@ -2,7 +2,7 @@ import {sortVersions} from '../sortVersions';
 import type {PreparedVersion} from '../types';
 
 describe('sortVersions', () => {
-    test('should sort versions by majorIndex in descending order', function () {
+    test('should sort versions by majorIndex in ascending order', function () {
         const versions: PreparedVersion[] = [
             {version: 'v2.0.0', majorIndex: 1},
             {version: 'v1.0.0', majorIndex: 2},
@@ -34,7 +34,7 @@ describe('sortVersions', () => {
         ]);
     });
 
-    test('should sort versions by minorIndex in descending order when majorIndex is the same', function () {
+    test('should sort versions by minorIndex in ascending order when majorIndex is the same', function () {
         const versions: PreparedVersion[] = [
             {version: 'v1.2.0', majorIndex: 2, minorIndex: 1},
             {version: 'v1.1.0', majorIndex: 2, minorIndex: 2},

src/utils/versions/clusterVersionColors.ts

@@ -1,5 +1,7 @@
 import type {MetaClusterVersion} from '../../types/api/meta';
 
+import {COLORS, DEFAULT_COLOR, getMinorVersionColorVariant, hashCode} from './getVersionsColors';
+import {getMinorVersion} from './parseVersion';
 import {sortVersions} from './sortVersions';
 import type {
     ColorIndexToVersionsMap,
@@ -8,8 +10,6 @@ import type {
     VersionsDataMap,
 } from './types';
 
-import {COLORS, DEFAULT_COLOR, getMinorVersion, getMinorVersionColorVariant, hashCode} from '.';
-
 const UNDEFINED_COLOR_INDEX = '__no_color__';
 
 export const getVersionMap = (
@@ -75,23 +75,24 @@ export const prepareClusterVersions = (
     filteredVersions.forEach((item) => {
         if (result[item.version]) {
             // Sum count for versions of different nodes types
+            // Do not recalculate version data
             const currentCount = result[item.version].count || 0;
             const itemCount = item.count || 0;
 
             result[item.version].count = currentCount + itemCount;
+        } else {
+            const minorVersion = getMinorVersion(item.version);
+            const data = versionsDataMap.get(minorVersion);
+
+            result[item.version] = {
+                version: item.version,
+                minorVersion,
+                color: data?.color,
+                majorIndex: data?.majorIndex,
+                minorIndex: data?.minorIndex,
+                count: item.count || 0,
+            };
         }
-
-        const minorVersion = getMinorVersion(item.version);
-        const data = versionsDataMap.get(minorVersion);
-
-        result[item.version] = {
-            version: item.version,
-            minorVersion,
-            color: data?.color,
-            majorIndex: data?.majorIndex,
-            minorIndex: data?.minorIndex,
-            count: item.count || 0,
-        };
     });
 
     return sortVersions(Object.values(result));