Update error reporting

bin/check.js

@@ -11,6 +11,7 @@ const branch = process.env.TESTOMATIO_BRANCH;
 const debug = require('debug')('testomatio:check');
 const { version } = require('../package.json');
 const { TEST_ID_REGEX } = require('../src/updateIds/constants');
+const { formatErrorMessage } = require('../src/lib/utils');
 console.log(chalk.cyan.bold(` 🤩 Tests checker by Testomat.io v${version}`));
 
 process.env.isTestomatioCli = true;
@@ -245,7 +246,7 @@ program
         console.log('\n✨ Pull completed successfully!');
       }
     } catch (error) {
-      console.error(' ✖️  Failed to pull files:', error.message);
+      console.error(' ✖️  Failed to pull files:', formatErrorMessage(error));
       process.exit(1);
     }
   });

src/lib/utils.js

@@ -260,6 +260,30 @@ function getAllSuiteTags(currentSuite) {
   return tags;
 }
 
+function parseErrorBody(body) {
+  try {
+    return typeof body === 'string' ? JSON.parse(body.trim()) : body;
+  } catch {
+    return body;
+  }
+}
+
+function formatErrorMessage(error) {
+  if (!error) return 'Unknown error';
+  if (typeof error === 'string') return error.trim() || 'Unknown error';
+  const response = error.response || error.res || {};
+  const status = error.statusCode || error.status || response.statusCode || response.status;
+  const statusText = error.statusMessage || error.statusText || response.statusMessage || response.statusText;
+  const body = parseErrorBody(error.body || error.data || response.body || response.data);
+  const bodyMessage = typeof body === 'string' ? body.trim() : body?.message || body?.error || body?.errors;
+  const message =
+    status &&
+    `Request ${Number(status) === 504 ? 'timed out' : 'failed'} (${status}${statusText ? ` ${statusText}` : ''})`;
+  if (message)
+    return bodyMessage ? `${message}: ${Array.isArray(bodyMessage) ? bodyMessage.join(', ') : bodyMessage}` : message;
+  return error.message || JSON.stringify(error) || String(error);
+}
+
 module.exports = {
   hasStringArgument,
   hasTemplateQuasi,
@@ -278,4 +302,5 @@ module.exports = {
   playwright,
   arrayCompare,
   getAllSuiteTags,
+  formatErrorMessage,
 };

src/pull.js

@@ -2,6 +2,7 @@ const fs = require('fs');
 const path = require('path');
 const { execSync } = require('child_process');
 const debug = require('debug')('testomatio:pull');
+const { formatErrorMessage } = require('./lib/utils');
 
 class Pull {
   constructor(reporter, workDir = '.', options = {}) {
@@ -81,7 +82,7 @@ class Pull {
         return createdFiles;
       }
     } catch (error) {
-      console.error('Error pulling files:', error.message);
+      console.error('Error pulling files:', formatErrorMessage(error));
       throw error;
     }
   }

src/reporter.js

@@ -4,6 +4,7 @@ const debug = require('debug')('testomatio:ids');
 const { request } = isHttps ? require('https') : require('http');
 const path = require('path');
 const fs = require('fs');
+const { formatErrorMessage } = require('./lib/utils');
 const { TEST_ID_REGEX } = require('./updateIds/constants');
 
 class Reporter {
@@ -61,7 +62,17 @@ class Reporter {
             debug('Files fetched from Testomat.io', message);
             if (resp.statusCode !== 200) {
               debug('Files fetch failed', resp.statusCode, resp.statusMessage, message);
-              rej(message);
+              const error = new Error(
+                formatErrorMessage({
+                  statusCode: resp.statusCode,
+                  statusMessage: resp.statusMessage,
+                  body: message,
+                }),
+              );
+              error.statusCode = resp.statusCode;
+              error.statusMessage = resp.statusMessage;
+              error.body = message;
+              rej(error);
             } else {
               res(JSON.parse(message));
             }

tests/error_message_test.js

@@ -0,0 +1,31 @@
+const { expect } = require('chai');
+const { formatErrorMessage } = require('../src/lib/utils');
+
+describe('formatErrorMessage', () => {
+  it('should format 504 responses without a body', () => {
+    const message = formatErrorMessage({
+      statusCode: 504,
+      statusMessage: 'Gateway Timeout',
+    });
+
+    expect(message).to.equal('Request timed out (504 Gateway Timeout)');
+  });
+
+  it('should use server body message when response has a body', () => {
+    const message = formatErrorMessage({
+      statusCode: 500,
+      statusMessage: 'Internal Server Error',
+      body: JSON.stringify({ message: 'Export failed' }),
+    });
+
+    expect(message).to.equal('Request failed (500 Internal Server Error): Export failed');
+  });
+
+  it('should keep regular Error messages', () => {
+    expect(formatErrorMessage(new Error('Server error'))).to.equal('Server error');
+  });
+
+  it('should not return undefined for plain objects', () => {
+    expect(formatErrorMessage({ code: 'ECONNRESET' })).to.equal('{"code":"ECONNRESET"}');
+  });
+});