Fix/skipped test (#237)

* fix report conditionals to show skipped reports when pending tests exist

* fix skipped tests display logic to include pending tests in summary

Author: Ma11hewThomas

__tests__/ctrf/report-preparation.test.ts

@@ -382,6 +382,20 @@ describe('addFooterDisplayFlags', () => {
         ?.reportConditionals as ReportConditionals
       expect(reportConditionals.showSkippedReports).toBe(true)
     })
+
+    it('should show skipped reports when pending tests exist', () => {
+      const report = createBaseReport()
+      report.results.summary.pending = 1
+      report.results.tests = [
+        { name: 'test1', status: 'pending', duration: 100 }
+      ]
+
+      addFooterDisplayFlags(report, createMultipleReportsInputs())
+
+      const reportConditionals = report.extra
+        ?.reportConditionals as ReportConditionals
+      expect(reportConditionals.showSkippedReports).toBe(true)
+    })
   })
 
   describe('Measured Over Footer', () => {

badges/coverage.svg

@@ -1,6 +1,6 @@
 {
   "name": "github-actions-ctrf",
-  "version": "1.0.23",
+  "version": "1.0.25",
   "description": "View test results directly within your GitHub workflow summary and Pull Requests",
   "main": "index.js",
   "scripts": {

dist/index.js

@@ -67,8 +67,9 @@ export function addFooterDisplayFlags(report: Report, inputs: Inputs): Report {
     0
 
   const skippedThisRun = report.results.summary.skipped > 0
+  const pendingThisRun = report.results.summary.pending > 0
 
-  if (skippedThisRun === false) {
+  if (skippedThisRun === false && pendingThisRun === false) {
     const conditionals = report.extra.reportConditionals as ReportConditionals
     conditionals.includeSkippedReportCurrentFooter =
       isAnySkippedReportEnabled(inputs) && numOfReportsEnabled > 1

dist/index.js.map

@@ -1,14 +1,8 @@
 {{#if (filterOtherTests ctrf.tests)}}
 | **Tests** | **Status** |
 | --- | --- |
-{{#each ctrf.tests}}
-{{#if (eq status "skipped")}}
+{{#each (filterOtherTests ctrf.tests)}}
 |       {{name}} | {{status}} {{getCtrfEmoji status}} |
-{{else if (eq status "pending")}}
-|       {{name}} | {{status}} {{getCtrfEmoji status}} |
-{{else if (eq status "other")}}
-|       {{name}} | {{status}} {{getCtrfEmoji status}} |
-{{/if}}
 {{/each}}
 {{else}}
 <p>No skipped tests ✨</p>