Merge pull request #6662 from dibarbet/enable_promise_reject_error

Enable eslint rule ensuring rejected promises throw an error

Author: dibarbet

.eslintrc.js

@@ -33,6 +33,7 @@ module.exports = {
         ],
         "@typescript-eslint/no-namespace": "off",
         "@typescript-eslint/promise-function-async": "error",
+        "prefer-promise-reject-errors": "error",
         "curly": "error",
         "prettier/prettier": [ "error", { "endOfLine": "auto" } ],
         "unicorn/filename-case": [

src/features/codeActionProvider.ts

@@ -78,7 +78,7 @@ export default class OmniSharpCodeActionProvider
                 };
             });
         } catch (error) {
-            return Promise.reject(`Problem invoking 'GetCodeActions' on OmniSharp server: ${error}`);
+            return Promise.reject(new Error(`Problem invoking 'GetCodeActions' on OmniSharp server: ${error}`));
         }
     }
 
@@ -107,7 +107,7 @@ export default class OmniSharpCodeActionProvider
                 return buildEditForResponse(response.Changes, this._languageMiddlewareFeature, token);
             }
         } catch (error) {
-            return Promise.reject(`Problem invoking 'RunCodeAction' on OmniSharp server: ${error}`);
+            return Promise.reject(new Error(`Problem invoking 'RunCodeAction' on OmniSharp server: ${error}`));
         }
 
         return undefined;

src/features/inlayHintProvider.ts

@@ -67,14 +67,14 @@ export default class OmniSharpInlayHintProvider extends AbstractProvider impleme
                 return mappedHint;
             });
         } catch (error) {
-            return Promise.reject(`Problem invoking 'GetInlayHints' on OmniSharpServer: ${error}`);
+            return Promise.reject(new Error(`Problem invoking 'GetInlayHints' on OmniSharpServer: ${error}`));
         }
     }
 
     async resolveInlayHint?(hint: vscode.InlayHint, token: vscode.CancellationToken): Promise<vscode.InlayHint> {
         const inlayHint = this._hintsMap.get(hint);
         if (inlayHint === undefined) {
-            return Promise.reject(`Outdated inlay hint was requested to be resolved, aborting.`);
+            return Promise.reject(new Error(`Outdated inlay hint was requested to be resolved, aborting.`));
         }
 
         const request: InlayHintResolveRequest = { Hint: inlayHint };
@@ -83,7 +83,7 @@ export default class OmniSharpInlayHintProvider extends AbstractProvider impleme
             const result = await serverUtils.resolveInlayHints(this._server, request, token);
             return this.toVSCodeHint(result);
         } catch (error) {
-            return Promise.reject(`Problem invoking 'ResolveInlayHints' on OmniSharpServer: ${error}`);
+            return Promise.reject(new Error(`Problem invoking 'ResolveInlayHints' on OmniSharpServer: ${error}`));
         }
     }
 

src/lsptoolshost/roslynLanguageServer.ts

@@ -572,8 +572,9 @@ export class RoslynLanguageServer {
             // Error information will be captured from the stdout/stderr streams above.
             childProcess.on('exit', (code) => {
                 if (code && code !== 0) {
-                    _channel.appendLine(`Language server process exited with ${code}`);
-                    reject();
+                    const message = `Language server process exited with ${code}`;
+                    _channel.appendLine(message);
+                    reject(new Error(message));
                 }
             });
         });