fix(@schematics/angular): warn about loose matching in arrayWithExactContents

This commit improves the transformation of `jasmine.arrayWithExactContents` in the `jasmine-vitest` schematic. While Vitest does not have a direct equivalent for multiset equality, the schematic previously transformed this into a check for length and subset containment, which could lead to false positives (e.g., `[1, 1]` matches `[1, 2]`).

Now, the schematic attaches a TODO comment to the generated code, explicitly warning the user that the transformation relies on `expect.arrayContaining` (a subset check) and advising them to verify strict content matching manually.

Author: clydin

packages/schematics/angular/refactor/jasmine-vitest/test-file-transformer.integration_spec.ts

@@ -275,6 +275,7 @@ describe('Jasmine to Vitest Transformer - Integration Tests', () => {
 
         it('should handle array contents checking', () => {
           const arr = [1, 2, 3];
+          // TODO: vitest-migration: Verify this matches strict array content (multiset equality). Vitest's arrayContaining is a subset check.
           expect(arr).toHaveLength(3);
           expect(arr).toEqual(expect.arrayContaining([3, 2, 1]));
         });

packages/schematics/angular/refactor/jasmine-vitest/test-file-transformer_spec.ts

@@ -85,10 +85,13 @@ describe('Jasmine to Vitest Transformer', () => {
       {
         description: 'should handle arrayWithExactContents containing nested asymmetric matchers',
         input: `expect(myArray).toEqual(jasmine.arrayWithExactContents([jasmine.objectContaining({ id: 1 })]));`,
+        /* eslint-disable max-len */
         expected: `
+          // TODO: vitest-migration: Verify this matches strict array content (multiset equality). Vitest's arrayContaining is a subset check.
           expect(myArray).toHaveLength(1);
           expect(myArray).toEqual(expect.arrayContaining([expect.objectContaining({ id: 1 })]));
         `,
+        /* eslint-enable max-len */
       },
       {
         description: 'should handle a spy rejecting with an asymmetric matcher',

packages/schematics/angular/refactor/jasmine-vitest/transformers/jasmine-matcher.ts

@@ -451,10 +451,14 @@ export function transformArrayWithExactContents(
     ],
   );
 
-  return [
-    ts.factory.createExpressionStatement(lengthCall),
-    ts.factory.createExpressionStatement(containingCall),
-  ];
+  const lengthStmt = ts.factory.createExpressionStatement(lengthCall);
+  const containingStmt = ts.factory.createExpressionStatement(containingCall);
+
+  const category = 'arrayWithExactContents-check';
+  reporter.recordTodo(category);
+  addTodoComment(lengthStmt, category);
+
+  return [lengthStmt, containingStmt];
 }
 
 export function transformCalledOnceWith(

packages/schematics/angular/refactor/jasmine-vitest/transformers/jasmine-matcher_spec.ts

@@ -225,19 +225,25 @@ expect(mySpyObj).toHaveSpyInteractions();`,
       {
         description: 'should transform toEqual(jasmine.arrayWithExactContents()) into two calls',
         input: `expect(myArray).toEqual(jasmine.arrayWithExactContents(['a', 'b']));`,
+        /* eslint-disable max-len */
         expected: `
+          // TODO: vitest-migration: Verify this matches strict array content (multiset equality). Vitest's arrayContaining is a subset check.
           expect(myArray).toHaveLength(2);
           expect(myArray).toEqual(expect.arrayContaining(['a', 'b']));
         `,
+        /* eslint-enable max-len */
       },
       {
         description:
           'should transform toEqual(jasmine.arrayWithExactContents()) with asymmetric matchers',
         input: `expect(myArray).toEqual(jasmine.arrayWithExactContents([jasmine.any(Number), 'a']));`,
+        /* eslint-disable max-len */
         expected: `
+          // TODO: vitest-migration: Verify this matches strict array content (multiset equality). Vitest's arrayContaining is a subset check.
           expect(myArray).toHaveLength(2);
           expect(myArray).toEqual(expect.arrayContaining([expect.any(Number), 'a']));
         `,
+        /* eslint-enable max-len */
       },
       {
         description:

packages/schematics/angular/refactor/jasmine-vitest/utils/todo-notes.ts

@@ -56,6 +56,10 @@ export const TODO_NOTES = {
     message:
       'Cannot transform jasmine.arrayWithExactContents with a dynamic variable. Please migrate this manually.',
   },
+  'arrayWithExactContents-check': {
+    message:
+      "Verify this matches strict array content (multiset equality). Vitest's arrayContaining is a subset check.",
+  },
   'expect-nothing': {
     message:
       'expect().nothing() has been removed because it is redundant in Vitest. Tests without assertions pass by default.',