froth-webdriverio-framework 6.0.77 → 6.0.78

package/froth_api_calls/getexecutionDetails.js

@@ -120,8 +120,6 @@ async function updateExecuitonDetails(frothUrl, token, id, resultdetails) {
     const url = `${frothUrl}/api/test-execution-update/${id}/`;
     const formData = new FormData();
 
-    console.log('PUT:', url);
-
     try {
         formData.append('updated_through_bot', true);
 
@@ -139,42 +137,34 @@ async function updateExecuitonDetails(frothUrl, token, id, resultdetails) {
         if (Array.isArray(resultdetails.comments) && resultdetails.comments.length) {
             formData.append('comments', resultdetails.comments.join('<br>'));
         }
-        // ✅ Log actual FormData keys
-        console.log('📤 FormData keys sent:');
-        for (const [key, value] of formData.entries()) {
-            console.log(`   ${key}: ${value}`);
-        }
+
+        console.log('📤 Payload sent to API:', JSON.stringify(resultdetails, null, 2));
         const response = await fetch(url, {
             method: 'PUT',
             headers: { 'Authorization': `Bearer ${token}` },
             body: formData
         });
-        // Log raw response info
-        console.log('📤 API Response Status:', response.status);
-        console.log('📤 API Response Headers:', JSON.stringify([...response.headers]));
-        let responseText;
-        try {
-            responseText = await response.text();
-            console.log('📤 API Response Body (text):', responseText);
-        } catch (err) {
-            console.error('❌ Error reading response body:', err.message);
-        }
 
-        // Then handle response as JSON if needed
+        // Read the response once and log it
+        const contentType = response.headers.get('content-type') || '';
         let data;
-        try {
-            data = JSON.parse(responseText);
-        } catch {
-            data = null;
+        if (contentType.includes('application/json')) {
+            data = await response.json();
+        } else {
+            data = await response.text();
         }
 
-        console.log('📤 Parsed Response:', data);
+        console.log('📤 API Response Status:', response.status);
+        console.log('📤 API Response Headers:', Array.from(response.headers.entries()));
+        console.log('📤 API Response Body:', data);
 
-        const result = await handleResponse(response, 'updateExecuitonDetails');
-        if (!result) return;
+        if (!response.ok) {
+            console.error(`❌ Failed to update execution details, status ${response.status}`);
+            return;
+        }
 
         BUFFER.setItem('FROTH_UPDATE_EXECUTION', 'TRUE');
-        console.log('✅ Execution details updated');
+        console.log('✅ Execution details updated successfully');
 
     } catch (error) {
         console.error('❌ updateExecuitonDetails error:', error.message);

package/froth_configs/commonhook.js

@@ -81,6 +81,12 @@ async function failExecution(reason) {
   await safeUpdateExecution();
   process.exit(1);
 }
+
+async function pushComment(msg) {
+  const comments = BUFFER.getItem('RESULT_COMMENTS') || [];
+  comments.push(msg);
+  BUFFER.setItem('RESULT_COMMENTS', comments);
+}
 /* ------------------ COMMON CONFIG ------------------ */
 
 const commonHooks = {
@@ -128,8 +134,7 @@ const commonHooks = {
 
       if (process.env.PLATFORM === 'browserstack' || process.env.PLATFORM === 'browserstacklocal') {
         const bsOpts = capabilities['bstack:options'] || {};
-        process.env.BROWSERSTACK_USERNAME = bsOpts.userName
-        process.env.BROWSERSTACK_ACCESS_KEY = bsOpts.accessKey
+
         console.log('✅ BS_USERNAME set from capabilities:', process.env.BROWSERSTACK_USERNAME);
 
         if (process.env.BS_SESSION_TYPE === 'app-automate') {
@@ -196,11 +201,8 @@ const commonHooks = {
 
     const fileName = path.basename(test.file);
     const status = passed ? 'PASSED' : 'FAILED';
-
-    if (!passed) {
-      resultdetails.comments.push(`${test.title} - ${error?.message || 'Failed'}`);
-    } else
-      resultdetails.comments.push(`${test.title} - 'Passed'}`);
+    const msg = `${test.title} - ${passed ? 'Passed' : error?.message || 'Failed'}`;
+    pushComment(msg);
 
     const suiteDetails = JSON.parse(BUFFER.getItem('FROTHE_SUITE_DETAILS'));
     const script = suiteDetails.find(s => s.scriptName === fileName.replace('.js', ''));
@@ -262,10 +264,17 @@ const commonHooks = {
     resultdetails.excution_time = await msToTime(totalTime);
     resultdetails.excution_status = exitCode === 0 ? 'PASSED' : 'FAILED';
 
+    const allComments = BUFFER.getItem('RESULT_COMMENTS') || [];
+
+    // In positive case, you can still push a success comment
+    if (allComments.length === 0) {
+      allComments.push(`All tests passed ✅`);
+    }
     if (exitCode !== 0 && resultdetails.comments.length === 0) {
       resultdetails.comments.push(`Execution failed with exit code ${exitCode}`);
     }
-
+    resultdetails.comments = allComments;
+    console.log('Comments being sent:', resultdetails.comments);
     console.log('⏱ Final execution time (ms):', totalTime);
     console.log('⏱ Final execution time (hh:mm:ss):', resultdetails.excution_time);
 

package/froth_configs/wdio.common.conf.js

@@ -18,7 +18,8 @@ try {
   }
   const ymlPath = path.resolve(process.cwd(), configFile);
   bsCaps = yaml.load(fs.readFileSync(ymlPath, 'utf8'));
- 
+  process.env.BROWSERSTACK_USERNAME = bsCaps.userName
+  process.env.BROWSERSTACK_ACCESS_KEY = Buffer.from(bsCaps.accessKey, 'base64').toString('utf-8'),
   // Merge chrome-specific options if applicable
 
 } catch (e) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "froth-webdriverio-framework",
-  "version": "6.0.77",
+  "version": "6.0.78",
   "readme": "WendriverIO Integration with [BrowserStack](https://www.browserstack.com)",
   "description": "Selenium examples for WebdriverIO and BrowserStack App Automate",
   "scripts": {