feat: change uptime format

Author: artemmufazalov

src/utils/dataFormatters/__test__/formatUptime.test.ts

@@ -1,21 +1,58 @@
-import {getDowntimeFromDateFormatted, getUptimeFromDateFormatted} from '../dataFormatters';
+import {EMPTY_DATA_PLACEHOLDER} from '../../constants';
+import {UNBREAKABLE_GAP} from '../../utils';
+import {
+    formatUptimeInSeconds,
+    getDowntimeFromDateFormatted,
+    getUptimeFromDateFormatted,
+} from '../dataFormatters';
 
 describe('getUptimeFromDateFormatted', () => {
     it('should calculate and format uptime', () => {
         expect(getUptimeFromDateFormatted(3_600_000, 7_200_000)).toBe('1:00:00');
     });
     it('should return 0 if dateFrom after dateTo', () => {
-        expect(getUptimeFromDateFormatted(3_600_000, 3_599_000)).toBe('0:00:00');
+        expect(getUptimeFromDateFormatted(3_600_000, 3_599_000)).toBe('0s');
     });
 });
 describe('getDowntimeFromDateFormatted', () => {
     it('should calculate and format downtime as -uptime', () => {
         expect(getDowntimeFromDateFormatted(3_600_000, 7_200_000)).toBe('-1:00:00');
     });
     it('should not add sign if downtime is 0', () => {
-        expect(getDowntimeFromDateFormatted(3_600_000, 3_600_000)).toBe('0:00:00');
+        expect(getDowntimeFromDateFormatted(3_600_000, 3_600_000)).toBe('0s');
     });
     it('should return 0 if dateFrom after dateTo', () => {
-        expect(getDowntimeFromDateFormatted(3_600_000, 3_599_000)).toBe('0:00:00');
+        expect(getDowntimeFromDateFormatted(3_600_000, 3_599_000)).toBe('0s');
+    });
+});
+describe('formatUptimeInSeconds', () => {
+    const M = 60;
+    const H = 60 * M;
+    const D = 24 * H;
+
+    it('should return days if value is more than 24h', () => {
+        expect(formatUptimeInSeconds(D)).toBe('1d' + UNBREAKABLE_GAP + '00:00:00');
+        expect(formatUptimeInSeconds(D + H + M + 12)).toBe('1d' + UNBREAKABLE_GAP + '01:01:12');
+        expect(formatUptimeInSeconds(12 * D + 12 * H + 12 * M + 12)).toBe(
+            '12d' + UNBREAKABLE_GAP + '12:12:12',
+        );
+        expect(formatUptimeInSeconds(1234 * D + 12 * H + 12 * M + 12)).toBe(
+            '1234d' + UNBREAKABLE_GAP + '12:12:12',
+        );
+    });
+    it('should return hours if value is less than 24h', () => {
+        expect(formatUptimeInSeconds(H + M + 12)).toBe('1:01:12');
+        expect(formatUptimeInSeconds(12 * H + 12 * M + 12)).toBe('12:12:12');
+    });
+    it('should return minutes if value is less than hour', () => {
+        expect(formatUptimeInSeconds(M + 12)).toBe('1:12');
+        expect(formatUptimeInSeconds(12 * M + 2)).toBe('12:02');
+    });
+    it('should return second if value is less than hour', () => {
+        expect(formatUptimeInSeconds(12)).toBe('12s');
+        expect(formatUptimeInSeconds(2)).toBe('2s');
+    });
+    it('should return empty placeholder on NaN', () => {
+        expect(formatUptimeInSeconds(Number.NaN)).toBe(EMPTY_DATA_PLACEHOLDER);
     });
 });

src/utils/dataFormatters/dataFormatters.ts

@@ -1,14 +1,14 @@
-import {dateTimeParse} from '@gravity-ui/date-utils';
+import {dateTimeParse, duration} from '@gravity-ui/date-utils';
 
 import type {TVDiskID, TVSlotId} from '../../types/api/vdisk';
 import {
     formatBytes as formatBytesCustom,
     getSizeWithSignificantDigits,
 } from '../bytesParsers/formatBytes';
 import type {BytesSizes} from '../bytesParsers/formatBytes';
-import {DAY_IN_SECONDS, HOUR_IN_SECONDS} from '../constants';
+import {EMPTY_DATA_PLACEHOLDER, HOUR_IN_SECONDS} from '../constants';
 import {configuredNumeral} from '../numeral';
-import {isNumeric} from '../utils';
+import {UNBREAKABLE_GAP, isNumeric} from '../utils';
 
 import {formatValues} from './common';
 import {formatNumberWithDigits, getNumberWithSignificantDigits} from './formatNumber';
@@ -40,19 +40,33 @@ export const stringifyVdiskId = (id?: TVDiskID | TVSlotId) => {
     return id ? Object.values(id).join('-') : '';
 };
 
-export const formatUptimeInSeconds = (seconds: number) => {
-    const days = Math.floor(seconds / DAY_IN_SECONDS);
-    const remain = seconds % DAY_IN_SECONDS;
+/**
+ * It works well only with positive values,
+ * if you want to get negative formatted uptime, use some wrapper like getDowntimeFromDateFormatted
+ */
+export function formatUptimeInSeconds(seconds: number) {
+    if (!isNumeric(seconds)) {
+        return EMPTY_DATA_PLACEHOLDER;
+    }
 
-    const uptime = [days && `${days}d`, configuredNumeral(remain).format('00:00:00')]
-        .filter(Boolean)
-        .join(' ');
+    const d = duration(seconds, 's').rescale();
 
-    return uptime;
-};
+    if (d.days() > 0) {
+        return d.format(`d[${i18n('d')}${UNBREAKABLE_GAP}]hh:mm:ss`);
+    }
+
+    if (d.hours() > 0) {
+        return d.format('h:mm:ss');
+    }
+    if (d.minutes() > 0) {
+        return d.format('m:ss');
+    }
+
+    return d.format(`s[${i18n('s')}]`);
+}
 
 export const formatMsToUptime = (ms?: number) => {
-    return ms && formatUptimeInSeconds(ms / 1000);
+    return formatUptimeInSeconds(Number(ms) / 1000);
 };
 
 export function getUptimeFromDateFormatted(dateFrom?: number | string, dateTo?: number | string) {

src/utils/dataFormatters/i18n/en.json

@@ -1,3 +1,6 @@
 {
-  "format-cpu.cores": ["core", "cores", "cores", "cores"]
+  "format-cpu.cores": ["core", "cores", "cores", "cores"],
+
+  "d": "d",
+  "s": "s"
 }

src/utils/dataFormatters/i18n/index.ts

@@ -1,8 +1,7 @@
 import {registerKeysets} from '../../i18n';
 
 import en from './en.json';
-import ru from './ru.json';
 
 const COMPONENT = 'ydb-format-cpu';
 
-export default registerKeysets(COMPONENT, {ru, en});
+export default registerKeysets(COMPONENT, {en});

src/utils/dataFormatters/i18n/ru.json

@@ -74,7 +74,7 @@ export function prepareNodeSystemState(
     const DC = systemState.Location?.DataCenter || systemState.DataCenter;
     const TenantName = systemState?.Tenants?.[0];
 
-    let Uptime: string;
+    let Uptime: PreparedNodeSystemState['Uptime'];
 
     if (systemState.DisconnectTime) {
         Uptime = getDowntimeFromDateFormatted(systemState.DisconnectTime);