fix(@schematics/angular): keep the default getter access type of spyOnProperty in refactor-jasmine-vitest

Jasmine spies on getter when `spyOnProperty` gets no access type, migration turned `spyOnProperty(obj, 'prop')` into `vi.spyOn(obj, 'prop')`, which Vitest treats as method spy, so converted spec failed with TS2769
now `'get'` is passed when access type is omited, explicit one is kept

Author: thekhegay

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

@@ -103,11 +103,18 @@ function transformPrimarySpy(node: ts.CallExpression, refactorCtx: RefactorConte
   ) {
     addVitestValueImport(pendingVitestValueImports, 'vi');
 
+    // `spyOnProperty` defaults to the getter while `vi.spyOn` needs the access type to spy on
+    // a property instead of a method.
+    const args =
+      node.expression.text === 'spyOnProperty' && node.arguments.length === 2
+        ? [...node.arguments, ts.factory.createStringLiteral('get', true)]
+        : node.arguments;
+
     const viSpyOnCall = ts.factory.updateCallExpression(
       node,
       createPropertyAccess('vi', 'spyOn'),
       node.typeArguments,
-      node.arguments,
+      args,
     );
 
     if (isChainedWithAnd(node)) {

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

@@ -60,9 +60,19 @@ describe('Jasmine to Vitest Transformer - transformSpies', () => {
     },
     {
       description:
-        'should transform spyOnProperty(object, "prop") to vi.spyOn(object, "prop").mockReturnValue(undefined)',
+        'should transform spyOnProperty(object, "prop") to vi.spyOn(object, "prop", "get").mockReturnValue(undefined)',
       input: `spyOnProperty(service, 'myProp');`,
-      expected: `vi.spyOn(service, 'myProp').mockReturnValue(undefined);`,
+      expected: `vi.spyOn(service, 'myProp', 'get').mockReturnValue(undefined);`,
+    },
+    {
+      description: 'should default the access type of a chained spyOnProperty to "get"',
+      input: `spyOnProperty(service, 'myProp').and.returnValue(42);`,
+      expected: `vi.spyOn(service, 'myProp', 'get').mockReturnValue(42);`,
+    },
+    {
+      description: 'should keep an explicit spyOnProperty access type',
+      input: `spyOnProperty(service, 'myProp', 'set');`,
+      expected: `vi.spyOn(service, 'myProp', 'set').mockReturnValue(undefined);`,
     },
     {
       description: 'should transform .and.stub() to .mockImplementation(() => {})',