From 5735f2347a4d1465033806e4648583a5a35a4e0f Mon Sep 17 00:00:00 2001
From: Ola Hungerford <olahungerford@gmail.com>
Date: Mon, 24 Mar 2025 08:52:45 -0700
Subject: [PATCH] Add tests related to issues/187 to confirm fixed

---
 .../__tests__/DynamicJsonForm.test.tsx        | 34 +++++++++++++++++++
 1 file changed, 34 insertions(+)

diff --git a/client/src/components/__tests__/DynamicJsonForm.test.tsx b/client/src/components/__tests__/DynamicJsonForm.test.tsx
index d140663..f18c15f 100644
--- a/client/src/components/__tests__/DynamicJsonForm.test.tsx
+++ b/client/src/components/__tests__/DynamicJsonForm.test.tsx
@@ -3,6 +3,40 @@ import { describe, it, expect, jest } from '@jest/globals';
 import DynamicJsonForm from '../DynamicJsonForm';
 import type { JsonSchemaType } from '../DynamicJsonForm';
 
+describe('DynamicJsonForm String Fields', () => {
+  const renderForm = (props = {}) => {
+    const defaultProps = {
+      schema: {
+        type: "string" as const,
+        description: "Test string field"
+      } satisfies JsonSchemaType,
+      value: undefined,
+      onChange: jest.fn()
+    };
+    return render(<DynamicJsonForm {...defaultProps} {...props} />);
+  };
+
+  describe('Type Validation', () => {
+    it('should handle numeric input as string type', () => {
+      const onChange = jest.fn();
+      renderForm({ onChange });
+      
+      const input = screen.getByRole('textbox');
+      fireEvent.change(input, { target: { value: '123321' } });
+      
+      expect(onChange).toHaveBeenCalledWith('123321');
+      // Verify the value is a string, not a number
+      expect(typeof onChange.mock.calls[0][0]).toBe('string');
+    });
+
+    it('should render as text input, not number input', () => {
+      renderForm();
+      const input = screen.getByRole('textbox');
+      expect(input).toHaveProperty('type', 'text');
+    });
+  });
+});
+
 describe('DynamicJsonForm Integer Fields', () => {
   const renderForm = (props = {}) => {
     const defaultProps = {