Add more documentation and tests

palantir · Jan 22, 2017 · c384109 · c384109
1 parent 5b31066
commit c384109
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 0 deletions.
diff --git a/src/rules/noLetUndefinedRule.ts b/src/rules/noLetUndefinedRule.ts
@@ -24,6 +24,7 @@ export class Rule extends Lint.Rules.AbstractRule {
     public static metadata: Lint.IRuleMetadata = {
         ruleName: "no-let-undefined",
         description: "Forbids a 'let' statement to be initialized to 'undefined'.",
+        hasFix: true,
         optionsDescription: "Not configurable.",
         options: null,
         optionExamples: ["true"],

diff --git a/test/rules/no-let-undefined/test.ts.fix b/test/rules/no-let-undefined/test.ts.fix
@@ -4,3 +4,7 @@ for (let y: number | undefined; y < 2; y++) {}
 
 let z;
 
+const x = undefined;
+
+function f(x: string | undefined = undefined) {}
+
diff --git a/test/rules/no-let-undefined/test.ts.lint b/test/rules/no-let-undefined/test.ts.lint
@@ -6,4 +6,8 @@ for (let y: number | undefined = undefined; y < 2; y++) {}
 
 let z;
 
+const x = undefined;
+
+function f(x: string | undefined = undefined) {}
+
 [0]: Unnecessary initialization to 'undefined'.
Original file line number	Diff line number	Diff line change
Expand Up		@@ -4,3 +4,7 @@ for (let y: number \| undefined; y < 2; y++) {}

		let z;

		const x = undefined;

		function f(x: string \| undefined = undefined) {}