Refactor all files to be consistent with deepsource, sonarcloud, code…

…beat and codeclimate

README.md

@@ -121,7 +121,7 @@ const { privateFunction } = jewire(
 ### 2.1. relativePath
 
 Path to the module relative to the current file, similar to CommonJS [require](https://nodejs.org/api/modules.html#requireid).
-- e.g. `'../animals/cats.js'`
+  - e.g. `'../animals/cats.js'`
 
 ### 2.2. options
 
@@ -150,7 +150,7 @@ process.cwd()
   </td>
     <td>
 <pre>
-(o) => JSON.parse(
+o => JSON.parse(
   JSON.stringify(
     o
   )

package.json

@@ -4,7 +4,7 @@
     "type": "git",
     "url": "https://github.com/nktnet1/jewire"
   },
-  "version": "0.0.2",
+  "version": "0.0.3",
   "files": [
     "dist"
   ],

src/clone.ts

@@ -68,10 +68,11 @@ const functionClone = <T extends (...args: any[]) => any>(fn: T, clone: CloneFn)
 
  * @param target object instance to decorate methods around
  */
-function decorateClassMethodClone<T extends { new (...args: any[]): Record<string, any> }>(target: T): T {
+function decorateClassMethodClone(target: any) {
   const decorateMethod = (obj: Record<string, any>, key: string | symbol) => {
     const descriptor = Reflect.getOwnPropertyDescriptor(obj, key);
-    if (!descriptor || !descriptor.configurable) {
+    /* istanbul ignore next */
+    if (!descriptor?.configurable) {
       return;
     }
     const { value } = descriptor;

src/files.ts

@@ -88,15 +88,15 @@ for errors: ${error}`
   const classes: string[] = [];
 
   ast.body.forEach((node) => {
-    if (node.type === 'FunctionDeclaration') {
-      functions.push(node.id!.name);
+    if (node.type === 'FunctionDeclaration' && node.id !== null) {
+      functions.push(node.id.name);
     } else if (node.type === 'VariableDeclaration') {
       node.declarations.forEach((declaration) => {
         if (declaration.id.type === 'Identifier') {
           variables.push(declaration.id.name);
         }
       });
-    } else if (node.type === 'ClassDeclaration' && node.id) {
+    } else if (node.type === 'ClassDeclaration' && node.id !== null) {
       classes.push(node.id.name);
     }
   });

tests/classes/classes.js

@@ -36,9 +36,7 @@ class TestClass {
   }
 
   static getObject() {
-    return {
-      key1: 'value1', key2: 2,
-    };
+    return { key1: 'value1', key2: 2 };
   }
 
   static getArrayOfObjects() {
@@ -63,3 +61,5 @@ class TestClass {
 Object.defineProperty(TestClass.prototype, 'makeThisNonConfigurable', {
   configurable: false,
 });
+
+Object.defineProperty(TestClass.prototype, 'null', {});

tests/classes/classes.test.js

@@ -3,8 +3,7 @@ import jewire from '../../src';
 const { TestClass } = jewire('./classes');
 
 describe('Instance methods', () => {
-  let instance;
-
+  let instance = null;
   beforeEach(() => {
     instance = new TestClass('Tam', ['Ham', 'Spam']);
   });

tests/functions/functions.js

@@ -29,10 +29,6 @@ const getDeeplyNestedObject = () => {
 };
 
 const getArrayOfObjects = () => [
-  {
-    name: 'Tam',
-  },
-  {
-    name: 'Ham',
-  },
+  { name: 'Tam' },
+  { name: 'Ham' },
 ];