-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Closed
Labels
accepting prsGo ahead, send a pull request that resolves this issueGo ahead, send a pull request that resolves this issuelocked due to agePlease open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing.Please open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing.repo maintenancethings to do with maintenance of the repo, and not with code/docsthings to do with maintenance of the repo, and not with code/docs
Description
Before You File a Proposal Please Confirm You Have Done The Following...
- I have searched for related issues and found none that match my proposal.
- I have searched the current rule list and found no rules that match my proposal.
- I have read the FAQ and my problem is not listed.
Relevant Package
rule-tester
My proposal is suitable for this project
- I believe my proposal would be useful to the broader TypeScript community (meaning it is not a niche proposal).
Description
// packages/rule-tester/tests/RuleTester.test.ts
it('passes with no output', () => {
expect(() => {
ruleTester.run('my-rule', rule, {
invalid: [
{
code: 'foo',
errors: [{ messageId: 'error' }],
},
],
valid: [],
});
}).not.toThrow();
});
it('passes with null output', () => {
expect(() => {
ruleTester.run('my-rule', rule, {
invalid: [
{
code: 'foo',
errors: [{ messageId: 'error' }],
},
],
valid: [],
});
}).not.toThrow();
});
These two test cases have almost the same meaning and are written in exactly the same code.
- Remove duplicates
OR - If duplication is absolutely necessary, please state in a comment why it is necessary.
OR - write it clearly.
it('passes with null output', () => {
expect(() => {
ruleTester.run('my-rule', rule, {
invalid: [
{
code: 'foo',
errors: [{ messageId: 'error' }],
output: null,
},
],
valid: [],
});
}).not.toThrow();
});
Additional Info
❤
Is this an issue with repo maintainance
?
Metadata
Metadata
Assignees
Labels
accepting prsGo ahead, send a pull request that resolves this issueGo ahead, send a pull request that resolves this issuelocked due to agePlease open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing.Please open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing.repo maintenancethings to do with maintenance of the repo, and not with code/docsthings to do with maintenance of the repo, and not with code/docs