@less-is-more/less-js 1.2.22 → 1.2.25

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@less-is-more/less-js",
-  "version": "1.2.22",
+  "version": "1.2.25",
   "description": "Fast develop kit for nodejs",
   "main": "src/index.js",
   "scripts": {

package/src/cache.js

@@ -50,12 +50,16 @@ module.exports = class Cache {
       if (!Param.isBlank(result)) {
         // 处理JSON
         if (result instanceof Object) {
-          await Redis.exec(
-            "setex",
-            fullKey,
-            timeSecond + "",
-            JSON.stringify(result)
-          );
+          if (result.success === false) {
+            console.log("Do not cache fail result");
+          } else {
+            await Redis.exec(
+              "setex",
+              fullKey,
+              timeSecond + "",
+              JSON.stringify(result)
+            );
+          }
         } else {
           await Redis.exec(
             "setex",
@@ -69,7 +73,7 @@ module.exports = class Cache {
     } else {
       console.log("Found cache", fullKey);
       // 优先转成json
-      if (savedData.startsWith("{")) {
+      if (savedData.startsWith("{") || savedData.startsWith("[")) {
         return JSON.parse(savedData);
       } else {
         return savedData;

package/src/ret.js

@@ -6,65 +6,64 @@ module.exports = class Ret {
   static CODE_SYSTEM = 1004;
 
   constructor(success, message, code) {
-    this.#instance.success = success;
+    this.success = success;
     if (message) {
-      this.#instance.message = message;
+      this.message = message;
     }
     if (code) {
-      this.#instance.code = code;
+      this.code = code;
     }
   }
 
-  #instance = {};
   ok() {
-    this.#instance.success = true;
+    this.success = true;
     return this;
   }
-  isOk(){
-    return this.#instance.success;
+  isOk() {
+    return this.success;
   }
   fail() {
-    this.#instance.success = false;
+    this.success = false;
     return this;
   }
-  isFail(){
-    return !this.#instance.success;
+  isFail() {
+    return !this.success;
   }
   setMessage(message) {
-    this.#instance.message = message;
+    this.message = message;
     return this;
   }
   getMessage() {
-    return this.#instance.message;
+    return this.message;
   }
   setCode(code) {
-    this.#instance.code = code;
+    this.code = code;
     return this;
   }
   setData(data) {
-    this.#instance.data = data;
+    this.data = data;
     return this;
   }
   getData() {
-    return this.#instance.data;
+    return this.data;
   }
   toString() {
-    return JSON.stringify(this.#instance);
+    return JSON.stringify(this);
   }
   getJson() {
-    return this.#instance;
+    return this;
   }
   toJson() {
-    return this.#instance;
+    return this;
   }
 
   filter(props = []) {
     if (props.length > 0) {
-      const data = this.#instance.data;
+      const data = this.data;
       if (data instanceof Array) {
-        this.#instance.data = this._filterArray(data, props);
+        this.data = this._filterArray(data, props);
       } else {
-        this.#instance.data = this._filterObject(data, props);
+        this.data = this._filterObject(data, props);
       }
     }
     return this;

package/test/test-cache.js

@@ -1,6 +1,7 @@
 const assert = require("assert");
 const Cache = require("../src/cache.js");
 const Redis = require("../src/redis.js");
+const Ret = require("../src/ret.js");
 
 describe("cache.js", () => {
   before(() => {
@@ -81,5 +82,26 @@ describe("cache.js", () => {
       const compare = { a: 1, b: 2 };
       assert(result.a === compare.a);
     });
+    it("return array", async () => {
+      async function test(a, b) {
+        return await Cache.auto("test4", 20, arguments, () => {
+          console.log(a, b);
+          return [{ a, b }];
+        });
+      }
+      const result = await test(1, 2);
+      console.log("result", JSON.stringify(result));
+      assert(result.length === 1);
+    });
+    it("no cache fail", async () => {
+      async function test(a, b) {
+        return await Cache.auto("test5", 20, arguments, () => {
+          return new Ret(false);
+        });
+      }
+      const result = await test(1, 2);
+      console.log("result", JSON.stringify(result));
+      assert(result.success === false);
+    });
   });
 });

package/test/test-ret.js

@@ -5,6 +5,7 @@ describe("ret.js", () => {
   describe("new()", () => {
     it("ok", () => {
       let ret = new Ret(true).setData({ name: "good" });
+      console.log(ret)
       assert(ret.getJson().data.name, "good");
     });
     it("mutli params", () => {