qase-javascript-commons 2.5.0 → 2.5.1

package/changelog.md

@@ -1,3 +1,9 @@
+# qase-javascript-commons@2.5.1
+
+## What's new
+
+Fixed issue with incorrect test status determination.
+
 # qase-javascript-commons@2.5.0
 
 ## What's new

package/dist/utils/test-status-utils.js

@@ -14,7 +14,7 @@ function determineTestStatus(error, originalStatus) {
         return mapOriginalStatus(originalStatus);
     }
     // Check if it's an assertion error
-    if (isAssertionError(error)) {
+    if (isAssertionError(error, originalStatus)) {
         return test_execution_1.TestStatusEnum.failed;
     }
     // For all other errors, return invalid
@@ -23,16 +23,19 @@ function determineTestStatus(error, originalStatus) {
 /**
  * Checks if error is an assertion error
  * @param error - Error object
+ * @param originalStatus - Original test status from test runner
  * @returns boolean - true if assertion error
  */
-function isAssertionError(error) {
+function isAssertionError(error, originalStatus) {
     const errorMessage = error.message.toLowerCase();
     const errorStack = error.stack?.toLowerCase() || '';
+    const normalizedOriginalStatus = originalStatus.toLowerCase();
     // Common assertion error patterns
     const assertionPatterns = [
         'expect',
         'assert',
         'matcher',
+        'objectcontaining',
         'assertion',
         'expected',
         'actual',
@@ -77,6 +80,14 @@ function isAssertionError(error) {
     const nonAssertionPatternsWithoutTimeout = nonAssertionPatterns.filter(pattern => pattern !== 'timeout');
     const hasNonAssertionPattern = nonAssertionPatternsWithoutTimeout.some(pattern => errorMessage.includes(pattern) || errorStack.includes(pattern));
     if (hasNonAssertionPattern) {
+        const hasAssertionContext = assertionPatterns.some(pattern => errorMessage.includes(pattern) || errorStack.includes(pattern));
+        const isRunnerFailed = normalizedOriginalStatus === 'failed' || normalizedOriginalStatus === 'timedout' || normalizedOriginalStatus === 'interrupted';
+        const isSyntaxError = errorMessage.includes('syntaxerror') || errorStack.includes('syntaxerror');
+        // When runner reported failure and error has assertion context (expect, ObjectContaining, diff),
+        // treat as Failed. Exception: SyntaxError (e.g. "Unexpected token") contains "expected" as false positive.
+        if (isRunnerFailed && hasAssertionContext && !isSyntaxError) {
+            return true;
+        }
         return false;
     }
     // For timeout errors without expect, treat as invalid
@@ -99,6 +110,7 @@ function isAssertionError(error) {
  * @returns TestStatusEnum
  */
 function mapOriginalStatus(originalStatus) {
+    // Keys must be lowercase to match normalizedStatus (case-insensitive matching)
     const statusMap = {
         'passed': test_execution_1.TestStatusEnum.passed,
         'failed': test_execution_1.TestStatusEnum.failed,
@@ -107,10 +119,9 @@ function mapOriginalStatus(originalStatus) {
         'pending': test_execution_1.TestStatusEnum.skipped,
         'todo': test_execution_1.TestStatusEnum.disabled,
         'focused': test_execution_1.TestStatusEnum.passed,
-        'timedOut': test_execution_1.TestStatusEnum.failed,
+        'timedout': test_execution_1.TestStatusEnum.failed,
         'interrupted': test_execution_1.TestStatusEnum.failed,
     };
-    // Convert to lowercase for case-insensitive matching
     const normalizedStatus = originalStatus.toLowerCase();
     return statusMap[normalizedStatus] || test_execution_1.TestStatusEnum.skipped;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "qase-javascript-commons",
-  "version": "2.5.0",
+  "version": "2.5.1",
   "description": "Qase JS Reporters",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",