Skip to content

Commit cfff9ce

Browse files
KingDarBojaJosh Goldberg
and
Josh Goldberg
authored
Add rxjs no-finnish converter (#1055)
Co-authored-by: Josh Goldberg <joshuakgoldberg@outlook.com>
1 parent 482cf1d commit cfff9ce

File tree

3 files changed

+32
-0
lines changed

3 files changed

+32
-0
lines changed

src/converters/lintConfigs/rules/ruleConverters.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -206,6 +206,7 @@ import { convertNoAsyncSubscribe } from "./ruleConverters/eslint-plugin-rxjs/no-
206206
import { convertNoImplicitAnyCatch } from "./ruleConverters/eslint-plugin-rxjs/no-implicit-any-catch";
207207
import { convertNoCreate } from "./ruleConverters/eslint-plugin-rxjs/no-create";
208208
import { convertNoExposedSubjects } from "./ruleConverters/eslint-plugin-rxjs/no-exposed-subjects";
209+
import { convertNoFinnish } from "./ruleConverters/eslint-plugin-rxjs/no-finnish";
209210
import { convertNoIgnoredNotifier } from "./ruleConverters/eslint-plugin-rxjs/no-ignored-notifier";
210211
import { convertNoIgnoredReplayBuffer } from "./ruleConverters/eslint-plugin-rxjs/no-ignored-replay-buffer";
211212
import { convertNoIgnoredSubscription } from "./ruleConverters/eslint-plugin-rxjs/no-ignored-subscription";
@@ -428,6 +429,7 @@ export const ruleConverters = new Map([
428429
["rxjs-no-implicit-any-catch", convertNoImplicitAnyCatch],
429430
["rxjs-no-create", convertNoCreate],
430431
["rxjs-no-exposed-subjects", convertNoExposedSubjects],
432+
["rxjs-no-finnish", convertNoFinnish],
431433
["rxjs-no-ignored-notifier", convertNoIgnoredNotifier],
432434
["rxjs-no-ignored-replay-buffer", convertNoIgnoredReplayBuffer],
433435
["rxjs-no-ignored-subscription", convertNoIgnoredSubscription],
Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
import { RuleConverter } from "../../ruleConverter";
2+
3+
export const convertNoFinnish: RuleConverter = () => {
4+
return {
5+
rules: [
6+
{
7+
ruleName: "rxjs/no-finnish",
8+
},
9+
],
10+
plugins: ["eslint-plugin-rxjs"],
11+
};
12+
};
Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
import { convertNoFinnish } from "../no-finnish";
2+
3+
describe(convertNoFinnish, () => {
4+
test("conversion without arguments", () => {
5+
const result = convertNoFinnish({
6+
ruleArguments: [],
7+
});
8+
9+
expect(result).toEqual({
10+
rules: [
11+
{
12+
ruleName: "rxjs/no-finnish",
13+
},
14+
],
15+
plugins: ["eslint-plugin-rxjs"],
16+
});
17+
});
18+
});

0 commit comments

Comments
 (0)