mirror of
https://github.com/idanoo/GoScrobble.git
synced 2024-11-24 09:25:15 +00:00
46 lines
992 B
Markdown
46 lines
992 B
Markdown
|
# Disallow using `exports` in files containing tests (`no-export`)
|
||
|
|
||
|
Prevents using `exports` if a file has one or more tests in it.
|
||
|
|
||
|
## Rule Details
|
||
|
|
||
|
This rule aims to eliminate duplicate runs of tests by exporting things from
|
||
|
test files. If you import from a test file, then all the tests in that file will
|
||
|
be run in each imported instance, so bottom line, don't export from a test, but
|
||
|
instead move helper functions into a separate file when they need to be shared
|
||
|
across tests.
|
||
|
|
||
|
Examples of **incorrect** code for this rule:
|
||
|
|
||
|
```js
|
||
|
export function myHelper() {}
|
||
|
|
||
|
module.exports = function () {};
|
||
|
|
||
|
module.exports = {
|
||
|
something: 'that should be moved to a non-test file',
|
||
|
};
|
||
|
|
||
|
describe('a test', () => {
|
||
|
expect(1).toBe(1);
|
||
|
});
|
||
|
```
|
||
|
|
||
|
Examples of **correct** code for this rule:
|
||
|
|
||
|
```js
|
||
|
function myHelper() {}
|
||
|
|
||
|
const myThing = {
|
||
|
something: 'that can live here',
|
||
|
};
|
||
|
|
||
|
describe('a test', () => {
|
||
|
expect(1).toBe(1);
|
||
|
});
|
||
|
```
|
||
|
|
||
|
## When Not To Use It
|
||
|
|
||
|
Don't use this rule on non-jest test files.
|