fix: correctly check for promise in useLoadData

hooks/useLoadData/useLoadData.ts

@@ -1,5 +1,5 @@
 import {useEffect, useState, useMemo} from 'react';
-import {ApiResponse, RetryResponse, ApiResponseBase, OptionalDependency, DependencyBase} from '../../types';
+import {ApiResponse, RetryResponse, ApiResponseBase, OptionalDependency, DependencyBase, Promisable} from '../../types';
 
 import {FetchData, NotUndefined} from './types';
 
@@ -30,6 +30,14 @@ function unboxApiResponse<T>(arg: ApiResponse<T> | T): T {
   }
 }
 
+/*
+  isPromise determines a promise by checking whether or not it is an instanceof 
+  native promise (preferred) or whether it has a then method.
+*/
+function isPromise<T>(promisable: Promisable<T>): promisable is Promise<T> {
+  return promisable && typeof promisable === 'object' && 'then' in promisable && typeof promisable.then === 'function';
+}
+
 export interface LoadDataConfig {
   fetchWhenDepsChange?: boolean;
   maxRetryCount?: number;
@@ -186,7 +194,8 @@ export function useLoadData<T extends NotUndefined, Deps extends any[]>(
     }
   }, [counter, localFetchWhenDepsChange]);
 
-  const nonPromiseResult = initialPromise.res instanceof Promise ? undefined : initialPromise.res;
+  const initialPromiseRes = initialPromise.res;
+  const nonPromiseResult = isPromise(initialPromiseRes) ? undefined : initialPromiseRes;
   const initialData = data || nonPromiseResult;
 
   // Initialize our pending data to one of three possible states:

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@optum/react-hooks",
-  "version": "1.0.4",
+  "version": "1.0.5-next.1",
   "description": "A reusable set of React hooks",
   "repository": "https://github.com/Optum/react-hooks",
   "license": "Apache 2.0",