Skip to content

Commit c9f27b7

Browse files
kbrillaKrzysztof BrillaJoshuaKGoldberg
authored
Adding the rule, mapping it and writing the tests (#802)
Co-authored-by: Krzysztof Brilla <brilla.krzysztof@gmail.com> Co-authored-by: Josh Goldberg <me@joshuakgoldberg.com>
1 parent a250fad commit c9f27b7

File tree

3 files changed

+33
-0
lines changed

3 files changed

+33
-0
lines changed

src/converters/lintConfigs/rules/ruleConverters.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -193,11 +193,13 @@ import { convertNoInternal } from "./ruleConverters/eslint-plugin-rxjs/no-intern
193193
import { convertNoNestedSubscribe } from "./ruleConverters/eslint-plugin-rxjs/no-nested-subscribe";
194194
import { convertNoRedundantNotify } from "./ruleConverters/eslint-plugin-rxjs/no-redundant-notify";
195195
import { convertNoShareReplay } from "./ruleConverters/eslint-plugin-rxjs/no-sharereplay";
196+
import { convertNoSubjectUnubscribe } from "./ruleConverters/eslint-plugin-rxjs/no-subject-unsubscribe";
196197
import { convertNoSubjectValue } from "./ruleConverters/eslint-plugin-rxjs/no-subject-value";
197198
import { convertNoUnboundMethods } from "./ruleConverters/eslint-plugin-rxjs/no-unbound-methods";
198199
import { convertNoUnsafeSubjectNext } from "./ruleConverters/eslint-plugin-rxjs/no-unsafe-subject-next";
199200

200201
/**
202+
* Keys TSLint rule names to their ESLint rule converters.
201203
*/
202204
export const ruleConverters = new Map([
203205
["adjacent-overload-signatures", convertAdjacentOverloadSignatures],
@@ -390,6 +392,7 @@ export const ruleConverters = new Map([
390392
["rxjs-no-nested-subscribe", convertNoNestedSubscribe],
391393
["rxjs-no-redundant-notify", convertNoRedundantNotify],
392394
["rxjs-no-sharereplay", convertNoShareReplay],
395+
["rxjs-no-subject-unsubscribe", convertNoSubjectUnubscribe],
393396
["rxjs-no-subject-value", convertNoSubjectValue],
394397
["rxjs-no-unbound-methods", convertNoUnboundMethods],
395398
["rxjs-no-unsafe-subject-next", convertNoUnsafeSubjectNext],
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 convertNoSubjectUnubscribe: RuleConverter = () => {
4+
return {
5+
rules: [
6+
{
7+
ruleName: "rxjs/no-subject-unsubscribe",
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 { convertNoSubjectUnubscribe } from "../no-subject-unsubscribe";
2+
3+
describe(convertNoSubjectUnubscribe, () => {
4+
test("conversion without arguments", () => {
5+
const result = convertNoSubjectUnubscribe({
6+
ruleArguments: [],
7+
});
8+
9+
expect(result).toEqual({
10+
rules: [
11+
{
12+
ruleName: "rxjs/no-subject-unsubscribe",
13+
},
14+
],
15+
plugins: ["eslint-plugin-rxjs"],
16+
});
17+
});
18+
});

0 commit comments

Comments
 (0)