Added options for classes to no-floating-promises

Users can now specify rules that are considered to be promises, such as
`"JQueryPromise"`.

src/rules/noFloatingPromisesRule.ts

@@ -24,9 +24,17 @@ export class Rule extends Lint.Rules.TypedRule {
     public static metadata: Lint.IRuleMetadata = {
         ruleName: "no-floating-promises",
         description: "Promises returned by functions must be handled appropriately.",
-        optionsDescription: "Not configurable.",
-        options: null,
-        optionExamples: ["true"],
+        optionsDescription: Lint.Utils.dedent`
+            A list of \'string\' names of any additional classes that should also be handled as Promises.
+        `,
+        options: {
+            type: "list",
+            listType: {
+                type: "array",
+                items: {type: "string"},
+            },
+        },
+        optionExamples: ["true", `[true, "JQueryPromise"]`],
         type: "functionality",
         requiresTypeInfo: true,
     };
@@ -35,7 +43,13 @@ export class Rule extends Lint.Rules.TypedRule {
     public static FAILURE_STRING = "Promises must be handled appropriately";
 
     public applyWithProgram(sourceFile: ts.SourceFile, program: ts.Program): Lint.RuleFailure[] {
-        return this.applyWithWalker(new NoFloatingPromisesWalker(sourceFile, this.getOptions(), program));
+        const walker = new NoFloatingPromisesWalker(sourceFile, this.getOptions(), program);
+
+        for (const className of this.getOptions().ruleArguments) {
+            walker.addPromiseClass(className);
+        }
+
+        return this.applyWithWalker(walker);
     }
 }
 
@@ -45,6 +59,12 @@ class NoFloatingPromisesWalker extends Lint.ProgramAwareRuleWalker {
         [ts.SyntaxKind.VariableDeclaration]: true,
     };
 
+    private promiseClasses = ["Promise"];
+
+    public addPromiseClass(className: string) {
+        this.promiseClasses.push(className);
+    }
+
     public visitCallExpression(node: ts.CallExpression): void {
         this.checkCallExpression(node);
         super.visitCallExpression(node);
@@ -68,7 +88,7 @@ class NoFloatingPromisesWalker extends Lint.ProgramAwareRuleWalker {
             return false;
         }
 
-        return symbol.name === "Promise";
+        return this.promiseClasses.indexOf(symbol.name) !== -1;
     }
 
     private kindCanContainPromise(kind: ts.SyntaxKind) {

test/rules/no-floating-promises/jquerypromise/test.ts.lint

@@ -0,0 +1,17 @@
+class Promise { }
+class JQueryPromise { }
+class NotAPromise { }
+
+const returnsPromise = () => new Promise();
+const returnsJQueryPromise = () => new JQueryPromise();
+const returnsNotAPromise = () => new NotAPromise();
+
+returnsPromise();
+~~~~~~~~~~~~~~~~       [0]
+
+returnsJQueryPromise();
+~~~~~~~~~~~~~~~~~~~~~~       [0]
+
+returnsNotAPromise();
+
+[0]: Promises must be handled appropriately

test/rules/no-floating-promises/jquerypromise/tslint.json

@@ -0,0 +1,8 @@
+{
+  "linterOptions": {
+    "typeCheck": true
+  },
+  "rules": {
+    "no-floating-promises": [true, "JQueryPromise"]
+  }
+}