@remoteoss/json-schema-form 0.1.1-dev.20230616085129 → 0.1.1-dev.20230619163824

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remoteoss/json-schema-form",
-  "version": "0.1.1-dev.20230616085129",
+  "version": "0.1.1-dev.20230619163824",
   "description": "Headless UI form powered by JSON Schemas",
   "author": "Remote.com <engineering@remote.com> (https://remote.com/)",
   "license": "MIT",
@@ -11,12 +11,10 @@
   },
   "main": "dist/index.cjs",
   "module": "dist/index.js",
-  "typings": "./json-schema-form.d.ts",
   "files": [
     "dist",
     "src/tests",
     "json-schema-form.schema.json",
-    "json-schema-form.d.ts",
     "README.md",
     "CHANGELOG.md",
     "LICENSE.md"
@@ -24,6 +22,7 @@
   "scripts": {
     "build": "node scripts/build.js",
     "test": "jest",
+    "test:watch": "jest --watchAll",
     "lint": "eslint \"src/**/*.{js,ts}\"",
     "format": "npm run format:prettier && npm run format:eslint",
     "prepare": "husky install",
@@ -48,7 +47,7 @@
   "dependencies": {
     "lodash": "^4.17.21",
     "randexp": "^0.5.3",
-    "yup": "^0.29.1"
+    "yup": "^0.30.0"
   },
   "devDependencies": {
     "@babel/core": "^7.21.5",

package/src/tests/createHeadlessForm.test.js

@@ -419,7 +419,7 @@ describe('createHeadlessForm', () => {
 
   describe('field support', () => {
     it('support "text" field type', () => {
-      const { fields } = createHeadlessForm(schemaInputTypeText);
+      const { fields, handleValidation } = createHeadlessForm(schemaInputTypeText);
 
       expect(fields[0]).toMatchObject({
         description: 'The number of your national identification (max 10 digits)',
@@ -435,10 +435,14 @@ describe('createHeadlessForm', () => {
       });
 
       const fieldValidator = fields[0].schema;
-      expect(fieldValidator.isValidSync('CI007')).toBe(true);
-      expect(fieldValidator.isValidSync(true)).toBe(true); // @BUG RMT-446 - cannot be a bool
-      expect(fieldValidator.isValidSync(1)).toBe(true); // @BUG RMT-446 - cannot be a number
-      expect(fieldValidator.isValidSync(0)).toBe(true); // @BUG RMT-446 - cannot be a number
+      expect(fieldValidator.isValidSync('CI007', { strict: true })).toBe(true);
+      expect(fieldValidator.isValidSync(true, { strict: true })).toBe(false);
+      expect(fieldValidator.isValidSync(1, { strict: true })).toBe(false);
+      expect(fieldValidator.isValidSync(0, { strict: true })).toBe(false);
+
+      expect(handleValidation({ id_number: 1 }).formErrors).toEqual({
+        id_number: 'id_number must be a `string` type, but the final value was: `1`.',
+      });
 
       expect(() => fieldValidator.validateSync('')).toThrowError('Required field');
     });
@@ -2313,7 +2317,7 @@ describe('createHeadlessForm', () => {
           validateForm({
             validate_tabs: 'no',
             a_fieldset: {
-              id_number: 123,
+              id_number: '123',
             },
             mandatory_group_array: 'no',
           })
@@ -2328,7 +2332,7 @@ describe('createHeadlessForm', () => {
           validateForm({
             validate_tabs: 'yes',
             a_fieldset: {
-              id_number: 123,
+              id_number: '123',
             },
             mandatory_group_array: 'no',
           })
@@ -2344,7 +2348,7 @@ describe('createHeadlessForm', () => {
           validateForm({
             validate_tabs: 'yes',
             a_fieldset: {
-              id_number: 123,
+              id_number: '123',
             },
             mandatory_group_array: 'yes',
             a_group_array: [{ full_name: 'adfs' }],
@@ -2355,7 +2359,7 @@ describe('createHeadlessForm', () => {
           validateForm({
             validate_tabs: 'yes',
             a_fieldset: {
-              id_number: 123,
+              id_number: '123',
               tabs: 2,
             },
             mandatory_group_array: 'no',
@@ -2446,7 +2450,7 @@ describe('createHeadlessForm', () => {
           validateForm({
             validate_fieldset: ['id_number'],
             a_fieldset: {
-              id_number: 123,
+              id_number: '123',
             },
           })
         ).toBeUndefined();
@@ -2455,7 +2459,7 @@ describe('createHeadlessForm', () => {
           validateForm({
             validate_fieldset: ['id_number', 'all'],
             a_fieldset: {
-              id_number: 123,
+              id_number: '123',
             },
           })
         ).toEqual({
@@ -2468,7 +2472,7 @@ describe('createHeadlessForm', () => {
           validateForm({
             validate_fieldset: ['id_number', 'all'],
             a_fieldset: {
-              id_number: 123,
+              id_number: '123',
               tabs: 2,
             },
           })
@@ -2483,7 +2487,7 @@ describe('createHeadlessForm', () => {
           validateForm({
             validate_fieldset: ['id_number'],
             a_fieldset: {
-              id_number: 123,
+              id_number: '123',
             },
           })
         ).toBeUndefined();
@@ -2492,7 +2496,7 @@ describe('createHeadlessForm', () => {
           validateForm({
             validate_fieldset: ['id_number', 'all'],
             a_fieldset: {
-              id_number: 123,
+              id_number: '123',
             },
           })
         ).toEqual({
@@ -2505,7 +2509,7 @@ describe('createHeadlessForm', () => {
           validateForm({
             validate_fieldset: ['id_number', 'all'],
             a_fieldset: {
-              id_number: 123,
+              id_number: '123',
               tabs: 2,
             },
           })

package/json-schema-form.d.ts

@@ -1,42 +0,0 @@
-/**
- * Shorthand to lookup for keys with `x-jsf-*` preffix.
- */
-export function pickXKey(node: Object, key: 'presentation' | 'error-message'): Object | undefined;
-
-type JSConfig = {
-  initialValues?: Record<string, unknown>;
-  strictInputType?: boolean;
-  customProperties?: {
-    description: Function | string;
-    [key: string]: unknown;
-  };
-  inputTypes?: {
-    errorMessage: Record<string, unknown>;
-  };
-};
-
-type Fields = Record<string, unknown>[]; // TODO: We don't know what type we have here, we need to investigate
-
-type $TsFixMe = any;
-
-/**
- * Returns the Yup schema structure of given fields.
- * These fields must be the same from
- * const { fields } = createHeadlessForm()
- */
-export function buildCompleteYupSchema(fields: Fields, config: JSConfig): $TsFixMe; // TODO: We don't what yup returns here, we'll fix it later
-
-type HeadlessFormOutput = {
-  fields: Fields;
-  handleValidation: () => $TsFixMe;
-  isError: boolean;
-  error?: undefined;
-};
-
-/**
- * Generates the Headless form based on the provided JSON schema
- */
-export function createHeadlessForm(
-  jsonSchema: Record<string, unknown>,
-  customConfig: JSConfig
-): HeadlessFormOutput;