fixup! test(@angular/build): verify coverage ignore comments are preserved during compilation

Author: clydin

packages/angular/build/src/builders/unit-test/tests/behavior/coverage-ignore-comments_spec.ts

@@ -102,54 +102,100 @@ describe('AppComponent', () => {
           'There should be no uncovered statements as the uncalled function was ignored',
         );
       });
-    }
 
-    // Note: V8 does not support 'ignore if' semantic comments; it only supports generic line/block ignores.
-    it('should respect istanbul ignore if comments when computing coverage', async () => {
-      harness.useTarget('test', {
-        ...BASE_OPTIONS,
-        coverage: true,
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        coverageReporters: ['json'] as any,
-      });
+      it(`should respect ${type} ignore if comments when computing coverage`, async () => {
+        harness.useTarget('test', {
+          ...BASE_OPTIONS,
+          coverage: true,
+          // eslint-disable-next-line @typescript-eslint/no-explicit-any
+          coverageReporters: ['json'] as any,
+        });
 
-      await harness.writeFile(
-        'src/app/app.component.ts',
-        `
-        import { Component } from '@angular/core';
+        await harness.writeFile(
+          'src/app/app.component.ts',
+          `
+          import { Component } from '@angular/core';
 
-        @Component({
-          selector: 'app-root',
-          template: '<h1>hello</h1>',
-          standalone: true,
-        })
-        export class AppComponent {
-          checkValue(val: boolean) {
-            /* istanbul ignore if -- @preserve */
-            if (val) {
-              return true;
+          @Component({
+            selector: 'app-root',
+            template: '<h1>hello</h1>',
+            standalone: true,
+          })
+          export class AppComponent {
+            checkValue(val: boolean) {
+              /* ${type} ignore if -- @preserve */
+              if (val) {
+                return true;
+              }
+              return false;
             }
-            return false;
           }
-        }
-        `,
-      );
+          `,
+        );
 
-      await harness.writeFile(
-        'src/app/app.component.spec.ts',
-        getSpecContent(`
-  it('should exercise the function but not the if block', () => {
-    const fixture = TestBed.createComponent(AppComponent);
-    const app = fixture.componentInstance;
-    app.checkValue(false);
-  });
-`),
-      );
+        await harness.writeFile(
+          'src/app/app.component.spec.ts',
+          getSpecContent(`
+    it('should exercise the function but not the if block', () => {
+      const fixture = TestBed.createComponent(AppComponent);
+      const app = fixture.componentInstance;
+      app.checkValue(false);
+    });
+  `),
+        );
 
-      await assertNoUncoveredStatements(
-        'There should be no uncovered statements as the uncalled branch was ignored',
-      );
+        await assertNoUncoveredStatements(
+          'There should be no uncovered statements as the uncalled branch was ignored',
+        );
+      });
+
+      it(`should respect ${type} ignore else comments when computing coverage`, async () => {
+        harness.useTarget('test', {
+          ...BASE_OPTIONS,
+          coverage: true,
+          // eslint-disable-next-line @typescript-eslint/no-explicit-any
+          coverageReporters: ['json'] as any,
+        });
+
+        await harness.writeFile(
+          'src/app/app.component.ts',
+          `
+          import { Component } from '@angular/core';
+
+          @Component({
+            selector: 'app-root',
+            template: '<h1>hello</h1>',
+            standalone: true,
+          })
+          export class AppComponent {
+            checkValue(val: boolean) {
+              /* ${type} ignore else -- @preserve */
+              if (val) {
+                return true;
+              } else {
+                return false;
+              }
+            }
+          }
+          `,
+        );
+
+        await harness.writeFile(
+          'src/app/app.component.spec.ts',
+          getSpecContent(`
+    it('should exercise the function but not the else block', () => {
+      const fixture = TestBed.createComponent(AppComponent);
+      const app = fixture.componentInstance;
+      app.checkValue(true);
     });
+  `),
+        );
+
+        await assertNoUncoveredStatements(
+          'There should be no uncovered statements as the uncalled else branch was ignored',
+        );
+      });
+    }
 
     it('should respect v8 ignore start/stop comments when computing coverage', async () => {
       harness.useTarget('test', {
@@ -187,52 +233,5 @@ describe('AppComponent', () => {
         'There should be no uncovered statements as the uncalled function was ignored',
       );
     });
-
-    it('should respect istanbul ignore else comments when computing coverage', async () => {
-      harness.useTarget('test', {
-        ...BASE_OPTIONS,
-        coverage: true,
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        coverageReporters: ['json'] as any,
-      });
-
-      await harness.writeFile(
-        'src/app/app.component.ts',
-        `
-        import { Component } from '@angular/core';
-
-        @Component({
-          selector: 'app-root',
-          template: '<h1>hello</h1>',
-          standalone: true,
-        })
-        export class AppComponent {
-          checkValue(val: boolean) {
-            /* istanbul ignore else -- @preserve */
-            if (val) {
-              return true;
-            } else {
-              return false;
-            }
-          }
-        }
-        `,
-      );
-
-      await harness.writeFile(
-        'src/app/app.component.spec.ts',
-        getSpecContent(`
-  it('should exercise the function but not the else block', () => {
-    const fixture = TestBed.createComponent(AppComponent);
-    const app = fixture.componentInstance;
-    app.checkValue(true);
-  });
-`),
-      );
-
-      await assertNoUncoveredStatements(
-        'There should be no uncovered statements as the uncalled else branch was ignored',
-      );
-    });
   });
 });