uses ruleArguments instead of getOptions()

palantir · Sep 29, 2017 · 688e168 · 688e168
1 parent 571c61f
commit 688e168
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/rules/noUnnecessaryTypeAssertionRule.ts b/src/rules/noUnnecessaryTypeAssertionRule.ts
@@ -42,12 +42,12 @@ export class Rule extends Lint.Rules.TypedRule {
     public static FAILURE_STRING = "This assertion is unnecessary since it does not change the type of the expression.";
 
     public applyWithProgram(sourceFile: ts.SourceFile, program: ts.Program): Lint.RuleFailure[] {
-        return this.applyWithWalker(new Walker(sourceFile, this.ruleName, this.getOptions(), program.getTypeChecker()));
+        return this.applyWithWalker(new Walker(sourceFile, this.ruleName, this.ruleArguments, program.getTypeChecker()));
     }
 }
 
-class Walker extends Lint.AbstractWalker<Lint.IOptions> {
-    constructor(sourceFile: ts.SourceFile, ruleName: string, options: Lint.IOptions, private readonly checker: ts.TypeChecker) {
+class Walker extends Lint.AbstractWalker<string[]> {
+    constructor(sourceFile: ts.SourceFile, ruleName: string, options: string[], private readonly checker: ts.TypeChecker) {
         super(sourceFile, ruleName, options);
     }
 
@@ -67,7 +67,7 @@ class Walker extends Lint.AbstractWalker<Lint.IOptions> {
     }
 
     private verifyCast(node: ts.TypeAssertion | ts.NonNullExpression | ts.AsExpression) {
-        if (isAssertionExpression(node) && this.options.ruleArguments.indexOf(node.type.getText(this.sourceFile)) !== -1) {
+        if (isAssertionExpression(node) && this.options.indexOf(node.type.getText(this.sourceFile)) !== -1) {
             return;
         }
         const castType = this.checker.getTypeAtLocation(node);