Skip to content

Commit e441aae

Browse files
committed
Account for newline indents
- Adjust old cases to use default 2 indents - Add new case to use optional rule for 4 indents
1 parent 5265126 commit e441aae

File tree

1 file changed

+21
-5
lines changed

1 file changed

+21
-5
lines changed

tests/lib/rules/jsx-first-prop-new-line.js

Lines changed: 21 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@ ruleTester.run('jsx-first-prop-new-line', rule, {
155155
code: '<Foo prop="one" />',
156156
output: [
157157
'<Foo',
158-
' prop="one" />'
158+
' prop="one" />'
159159
].join('\n'),
160160
options: ['always'],
161161
errors: [{message: 'Property should be placed on a new line'}],
@@ -169,24 +169,40 @@ ruleTester.run('jsx-first-prop-new-line', rule, {
169169
].join('\n'),
170170
output: [
171171
'<Foo',
172-
' propOne="one"',
172+
' propOne="one"',
173173
' propTwo="two"',
174174
'/>'
175175
].join('\n'),
176176
options: ['always'],
177177
errors: [{message: 'Property should be placed on a new line'}],
178178
parser: parserOptions
179179
},
180+
{
181+
code: [
182+
' <Foo propOne="one"',
183+
' propTwo="two"',
184+
' />'
185+
].join('\n'),
186+
output: [
187+
' <Foo',
188+
' propOne="one"',
189+
' propTwo="two"',
190+
' />'
191+
].join('\n'),
192+
options: ['always', 4],
193+
errors: [{message: 'Property should be placed on a new line'}],
194+
parser: parserOptions
195+
},
180196
{
181197
code: [
182198
'<Foo',
183-
' propOne="one"',
184-
' propTwo="two"',
199+
' propOne="one"',
200+
' propTwo="two"',
185201
'/>'
186202
].join('\n'),
187203
output: [
188204
'<Foo propOne="one"',
189-
' propTwo="two"',
205+
' propTwo="two"',
190206
'/>'
191207
].join('\n'),
192208
options: ['never'],

0 commit comments

Comments
 (0)