Skip to content

Commit 5211a2b

Browse files
committed
Add test for jsx-eslint#1865
1 parent 4d1b833 commit 5211a2b

File tree

1 file changed

+27
-0
lines changed

1 file changed

+27
-0
lines changed

tests/lib/rules/require-default-props.js

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -834,6 +834,19 @@ ruleTester.run('require-default-props', rule, {
834834
].join('\n'),
835835
parser: 'babel-eslint',
836836
options: [{forbidDefaultForRequired: true}]
837+
}, {
838+
code: `
839+
type Props = {
840+
+name?: string,
841+
};
842+
function Hello(props: Props) {
843+
return <div>Hello {props.name}</div>;
844+
}
845+
Hello.defaultProps = {
846+
name: 'foo'
847+
};
848+
`,
849+
parser: 'babel-eslint'
837850
}
838851
],
839852

@@ -2174,6 +2187,20 @@ ruleTester.run('require-default-props', rule, {
21742187
}, {
21752188
message: 'propType "bar" is not required, but has no corresponding defaultProp declaration.'
21762189
}]
2190+
},
2191+
{
2192+
code: `
2193+
type Props = {
2194+
+name?: string,
2195+
};
2196+
function Hello(props: Props) {
2197+
return <div>Hello {props.name}</div>;
2198+
}
2199+
`,
2200+
parser: 'babel-eslint',
2201+
errors: [{
2202+
message: 'propType "name" is not required, but has no corresponding defaultProp declaration.'
2203+
}]
21772204
}
21782205
]
21792206
});

0 commit comments

Comments
 (0)