qase-javascript-commons 2.4.6 → 2.4.8

package/changelog.md

@@ -1,3 +1,15 @@
+# qase-javascript-commons@2.4.8
+
+## What's new
+
+Improved error handling for timeout errors.
+
+# qase-javascript-commons@2.4.7
+
+## Bug fixes
+
+Fixed issue where test results with unknown statuses were incorrectly marked as `skipped` instead of `passed` by default. Now status matching is case-insensitive for better compatibility with different test frameworks.
+
 # qase-javascript-commons@2.4.6
 
 ## What's new

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

@@ -67,11 +67,22 @@ function isAssertionError(error) {
         'server error',
         'internal error'
     ];
-    // Check for non-assertion patterns first
-    const hasNonAssertionPattern = nonAssertionPatterns.some(pattern => errorMessage.includes(pattern) || errorStack.includes(pattern));
+    // Special case: timeout errors with expect should be treated as assertion errors
+    const isTimeoutWithExpect = errorMessage.includes('timeout') &&
+        errorMessage.includes('expect');
+    if (isTimeoutWithExpect) {
+        return true;
+    }
+    // Check for non-assertion patterns (excluding timeout for special handling above)
+    const nonAssertionPatternsWithoutTimeout = nonAssertionPatterns.filter(pattern => pattern !== 'timeout');
+    const hasNonAssertionPattern = nonAssertionPatternsWithoutTimeout.some(pattern => errorMessage.includes(pattern) || errorStack.includes(pattern));
     if (hasNonAssertionPattern) {
         return false;
     }
+    // For timeout errors without expect, treat as invalid
+    if (errorMessage.includes('timeout')) {
+        return false;
+    }
     // Check error message and stack for assertion patterns
     const hasAssertionPattern = assertionPatterns.some(pattern => errorMessage.includes(pattern) || errorStack.includes(pattern));
     // Check for specific error types that indicate assertions
@@ -99,5 +110,7 @@ function mapOriginalStatus(originalStatus) {
         'timedOut': test_execution_1.TestStatusEnum.failed,
         'interrupted': test_execution_1.TestStatusEnum.failed,
     };
-    return statusMap[originalStatus] || test_execution_1.TestStatusEnum.skipped;
+    // 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.4.6",
+  "version": "2.4.8",
   "description": "Qase JS Reporters",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",