Fix abbreviateNumber result.trim() return value not assigned back
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -23,7 +23,7 @@ export default function abbreviateNumber(totalSeconds) {
|
|||||||
for(let i = 0; i < units.length; i++) {
|
for(let i = 0; i < units.length; i++) {
|
||||||
if(units[i] > 0) result += units[i] + symbols[i] + " ";
|
if(units[i] > 0) result += units[i] + symbols[i] + " ";
|
||||||
}
|
}
|
||||||
result.trim();
|
result = result.trim();
|
||||||
if(totalSeconds === 0) result = '0';
|
if(totalSeconds === 0) result = '0';
|
||||||
|
|
||||||
return result;
|
return result;
|
||||||
|
|||||||
@@ -43,4 +43,9 @@ describe('abbreviateNumber', () => {
|
|||||||
it('handles NaN input', () => {
|
it('handles NaN input', () => {
|
||||||
expect(abbreviateNumber('abc').trim()).toBe('');
|
expect(abbreviateNumber('abc').trim()).toBe('');
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it('does not have trailing whitespace', () => {
|
||||||
|
expect(abbreviateNumber(45)).toBe('45s');
|
||||||
|
expect(abbreviateNumber(3661)).toBe('1h 1m 1s');
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|||||||
Reference in New Issue
Block a user