fixup: address review feedback

- Simplify enforceArray tests to two cases (array vs non-array)
- Remove redundant multiple-substitutions test for populate

Author: wilsonwangdev

src/utils/__tests__/array.test.mjs

@@ -13,17 +13,5 @@ describe('enforceArray', () => {
 
   it('should wrap a non-array value in an array', () => {
     assert.deepStrictEqual(enforceArray('hello'), ['hello']);
-    assert.deepStrictEqual(enforceArray(42), [42]);
-    assert.deepStrictEqual(enforceArray(true), [true]);
-  });
-
-  it('should wrap null and undefined in an array', () => {
-    assert.deepStrictEqual(enforceArray(null), [null]);
-    assert.deepStrictEqual(enforceArray(undefined), [undefined]);
-  });
-
-  it('should return an empty array as-is', () => {
-    const input = [];
-    assert.strictEqual(enforceArray(input), input);
   });
 });

src/utils/configuration/__tests__/templates.test.mjs

@@ -23,9 +23,4 @@ describe('populate', () => {
     const result = populate('{missing}', {});
     assert.strictEqual(result, '{missing}');
   });
-
-  it('should handle multiple substitutions in one template', () => {
-    const result = populate('{a}-{b}-{c}', { a: '1', b: '2', c: '3' });
-    assert.strictEqual(result, '1-2-3');
-  });
 });