Fix @typescript-eslint/no-redundant-type-constituents

Author: robertbrignull

extensions/ql-vscode/src/databases/database-fetcher.ts

@@ -612,7 +612,7 @@ export class DatabaseFetcher {
   private async checkForFailingResponse(
     response: Response,
     errorMessage: string,
-  ): Promise<Response | never> {
+  ): Promise<Response> {
     if (response.ok) {
       return response;
     }

extensions/ql-vscode/test/vscode-tests/test-dispose-handler.ts

@@ -1,11 +1,11 @@
 import type { Disposable } from "vscode";
 import { DisposableObject } from "../../src/common/disposable-object";
 
-export function testDisposeHandler(disposable: any & Disposable) {
+export function testDisposeHandler(disposable: Disposable) {
   if (
-    disposable.onDidExpandElement &&
-    disposable.onDidCollapseElement &&
-    disposable.reveal
+    (disposable as any).onDidExpandElement &&
+    (disposable as any).onDidCollapseElement &&
+    (disposable as any).reveal
   ) {
     // This looks like a treeViewer. Don't dispose
     return;