Revert to old coverage reporter

Author: leeyi45

lib/modules-lib/src/types/index.ts

@@ -19,7 +19,7 @@ export abstract class glAnimation {
 
   public abstract getFrame(timestamp: number): AnimFrame;
 
-  public static isAnimation = (obj: any): obj is glAnimation => obj.fps !== undefined;
+  public static isAnimation = (obj: any): obj is glAnimation => obj instanceof glAnimation;
 }
 export interface AnimFrame {
   draw: (canvas: HTMLCanvasElement) => void;

lib/vitest-reporter/src/__tests__/coverage.test.ts

@@ -29,8 +29,8 @@ describe(CoverageReporter, () => {
     }
   });
 
-  it('doesn\'t open the summary file if GITHUB_STEP_SUMMARY isn\'t defined', ({ reporter }) => {
-    reporter.onStart({} as any, { watermarks: {} } as any);
-    expect(fs.createWriteStream).not.toHaveBeenCalled();
+  it.skipIf(!!process.env.GITHUB_STEP_SUMMARY)('doesn\'t open the summary file if GITHUB_STEP_SUMMARY isn\'t defined', ({ reporter }) => {
+      reporter.onStart({} as any, { watermarks: {} } as any);
+      expect(fs.createWriteStream).not.toHaveBeenCalled();
   });
 });

lib/vitest-reporter/src/__tests__/test-reporter.test.ts

@@ -26,7 +26,7 @@ describe(TestReporter, () => {
     }
   });
 
-  it('doesn\'t open the summary file if GITHUB_STEP_SUMMARY isn\'t defined', ({ reporter }) => {
+  it.skipIf(!!process.env.GITHUB_STEP_SUMMARY)('doesn\'t open the summary file if GITHUB_STEP_SUMMARY isn\'t defined', ({ reporter }) => {
     reporter.onInit();
     expect(fs.createWriteStream).not.toHaveBeenCalled();
   });

src/bundles/rune/src/functions.ts

@@ -650,7 +650,7 @@ export class HollusionRune extends DrawnRune {
 }
 
 /** @hidden */
-export const isHollusionRune = (rune: DrawnRune): rune is HollusionRune => rune.isHollusion;
+export const isHollusionRune = (rune: DrawnRune): rune is HollusionRune => rune instanceof HollusionRune;
 
 /**
  * Makes a new Rune from two given Runes by