fix(isUppercase): assume characters lowercase by default (#71)

unjs · Jan 10, 2024 · 57cfd15 · 57cfd15
1 parent d2c281f
commit 57cfd15
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/index.ts b/src/index.ts
@@ -16,7 +16,7 @@ export function isUppercase(char = ""): boolean | undefined {
   if (NUMBER_CHAR_RE.test(char)) {
     return undefined;
   }
-  return char.toUpperCase() === char;
+  return char !== char.toLowerCase();
 }
 
 export function splitByCase<T extends string>(str: T): SplitByCase<T>;

diff --git a/test/scule.test.ts b/test/scule.test.ts
@@ -30,6 +30,7 @@ describe("splitByCase", () => {
       ["foo", "Bar", "fuzz", "FI", "Zz"],
       ["\\", ".", "-"],
     ],
+    ["new-name-value", ["new-name-value"], ["_"]],
   ])("%s => %s", (input, expected, customSplitters?) => {
     if (customSplitters) {
       expect(splitByCase(input, customSplitters)).toMatchObject(expected);