From 6178f09cfe5ef705be7760db8c765b2d196e943a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E4=BE=9D=E7=91=AA=E8=B2=93?= Date: Fri, 6 Mar 2026 08:08:43 +0800 Subject: [PATCH] Fix abbreviateNumber result.trim() return value not assigned back Co-Authored-By: Claude Opus 4.6 --- src/module/abbreviateNumber.js | 2 +- tests/unit/module/abbreviateNumber.test.js | 5 +++++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/src/module/abbreviateNumber.js b/src/module/abbreviateNumber.js index 63229f1..42dc195 100644 --- a/src/module/abbreviateNumber.js +++ b/src/module/abbreviateNumber.js @@ -23,7 +23,7 @@ export default function abbreviateNumber(totalSeconds) { for(let i = 0; i < units.length; i++) { if(units[i] > 0) result += units[i] + symbols[i] + " "; } - result.trim(); + result = result.trim(); if(totalSeconds === 0) result = '0'; return result; diff --git a/tests/unit/module/abbreviateNumber.test.js b/tests/unit/module/abbreviateNumber.test.js index f755c42..10d218b 100644 --- a/tests/unit/module/abbreviateNumber.test.js +++ b/tests/unit/module/abbreviateNumber.test.js @@ -43,4 +43,9 @@ describe('abbreviateNumber', () => { it('handles NaN input', () => { expect(abbreviateNumber('abc').trim()).toBe(''); }); + + it('does not have trailing whitespace', () => { + expect(abbreviateNumber(45)).toBe('45s'); + expect(abbreviateNumber(3661)).toBe('1h 1m 1s'); + }); });