Skip to content

Commit da37aeb

Browse files
committed
fix(utils): reuse capitalize
1 parent fa7485b commit da37aeb

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

packages/utils/src/lib/case-conversions.ts

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,7 @@ export function kebabCaseToCamelCase<T extends string>(
1010
): KebabCaseToCamelCase<T> {
1111
return string
1212
.split('-')
13-
.map((segment, index) =>
14-
index === 0
15-
? segment
16-
: segment.charAt(0).toUpperCase() + segment.slice(1),
17-
)
13+
.map((segment, index) => (index === 0 ? segment : capitalize(segment)))
1814
.join('') as KebabCaseToCamelCase<T>;
1915
}
2016

packages/utils/src/lib/case-conversions.unit.test.ts

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { describe, expect, it } from 'vitest';
1+
import { beforeEach, describe, expect, it } from 'vitest';
22
import {
33
camelCaseToKebabCase,
44
capitalize,
@@ -9,11 +9,15 @@ import {
99

1010
describe('capitalize', () => {
1111
it('should transform the first string letter to upper case', () => {
12-
expect(capitalize('code PushUp')).toBe('Code PushUp');
12+
expect(capitalize('code')).toBe('Code');
13+
});
14+
15+
it('should lowercase all but the the first string letter', () => {
16+
expect(capitalize('PushUp')).toBe('Pushup');
1317
});
1418

1519
it('should leave the first string letter in upper case', () => {
16-
expect(capitalize('Code PushUp')).toBe('Code PushUp');
20+
expect(capitalize('Code')).toBe('Code');
1721
});
1822

1923
it('should accept empty string', () => {

0 commit comments

Comments
 (0)