Skip to content

Commit 4a90061

Browse files
committed
Holy lint violations, batman
1 parent b9443ae commit 4a90061

4 files changed

Lines changed: 49 additions & 2 deletions

File tree

test-app/.eslintrc.js

Lines changed: 31 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,8 +49,38 @@ module.exports = {
4949
},
5050
{
5151
// test files
52-
files: ['tests/**/*-test.{js,ts}'],
52+
files: ['tests/**/*'],
5353
extends: ['plugin:qunit/recommended'],
54+
rules: {
55+
// TODO items from upgrades (but also, test-helpers may need to support all these things)
56+
57+
// eslint-plugin-ember
58+
'ember/avoid-leaking-state-in-ember-objects': 'off',
59+
'ember/new-module-imports': 'off',
60+
'ember/no-actions-hash': 'off',
61+
'ember/no-classic-classes': 'off',
62+
'ember/no-classic-components': 'off',
63+
'ember/no-ember-testing-in-module-scope': 'off',
64+
'ember/no-empty-glimmer-component-classes': 'off',
65+
'ember/no-get': 'off',
66+
'ember/require-super-in-lifecycle-hooks': 'off',
67+
'ember/no-global-jquery': 'off',
68+
'ember/no-jquery': 'off',
69+
'ember/no-legacy-test-waiters': 'off',
70+
'ember/no-pause-test': 'off',
71+
'ember/no-settled-after-test-helper': 'off',
72+
'ember/require-valid-css-selector-in-test-helpers': 'off',
73+
74+
// eslint-plugin-qunit
75+
'qunit/no-assert-equal': 'off',
76+
'qunit/no-assert-equal-boolean': 'off',
77+
'qunit/no-conditional-assertions': 'off',
78+
'qunit/no-early-return': 'off',
79+
'qunit/no-identical-names': 'off',
80+
'qunit/no-negated-ok': 'off',
81+
'qunit/require-expect': 'off',
82+
'qunit/resolve-async': 'off',
83+
},
5484
},
5585
],
5686
};

test-app/.template-lintrc.js

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,20 @@
22

33
module.exports = {
44
extends: 'recommended',
5+
overrides: [
6+
{
7+
files: ['tests/**/*'],
8+
rules: {
9+
// These probably don't need to be enabled for tests
10+
'no-action': 'off',
11+
'no-curly-component-invocation': 'off',
12+
'no-empty-headings': 'off',
13+
'no-inline-styles': 'off',
14+
'no-link-to-positional-params': 'off',
15+
'no-unnecessary-component-helper': 'off',
16+
'require-button-type': 'off',
17+
'require-input-label': 'off',
18+
},
19+
},
20+
],
521
};

test-app/tests/helpers/events.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
import { get } from '@ember/object';
2+
import QUnit from 'qunit';
23

34
// from https://mdn.mozilla.org/en-US/docs/Web/Events
45
export const KNOWN_EVENTS = Object.freeze([

test-app/tests/unit/setup-application-context-test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { module, test } from 'qunit';
1+
import QUnit, { module, test } from 'qunit';
22
import EmberRouter from '@ember/routing/router';
33
import Route from '@ember/routing/route';
44
import Service from '@ember/service';

0 commit comments

Comments
 (0)