[Pages] Stub child_process.execSync directly in validation-helpers test (#133)

Follow-up to #132. The initial test stubbed `child_process` by swapping
the entry in `require.cache`, which relies on Node's caching of built-in
modules — implementation detail that may not hold across versions.

Switch to monkeypatching `childProcess.execSync` directly before
requiring `validation-helpers.js`, with cleanup via `t.after()`. This
exercises the helper through its real `require('child_process')` call
and is the idiomatic node:test approach.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: priyanshu92

plugins/power-pages/scripts/tests/validation-helpers.test.js

@@ -1,39 +1,31 @@
 const test = require('node:test');
 const assert = require('node:assert/strict');
 const path = require('path');
+const childProcess = require('child_process');
 
 const helpersPath = path.join(__dirname, '..', 'lib', 'validation-helpers.js');
-const childProcessId = require.resolve('child_process');
 
-test('getAuthToken passes --allow-no-subscriptions to az', () => {
-  const originalChildProcess = require.cache[childProcessId];
+test('getAuthToken passes --allow-no-subscriptions to az', (t) => {
+  const originalExecSync = childProcess.execSync;
   let capturedCommand = null;
 
-  require.cache[childProcessId] = {
-    id: childProcessId,
-    filename: childProcessId,
-    loaded: true,
-    exports: {
-      execSync: (command, options) => {
-        capturedCommand = command;
-        const out = 'fake-token-value\n';
-        return options && options.encoding ? out : Buffer.from(out);
-      },
-    },
+  childProcess.execSync = (command, options) => {
+    capturedCommand = command;
+    const out = 'fake-token-value\n';
+    return options && options.encoding ? out : Buffer.from(out);
   };
   delete require.cache[require.resolve(helpersPath)];
 
-  try {
-    const { getAuthToken } = require(helpersPath);
-    const token = getAuthToken('https://example.crm.dynamics.com');
-
-    assert.equal(token, 'fake-token-value');
-    assert.match(capturedCommand, /^az account get-access-token /);
-    assert.match(capturedCommand, /--allow-no-subscriptions/);
-    assert.match(capturedCommand, /--resource "https:\/\/example\.crm\.dynamics\.com"/);
-  } finally {
-    if (originalChildProcess) require.cache[childProcessId] = originalChildProcess;
-    else delete require.cache[childProcessId];
+  t.after(() => {
+    childProcess.execSync = originalExecSync;
     delete require.cache[require.resolve(helpersPath)];
-  }
+  });
+
+  const { getAuthToken } = require(helpersPath);
+  const token = getAuthToken('https://example.crm.dynamics.com');
+
+  assert.equal(token, 'fake-token-value');
+  assert.match(capturedCommand, /^az account get-access-token /);
+  assert.match(capturedCommand, /--allow-no-subscriptions/);
+  assert.match(capturedCommand, /--resource "https:\/\/example\.crm\.dynamics\.com"/);
 });