Skip to content

Commit 2449b44

Browse files
lourencignapse
andauthored
chore: Disable max-lines-per-function for tests files (#198)
Co-authored-by: Ernesto García <[email protected]>
1 parent 5e39222 commit 2449b44

File tree

5 files changed

+14
-14
lines changed

5 files changed

+14
-14
lines changed

package.json

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,15 @@
5252
"extends": "./node_modules/kcd-scripts/eslint.js",
5353
"rules": {
5454
"babel/no-invalid-this": "off"
55-
}
55+
},
56+
"overrides": [
57+
{
58+
"files": ["src/__tests__/*.js"],
59+
"rules": {
60+
"max-lines-per-function": "off"
61+
}
62+
}
63+
]
5664
},
5765
"eslintIgnore": [
5866
"node_modules",

src/__tests__/to-be-checked.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -194,5 +194,3 @@ describe('.toBeChecked', () => {
194194
)
195195
})
196196
})
197-
198-
/* eslint max-lines-per-function:0 */

src/__tests__/to-be-visible.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import {render} from './helpers/test-utils'
22

3-
// eslint-disable-next-line max-lines-per-function
43
describe('.toBeVisible', () => {
54
it('returns the visibility of an element', () => {
65
const {container} = render(`
@@ -36,7 +35,6 @@ describe('.toBeVisible', () => {
3635
).toThrowError()
3736
})
3837

39-
// eslint-disable-next-line max-lines-per-function
4038
describe('with a <details /> element', () => {
4139
let subject
4240

src/__tests__/to-have-form-values.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -349,5 +349,3 @@ function renderSelectMultiple(name, label, options, value = []) {
349349
function renderList(items, mapper) {
350350
return items.map(mapper).join('')
351351
}
352-
353-
/* eslint max-lines-per-function:0 */

src/__tests__/to-have-value.js

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -50,20 +50,20 @@ describe('.toHaveValue', () => {
5050
test('handles value of select element', () => {
5151
const {queryByTestId} = render(`
5252
<select data-testid="single">
53-
<option value="first">First Value</option>
53+
<option value="first">First Value</option>
5454
<option value="second" selected>Second Value</option>
5555
<option value="third">Third Value</option>
5656
</select>
57-
57+
5858
<select data-testid="multiple" multiple>
59-
<option value="first">First Value</option>
59+
<option value="first">First Value</option>
6060
<option value="second" selected>Second Value</option>
6161
<option value="third" selected>Third Value</option>
6262
</select>
63-
63+
6464
<select data-testid="not-selected" >
6565
<option value="" disabled selected>- Select some value - </option>
66-
<option value="first">First Value</option>
66+
<option value="first">First Value</option>
6767
<option value="second">Second Value</option>
6868
<option value="third">Third Value</option>
6969
</select>
@@ -204,5 +204,3 @@ describe('.toHaveValue', () => {
204204
`)
205205
})
206206
})
207-
208-
/* eslint max-lines-per-function:0 */

0 commit comments

Comments
 (0)