chore: remove non-schema properties from insights

Author: Ma11hewThomas

README.md

@@ -27,7 +27,7 @@ Explore more <a href="https://www.ctrf.io/integrations">integrations</a>
 ## Installation
 
 ```sh
-npm install ctrf@0.0.13-next.5
+npm install ctrf
 ```
 
 ## API reference

src/methods/run-insights.test.ts

@@ -322,7 +322,7 @@ describe('enrichReportWithInsights - Main API', () => {
   })
 
   describe('added/removed tests tracking', () => {
-    it('should track tests added since baseline', () => {
+    it.skip('should track tests added since baseline', () => {
       const currentTests = [
         createMockTest({ name: 'test1', status: 'passed' }),
         createMockTest({ name: 'test2', status: 'passed' }),
@@ -350,7 +350,7 @@ describe('enrichReportWithInsights - Main API', () => {
       )
     })
 
-    it('should track tests removed since baseline', () => {
+    it.skip('should track tests removed since baseline', () => {
       const currentTests = [createMockTest({ name: 'test1', status: 'passed' })]
       const previousTests = [
         createMockTest({ name: 'test1', status: 'passed' }),
@@ -376,7 +376,7 @@ describe('enrichReportWithInsights - Main API', () => {
       )
     })
 
-    it('should handle single report no added/removed tests', () => {
+    it.skip('should handle single report no added/removed tests', () => {
       const currentTests = [createMockTest({ name: 'test1', status: 'passed' })]
 
       const currentReport = createMockReport(currentTests)

src/methods/run-insights.ts

@@ -859,6 +859,14 @@ export function enrichReportWithInsights(
     baseline
   )
 
+  // Remove testsAdded and testsRemoved as they're not part of the official schema yet
+  if (baselineInsights.extra?.testsAdded) {
+    delete baselineInsights.extra.testsAdded
+  }
+  if (baselineInsights.extra?.testsRemoved) {
+    delete baselineInsights.extra.testsRemoved
+  }
+
   return {
     ...currentReportWithTestInsights,
     insights: baselineInsights,

src/test-utils/report-enrichment-helper.ts

@@ -54,6 +54,8 @@ export class ReportEnrichmentTestHelper {
           other: tests.filter(
             t => !['passed', 'failed', 'skipped', 'pending'].includes(t.status)
           ).length,
+          flaky: tests.filter(t => t.flaky === true).length,
+          duration: tests.reduce((sum, t) => sum + t.duration, 0),
           start: startTime,
           stop: startTime + 5000,
         },