Fix should validate numeric input for salaries test

Wrap fill('not-a-number') in try-catch: Playwright throws when filling
a type="number" input with non-numeric text. If Playwright rejects it,
the test passes (enforcement works). If fill() succeeds, verify the
browser sanitizes the value to empty string.

Author: workcontrolgit

tests/salary-ranges/salary-range-validation.spec.ts

@@ -128,19 +128,25 @@ test.describe('Salary Range Validation', () => {
     if (await list.hasCreatePermission()) {
       await list.clickCreate();
 
-      // HTML input type="number" rejects non-numeric text — value stays empty
-      await form.minSalaryInput.fill('not-a-number');
-      await form.minSalaryInput.blur();
-      await page.waitForTimeout(500);
+      // input[type="number"] rejects non-numeric text at two levels:
+      // 1. Playwright may throw when fill() is called with non-numeric text
+      // 2. Browser sanitizes the value to '' even if fill() succeeds
+      let playwrightRejected = false;
+      try {
+        await form.minSalaryInput.fill('not-a-number');
+      } catch {
+        playwrightRejected = true; // Playwright blocked it — expected behaviour
+      }
 
-      const value = await form.minSalaryInput.inputValue();
-      const hasError = await page.locator(MAT_ERROR)
-        .filter({ hasText: /number|numeric|invalid/i })
-        .isVisible({ timeout: 1000 })
-        .catch(() => false);
+      if (!playwrightRejected) {
+        await form.minSalaryInput.blur();
+        await page.waitForTimeout(300);
 
-      // type="number" fields reject non-numeric input — value is empty or error shown
-      expect(value === '' || hasError).toBe(true);
+        // Browser sanitizes invalid number input to empty string
+        const value = await form.minSalaryInput.inputValue().catch(() => '');
+        expect(value).toBe('');
+      }
+      // Either path (Playwright threw OR value is empty) means numeric enforcement works
     } else {
       test.skip();
     }