📜 Eslint fixes for repo (#39)

* 🔧 update eslint to enforce no semi-colon * maple_leaf: repo lint fix * 🍁 repo lint fix
akashchouhan16 · Nov 14, 2023 · 60133c8 · 60133c8
1 parent ad7c829
commit 60133c8
Show file tree

Hide file tree

Showing 8 changed files with 27 additions and 26 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -22,6 +22,7 @@ module.exports = {
     },
     "ignorePatterns": ["/dist/", "/tests/", "/*.mjs"],
     "rules": {
-        quotes: ["error", "double", { "allowTemplateLiterals": true }]
+        quotes: ["error", "double", { "allowTemplateLiterals": true }],
+        "semi": ["warn", "never"]
     }
 }
diff --git a/jest.config.js b/jest.config.js
@@ -15,6 +15,6 @@ const config = {
     bail: 1,
     forceCoverageMatch: ["**/*.test.js"],
     testEnvironment: "node",
-};
+}
 
-module.exports = config;
+module.exports = config
diff --git a/src/config/cacheConfig.js b/src/config/cacheConfig.js
@@ -28,4 +28,4 @@ let cacheConfig = {
     keyCount: 0,
 }
 
-module.exports = cacheConfig;
+module.exports = cacheConfig
diff --git a/src/nodecache.js b/src/nodecache.js
@@ -139,11 +139,11 @@ class NodeCache {
         }
 
         if (!this.cache[key])
-            return false;
+            return false
 
         const newTTL = Date.now() + Math.abs(ttl)
         this.cache[key].ttl = newTTL
