Added tests

Author: PeterHasitschka

jest.config.js

@@ -0,0 +1,7 @@
+/** @type {import('ts-jest').JestConfigWithTsJest} */
+// eslint-disable-next-line no-undef
+module.exports = {
+  preset: 'ts-jest',
+  testEnvironment: 'node',
+  testRegex: '.*.test.ts',
+}

package.json

@@ -7,18 +7,22 @@
     "start:dev": "ts-node-dev --inspect --respawn .",
     "lint": "tsc --noEmit && eslint",
     "build": "tsc",
-    "start": "node dist/index.js"
+    "start": "node dist/index.js",
+    "test": "jest"
   },
   "dependencies": {
     "axios": "^0.27.2",
     "dotenv": "^16.0.2"
   },
   "devDependencies": {
     "@types/dotenv": "^8.2.0",
+    "@types/jest": "^29.0.3",
     "@types/node": "^18.7.23",
     "@typescript-eslint/eslint-plugin": "^5.38.1",
     "@typescript-eslint/parser": "^5.38.1",
     "eslint": "^8.24.0",
+    "jest": "^29.0.3",
+    "ts-jest": "^29.0.2",
     "ts-node": "^10.9.1",
     "ts-node-dev": "^2.0.0",
     "typescript": "^4.8.3"

src/aggregation.ts

@@ -1,6 +1,6 @@
 import { ReportLine } from './api'
 
-interface Epic {
+export interface Epic {
   name: string
   tasks: string[]
   durationSecs: number
@@ -26,7 +26,7 @@ export const aggregateReports = (reportLines: ReportLine[]): Epic[] => {
     }
 
     if (splitted.length > 1) {
-      epic.tasks.push(...epicRest)
+      epic.tasks.push(...epicRest.map((t) => t.trim()))
     }
     epic.durationSecs += line.durationSec
   })
@@ -40,16 +40,15 @@ export const printEpics = (epics: Epic[]): void => {
   const DURATION_LENGTH = 14
 
   const uniqueTasks = (value: string, idx: number, self: string[]) =>
-    self.indexOf(value) === idx
+    +self.indexOf(value) === idx
 
   const prettyTable = epics.map((epic) => ({
     name: epic.name.padEnd(EPIC_LENGTH),
-    tasks: epic.tasks
-      .map((t) => t.trim())
-      .filter(uniqueTasks)
-      .join(', ')
-      .padEnd(TASKS_LENGTH),
-    minutes: (epic.durationSecs / 60).toFixed(2).padStart(DURATION_LENGTH),
+    tasks: epic.tasks.filter(uniqueTasks).join(', ').padEnd(TASKS_LENGTH),
+    minutes: (epic.durationSecs / 60)
+      .toFixed(2)
+      .replace('.', ',')
+      .padStart(DURATION_LENGTH),
   }))
 
   console.log('\n\n')
@@ -75,6 +74,7 @@ export const printEpics = (epics: Epic[]): void => {
       }, 0) / 60
     )
       .toFixed(2)
+      .replace('.', ',')
       .padStart(DURATION_LENGTH),
   }
   console.log(''.padEnd(EPIC_LENGTH + TASKS_LENGTH + DURATION_LENGTH, '='))

tests/aggregation.test.ts

@@ -0,0 +1,42 @@
+import { aggregateReports } from '../src/aggregation'
+import { ReportLine } from '../src/api'
+
+describe('Test aggregation', () => {
+  let report: ReportLine[]
+
+  beforeEach(() => {
+    report = [
+      {
+        description: 'E1',
+        durationSec: 10,
+      },
+      {
+        description: 'E1, T1',
+        durationSec: 10,
+      },
+      {
+        description: 'E1, T2',
+        durationSec: 10,
+      },
+      {
+        description: 'E2, T1',
+        durationSec: 10,
+      },
+    ]
+  })
+
+  it('should aggregate epics correctly', () => {
+    const aggregation = aggregateReports(report)
+    expect(aggregation.length).toBe(2)
+    expect(aggregation[1].name).toBe('E2')
+  })
+  it('should aggregate tasks correctly', () => {
+    const aggregation = aggregateReports(report)
+    expect(aggregation[0].tasks).toEqual(['T1', 'T2'])
+    expect(aggregation[1].tasks).toEqual(['T1'])
+  })
+  it('should aggregate time correctly', () => {
+    const aggregation = aggregateReports(report)
+    expect(aggregation[0].durationSecs).toBe(30)
+  })
+})