squash: rename from can to must

Author: LiviaMedeiros

test/common/README.md

@@ -299,7 +299,7 @@ If `fn` is not provided, an empty function will be used.
 Returns a function that triggers an `AssertionError` if it is invoked. `msg` is
 used as the error message for the `AssertionError`.
 
-### `canNotMutateObjectDeep([target])`
+### `mustNotMutateObjectDeep([target])`
 
 * `target` [\<any>][<any>] default = `undefined`
 * return [\<any>][<any>]
@@ -313,10 +313,10 @@ APIs that work with user-provided objects.
 
 ```mjs
 import { open } from 'node:fs/promises';
-import { canNotMutateObjectDeep } from '../common/index.mjs';
+import { mustNotMutateObjectDeep } from '../common/index.mjs';
 
 const _mutableOptions = { length: 4, position: 8 };
-const options = canNotMutateObjectDeep(_mutableOptions);
+const options = mustNotMutateObjectDeep(_mutableOptions);
 
 // In filehandle.read or filehandle.write, attempt to mutate options will throw
 // In the test code, options can still be mutated via _mutableOptions
@@ -326,7 +326,7 @@ _mutableOptions.position = 4;
 await fh.write(buffer, options);
 
 // Inline usage
-const stats = await fh.stat(canNotMutateObjectDeep({ bigint: true }));
+const stats = await fh.stat(mustNotMutateObjectDeep({ bigint: true }));
 console.log(stats.size);
 ```
 

test/common/index.js

@@ -519,22 +519,22 @@ function mustNotCall(msg) {
   };
 }
 
-const _canNotMutateObjectDeepProxies = new WeakMap();
+const _mustNotMutateObjectDeepProxies = new WeakMap();
 
-function canNotMutateObjectDeep(original) {
+function mustNotMutateObjectDeep(original) {
   // Return primitives and functions directly. Primitives are immutable, and
   // proxied functions are impossible to compare against originals, e.g. with
   // `assert.deepEqual()`.
   if (original === null || typeof original !== 'object') {
     return original;
   }
 
-  const cachedProxy = _canNotMutateObjectDeepProxies.get(original);
+  const cachedProxy = _mustNotMutateObjectDeepProxies.get(original);
   if (cachedProxy) {
     return cachedProxy;
   }
 
-  const _canNotMutateObjectDeepHandler = {
+  const _mustNotMutateObjectDeepHandler = {
     __proto__: null,
     defineProperty(target, property, descriptor) {
       assert.fail(`Expected no side effects, got ${inspect(property)} ` +
@@ -545,7 +545,7 @@ function canNotMutateObjectDeep(original) {
                   'deleted');
     },
     get(target, prop, receiver) {
-      return canNotMutateObjectDeep(Reflect.get(target, prop, receiver));
+      return mustNotMutateObjectDeep(Reflect.get(target, prop, receiver));
     },
     preventExtensions(target) {
       assert.fail('Expected no side effects, got extensions prevented on ' +
@@ -560,8 +560,8 @@ function canNotMutateObjectDeep(original) {
     }
   };
 
-  const proxy = new Proxy(original, _canNotMutateObjectDeepHandler);
-  _canNotMutateObjectDeepProxies.set(original, proxy);
+  const proxy = new Proxy(original, _mustNotMutateObjectDeepHandler);
+  _mustNotMutateObjectDeepProxies.set(original, proxy);
   return proxy;
 }
 
@@ -873,7 +873,7 @@ const common = {
   mustCall,
   mustCallAtLeast,
   mustNotCall,
-  canNotMutateObjectDeep,
+  mustNotMutateObjectDeep,
   mustSucceed,
   nodeProcessAborted,
   PIPE,

test/common/index.mjs

@@ -35,7 +35,7 @@ const {
   canCreateSymLink,
   getCallSite,
   mustNotCall,
-  canNotMutateObjectDeep,
+  mustNotMutateObjectDeep,
   printSkipMessage,
   skip,
   nodeProcessAborted,
@@ -82,7 +82,7 @@ export {
   canCreateSymLink,
   getCallSite,
   mustNotCall,
-  canNotMutateObjectDeep,
+  mustNotMutateObjectDeep,
   printSkipMessage,
   skip,
   nodeProcessAborted,

…st-common-can-not-mutate-object-deep.mjs …t-common-must-not-mutate-object-deep.mjstest/parallel/test-common-can-not-mutate-object-deep.mjs renamed to test/parallel/test-common-must-not-mutate-object-deep.mjs test/parallel/test-common-can-not-mutate-object-deep.mjs renamed to test/parallel/test-common-must-not-mutate-object-deep.mjs

@@ -1,8 +1,8 @@
-import { canNotMutateObjectDeep } from '../common/index.mjs';
+import { mustNotMutateObjectDeep } from '../common/index.mjs';
 import assert from 'node:assert';
 import { promisify } from 'node:util';
 
-// Test common.canNotMutateObjectDeep()
+// Test common.mustNotMutateObjectDeep()
 
 const original = {
   foo: { bar: 'baz' },
@@ -17,7 +17,7 @@ const original = {
 const backup = structuredClone(original);
 
 // Wrapper for convenience
-const obj = () => canNotMutateObjectDeep(original);
+const obj = () => mustNotMutateObjectDeep(original);
 
 function testOriginal(root) {
   assert.deepStrictEqual(root, backup);
@@ -194,16 +194,16 @@ function setPrototypeOfQuux(root) {
   undefined, null, false, true, 42, 42n, Symbol('42'), NaN, Infinity, {}, [],
   () => {}, async () => {}, Promise.resolve(), Math, Object.create(null),
 ].forEach((target) => {
-  assert.deepStrictEqual(canNotMutateObjectDeep(target), target);
-  assert.deepStrictEqual(canNotMutateObjectDeep({ target }), { target });
-  assert.deepStrictEqual(canNotMutateObjectDeep([ target ]), [ target ]);
+  assert.deepStrictEqual(mustNotMutateObjectDeep(target), target);
+  assert.deepStrictEqual(mustNotMutateObjectDeep({ target }), { target });
+  assert.deepStrictEqual(mustNotMutateObjectDeep([ target ]), [ target ]);
 });
 
 // Test that passed functions keep working correctly
 {
   const fn = () => 'blep';
   fn.foo = {};
-  const fnImmutableView = canNotMutateObjectDeep(fn);
+  const fnImmutableView = mustNotMutateObjectDeep(fn);
   assert.deepStrictEqual(fnImmutableView, fn);
 
   // The function works