feat: Update SonarQube exclusions and add unit tests for main worker functions (#38)

- Modified `sonar-project.properties` to exclude test files with the
pattern `**/*.test.ts` instead of `**/src/__tests__/**`.
- Introduced a new test file `index.test.ts` containing unit tests for
the `run` and `handleRunError` functions, ensuring proper error handling
and logging.

These changes improve the project's test coverage and ensure that
SonarQube analysis focuses on relevant source files.

Author: anatolyshipitz

sonar-project.properties

@@ -1,4 +1,4 @@
 sonar.projectKey=speedandfunction_automatization
 sonar.organization=speedandfunction
 sonar.javascript.lcov.reportPaths=workers/main/coverage/lcov.info
-sonar.exclusions=**/src/__tests__/**,**/src/dist/**
+sonar.exclusions=**/*.test.ts,**/src/dist/**

workers/main/src/__tests__/index.test.ts

@@ -0,0 +1,36 @@
+import { describe, expect, it, vi } from 'vitest';
+
+import { handleRunError, logger } from './index';
+
+describe('handleRunError', () => {
+  it('should log the error', () => {
+    const error = new Error('test error');
+    const logSpy = vi.spyOn(logger, 'error').mockImplementation(() => {});
+
+    handleRunError(error);
+    expect(logSpy).toHaveBeenCalledWith(
+      `Error in main worker: ${error.message}`,
+    );
+    logSpy.mockRestore();
+  });
+
+  it('should log when error is a string', () => {
+    const error = 'string error';
+    const logSpy = vi.spyOn(logger, 'error').mockImplementation(() => {});
+
+    handleRunError(error);
+    expect(logSpy).toHaveBeenCalledWith(`Error in main worker: ${error}`);
+    logSpy.mockRestore();
+  });
+
+  it('should log when error is an Error object', () => {
+    const error = new Error('error from Error object');
+    const logSpy = vi.spyOn(logger, 'error').mockImplementation(() => {});
+
+    handleRunError(error);
+    expect(logSpy).toHaveBeenCalledWith(
+      `Error in main worker: ${error.message}`,
+    );
+    logSpy.mockRestore();
+  });
+});

workers/main/src/index.test.ts

@@ -1,6 +1,6 @@
 import { DefaultLogger, NativeConnection, Worker } from '@temporalio/worker';
 
-import { logWorkerError, validateEnv } from '../../common/utils';
+import { validateEnv } from './common/utils';
 import { temporalConfig } from './configs/temporal';
 import { workerConfig } from './configs/worker';
 
@@ -37,10 +37,11 @@ export async function run(): Promise<void> {
   }
 }
 
-export function handleRunError(err: unknown): never {
-  logWorkerError('main', err);
+export function handleRunError(error: unknown): void {
+  logger.error(
+    `Error in main worker: ${error instanceof Error ? error.message : String(error)}`,
+  );
   setTimeout(() => process.exit(1), 100);
-  throw err;
 }
 
 export function mainEntry() {

workers/main/src/index.ts

@@ -7,7 +7,7 @@ export default defineConfig({
   test: {
     globals: true,
     environment: 'node',
-    include: ['src/__tests__/**/*.test.ts'],
+    include: ['src/**/*.test.ts'],
     coverage: {
       provider: 'v8',
       reporter: ['text', 'lcov'],