Should check for staleTime

Author: k-ode

packages/mst-query/src/MstQueryHandler.ts

@@ -136,16 +136,14 @@ export class QueryObserver {
                     this.handler.type,
                     options.cacheKey,
                 );
-                const isStale = !cacheEntry || isDataStale(cacheEntry.cachedAt, options.cacheTime);
+                if (cacheEntry) {
+                    this.handler.hydrate(cacheEntry.data, options);
+                }                
+                
+                const isStale = !cacheEntry || isDataStale(cacheEntry.cachedAt, options.staleTime);
                 if (isStale) {
                     this.handler.queryWhenChanged(options);
-                } else {
-                    try {
-                        this.handler.hydrate(cacheEntry.data, options);
-                    } catch {
-                        this.handler.queryWhenChanged(options);
-                    }
-                }
+                }                
             } else {
                 if (!options.isRequestEqual) {
                     this.query.setData(null);

packages/mst-query/tests/mstQuery.test.tsx

@@ -242,6 +242,7 @@ test('useQuery - cacheKey and cacheTime', async () => {
             request: { id: id.get() },
             cacheKey: id.get(),
             cacheTime: 25,
+            staleTime: 25,
             meta: { getItem: testApi.getItem },
         });
         return <div></div>;