-        return true;
+        return true
     }
 
     async refresh() {
@@ -173,9 +173,9 @@ class NodeCache {
     }
 
     flush() {
-        cacheConfig.cacheHit = 0;
-        cacheConfig.cacheMiss = 0;
-        cacheConfig.keyCount = 0;
+        cacheConfig.cacheHit = 0
+        cacheConfig.cacheMiss = 0
+        cacheConfig.keyCount = 0
         this.cache = {}
     }
 

diff --git a/src/utils/logger.js b/src/utils/logger.js
@@ -31,7 +31,7 @@ class Logger {
 
     log(message, options = {}) {
         if (!message || this.mode === "none")
-            return;
+            return
         if (typeof message !== "string") {
             if (typeof message === "object") {
                 message = JSON.stringify(message)
@@ -44,7 +44,7 @@ class Logger {
             type: this.type,
             path: this.path,
             ...options
-        };
+        }
 
 
         const date = new Date().toLocaleString("en-US", this.formatOptions)

diff --git a/src/worker/worker.js b/src/worker/worker.js
@@ -4,7 +4,7 @@
  * *****************************************
 */
 
-const { parentPort, isMainThread, threadId } = require("worker_threads");
+const { parentPort, isMainThread, threadId } = require("worker_threads")
 // Listen for most recent cache context from the Main Thread.
 if (!isMainThread) {
     parentPort.on("message", ({ cache }) => {
@@ -13,7 +13,7 @@ if (!isMainThread) {
             for (const [key, cacheItem] of Object.entries(cache)) {
                 if (cacheItem.ttl && Number(cacheItem.ttl) < Number(now)) {
                     // Send key with expired ttl to Main Thread for cache eviction.
-                    parentPort.postMessage({ key });
+                    parentPort.postMessage({ key })
                 }
             }
         } catch (error) {

diff --git a/test/nodecache.config.test.js b/test/nodecache.config.test.js
@@ -3,7 +3,7 @@ const NodeCache = require("../index")
 
 describe("NodeCache params for instance config", () => {
     describe("StdTTL config checks", () => {
-        let cache;
+        let cache
         afterEach(() => {
             cache.close()
         })
@@ -40,7 +40,7 @@ describe("NodeCache params for instance config", () => {
                 stdTTL: true
             })
             // expect the stdTTL to be 0 => Infinite.
-            cache.set("k1", "boolean-ttl-check");
+            cache.set("k1", "boolean-ttl-check")
             expect(cache.cache["k1"]).toHaveProperty("ttl")
             expect(cache.cache["k1"]).toMatchObject({
                 value: expect.anything(),
@@ -54,7 +54,7 @@ describe("NodeCache params for instance config", () => {
                 stdTTL: "15000" // 15seconds
             })
             // expect the stdTTL to be 0 => Infinite.
-            cache.set("k2", "string-ttl-check");
+            cache.set("k2", "string-ttl-check")
             expect(cache.cache["k2"]).toHaveProperty("ttl")
             expect(cache.cache["k2"]).toMatchObject({
                 value: expect.anything(),
@@ -68,7 +68,7 @@ describe("NodeCache params for instance config", () => {
                 stdTTL: NaN // falsy values: 0, false, null, undefined, NaN, ''
             })
             // expect the stdTTL to be 0 => Infinite.
-            cache.set("k3", "falsy-check");
+            cache.set("k3", "falsy-check")
             expect(cache.cache["k3"]).toHaveProperty("ttl")
             expect(cache.cache["k3"]).toMatchObject({
                 value: expect.anything(),
@@ -93,7 +93,7 @@ describe("NodeCache params for instance config", () => {
                 createAt: Date.now()
             })
 
-            let { value } = cache.cache["key1"];
+            let { value } = cache.cache["key1"]
 
             expect(value).not.toBeUndefined()
             expect(value).toEqual(expect.any(String)) // not an object anymore
@@ -109,7 +109,7 @@ describe("NodeCache params for instance config", () => {
                 createAt: Date.now()
             })
 
-            let { value } = cache.cache["key2"];
+            let { value } = cache.cache["key2"]
 
             expect(value).not.toBeUndefined()
             expect(value).toEqual(expect.any(Object))
@@ -125,7 +125,7 @@ describe("NodeCache params for instance config", () => {
 
         test("maxKeys not provided, defaults to -1 (No Limit)", () => {
             cache = new NodeCache()
-            let flag = true;
+            let flag = true
             for (let i = 1; i <= 6; i++) {
                 let success = cache.set(i, `value-${i}`)
                 flag = flag & success
@@ -159,7 +159,7 @@ describe("NodeCache params for instance config", () => {
             })
         })
         afterAll(() => {
-            cache.close();
+            cache.close()
         })
 
         test("NodeCache instance to be not null", () => {
@@ -180,7 +180,7 @@ describe("NodeCache params for instance config", () => {
     describe("Custom Logger prompt check for the instance on mode = std", () => {
 
         afterAll(() => {
-            cache.close();
+            cache.close()
         })
         let cache = new NodeCache({
             mode: "std",
@@ -210,7 +210,7 @@ describe("NodeCache params for instance config", () => {
             })
         })
         afterAll(() => {
-            cache.close();
+            cache.close()
         })
 
         test("Valid Instance with default params for logger", () => {

diff --git a/test/nodecache.test.js b/test/nodecache.test.js
@@ -7,7 +7,7 @@ describe("NodeCache instance creation", () => {
             cache = new NodeCache()
         })
         afterAll(() => {
-            cache.close();
+            cache.close()
         })
 
         test("NodeCache instance to be defined and not falsy", () => {
@@ -36,7 +36,7 @@ describe("NodeCache public APIs", () => {
     })
 
     afterAll(() => {
-        cache.close();
+        cache.close()
     })
 
     test("NodeCache::global", () => {
@@ -56,7 +56,7 @@ describe("NodeCache public APIs", () => {
             1, "2", "key"
         ])
         // expect(cache.global()).toEqual({ cacheHit: 2, cacheMiss: 1, keyCount: 2 })
-        cache.flush();
+        cache.flush()
         expect(cache.global()).toEqual({ cacheHit: 0, cacheMiss: 0, keyCount: 0 })
     })