Skip to content

Commit 778bc74

Browse files
committed
Update tests
1 parent 0d450d1 commit 778bc74

39 files changed

+2234
-8382
lines changed

packages/ant-component-mapper/src/tests/components.test.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,6 @@ describe('formFields generated tests', () => {
107107
const errorField = {
108108
...field,
109109
validate: [{ type: validatorTypes.REQUIRED, warning: true }],
110-
useWarnings: true,
111110
validateOnMount: true
112111
};
113112
let wrapper;

packages/ant-component-mapper/src/tests/dual-list-select.test.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,6 +123,8 @@ describe('DualListSelect', () => {
123123
wrapper.find('form').simulate('submit');
124124
});
125125

126-
expect(onSubmit).toHaveBeenCalledWith({});
126+
expect(onSubmit).toHaveBeenCalledWith({
127+
'dual-Menu': []
128+
});
127129
});
128130
});

packages/blueprint-component-mapper/src/tests/components.test.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,6 @@ describe('formFields generated tests', () => {
112112
const errorField = {
113113
...field,
114114
validate: [{ type: validatorTypes.REQUIRED, warning: true }],
115-
useWarnings: true,
116115
validateOnMount: true
117116
};
118117
let wrapper;

packages/blueprint-component-mapper/src/tests/dual-list-select.test.js

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -254,7 +254,9 @@ describe('DualListSelect', () => {
254254
wrapper.find('form').simulate('submit');
255255
});
256256

257-
expect(onSubmit).toHaveBeenCalledWith({});
257+
expect(onSubmit).toHaveBeenCalledWith({
258+
'dual-Menu': []
259+
});
258260
});
259261

260262
it('switch all to right', async () => {
@@ -288,7 +290,9 @@ describe('DualListSelect', () => {
288290
wrapper.find('form').simulate('submit');
289291
});
290292

291-
expect(onSubmit).toHaveBeenCalledWith({});
293+
expect(onSubmit).toHaveBeenCalledWith({
294+
'dual-Menu': []
295+
});
292296
});
293297

