feat(financial): implement financial report formatting and enhance project unit retrieval

- Added `FinancialReportFormatter` to format weekly financial reports based on project units.
- Introduced tests for `formatFinancialReport` to ensure correct output structure and content.
- Updated `weeklyFinancialReportsWorkflow` to utilize the new formatter, improving code clarity and maintainability.
- Created a new test file for `RedmineRepository` to validate the behavior of `getProjectUnits` and ensure accurate data retrieval.

These changes enhance the financial reporting capabilities and improve the overall structure of the financial module.

Author: anatolyshipitz

workers/main/src/repositories/financial/IProjectUnitRepository.ts

@@ -1,4 +1,4 @@
-import type { ProjectUnit } from '../../../common/types';
+import { ProjectUnit } from '../../common/types';
 
 export interface IProjectUnitRepository {
   getProjectUnits(): Promise<ProjectUnit[]>;

workers/main/src/services/redmine/RedmineRepository.test.ts

@@ -0,0 +1,53 @@
+import type { Pool } from 'mysql2/promise';
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+
+import { RedminePool } from '../../common/RedminePool';
+import { RedmineRepository } from './RedmineRepository';
+
+// Mock RedminePool and Pool
+const mockExecute = vi.fn();
+const mockPool: Partial<Pool> = {
+  execute: mockExecute,
+};
+
+const mockRedminePool: Partial<RedminePool> = {
+  getPool: () => mockPool as Pool,
+};
+
+describe('RedmineRepository', () => {
+  let repo: RedmineRepository;
+
+  beforeEach(() => {
+    vi.clearAllMocks();
+    repo = new RedmineRepository(mockRedminePool as RedminePool);
+  });
+
+  it('getProjectUnitsQuery returns correct SQL', () => {
+    // @ts-expect-error: access private method for test
+    const query = repo.getProjectUnitsQuery();
+
+    expect(query).toContain('SELECT');
+    expect(query).toContain('group_id');
+    expect(query).toContain('SUM(total_hours) AS total_hours');
+    expect(query).toContain('ORDER BY group_name ASC');
+  });
+
+  it('getProjectUnits returns rows from pool', async () => {
+    const mockRows = [
+      { group_id: 1, group_name: 'A', project_id: 2, project_name: 'B' },
+    ];
+
+    mockExecute.mockResolvedValueOnce([mockRows]);
+    const result = await repo.getProjectUnits();
+
+    expect(result).toEqual(mockRows);
+    expect(mockExecute).toHaveBeenCalled();
+  });
+
+  it('getProjectUnits returns empty array if no rows', async () => {
+    mockExecute.mockResolvedValueOnce([[]]);
+    const result = await repo.getProjectUnits();
+
+    expect(result).toEqual([]);
+  });
+});

workers/main/src/workflows/financial/FinancialReportFormatter.ts

@@ -0,0 +1,7 @@
+import { ProjectUnit } from '../../common/types';
+
+export function formatFinancialReport(units: ProjectUnit[]) {
+  const reportTitle = 'Weekly Financial Report';
+
+  return `${reportTitle}\n${JSON.stringify(units, null, 2)}`;
+}

workers/main/src/workflows/financial/weeklyFinancialReports.workflow.test.ts

@@ -0,0 +1,46 @@
+import { describe, expect, it } from 'vitest';
+import { vi } from 'vitest';
+
+import { formatFinancialReport } from './FinancialReportFormatter';
+
+describe('formatFinancialReport', () => {
+  it('should format report with given project units', () => {
+    const result = formatFinancialReport([
+      {
+        group_id: 1,
+        group_name: 'Engineering',
+        project_id: 101,
+        project_name: 'Project Alpha',
+      },
+    ]);
+
+    expect(result).toContain('Weekly Financial Report');
+    expect(result).toContain('Engineering');
+    expect(result).toContain('Project Alpha');
+  });
+});
+
+vi.mock('@temporalio/workflow', () => ({
+  proxyActivities: () => ({
+    getProjectUnits: vi.fn().mockResolvedValue([
+      {
+        group_id: 1,
+        group_name: 'Engineering',
+        project_id: 101,
+        project_name: 'Project Alpha',
+      },
+    ]),
+  }),
+}));
+
+import { weeklyFinancialReportsWorkflow } from './weeklyFinancialReports.workflow';
+
+describe('weeklyFinancialReportsWorkflow', () => {
+  it('should return formatted report from workflow', async () => {
+    const result = await weeklyFinancialReportsWorkflow();
+
+    expect(result).toContain('Weekly Financial Report');
+    expect(result).toContain('Engineering');
+    expect(result).toContain('Project Alpha');
+  });
+});

workers/main/src/workflows/financial/weeklyFinancialReports.workflow.ts

@@ -1,14 +1,14 @@
 import { proxyActivities } from '@temporalio/workflow';
 
 import type * as activities from '../../activities/financial';
+import { formatFinancialReport } from './FinancialReportFormatter';
 
 const { getProjectUnits } = proxyActivities<typeof activities>({
   startToCloseTimeout: '10 minutes',
 });
 
 export async function weeklyFinancialReportsWorkflow(): Promise<string> {
-  const reportTitle = 'Weekly Financial Report';
   const projectUnits = await getProjectUnits();
 
-  return `${reportTitle}\n${JSON.stringify(projectUnits, null, 2)}`;
+  return formatFinancialReport(projectUnits);
 }