squash: move caching higher

Author: LiviaMedeiros

test/common/index.js

@@ -522,6 +522,11 @@ function mustNotCall(msg) {
 const _mustNotMutateProxies = new WeakMap();
 
 function mustNotMutate(object) {
+  const cachedProxy = _mustNotMutateProxies.get(object);
+  if (cachedProxy) {
+    return cachedProxy;
+  }
+
   const _mustNotMutateHandler = {
     defineProperty(target, property, descriptor) {
       assert.fail(`Expected no side effects, got ${property} defined`);
@@ -548,13 +553,9 @@ function mustNotMutate(object) {
     }
   };
 
-  const cachedProxy = _mustNotMutateProxies.get(object);
-  if (cachedProxy) {
-    return cachedProxy;
-  }
-  const newProxy = new Proxy(object, _mustNotMutateHandler);
-  _mustNotMutateProxies.set(object, newProxy);
-  return newProxy;
+  const proxy = new Proxy(object, _mustNotMutateHandler);
+  _mustNotMutateProxies.set(object, proxy);
+  return proxy;
 }
 
 function printSkipMessage(msg) {