294298
it('filters options', async () => {

packages/carbon-component-mapper/src/tests/components.test.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,6 @@ describe('component tests', () => {
103103
const errorField = {
104104
...field,
105105
validate: [{ type: validatorTypes.REQUIRED, warning: true }],
106-
useWarnings: true,
107106
validateOnMount: true
108107
};
109108
let wrapper;

packages/carbon-component-mapper/src/tests/dual-list-select.test.js

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -249,7 +249,9 @@ describe('DualListSelect', () => {
249249
wrapper.find('form').simulate('submit');
250250
});
251251

252-
expect(onSubmit).toHaveBeenCalledWith({});
252+
expect(onSubmit).toHaveBeenCalledWith({
253+
'dual-StructuredListWrapper': []
254+
});
253255
});
254256

255257
it('switch all to right', async () => {
@@ -283,7 +285,9 @@ describe('DualListSelect', () => {
283285
wrapper.find('form').simulate('submit');
284286
});
285287

286-
expect(onSubmit).toHaveBeenCalledWith({});
288+
expect(onSubmit).toHaveBeenCalledWith({
289+
'dual-StructuredListWrapper': []
290+
});
287291
});
288292

289293
it('filters options', async () => {

packages/form-state-manager/src/tests/files/use-field.test.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -551,6 +551,7 @@ describe('useField', () => {
551551
expect(checkEmpty(false)).toEqual(false);
552552
expect(checkEmpty('A')).toEqual(false);
553553
expect(checkEmpty(0)).toEqual(false);
554+
expect(checkEmpty(new Date())).toEqual(false);
554555

555556
expect(checkEmpty([])).toEqual(true);
556557
expect(checkEmpty('')).toEqual(true);

packages/form-state-manager/src/tests/utils/manager-api.test.js

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -573,7 +573,7 @@ describe('managerApi', () => {
573573
array: [{ name: 'array[0].name', age: 'array[0].age' }]
574574
};
575575
const onSubmit = jest.fn();
576-
const managerApi = createManagerApi({ onSubmit });
576+
const managerApi = createManagerApi({ onSubmit: (values) => onSubmit(values) });
577577
const { registerField, change } = managerApi();
578578
registerField({ name: 'field', render: jest.fn() });
579579
registerField({ name: 'nested.name', render: jest.fn() });
@@ -765,7 +765,7 @@ describe('managerApi', () => {
765765
it('should send initialvalues to submit', () => {
766766
const onSubmit = jest.fn();
767767
const initialValues = { field: 'value1', nested: { level: 'value2' } };
768-
const managerApi = createManagerApi({ onSubmit, initialValues });
768+
const managerApi = createManagerApi({ onSubmit: (values) => onSubmit(values), initialValues });
769769

770770
managerApi().handleSubmit({ preventDefault: jest.fn() });
771771

@@ -808,7 +808,7 @@ describe('managerApi', () => {
808808
it('global subscription', () => {
809809
const managerApi = createManagerApi({ subscription: { valid: true } });
810810

811-
managerApi().registerField({ ...field1 });
811+
managerApi().registerField({ ...field1, subscription: { valid: true } }, false, true);
812812
managerApi().registerField({ ...field2 });
813813

814814
expect(renderField1).not.toHaveBeenCalled();
@@ -822,7 +822,7 @@ describe('managerApi', () => {
822822
managerApi().rerender(['valid']);
823823

824824
expect(renderField1).toHaveBeenCalled();
825-
expect(renderField2).toHaveBeenCalled();
825+
expect(renderField2).not.toHaveBeenCalled();
826826
});
827827

828828
it('field subscription only on valid', () => {
@@ -847,7 +847,7 @@ describe('managerApi', () => {
847847
expect(renderField2).toHaveBeenCalled();
848848
});
849849

850-
it('merge global and field subscription correctly', () => {
850+
it('do not render when global subscription is set', () => {
851851
const managerApi = createManagerApi({ subscription: { validating: true } });
852852

853853
managerApi().registerField({ ...field1 });
@@ -858,8 +858,8 @@ describe('managerApi', () => {
858858

859859
managerApi().rerender(['validating']);
860860

861-
expect(renderField1).toHaveBeenCalled();
862-
expect(renderField2).toHaveBeenCalled();
861+
expect(renderField1).not.toHaveBeenCalled();
862+
expect(renderField2).not.toHaveBeenCalled();
863863

864864
renderField1.mockReset();
865865
renderField2.mockReset();

packages/mui-component-mapper/src/tests/dual-list-select.test.js

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -257,7 +257,9 @@ describe('DualListSelect', () => {
257257
wrapper.find('form').simulate('submit');
258258
});
259259

260-
expect(onSubmit).toHaveBeenCalledWith({});
260+
expect(onSubmit).toHaveBeenCalledWith({
261+
'dual-list': []
262+
});
261263
});
262264

263265
it('switch all to right', async () => {
@@ -291,7 +293,9 @@ describe('DualListSelect', () => {
291293
wrapper.find('form').simulate('submit');
292294
});
293295

294-
expect(onSubmit).toHaveBeenCalledWith({});
296+
expect(onSubmit).toHaveBeenCalledWith({
297+
'dual-list': []
298+
});
295299
});
296300

297301
it('filters options', async () => {

packages/mui-component-mapper/src/tests/form-fields.test.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -102,8 +102,7 @@ describe('formFields', () => {
102102
it('renders with warning', async () => {
103103
const errorField = {
104104
...field,
105-
validate: [{ type: validatorTypes.REQUIRED, warning: true }],
106-
useWarnings: true
105+
validate: [{ type: validatorTypes.REQUIRED, warning: true }]
107106
};
108107
let wrapper;
109108

0 commit comments

Comments
 (0)