fix: alt space highlighting correct word as incorrect

also updates tests to just use the implementation instead of mocking it

Author: Miodec

frontend/__tests__/input/helpers/validation.spec.ts

@@ -14,10 +14,15 @@ vi.mock("../../../src/ts/test/funbox/list", () => ({
   findSingleActiveFunboxWithFunction: vi.fn(),
 }));
 
-vi.mock("../../../src/ts/utils/strings", () => ({
-  areCharactersVisuallyEqual: vi.fn(),
-  isSpace: vi.fn(),
-}));
+vi.mock("../../../src/ts/utils/strings", async () => {
+  const actual = await vi.importActual<typeof Strings>(
+    "../../../src/ts/utils/strings"
+  );
+  return {
+    ...actual,
+    areCharactersVisuallyEqual: vi.fn(),
+  };
+});
 
 describe("isCharCorrect", () => {
   beforeEach(() => {
@@ -34,7 +39,6 @@ describe("isCharCorrect", () => {
       null
     );
     (Strings.areCharactersVisuallyEqual as any).mockReturnValue(false);
-    (Strings.isSpace as any).mockReturnValue(false);
   });
 
   afterAll(() => {
@@ -133,7 +137,6 @@ describe("isCharCorrect", () => {
 
 describe("shouldInsertSpaceCharacter", () => {
   beforeEach(() => {
-    (Strings.isSpace as any).mockReturnValue(true);
     replaceConfig({
       mode: "time",
       stopOnError: "off",
@@ -147,7 +150,6 @@ describe("shouldInsertSpaceCharacter", () => {
   });
 
   it("returns null if data is not a space", () => {
-    (Strings.isSpace as any).mockReturnValue(false);
     expect(
       shouldInsertSpaceCharacter({
         data: "a",

frontend/src/ts/input/helpers/validation.ts

@@ -25,7 +25,7 @@ export function isCharCorrect(options: {
     throw new Error("Failed to check if char is correct - data is undefined");
   }
 
-  if (data === " ") {
+  if (isSpace(data)) {
     return inputValue === targetWord;
   }