test: rm underscores from test result assertions

Nerdware-LLC · Jun 17, 2024 · 024d2a4 · 024d2a4
1 parent 038d9ed
commit 024d2a4
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/src/sanitize/id.test.ts b/src/sanitize/id.test.ts
@@ -9,7 +9,7 @@ describe("sanitize: id", () => {
   test("removes all non-ID characters", () => {
     const input = `ab c${ZERO_WIDTH_SPACE}1 23${NULL_CONTROL_CHAR}~!@#$%^&*()_+-={}|:"<>?[]\\;',./`;
     const result = sanitizeID(input);
-    expect(result).toBe("abc123#_-");
+    expect(result).toBe("abc123#-");
     expect(result).not.toMatch(ZERO_WIDTH_AND_CONTROL_CHAR_REGEX);
   });
 });
diff --git a/src/validate/id.test.ts b/src/validate/id.test.ts
@@ -3,8 +3,8 @@ import { isValidID } from "./id.js";
 describe("validate: id", () => {
   // POSITIVE TEST CASES:
 
-  test(`returns true when given a string containing only alphanumeric characters, "-", "_", and/or "#"`, () => {
-    expect(isValidID("fake-id#abc_123")).toBe(true);
+  test(`returns true when given a string containing only alphanumeric characters, "#", and/or "-"`, () => {
+    expect(isValidID("fake-id#abc123")).toBe(true);
   });
 
   // NEGATIVE TEST CASES:
@@ -19,9 +19,11 @@ describe("validate: id", () => {
     expect(isValidID("fake-id#abc 123")).toBe(false);
   });
   test("returns false when given a string containing a disallowed special character", () => {
-    expect(isValidID("fake-id#abc_123.456!")).toBe(false);
+    expect(isValidID("fake-id#abc123_456")).toBe(false);
+    expect(isValidID("fake-id#abc123.456")).toBe(false);
+    expect(isValidID("fake-id#abc123456!")).toBe(false);
   });
   test("returns false when given a string containing a null unicode character", () => {
-    expect(isValidID("fake-id#abc_123\u0000")).toBe(false);
+    expect(isValidID("fake-id#abc123\u0000")).toBe(false);
   });
 });