Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(text-validation) - pass only values that are strings #12

Merged
merged 12 commits into from
Jun 21, 2023
Merged

fix(text-validation) - pass only values that are strings #12

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
510b0e7
fix(text-validation) - pass only values that are strings
gabrielremote Jun 9, 2023
7353ade
Release 0.1.1-dev.20230613092121
gabrielremote Jun 13, 2023
a45443e
docs(text) - add assertion for better confidence
gabrielremote Jun 14, 2023
d8ba813
fix(text) - fix text coercion with strict
gabrielremote Jun 14, 2023
c55c71c
Release 0.1.1-dev.20230614173637
gabrielremote Jun 14, 2023
d1f3c27
fix(text) - update yup and use context.originalValue
gabrielremote Jun 15, 2023
12b420b
remove strict references
gabrielremote Jun 15, 2023
e2f7e59
Release 0.1.1-dev.20230616112349
gabrielremote Jun 16, 2023
6fbd8ce
docs(json-schemas) - fix nullables
gabrielremote Jun 19, 2023
3115f87
Release 0.1.1-dev.20230619163824
gabrielremote Jun 19, 2023
af4df92
revert package
gabrielremote Jun 19, 2023
31b468a
allow 0 values
gabrielremote Jun 19, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,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",
Expand Down
26 changes: 13 additions & 13 deletions src/tests/createHeadlessForm.test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -436,9 +436,9 @@ 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(true)).toBe(false);
expect(fieldValidator.isValidSync(1)).toBe(false);
expect(fieldValidator.isValidSync(0)).toBe(false);

expect(() => fieldValidator.validateSync('')).toThrowError('Required field');
});
Expand Down Expand Up @@ -2313,7 +2313,7 @@ describe('createHeadlessForm', () => {
validateForm({
validate_tabs: 'no',
a_fieldset: {
id_number: 123,
id_number: '123',
},
mandatory_group_array: 'no',
})
Expand All @@ -2328,7 +2328,7 @@ describe('createHeadlessForm', () => {
validateForm({
validate_tabs: 'yes',
a_fieldset: {
id_number: 123,
id_number: '123',
},
mandatory_group_array: 'no',
})
Expand All @@ -2344,7 +2344,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' }],
Expand All @@ -2355,7 +2355,7 @@ describe('createHeadlessForm', () => {
validateForm({
validate_tabs: 'yes',
a_fieldset: {
id_number: 123,
id_number: '123',
tabs: 2,
},
mandatory_group_array: 'no',
Expand Down Expand Up @@ -2446,7 +2446,7 @@ describe('createHeadlessForm', () => {
validateForm({
validate_fieldset: ['id_number'],
a_fieldset: {
id_number: 123,
id_number: '123',
},
})
).toBeUndefined();
Expand All @@ -2455,7 +2455,7 @@ describe('createHeadlessForm', () => {
validateForm({
validate_fieldset: ['id_number', 'all'],
a_fieldset: {
id_number: 123,
id_number: '123',
},
})
).toEqual({
Expand All @@ -2468,7 +2468,7 @@ describe('createHeadlessForm', () => {
validateForm({
validate_fieldset: ['id_number', 'all'],
a_fieldset: {
id_number: 123,
id_number: '123',
tabs: 2,
},
})
Expand All @@ -2483,7 +2483,7 @@ describe('createHeadlessForm', () => {
validateForm({
validate_fieldset: ['id_number'],
a_fieldset: {
id_number: 123,
id_number: '123',
},
})
).toBeUndefined();
Expand All @@ -2492,7 +2492,7 @@ describe('createHeadlessForm', () => {
validateForm({
validate_fieldset: ['id_number', 'all'],
a_fieldset: {
id_number: 123,
id_number: '123',
},
})
).toEqual({
Expand All @@ -2505,7 +2505,7 @@ describe('createHeadlessForm', () => {
validateForm({
validate_fieldset: ['id_number', 'all'],
a_fieldset: {
id_number: 123,
id_number: '123',
tabs: 2,
},
})
Expand Down
9 changes: 8 additions & 1 deletion src/yupSchema.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,16 @@ export const baseString = string().trim();
const todayDateHint = new Date().toISOString().substring(0, 10);
const convertBytesToKB = convertDiskSizeFromTo('Bytes', 'KB');
const convertKbBytesToMB = convertDiskSizeFromTo('KB', 'MB');
const useOnlyStrings = (value, originalValue) => {
if (typeof originalValue === 'string') {
return originalValue;
}

return null;
};

const yupSchemas = {
text: string().trim().nullable(),
text: string().transform(useOnlyStrings).trim().nullable(),
gabrielseco marked this conversation as resolved.
Show resolved Hide resolved
select: string().trim().nullable(),
radio: string().trim().nullable(),
date: string()
Expand Down