Add missing converter: newline-before-return (#203)

To #164

Author: da-snap

src/rules/converters.ts

@@ -25,6 +25,7 @@ import { convertMaxClassesPerFile } from "./converters/max-classes-per-file";
 import { convertMaxFileLineCount } from "./converters/max-file-line-count";
 import { convertMaxLineLength } from "./converters/max-line-length";
 import { convertMemberOrdering } from "./converters/member-ordering";
+import { convertNewlineBeforeReturn } from "./converters/newline-before-return";
 import { convertNewlinePerChainedCall } from "./converters/newline-per-chained-call";
 import { convertNewParens } from "./converters/new-parens";
 import { convertNoAngleBracketTypeAssertion } from "./converters/no-angle-bracket-type-assertion";
@@ -133,6 +134,7 @@ export const converters = new Map([
     ["member-access", convertMemberAccess],
     ["member-ordering", convertMemberOrdering],
     ["new-parens", convertNewParens],
+    ["newline-before-return", convertNewlineBeforeReturn],
     ["newline-per-chained-call", convertNewlinePerChainedCall],
     ["no-angle-bracket-type-assertion", convertNoAngleBracketTypeAssertion],
     ["no-any", convertNoExplicitAny],
@@ -232,7 +234,6 @@ export const converters = new Map([
     // TSLint core rules:
     // ["ban", convertBan], // no-restricted-properties
     // ["import-blacklist", convertImportBlacklist], // no-restricted-imports
-    // ["newline-before-return", convertNewlineBeforeReturn],
     // ["no-duplicate-variable", convertNoDuplicateVariable], // no-redeclare
     // ["no-shadowed-variable", convertNoShadowedVariable], // no-shadow
     // ["no-trailing-whitespace", convertNoTrailingWhitespace], //  no-trailing-spaces

src/rules/converters/newline-before-return.ts

@@ -0,0 +1,19 @@
+import { RuleConverter } from "../converter";
+
+export const convertNewlineBeforeReturn: RuleConverter = () => {
+    return {
+        rules: [
+            {
+                ruleName: "padding-line-between-statements",
+                ruleArguments: [
+                    "error",
+                    {
+                        blankLine: "always",
+                        prev: "*",
+                        next: "return",
+                    },
+                ],
+            },
+        ],
+    };
+};

src/rules/converters/tests/newline-before-return.test.ts

@@ -0,0 +1,25 @@
+import { convertNewlineBeforeReturn } from "../newline-before-return";
+
+describe(convertNewlineBeforeReturn, () => {
+    test("conversion without arguments", () => {
+        const result = convertNewlineBeforeReturn({
+            ruleArguments: [],
+        });
+
+        expect(result).toEqual({
+            rules: [
+                {
+                    ruleName: "padding-line-between-statements",
+                    ruleArguments: [
+                        "error",
+                        {
+                            blankLine: "always",
+                            next: "return",
+                            prev: "*",
+                        },
+                    ],
+                },
+            ],
+        });
+    });
+});