Chore/minor report improvements (#171)

Author: Ma11hewThomas

__tests__/handlebars/string.test.ts

@@ -0,0 +1,67 @@
+import Handlebars from 'handlebars'
+import { splitLinesHelper } from '../../src/handlebars/helpers/string'
+
+describe('String Helpers', () => {
+  beforeEach(() => {
+    // Register the helper before each test
+    splitLinesHelper()
+  })
+
+  describe('splitLines', () => {
+    it('should split lines correctly for multiline strings', () => {
+      const template = Handlebars.compile(
+        '{{#each (splitLines text)}}{{this}}{{#unless @last}},{{/unless}}{{/each}}'
+      )
+      const result = template({ text: 'Line one\nLine two\nLine three' })
+      expect(result).toBe('Line one,Line two,Line three')
+    })
+
+    it('should filter out empty lines', () => {
+      const template = Handlebars.compile(
+        '{{#each (splitLines text)}}{{this}}{{#unless @last}},{{/unless}}{{/each}}'
+      )
+      const result = template({ text: 'Line one\n\nLine two\n\nLine three' })
+      expect(result).toBe('Line one,Line two,Line three')
+    })
+
+    it('should handle undefined values gracefully', () => {
+      const template = Handlebars.compile(
+        '{{#each (splitLines text)}}{{this}}{{#unless @last}},{{/unless}}{{/each}}'
+      )
+      const result = template({ text: undefined })
+      expect(result).toBe('')
+    })
+
+    it('should handle null values gracefully', () => {
+      const template = Handlebars.compile(
+        '{{#each (splitLines text)}}{{this}}{{#unless @last}},{{/unless}}{{/each}}'
+      )
+      const result = template({ text: null })
+      expect(result).toBe('')
+    })
+
+    it('should handle empty strings', () => {
+      const template = Handlebars.compile(
+        '{{#each (splitLines text)}}{{this}}{{#unless @last}},{{/unless}}{{/each}}'
+      )
+      const result = template({ text: '' })
+      expect(result).toBe('')
+    })
+
+    it('should handle strings with only whitespace', () => {
+      const template = Handlebars.compile(
+        '{{#each (splitLines text)}}{{this}}{{#unless @last}},{{/unless}}{{/each}}'
+      )
+      const result = template({ text: '   \n  \n   ' })
+      expect(result).toBe('')
+    })
+
+    it('should handle single line strings', () => {
+      const template = Handlebars.compile(
+        '{{#each (splitLines text)}}{{this}}{{#unless @last}},{{/unless}}{{/each}}'
+      )
+      const result = template({ text: 'Single line' })
+      expect(result).toBe('Single line')
+    })
+  })
+})

badges/coverage.svg

@@ -53,6 +53,9 @@ export function escapeMarkdownHelper(): void {
  */
 export function splitLinesHelper(): void {
   Handlebars.registerHelper('splitLines', (str: string) => {
+    if (!str) {
+      return []
+    }
     return str.split('\n').filter((line: string) => line.trim() !== '')
   })
 }

dist/index.js

@@ -1,6 +1,6 @@
-| **Build {{getCtrfEmoji "build"}}** | **Result {{getCtrfEmoji "result"}}** | **Tests {{getCtrfEmoji "tests"}}** | **Passed {{getCtrfEmoji "passed"}}** | **Failed {{getCtrfEmoji "failed"}}** | **Skipped {{getCtrfEmoji "skipped"}}** | **Pending {{getCtrfEmoji "pending"}}** | **Other {{getCtrfEmoji "other"}}** | **Flaky {{getCtrfEmoji "flaky"}}** | **Duration {{getCtrfEmoji "duration"}}** |
-| --- | --- | --- | --- | --- | --- | --- | --- | --- | --- |
-| [#{{ctrf.environment.extra.runNumber}}]({{ctrf.environment.extra.buildUrl}}) | {{getCtrfEmoji ctrf.summary.extra.result}} | {{ctrf.summary.tests}} | {{ctrf.summary.passed}} | {{ctrf.summary.failed}} | {{ctrf.summary.skipped}} | {{ctrf.summary.pending}} | {{ctrf.summary.other}} | {{countFlaky ctrf.tests}} | {{formatDuration ctrf.summary.start ctrf.summary.stop}} |
+| **Build {{getCtrfEmoji "build"}}** | **Result {{getCtrfEmoji "result"}}** | **Tests {{getCtrfEmoji "tests"}}** | **Passed {{getCtrfEmoji "passed"}}** | **Failed {{getCtrfEmoji "failed"}}** | **Skipped {{getCtrfEmoji "skipped"}}** | **Other {{getCtrfEmoji "other"}}** | **Flaky {{getCtrfEmoji "flaky"}}** | **Duration {{getCtrfEmoji "duration"}}** |
+| --- | --- | --- | --- | --- | --- | --- | --- | --- |
+| [#{{ctrf.environment.extra.runNumber}}]({{ctrf.environment.extra.buildUrl}}) | {{getCtrfEmoji ctrf.summary.extra.result}} | {{ctrf.summary.tests}} | {{ctrf.summary.passed}} | {{ctrf.summary.failed}} | {{add ctrf.summary.skipped ctrf.summary.pending}} | {{ctrf.summary.other}} | {{countFlaky ctrf.tests}} | {{formatDuration ctrf.summary.start ctrf.summary.stop}} |
 {{#each ctrf.extra.previousReports}}
-| [#{{this.results.environment.extra.runNumber}}]({{this.results.environment.extra.buildUrl}}) | {{getCtrfEmoji this.results.summary.extra.result}} | {{this.results.summary.tests}} | {{this.results.summary.passed}} | {{this.results.summary.failed}} | {{this.results.summary.skipped}} | {{this.results.summary.pending}} | {{this.results.summary.other}} | {{this.results.summary.extra.flaky}} | {{formatDuration this.results.summary.start this.results.summary.stop}} |
+| [#{{this.results.environment.extra.runNumber}}]({{this.results.environment.extra.buildUrl}}) | {{getCtrfEmoji this.results.summary.extra.result}} | {{this.results.summary.tests}} | {{this.results.summary.passed}} | {{this.results.summary.failed}} | {{add this.results.summary.skipped this.results.summary.pending}} | {{this.results.summary.other}} | {{this.results.summary.extra.flaky}} | {{formatDuration this.results.summary.start this.results.summary.stop}} |
 {{/each}}

dist/index.js.map

@@ -1,3 +1,3 @@
-| **Tests {{getCtrfEmoji "tests"}}** | **Passed {{getCtrfEmoji "passed"}}** | **Failed {{getCtrfEmoji "failed"}}** | **Skipped {{getCtrfEmoji "skipped"}}** | **Pending {{getCtrfEmoji "pending"}}** | **Other {{getCtrfEmoji "other"}}** | **Flaky {{getCtrfEmoji "flaky"}}** | **Duration {{getCtrfEmoji "duration"}}** |
-| --- | --- | --- | --- | --- | --- | --- | --- |
-| {{ctrf.summary.tests}} | {{ctrf.summary.passed}} | {{ctrf.summary.failed}} | {{ctrf.summary.skipped}} | {{ctrf.summary.pending}} | {{ctrf.summary.other}} | {{countFlaky ctrf.tests}} | {{formatDuration ctrf.summary.start ctrf.summary.stop}} |
+| **Tests {{getCtrfEmoji "tests"}}** | **Passed {{getCtrfEmoji "passed"}}** | **Failed {{getCtrfEmoji "failed"}}** | **Skipped {{getCtrfEmoji "skipped"}}** | **Other {{getCtrfEmoji "other"}}** | **Flaky {{getCtrfEmoji "flaky"}}** | **Duration {{getCtrfEmoji "duration"}}** |
+| --- | --- | --- | --- | --- | --- | --- |
+| {{ctrf.summary.tests}} | {{ctrf.summary.passed}} | {{ctrf.summary.failed}} | {{add ctrf.summary.skipped ctrf.summary.pending}} | {{ctrf.summary.other}} | {{countFlaky ctrf.tests}} | {{formatDuration ctrf.summary.start ctrf.summary.stop}} |