Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(utils): add dependency constraint filtering for RuleTester #5750

Merged
merged 5 commits into from Oct 9, 2022
+1,058 −135
Merged

feat(utils): add dependency constraint filtering for RuleTester #5750

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
74e5ba7
feat(utils): add dependency constraint filtering for eslint plugin Ru…
bradzacher Oct 6, 2022
6bb23c3
address feedback and add more tests
bradzacher Oct 7, 2022
3e5249e
Update packages/utils/src/eslint-utils/rule-tester/dependencyConstrai…
bradzacher Oct 9, 2022
42ae133
Update packages/utils/src/eslint-utils/rule-tester/RuleTester.ts
bradzacher Oct 9, 2022
98b6eb4
Merge branch 'main' into rule-tester-dependency-constraints
bradzacher Oct 9, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
1 change: 1 addition & 0 deletions packages/eslint-plugin-internal/jest.config.js
View file
Open in desktop
Expand Up @@ -4,4 +4,5 @@
/** @type {import('@jest/types').Config.InitialOptions} */
module.exports = {
...require('../../jest.config.base.js'),
coveragePathIgnorePatterns: ['src/index.ts$', 'src/configs/.*.ts$'],
};
1 change: 1 addition & 0 deletions packages/eslint-plugin/jest.config.js
View file
Open in desktop
Expand Up @@ -4,4 +4,5 @@
/** @type {import('@jest/types').Config.InitialOptions} */
module.exports = {
...require('../../jest.config.base.js'),
coveragePathIgnorePatterns: ['src/index.ts$', 'src/configs/.*.ts$'],
};
4 changes: 3 additions & 1 deletion packages/utils/package.json
View file
Open in desktop
Expand Up @@ -44,12 +44,14 @@
"@typescript-eslint/types": "5.39.0",
"@typescript-eslint/typescript-estree": "5.39.0",
"eslint-scope": "^5.1.1",
"eslint-utils": "^3.0.0"
"eslint-utils": "^3.0.0",
"semver": "^7.3.7"
},
"peerDependencies": {
"eslint": "^6.0.0 || ^7.0.0 || ^8.0.0"
},
"devDependencies": {
"@typescript-eslint/parser": "5.39.0",
"typescript": "*"
},
"funding": {
Expand Down
130 changes: 0 additions & 130 deletions packages/utils/src/eslint-utils/RuleTester.ts
View file
Open in desktop

This file was deleted.

5 changes: 4 additions & 1 deletion packages/utils/src/eslint-utils/batchedSingleLineTests.ts
View file
Open in desktop
@@ -1,4 +1,7 @@
import type { InvalidTestCase, ValidTestCase } from '../ts-eslint';
import type {
InvalidTestCase,
ValidTestCase,
} from '../eslint-utils/rule-tester/RuleTester';

/**
* Converts a batch of single line tests into a number of separate test cases.
Expand Down
2 changes: 1 addition & 1 deletion packages/utils/src/eslint-utils/index.ts
View file
Open in desktop
Expand Up @@ -3,6 +3,6 @@ export * from './batchedSingleLineTests';
export * from './getParserServices';
export * from './InferTypesFromRule';
export * from './RuleCreator';
export * from './RuleTester';
export * from './rule-tester/RuleTester';
export * from './deepMerge';
export * from './nullThrows';