fix(Cache): remove cached items when expired

Author: leeyeh

src/cache.js

@@ -2,6 +2,8 @@
 const storage = require('./localstorage');
 const AV = require('./av');
 
+const remove = exports.remove = storage.removeItemAsync.bind(storage);
+
 exports.get = (key) => {
   return storage.getItemAsync(`${AV.applicationId}/${key}`)
     .then(cache => {
@@ -15,6 +17,7 @@ exports.get = (key) => {
         if (!expired) {
           return cache.value;
         }
+        return remove(key).then(() => null);
       }
       return null;
     });

src/utils.js

@@ -131,7 +131,7 @@ const init = (AV) => {
 
   const setRegionServer = (region = 'cn') => {
     AVConfig.region = region;
-    // 如果用户在 init 之前设置了 APIServerURL，则跳请求 router
+    // 如果用户在 init 之前设置了 APIServerURL，则跳过请求 router
     if (AVConfig.APIServerURL) {
       return;
     }
@@ -144,7 +144,10 @@ const init = (AV) => {
           return ajax('get', `https://app-router.leancloud.cn/1/route?appId=${AV.applicationId}`)
             .then(servers => {
               if (servers.api_server) {
-                Cache.set('APIServerURL', servers.api_server, (servers.ttl || 3600) * 1000);
+                Cache.set(
+                  'APIServerURL',
+                  servers.api_server,
+                  (typeof servers.ttl ==='number' ? servers.ttl : 3600) * 1000);
                 return servers.api_server;
               }
             });

test/cache.js

@@ -1,17 +1,23 @@
 var Cache = AV.Cache;
-const wait = time => new Promise(resolve => setTimeout(resolve, time));
+var wait = function wait(time) {
+  return new Promise(function (resolve) {
+    return setTimeout(resolve, time);
+  });
+};
 
-describe('Cache', () => {
-  const getValue = () => Cache.get('__test');
-  it('get/set', () =>
-    Cache.set('__test', 1).then(getValue).then(value => {
+describe('Cache', function () {
+  var getValue = function getValue() {
+    return Cache.get('__test');
+  };
+  it('get/set', function () {
+    return Cache.set('__test', 1).then(getValue).then(function (value) {
       expect(value).to.be(1);
       return Cache.set('__test', '1', 100).then(getValue);
-    }).then(value => {
+    }).then(function (value) {
       expect(value).to.be('1');
       return wait(110).then(getValue);
-    }).then(value => {
+    }).then(function (value) {
       expect(value).to.be(null);
-    })
-  );
+    });
+  });
 });