Bugix/undefined promise (#21)

* return undefined vs null / clear() returns promise

* update changelog

CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.9.2
+
+- Fix `cache.clear()` to return a promise
+- Return `undefined` when the cache has no results for a given key
+
 ## 0.9.1
 
 - Set `__fromCache = true` when returning results from cache

lib/cache.js

@@ -38,15 +38,15 @@ var set = exports.set = function set(cacheKey, value) {
 
 var getSync = exports.getSync = function getSync(cacheKey, defaultValue) {
   var result = _store2.default.get(cacheKey) || defaultValue;
-  return result ? _extends({}, result, { __cacheKey: cacheKey, __fromCache: true }) : null;
+  return result ? _extends({}, result, { __cacheKey: cacheKey, __fromCache: true }) : undefined;
 };
 
 var get = exports.get = function get(cacheKey, defaultValue) {
   return Promise.resolve(getSync(cacheKey, defaultValue));
 };
 
 var clear = exports.clear = function clear() {
-  return _store2.default.clearAll();
+  return Promise.resolve(_store2.default.clearAll());
 };
 
 var observeData = function observeData(actionName, dataFn, onNext, onError) {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "withdata",
-  "version": "0.9.1",
+  "version": "0.9.2",
   "description": "Utilities for managing data via props. Inspired by apollo-client.",
   "main": "index.js",
   "scripts": {

src/cache.js

@@ -15,13 +15,13 @@ export const set = (cacheKey, value, maxAge = defaultConfig.maxAge) => {
 
 export const getSync = (cacheKey, defaultValue) => {
   const result = store.get(cacheKey) || defaultValue;
-  return result ? { ...result, __cacheKey: cacheKey, __fromCache: true } : null;
+  return result ? { ...result, __cacheKey: cacheKey, __fromCache: true } : undefined;
 };
 
 export const get = (cacheKey, defaultValue) =>
   Promise.resolve(getSync(cacheKey, defaultValue));
 
-export const clear = () => store.clearAll();
+export const clear = () => Promise.resolve(store.clearAll());
 
 export const observeData = (
   actionName,