Skip to content

Commit f74e30c

Browse files
committed
refactor(jsx-no-leaked-render): move tests options after code
1 parent b2e45f0 commit f74e30c

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

tests/lib/rules/jsx-no-leaked-render.js

+8-8
Original file line numberDiff line numberDiff line change
@@ -93,52 +93,51 @@ ruleTester.run('jsx-no-leaked-render', rule, {
9393
`,
9494
},
9595
{
96-
options: [{ validStrategies: ['ternary'] }],
9796
code: `
9897
const Component = ({ elements, count }) => {
9998
return <div>{count ? <List elements={elements}/> : null}</div>
10099
}
101100
`,
101+
options: [{ validStrategies: ['ternary'] }],
102102
},
103103
{
104-
options: [{ validStrategies: ['coerce'] }],
105104
code: `
106105
const Component = ({ elements, count }) => {
107106
return <div>{!!count && <List elements={elements}/>}</div>
108107
}
109108
`,
109+
options: [{ validStrategies: ['coerce'] }],
110110
},
111111
{
112-
options: [{ validStrategies: ['coerce', 'ternary'] }],
113112
code: `
114113
const Component = ({ elements, count }) => {
115114
return <div>{count ? <List elements={elements}/> : null}</div>
116115
}
117116
`,
117+
options: [{ validStrategies: ['coerce', 'ternary'] }],
118118
},
119119
{
120-
options: [{ validStrategies: ['coerce', 'ternary'] }],
121120
code: `
122121
const Component = ({ elements, count }) => {
123122
return <div>{!!count && <List elements={elements}/>}</div>
124123
}
125124
`,
125+
options: [{ validStrategies: ['coerce', 'ternary'] }],
126126
},
127127
{
128-
options: [{ validStrategies: ['coerce'] }],
129128
code: `
130129
const Component = ({ elements, count }) => {
131130
return <div>{!!count && <List elements={elements}/>}</div>
132131
}
133132
`,
133+
options: [{ validStrategies: ['coerce'] }],
134134
},
135135

136136
// Fixes for:
137137
// - https://github.com/jsx-eslint/eslint-plugin-react/issues/3292
138138
// - https://github.com/jsx-eslint/eslint-plugin-react/issues/3297
139139
{
140140
// It shouldn't delete valid alternate from ternary expressions when "coerce" is the only valid strategy
141-
options: [{ validStrategies: ['coerce'] }],
142141
code: `
143142
const Component = ({ elements, count }) => {
144143
return (
@@ -149,19 +148,19 @@ ruleTester.run('jsx-no-leaked-render', rule, {
149148
)
150149
}
151150
`,
151+
options: [{ validStrategies: ['coerce'] }],
152152
},
153153
{
154154
// It shouldn't delete valid branches from ternary expressions when ["coerce", "ternary"] are only valid strategies
155-
options: [{ validStrategies: ['coerce', 'ternary'] }],
156155
code: `
157156
const Component = ({ elements, count }) => {
158157
return <div>{direction ? (direction === "down" ? "▼" : "▲") : ""}</div>
159158
}
160159
`,
160+
options: [{ validStrategies: ['coerce', 'ternary'] }],
161161
},
162162
{
163163
// It shouldn't report nested logical expressions when "coerce" is the only valid strategy
164-
options: [{ validStrategies: ['coerce'] }],
165164
code: `
166165
const Component = ({ direction }) => {
167166
return (
@@ -174,6 +173,7 @@ ruleTester.run('jsx-no-leaked-render', rule, {
174173
)
175174
}
176175
`,
176+
options: [{ validStrategies: ['coerce'] }],
177177
},
178178
]),
179179

0 commit comments

Comments
 (0)