Fix returning promise from complete() method

Kapil Borle · daviwil · commit 2cfb13da7a51 · 2017-05-17T21:31:07.000-07:00
diff --git a/src/features/HelpCompletion.ts b/src/features/HelpCompletion.ts
@@ -51,8 +51,7 @@ export class HelpCompletionFeature implements IFeature {
 
         // todo raise an event when trigger is found, and attach complete() to the event.
         if (this.helpCompletionProvider.triggerFound) {
-            this.helpCompletionProvider.reset();
-            this.helpCompletionProvider.complete();
+            this.helpCompletionProvider.complete().then(() => this.helpCompletionProvider.reset());
         }
 
     }
@@ -148,7 +147,7 @@ class HelpCompletionProvider {
         let triggerStartPos = this.lastChangeRange.start;
         let triggerEndPos = this.lastChangeRange.end;
         let doc = this.lastDocument;
-        this.langClient.sendRequest(
+        return this.langClient.sendRequest(
             CommentHelpRequest.type,
             {
                 documentUri: doc.uri.toString(),

Original file line number	Diff line number	Diff line change
`@@ -51,8 +51,7 @@ export class HelpCompletionFeature implements IFeature {`
`51`	`51`
`52`	`52`	`// todo raise an event when trigger is found, and attach complete() to the event.`
`53`	`53`	`if (this.helpCompletionProvider.triggerFound) {`
`54`		`- this.helpCompletionProvider.reset();`
`55`		`- this.helpCompletionProvider.complete();`
	`54`	`+ this.helpCompletionProvider.complete().then(() => this.helpCompletionProvider.reset());`
`56`	`55`	`}`
`57`	`56`
`58`	`57`	`}`
`@@ -148,7 +147,7 @@ class HelpCompletionProvider {`
`148`	`147`	`let triggerStartPos = this.lastChangeRange.start;`
`149`	`148`	`let triggerEndPos = this.lastChangeRange.end;`
`150`	`149`	`let doc = this.lastDocument;`
`151`		`- this.langClient.sendRequest(`
	`150`	`+ return this.langClient.sendRequest(`
`152`	`151`	`CommentHelpRequest.type,`
`153`	`152`	`{`
`154`	`153`	`documentUri: doc.uri.toString(),`