Merge pull request #78 from stbestichhh/refactor

Patch 1.0.7

Author: stbestichhh

SECURITY.md

@@ -4,6 +4,7 @@
 
 | Version | Supported          |
 |---------|--------------------|
+| 1.0.7   | :white_check_mark: |
 | 1.0.6   | :white_check_mark: |
 | 1.0.5   | :white_check_mark: |
 | 1.0.4   | :x:                |

bin/cli/cli.ts

@@ -20,7 +20,7 @@ const handleOptionsAndExecuteAction = async (options: StestOptions) => {
 program
   .name('stest')
   .description('Testing framework for TypeScript Node.js applications')
-  .version('1.0.7.beta.1');
+  .version('1.0.7');
 
 program
   .option(

bin/errorInfo/error.info.ts

@@ -1,4 +1,7 @@
-export function findWhereErrorHasBeenThrown(e: Error, testClassName: string) {
+export function findWhereErrorHasBeenThrown(
+  e: Error,
+  testClassName: string,
+): string | undefined {
   const stackLines = e.stack?.split('\n');
   if (stackLines) {
     const relevantLine = stackLines.find((line) =>

bin/logger/testRunner.logger.ts

@@ -21,7 +21,9 @@ export class TestRunnerLogger extends LoggerService {
     if (e instanceof Error) {
       const testClassName = testSuite.constructor.name;
       const errorInfo = findWhereErrorHasBeenThrown(e, testClassName);
-      console.error(`    ⚠︎ ${errorInfo || e}`.brightRed);
+      const errorLog = `    ⚠︎ ${errorInfo || e}`;
+      console.error(errorLog.brightRed);
+      return errorLog;
     }
   }
 }

bin/runner/test.runner.ts

@@ -69,10 +69,14 @@ export class TestRunner {
     const testSuite = new target();
     console.log(colors.white.bold(`\n${testName}`));
 
-    await this.runTestCycle(target, testSuite);
+    await this.runTestCycle(target, testSuite, testName);
   }
 
-  private static async runTestCycle(target: any, testSuite: any) {
+  private static async runTestCycle(
+    target: any,
+    testSuite: any,
+    testSuiteName: string,
+  ) {
     const testCases: ITestCase[] = target.testCases || [];
     const dataSetsArray: IDataSet[] = target.testCasesDataSets || [];
     const dataTableArray: IDataTableArray[] = target.testCasesDataTable || [];
@@ -88,6 +92,7 @@ export class TestRunner {
       await this.runLifecycleMethods(testSuite, beforeEach, 'beforeEach');
       await this.runTestCase(
         testSuite,
+        testSuiteName,
         methodName,
         caseDescription,
         dataSetsArray,
@@ -132,6 +137,7 @@ export class TestRunner {
 
   private static async runTestCase(
     testSuiteInstance: any,
+    testSuiteName: string,
     methodName: string,
     caseDescription: string,
     dataSetsArray: IDataSet[],
@@ -140,7 +146,7 @@ export class TestRunner {
   ) {
     const startTime = performance.now();
     let status: 'PASSED' | 'FAILED' = 'FAILED';
-    let error: string = '';
+    let error: string | undefined = '';
     let duration: number = 0;
 
     try {
@@ -162,14 +168,18 @@ export class TestRunner {
       this.log.logTestResult(caseDescription || methodName, status, 'grey');
     } catch (e: unknown) {
       status = 'FAILED';
-      error = e instanceof Error ? e.message : `Unknown error: ${e}`;
       this.isAllPassed = false;
-      this.log.handleError(e, methodName, caseDescription, testSuiteInstance);
+      error = this.log.handleError(
+        e,
+        methodName,
+        caseDescription,
+        testSuiteInstance,
+      );
     } finally {
       duration = performance.now() - startTime;
       this.reportTestResult(
-        testSuiteInstance,
-        methodName,
+        testSuiteName,
+        caseDescription || methodName,
         status,
         duration,
         error,
@@ -178,19 +188,19 @@ export class TestRunner {
   }
 
   private static reportTestResult(
-    testSuiteInstance: any,
+    testSuiteName: string,
     methodName: string,
     status: 'PASSED' | 'FAILED',
     duration: number,
-    error: string,
+    error?: string,
   ) {
     if (
       this.isReportingEnabled &&
       this.reporterModule?.ReportBuilder &&
       this.reporterModule?.ReportsRegistry
     ) {
       const reportBuilder = new this.reporterModule.ReportBuilder(
-        testSuiteInstance.constructor.name,
+        testSuiteName,
         methodName,
       )
         .status(status)

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stlib/testing",
-  "version": "1.0.7-beta.1",
+  "version": "1.0.7",
   "description": "Testing framework for TypeScript Node.js applications",
   "main": "stutil.ts",
   "types": "stutil.d.ts",
@@ -42,7 +42,7 @@
     "@commitlint/cli": "^19.5.0",
     "@commitlint/config-conventional": "^19.5.0",
     "@commitlint/types": "^19.5.0",
-    "@stlib/testing-reporter": "^0.0.1",
+    "@stlib/testing-reporter": "^0.0.2",
     "@types/jest": "^29.5.12",
     "@types/node": "^20.12.10",
     "@typescript-eslint/eslint-plugin": "^7.8.0",

yarn.lock

@@ -1044,10 +1044,10 @@ __metadata:
   languageName: node
   linkType: hard
 
-"@stlib/testing-reporter@npm:^0.0.1":
-  version: 0.0.1
-  resolution: "@stlib/testing-reporter@npm:0.0.1"
-  checksum: 10c0/c5f23f783184f7d001017b925bff9f5e05f131de7b0d6b3baa9eb6dbbdcfceb88c1b9c97c040b829da20837e29d2ba58e4b304a98ecaf24396830fd66e61e38e
+"@stlib/testing-reporter@npm:^0.0.2":
+  version: 0.0.2
+  resolution: "@stlib/testing-reporter@npm:0.0.2"
+  checksum: 10c0/1ad11327967530f7438c9140603bd5eeec0b984a12d4c79ad85ea082e3ffc1c25fc757bd55482a8546d68fe8a3268ee4ae2e4d6ea13fb927c3ac70cf918e22ce
   languageName: node
   linkType: hard
 
@@ -1059,7 +1059,7 @@ __metadata:
     "@commitlint/cli": "npm:^19.5.0"
     "@commitlint/config-conventional": "npm:^19.5.0"
     "@commitlint/types": "npm:^19.5.0"
-    "@stlib/testing-reporter": "npm:^0.0.1"
+    "@stlib/testing-reporter": "npm:^0.0.2"
     "@types/exit": "npm:^0.1.33"
     "@types/jest": "npm:^29.5.12"
     "@types/node": "npm:^20.12.10"