eslint-plugin-ember
eslint-plugin-ember copied to clipboard
ember/template-no-let-reference
Is it ok to disable this rule in tests? This is a contrived test case
let someThing;
hooks.beforeEach(function (assert) {
someThing = null;
});
test('foo', async function (assert) {
someThing = 'foo'
await render(<template>
<MyComponent @arg={{someThing}} />
</template>);
});
test('bar', async function (assert) {
someThing = 'bar'
await render(<template>
<MyComponent @arg={{someThing}} />
</template>);
});
Is it ok to disable this rule in tests?
yea, I'd say so.
@bmish should we have a config for tests?
Re-opening, as this could affect many
I'm opposed to adding additional configs. I'd like everything to work properly with simply the recommended config. We have other rules that bail-out in test files.