From b52f3b8cb43d888d132d2a08773a316733b67e2d Mon Sep 17 00:00:00 2001 From: Kenneth Chung Date: Mon, 3 Oct 2016 22:59:43 -0700 Subject: [PATCH] jsx-max-props-per-line: Add additional test cases Make sure it works when a prop is on the same line as the start line of the tag, as well as with spread props that spans multiple lines. --- tests/lib/rules/jsx-max-props-per-line.js | 44 +++++++++++++++++++++++ 1 file changed, 44 insertions(+) diff --git a/tests/lib/rules/jsx-max-props-per-line.js b/tests/lib/rules/jsx-max-props-per-line.js index b322c367f9..55ce2d2e79 100644 --- a/tests/lib/rules/jsx-max-props-per-line.js +++ b/tests/lib/rules/jsx-max-props-per-line.js @@ -38,6 +38,10 @@ ruleTester.run('jsx-max-props-per-line', rule, { code: '', options: [{when: 'multiline'}], parserOptions: parserOptions + }, { + code: '', + options: [{when: 'multiline'}], + parserOptions: parserOptions }, { code: '', options: [{maximum: 2, when: 'multiline'}], @@ -109,6 +113,46 @@ ruleTester.run('jsx-max-props-per-line', rule, { ].join('\n'), errors: [{message: 'Prop `bar` must be placed on a new line'}], parserOptions: parserOptions + }, { + code: [ + '' + ].join('\n'), + errors: [{message: 'Prop `bar` must be placed on a new line'}], + parserOptions: parserOptions + }, { + code: [ + '' + ].join('\n'), + options: [{maximum: 2}], + errors: [{message: 'Prop `baz` must be placed on a new line'}], + parserOptions: parserOptions + }, { + code: [ + '' + ].join('\n'), + errors: [{message: 'Prop `rest` must be placed on a new line'}], + parserOptions: parserOptions + }, { + code: [ + '' + ].join('\n'), + errors: [{message: 'Prop `bar` must be placed on a new line'}], + parserOptions: parserOptions + }, { + code: [ + '' + ].join('\n'), + errors: [{message: 'Prop `rest` must be placed on a new line'}], + parserOptions: parserOptions }, { code: [ '