test(helpers): use early return in decycle

Author: remarkablemark

__tests__/helpers/decycle.ts

@@ -79,35 +79,25 @@ function deepCopy(
     value = replacer(value);
   }
 
-  if (isPlainObjectOrArray(value)) {
-    const existingPath = visitedObjects.get(value);
-
-    if (existingPath !== undefined) {
-      return { $ref: existingPath };
-    }
-
-    visitedObjects.set(value, path);
-
-    if (Array.isArray(value)) {
-      const copy: unknown[] = [];
-      value.forEach((element, index) => {
-        copy[index] = deepCopy(
-          element,
-          `${path}[${index.toString()}]`,
-          visitedObjects,
-          replacer,
-        );
-      });
-      return copy;
-    }
-
-    const record = value as Record<string, unknown>;
-    const copy: DecycledObject = {};
-
-    Object.keys(record).forEach((key) => {
-      copy[key] = deepCopy(
-        record[key],
-        `${path}[${JSON.stringify(key)}]`,
+  if (!isPlainObjectOrArray(value)) {
+    return value;
+  }
+
+  const existingPath = visitedObjects.get(value);
+
+  if (existingPath !== undefined) {
+    return { $ref: existingPath };
+  }
+
+  visitedObjects.set(value, path);
+
+  if (Array.isArray(value)) {
+    const copy: unknown[] = [];
+
+    value.forEach((element, index) => {
+      copy[index] = deepCopy(
+        element,
+        `${path}[${index.toString()}]`,
         visitedObjects,
         replacer,
       );
@@ -116,7 +106,19 @@ function deepCopy(
     return copy;
   }
 
-  return value;
+  const record = value as Record<string, unknown>;
+  const copy: DecycledObject = {};
+
+  Object.keys(record).forEach((key) => {
+    copy[key] = deepCopy(
+      record[key],
+      `${path}[${JSON.stringify(key)}]`,
+      visitedObjects,
+      replacer,
+    );
+  });
+
+  return copy;
 }
 
 /**