Skip to content

Commit 3a26ed3

Browse files
committed
Add tests
1 parent f8fcd75 commit 3a26ed3

File tree

6 files changed

+1002
-485
lines changed

6 files changed

+1002
-485
lines changed

packages/pf4-component-mapper/src/form-fields/fieldArray/index.js

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,11 @@ ArrayItem.propTypes = {
6565
fieldIndex: PropTypes.number.isRequired,
6666
fields: PropTypes.arrayOf(PropTypes.object),
6767
remove: PropTypes.func.isRequired,
68-
formOptions: PropTypes.object.isRequired,
68+
formOptions: PropTypes.shape({
69+
renderForm: PropTypes.func.isRequired,
70+
}).isRequired,
71+
length: PropTypes.number,
72+
minItems: PropTypes.number,
6973
};
7074

7175
const DynamicArray = ({
@@ -76,7 +80,6 @@ const DynamicArray = ({
7680
defaultItem,
7781
formOptions,
7882
meta,
79-
reactFinalForm,
8083
FieldArrayProvider,
8184
minItems,
8285
maxItems,
@@ -128,11 +131,16 @@ DynamicArray.propTypes = {
128131
arrayValidator: PropTypes.func,
129132
label: PropTypes.node,
130133
description: PropTypes.node,
131-
fields: PropTypes.arrayOf(PropTypes.object),
134+
fields: PropTypes.arrayOf(PropTypes.object).isRequired,
132135
defaultItem: PropTypes.any,
133136
minItems: PropTypes.number,
134137
maxItems: PropTypes.number,
135138
noItemsMessage: PropTypes.node,
139+
formOptions: PropTypes.shape({
140+
renderForm: PropTypes.func.isRequired,
141+
}).isRequired,
142+
FieldArrayProvider: PropTypes.node.isRequired,
143+
meta: PropTypes.object.isRequired,
136144
};
137145

138146
DynamicArray.defaultProps = {

0 commit comments

Comments
 (0)