fix(web): admin settings number input validation (#9470)

* fix(web): admin settings number input validation

* fix import by creating *.ts file

* just ignore import error

web/package.json

@@ -31,6 +31,7 @@
     "@sveltejs/vite-plugin-svelte": "^3.0.2",
     "@testing-library/jest-dom": "^6.4.2",
     "@testing-library/svelte": "^5.0.0",
+    "@testing-library/user-event": "^14.5.2",
     "@types/dom-to-image": "^2.6.7",
     "@types/justified-layout": "^4.1.4",
     "@types/lodash-es": "^4.17.12",

web/src/lib/components/shared-components/settings/setting-input-field.spec.ts

@@ -0,0 +1,30 @@
+import { render } from '@testing-library/svelte';
+import userEvent from '@testing-library/user-event';
+// @ts-expect-error the import works but tsc check errors
+import SettingInputField, { SettingInputFieldType } from './setting-input-field.svelte';
+
+describe('SettingInputField component', () => {
+  it('validates number input on blur', async () => {
+    const { getByRole } = render(SettingInputField, {
+      props: {
+        label: 'test-number-input',
+        inputType: SettingInputFieldType.NUMBER,
+        value: 0,
+        min: 0,
+        max: 100,
+        step: '0.1',
+      },
+    });
+    const user = userEvent.setup();
+
+    const numberInput = getByRole('spinbutton') as HTMLInputElement;
+    expect(numberInput.value).toEqual('0');
+
+    await user.click(numberInput);
+    await user.keyboard('100.1');
+    expect(numberInput.value).toEqual('100.1');
+
+    await user.click(document.body);
+    expect(numberInput.value).toEqual('100');
+  });
+});

web/src/lib/components/shared-components/settings/setting-input-field.svelte

@@ -25,9 +25,7 @@
   export let isEdited = false;
   export let passwordAutocomplete: string = 'current-password';
 
-  const handleInput = (e: Event) => {
-    value = (e.target as HTMLInputElement).value;
-
+  const validateInput = () => {
     if (inputType === SettingInputFieldType.NUMBER) {
       let newValue = Number(value) || 0;
       if (newValue < min) {
@@ -79,7 +77,8 @@
       {step}
       {required}
       {value}
-      on:input={handleInput}
+      on:input={(e) => (value = e.currentTarget.value)}
+      on:blur={validateInput}
       {disabled}
       {title}
     />