fix(no-misused-observables): accept Observable-returning function for union types.

Author: JasonWeinzierl

src/rules/no-misused-observables.ts

@@ -9,6 +9,7 @@ import {
   isJSXExpressionContainer,
   isMethodDefinition,
   isPropertyDefinition,
+  couldBeType as tsutilsEtcCouldBeType,
 } from '../etc';
 import { ruleCreator } from '../utils';
 
@@ -390,6 +391,12 @@ function isVoidReturningFunctionType(
     for (const signature of subType.getCallSignatures()) {
       const returnType = signature.getReturnType();
 
+      // If a certain positional argument accepts both Observable and void returns,
+      // an Observable-returning function is valid.
+      if (tsutilsEtcCouldBeType(returnType, 'Observable')) {
+        return false;
+      }
+
       hasVoidReturn ||= tsutils.isTypeFlagSet(returnType, ts.TypeFlags.Void);
     }
   }

tests/rules/no-misused-observables.test.ts

@@ -197,6 +197,18 @@ ruleTester({ types: true }).run('no-misused-observables', noMisusedObservablesRu
         bar,
       };
     `,
+    stripIndent`
+      // void return property; union type
+      import { Observable, of } from "rxjs";
+
+      interface Hook {
+        onInit?: ((field: string) => void) | ((field: string) => Observable<any>);
+      }
+
+      const hook: Hook = {
+        onInit: field => of(field),
+      };
+    `,
     // #endregion valid; void return property
     // #region valid; void return return value
     {