@testomatio/reporter 2.3.8 → 2.3.9-beta-bin-fix

package/src/bin/cli.js

@@ -35,20 +35,14 @@ program
 program
   .command('start')
   .description('Start a new run and return its ID')
-  .option('--kind <type>', 'Specify run type: automated, manual, or mixed')
-  .action(async (opts) => {
+  .action(async () => {
     cleanLatestRunId();
 
     console.log('Starting a new Run on Testomat.io...');
     const apiKey = process.env['INPUT_TESTOMATIO-KEY'] || config.TESTOMATIO;
     const client = new TestomatClient({ apiKey });
 
-    const createRunParams = {};
-    if (opts.kind) {
-      createRunParams.kind = opts.kind;
-    }
-
-    client.createRun(createRunParams).then(() => {
+    client.createRun().then(() => {
       console.log(process.env.runId);
       process.exit(0);
     });
@@ -81,7 +75,6 @@ program
   .description('Run tests with the specified command')
   .argument('<command>', 'Test runner command')
   .option('--filter <filter>', 'Additional execution filter')
-  .option('--kind <type>', 'Specify run type: automated, manual, or mixed')
   .action(async (command, opts) => {
     const apiKey = process.env['INPUT_TESTOMATIO-KEY'] || config.TESTOMATIO;
     const title = process.env.TESTOMATIO_TITLE;
@@ -127,13 +120,8 @@ program
       });
     };
 
-    const createRunParams = {};
-    if (opts.kind) {
-      createRunParams.kind = opts.kind;
-    }
-
     if (apiKey) {
-      await client.createRun(createRunParams).then(runTests);
+      await client.createRun().then(runTests);
     } else {
       await runTests();
     }
@@ -170,7 +158,7 @@ program
   .option('--lang <lang>', 'Language used (python, ruby, java)')
   .option('--timelimit <time>', 'default time limit in seconds to kill a stuck process')
   .action(async (pattern, opts) => {
-    if (!pattern.endsWith('.xml') && !pattern.includes('*')) {
+    if (!pattern.endsWith('.xml')) {
       pattern += '.xml';
     }
     let { javaTests, lang } = opts;

package/src/bin/reportXml.js

@@ -23,7 +23,7 @@ program
   .option('--timelimit <time>', 'default time limit in seconds to kill a stuck process')
   .option('--env-file <envfile>', 'Load environment variables from env file')
   .action(async (pattern, opts) => {
-    if (!pattern.endsWith('.xml') && !pattern.includes('*')) {
+    if (!pattern.endsWith('.xml')) {
       pattern += '.xml';
     }
     let { javaTests, lang } = opts;
@@ -34,10 +34,7 @@ program
     }
     lang = lang?.toLowerCase();
     if (javaTests === true || (lang === 'java' && !javaTests)) javaTests = 'src/test/java';
-    const runReader = new XmlReader({
-      javaTests,
-      lang,
-    });
+    const runReader = new XmlReader({ javaTests, lang });
     const files = glob.sync(pattern, { cwd: opts.dir || process.cwd() });
     if (!files.length) {
       console.log(APP_PREFIX, `Report can't be created. No XML files found 😥`);

package/src/client.js

@@ -10,21 +10,11 @@ import { glob } from 'glob';
 import path, { sep } from 'path';
 import { fileURLToPath } from 'node:url';
 import { S3Uploader } from './uploader.js';
-import {
-  formatStep,
-  truncate,
-  readLatestRunId,
-  storeRunId,
-  validateSuiteId,
-  transformEnvVarToBoolean
-} from './utils/utils.js';
+import { formatStep, truncate, readLatestRunId, storeRunId, validateSuiteId } from './utils/utils.js';
 import { filesize as prettyBytes } from 'filesize';
-import { stripVTControlCharacters } from 'util';
 
 const debug = createDebugMessages('@testomatio/reporter:client');
 
-const stripColors = stripVTControlCharacters || ((str) => str?.replace(/\x1b\[[0-9;]*m/g, '') || '');
-
 // removed __dirname usage, because:
 // 1. replaced with ESM syntax (import.meta.url), but it throws an error on tsc compilation;
 // 2. got error "__dirname already defined" in compiles js code (cjs dir)
@@ -120,7 +110,7 @@ class Client {
    *
    * @returns {Promise<any>} - resolves to Run id which should be used to update / add test
    */
-  async createRun(params = {}) {
+  async createRun(params) {
     if (!this.pipes || !this.pipes.length)
       this.pipes = await pipesFactory(params || this.paramsForPipesFactory || {}, this.pipeStore);
     debug('Creating run...');
@@ -128,7 +118,7 @@ class Client {
     if (!this.pipes?.filter(p => p.isEnabled).length) return Promise.resolve();
 
     this.queue = this.queue
-      .then(() => Promise.all(this.pipes.map(p => p.createRun(params))))
+      .then(() => Promise.all(this.pipes.map(p => p.createRun())))
       .catch(err => console.log(APP_PREFIX, err))
       .then(() => {
         const runId = this.pipeStore?.runId;
@@ -149,6 +139,19 @@ class Client {
    * @returns {Promise<PipeResult[]>}
    */
   async addTestRun(status, testData) {
+    if (!this.pipes || !this.pipes.length)
+      this.pipes = await pipesFactory(this.paramsForPipesFactory || {}, this.pipeStore);
+
+    // all pipes disabled, skipping
+    if (!this.pipes?.filter(p => p.isEnabled).length) return [];
+
+    if (isTestShouldBeExculedFromReport(testData)) return [];
+
+    if (status === STATUS.SKIPPED && process.env.TESTOMATIO_EXCLUDE_SKIPPED) {
+      debug('Skipping test from report', testData?.title);
+      return []; // do not log skipped tests
+    }
+
     if (!testData)
       testData = {
         title: 'Unknown test',
@@ -166,23 +169,15 @@ class Client {
     const {
       rid,
       error = null,
-      steps: originalSteps,
-      title,
-      suite_title,
-    } = testData;
-    let steps = originalSteps;
-
-    const uploadedFiles = [];
-    const stackArtifactsEnabled = transformEnvVarToBoolean(process.env.TESTOMATIO_STACK_ARTIFACTS);
-
-
-    const {
       time = 0,
       example = null,
       files = [],
       filesBuffers = [],
+      steps,
       code = null,
+      title,
       file,
+      suite_title,
       suite_id,
       test_id,
       timestamp,
@@ -193,6 +188,7 @@ class Client {
     } = testData;
     let { message = '', meta = {} } = testData;
 
+    // stringify meta values and limit keys and values length to 255
     meta = Object.entries(meta)
       .filter(([, value]) => value !== null && value !== undefined)
       .reduce((acc, [key, value]) => {
@@ -200,6 +196,7 @@ class Client {
         return acc;
       }, {});
 
+    // Get links from storage using the test context
     const testContext = suite_title ? `${suite_title} ${title}` : title;
 
     let errorFormatted = '';
@@ -208,39 +205,14 @@ class Client {
       message = error?.message;
     }
 
-    let fullLogs = this.formatLogs({ error: errorFormatted, steps, logs: testData.logs });
-
-    if (stackArtifactsEnabled && fullLogs?.trim()?.length > 0) {
-      uploadedFiles.push(
-        this.uploader.uploadFileAsBuffer(
-          Buffer.from(stripColors(fullLogs), 'utf8'),
-          [this.runId, rid, `logs_${+new Date}.log`]
-        )
-      );
-      fullLogs = '';
-      steps = null;
-    }
-
-
-    if (!this.pipes || !this.pipes.length)
-      this.pipes = await pipesFactory(this.paramsForPipesFactory || {}, this.pipeStore);
-
-    if (!this.pipes?.filter(p => p.isEnabled).length) {
-      if (uploadedFiles.length > 0) {
-        await Promise.all(uploadedFiles);
-      }
-      return [];
-    }
-
-    if (isTestShouldBeExculedFromReport(testData)) return [];
-
-    if (status === STATUS.SKIPPED && process.env.TESTOMATIO_EXCLUDE_SKIPPED) {
-      debug('Skipping test from report', testData?.title);
-      return [];
-    }
+    // Attach logs
+    const fullLogs = this.formatLogs({ error: errorFormatted, steps, logs: testData.logs });
 
+    // add artifacts
     if (manuallyAttachedArtifacts?.length) files.push(...manuallyAttachedArtifacts);
 
+    const uploadedFiles = [];
+
     for (let f of files) {
       if (!f) continue; // f === null
       if (typeof f === 'object') {
@@ -336,7 +308,7 @@ class Client {
           const uploadedArtifacts = this.uploader.successfulUploads.map(file => ({
             relativePath: file.path.replace(process.cwd(), ''),
             link: file.link,
-            sizePretty: file.size == null ? 'unknown' : prettyBytes(file.size, { round: 0 }).toString(),
+            sizePretty: prettyBytes(file.size, { round: 0 }).toString(),
           }));
 
           uploadedArtifacts.forEach(upload => {
@@ -358,7 +330,7 @@ class Client {
           );
           const failedUploads = this.uploader.failedUploads.map(file => ({
             relativePath: file.path.replace(process.cwd(), ''),
-            sizePretty: file.size == null ? 'unknown' : prettyBytes(file.size, { round: 0 }).toString(),
+            sizePretty: prettyBytes(file.size, { round: 0 }).toString(),
           }));
 
           const pathPadding = Math.max(...failedUploads.map(upload => upload.relativePath.length)) + 1;

package/src/junit-adapter/csharp.js

@@ -3,50 +3,18 @@ import Adapter from './adapter.js';
 
 class CSharpAdapter extends Adapter {
   formatTest(t) {
-    // Extract example from title if not already present
-    if (!t.example) {
-      const exampleMatch = t.title.match(/\((.*?)\)/);
-      if (exampleMatch) {
-        // Extract parameters as object with numeric keys for API
-        const params = exampleMatch[1].split(',').map(param => param.trim()).filter(param => param !== '');
-        t.example = {};
-        params.forEach((param, index) => {
-          t.example[index] = param;
-        });
-      }
-    }
-
-    // Remove parameters from title to avoid duplicates in Test Suite
-    // The example field will be used for grouping on import
-    t.title = t.title.replace(/\(.*?\)/, '').trim();
-
+    const title = t.title.replace(/\(.*?\)/, '').trim();
+    const example = t.title.match(/\((.*?)\)/);
+    if (example) t.example = { ...example[1].split(',') };
     const suite = t.suite_title.split('.');
     t.suite_title = suite.pop();
     t.file = namespaceToFileName(t.file);
+    t.title = title.trim();
     return t;
   }
 
   getFilePath(t) {
-    if (!t.file) return null;
-
-    // Normalize path separators for cross-platform compatibility
-    let filePath = t.file.replace(/\\/g, '/');
-
-    // If file already has .cs extension, use it directly
-    if (filePath.endsWith('.cs')) {
-      // Make relative path if it's absolute
-      if (path.isAbsolute(filePath)) {
-        // Try to find project-relative path
-        const cwd = process.cwd().replace(/\\/g, '/');
-        if (filePath.startsWith(cwd)) {
-          filePath = path.relative(cwd, filePath).replace(/\\/g, '/');
-        }
-      }
-      return filePath;
-    }
-
-    // Convert namespace path to file path
-    const fileName = namespaceToFileName(filePath);
+    const fileName = namespaceToFileName(t.file);
     return fileName;
   }
 }
@@ -54,14 +22,7 @@ class CSharpAdapter extends Adapter {
 export default CSharpAdapter;
 
 function namespaceToFileName(fileName) {
-  if (!fileName) return '';
-
-  // If already a .cs file path, clean it up
-  if (fileName.endsWith('.cs')) {
-    return fileName.replace(/\\/g, '/');
-  }
-
   const fileParts = fileName.split('.');
   fileParts[fileParts.length - 1] = fileParts[fileParts.length - 1]?.replace(/\$.*/, '');
-  return `${fileParts.join('/')}.cs`;
+  return `${fileParts.join(path.sep)}.cs`;
 }

package/src/pipe/testomatio.js

@@ -163,7 +163,7 @@ class TestomatioPipe {
 
   /**
    * Creates a new run on Testomat.io
-   * @param {{isBatchEnabled?: boolean, kind?: string}} params
+   * @param {{isBatchEnabled?: boolean}} params
    * @returns Promise<void>
    */
   async createRun(params = {}) {
@@ -204,7 +204,6 @@ class TestomatioPipe {
         label: this.label,
         shared_run: this.sharedRun,
         shared_run_timeout: this.sharedRunTimeout,
-        kind: params.kind,
       }).filter(([, value]) => !!value),
     );
     debug(' >>>>>> Run params', JSON.stringify(runParams, null, 2));

package/src/reporter.js

@@ -1,10 +1,8 @@
-import Client from './client.js';
-import * as TestomatioConstants   from './constants.js';
+// import TestomatClient from './client.js';
+// import * as TRConstants from './constants.js';
 import { services } from './services/index.js';
 import reporterFunctions from './reporter-functions.js';
 
-export { Client };
-export const STATUS = TestomatioConstants.STATUS;
 export const artifact = reporterFunctions.artifact;
 export const log = reporterFunctions.log;
 export const logger = services.logger;
@@ -37,7 +35,6 @@ export default {
   linkTest: reporterFunctions.linkTest,
   linkJira: reporterFunctions.linkJira,
 
-  TestomatioClient: Client,
-  STATUS,
-
+  // TestomatClient,
+  // TRConstants,
 };

package/src/uploader.js

@@ -194,11 +194,6 @@ export class S3Uploader {
       filePath = path.join(process.cwd(), filePath);
     }
 
-    // Normalize path separators for cross-platform compatibility
-    if (typeof filePath === 'string') {
-      filePath = filePath.replace(/\\/g, '/');
-    }
-
     const data = { rid, file: filePath, uploaded };
     const jsonLine = `${JSON.stringify(data)}\n`;
     fs.appendFileSync(tempFilePath, jsonLine);

package/src/utils/utils.js

@@ -76,29 +76,21 @@ const isValidUrl = s => {
   }
 };
 
-const fileMatchRegex = /file:(\/*)([A-Za-z]:[\\/].*?|\/.*?)\.(png|avi|webm|jpg|html|txt)/gi;
+const fileMatchRegex = /file:(\/+(?:[A-Za-z]:[\\/]|\/)?[^\s]*?\.(png|avi|webm|jpg|html|txt))/gi;
 
 const fetchFilesFromStackTrace = (stack = '', checkExists = true) => {
-  let files = Array.from(stack.matchAll(fileMatchRegex))
-    .map(match => {
-      // match[0] is full match, match[1] is slashes, match[2] is path, match[3] is extension
-      const slashes = match[1] || '';
-      const path = match[2];
-      const extension = match[3];
-      return `${slashes}${path}.${extension}`;
-    })
-    .map(f => f.trim())
+  const files = Array.from(stack.matchAll(fileMatchRegex))
+    .map(f => f[1].trim())
     .map(f => f.replace(/^\/+/, '/').replace(/^\/([A-Za-z]:)/, '$1')) // Remove extra slashes, handle Windows paths
     .map(f => {
-      // Normalize path separators for cross-platform compatibility
-      return f.replace(/\\/g, '/');
+      // Convert Windows paths to Linux paths for testing purposes
+      if (f.match(/^[A-Za-z]:[\\\/]/)) {
+        // Convert Windows path to Linux equivalent for test scenarios
+        return f.replace(/^[A-Za-z]:[\\\/]/, '/').replace(/\\/g, '/');
+      }
+      return f;
     });
 
-  // If we're not checking file existence, remove Windows drive letters for consistency
-  if (!checkExists) {
-    files = files.map(f => f.replace(/^([A-Za-z]):/, ''));
-  }
-
   debug('Found files in stack trace: ', files);
 
   return files.filter(f => {
@@ -113,92 +105,21 @@ const fetchSourceCodeFromStackTrace = (stack = '') => {
   const stackLines = stack
     .split('\n')
     .filter(l => l.includes(':'))
+    // .map(l => l.match(/\[(.*?)\]/)?.[1] || l) // minitest format
+    // .map(l => l.split(':')[0])
     .map(l => l.trim())
-    .map(l => {
-      // Remove 'at ' prefix if present
-      if (l.startsWith('at ')) {
-        return l.substring(3).trim();
-      }
-      // Find the part that looks like a file path with line number
-      const parts = l.split(' ');
-      for (const part of parts) {
-        // Check if this part has a colon
-        if (part.includes(':')) {
-          // For Windows paths, we need to handle drive letters (C:, D:, etc.)
-          // Split by colon but keep drive letter with the path
-          const colonParts = part.split(':');
-          let filePath;
-
-          // Check if first part is a Windows drive letter (single letter)
-          if (colonParts.length >= 2 && colonParts[0].length === 1 && /[A-Za-z]/.test(colonParts[0])) {
-            // Windows path like D:\path\file.php:24
-            // Reconstruct as D:\path\file.php
-            filePath = colonParts[0] + ':' + colonParts[1];
-          } else {
-            // Unix path like /path/file.php:24
-            filePath = colonParts[0];
-          }
-
-          // Only consider it valid if the file exists
-          if (fs.existsSync(filePath)) {
-            return part;
-          }
-        }
-      }
-      // If no valid file path found in parts, return the whole line
-      // It will be filtered out later if it's not a valid file path
-      return parts.find(p => p.includes(':')) || l;
-    })
-    .filter(l => {
-      // Extract file path from line (accounting for Windows drive letters)
-      if (!l) return false;
-      const colonParts = l.split(':');
-      let filePath;
-
-      if (colonParts.length >= 2 && colonParts[0].length === 1 && /[A-Za-z]/.test(colonParts[0])) {
-        // Windows path
-        filePath = colonParts[0] + ':' + colonParts[1];
-      } else {
-        // Unix path
-        filePath = colonParts[0];
-      }
-
-      return filePath && fs.existsSync(filePath);
-    })
+    .map(l => l.split(' ').find(p => p.includes(':')) || '')
+    .filter(l => isValid(l?.split(':')[0]))
 
     // // filter out 3rd party libs
     .filter(l => !l?.includes(`vendor${sep}`))
     .filter(l => !l?.includes(`node_modules${sep}`))
-    .filter(l => {
-      // Extract file path for final check (accounting for Windows drive letters)
-      const colonParts = l.split(':');
-      let filePath;
-
-      if (colonParts.length >= 2 && colonParts[0].length === 1 && /[A-Za-z]/.test(colonParts[0])) {
-        filePath = colonParts[0] + ':' + colonParts[1];
-      } else {
-        filePath = colonParts[0];
-      }
-
-      return fs.lstatSync(filePath).isFile();
-    });
+    .filter(l => fs.existsSync(l.split(':')[0]))
+    .filter(l => fs.lstatSync(l.split(':')[0]).isFile());
 
   if (!stackLines.length) return '';
 
-  // Extract file and line number (accounting for Windows drive letters)
-  const firstLine = stackLines[0];
-  const colonParts = firstLine.split(':');
-  let file, line;
-
-  if (colonParts.length >= 3 && colonParts[0].length === 1 && /[A-Za-z]/.test(colonParts[0])) {
-    // Windows path like D:\path\file.php:24
-    file = colonParts[0] + ':' + colonParts[1];
-    line = colonParts[2];
-  } else {
-    // Unix path like /path/file.php:24
-    file = colonParts[0];
-    line = colonParts[1];
-  }
+  const [file, line] = stackLines[0].split(':');
 
   const prepend = 3;
   const source = fetchSourceCode(fs.readFileSync(file).toString(), { line, prepend, limit: 7 });
@@ -218,8 +139,6 @@ export const TEST_ID_REGEX = /@T([\w\d]{8})/;
 export const SUITE_ID_REGEX = /@S([\w\d]{8})/;
 
 const fetchIdFromCode = (code, opts = {}) => {
-  if (!code) return null;
-
   const comments = code
     .split('\n')
     .map(l => l.trim())
@@ -261,65 +180,8 @@ const fetchSourceCode = (contents, opts = {}) => {
       if (lineIndex === -1) lineIndex = lines.findIndex(l => l.includes(`public void ${title}`));
       if (lineIndex === -1) lineIndex = lines.findIndex(l => l.includes(`${title}(`));
     } else if (opts.lang === 'csharp') {
-      // Find the method declaration line
-      let methodLineIndex = lines.findIndex(l => l.includes(`public void ${title}(`));
-
-      if (methodLineIndex === -1) {
-        methodLineIndex = lines.findIndex(l => l.includes(`public async Task ${title}(`));
-      }
-
-      if (methodLineIndex === -1) {
-        methodLineIndex = lines.findIndex(l => l.includes(`${title}(`));
-      }
-
-      // If found, scan upwards to find [TestCase], [Test] attributes and XML comments
-      if (methodLineIndex !== -1) {
-        lineIndex = methodLineIndex;
-
-        // Scan upwards to find the start of attributes and comments
-        for (let i = methodLineIndex - 1; i >= 0; i--) {
-          const trimmedLine = lines[i].trim();
-
-          // Include [TestCase], [Test], and other attributes
-          if (trimmedLine.startsWith('[')) {
-            lineIndex = i;
-            continue;
-          }
-
-          // Include XML documentation comments
-          if (trimmedLine.startsWith('///')) {
-            lineIndex = i;
-            continue;
-          }
-
-          // Stop at empty lines (with some tolerance)
-          if (trimmedLine === '') {
-            // Check if next non-empty line is an attribute or comment
-            let hasMoreAttributes = false;
-            for (let j = i - 1; j >= 0; j--) {
-              const nextTrimmed = lines[j].trim();
-              if (nextTrimmed === '') continue;
-              if (nextTrimmed.startsWith('[') || nextTrimmed.startsWith('///')) {
-                hasMoreAttributes = true;
-                lineIndex = j;
-              }
-              break;
-            }
-            if (!hasMoreAttributes) break;
-            continue;
-          }
-
-          // Stop at other method declarations or class-level elements
-          if (
-            trimmedLine.includes('public ') ||
-            trimmedLine.includes('private ') ||
-            trimmedLine.includes('protected ') ||
-            trimmedLine.includes('internal ')
-          ) {
-            if (!trimmedLine.startsWith('[')) break;
-          }
-        }
-      }
+      if (lineIndex === -1) lineIndex = lines.findIndex(l => l.includes(`public void ${title}`));
+      if (lineIndex === -1) lineIndex = lines.findIndex(l => l.includes(`${title}(`));
     } else {
       lineIndex = lines.findIndex(l => l.includes(title));
     }
@@ -329,31 +191,11 @@ const fetchSourceCode = (contents, opts = {}) => {
     lineIndex -= opts.prepend;
   }
 
-  if (lineIndex !== -1 && lineIndex !== undefined) {
+  if (lineIndex) {
     const result = [];
-    let braceDepth = 0; // Track brace depth for C# methods
-    let methodStartFound = false; // Flag to indicate we've found the method opening brace
-
     for (let i = lineIndex; i < lineIndex + limit; i++) {
       if (lines[i] === undefined) continue;
 
-      // Track brace depth for C# to stop after method closes
-      if (opts.lang === 'csharp') {
-        const line = lines[i];
-        // Count opening and closing braces
-        const openBraces = (line.match(/\{/g) || []).length;
-        const closeBraces = (line.match(/\}/g) || []).length;
-
-        if (openBraces > 0) methodStartFound = true;
-        braceDepth += openBraces - closeBraces;
-
-        // If we've started the method and depth returns to 0, method is complete
-        if (methodStartFound && braceDepth === 0 && closeBraces > 0) {
-          // Don't include the closing brace - just break
-          break;
-        }
-      }
-
       if (i > lineIndex + 2 && !opts.prepend) {
         // annotation
         if (opts.lang === 'php' && lines[i].trim().startsWith('#[')) break;
@@ -374,25 +216,7 @@ const fetchSourceCode = (contents, opts = {}) => {
         if (opts.lang === 'java' && lines[i].trim().match(/^@\w+/)) break;
         if (opts.lang === 'java' && lines[i].includes(' public void ')) break;
         if (opts.lang === 'java' && lines[i].includes(' class ')) break;
-        // For C#, additional checks if brace tracking didn't stop us
-        if (opts.lang === 'csharp') {
-          const trimmed = lines[i].trim();
-          // Stop at attribute that marks beginning of next test (but not if we're still in the current method)
-          if (trimmed.match(/^\[(Test|TestCase|Theory|Fact)/) && methodStartFound && braceDepth === 0) break;
-          // Stop at XML documentation comments that belong to next method
-          if (trimmed.startsWith('///') && methodStartFound && braceDepth === 0) break;
-          // Stop at another method declaration (but not if we're still in the current method)
-          if (
-            trimmed.match(/^\s*(public|private|protected|internal)\s+(\w+|async\s+\w+)\s+\w+\s*\(/) &&
-            methodStartFound &&
-            braceDepth === 0
-          )
-            break;
-          // Stop at class declaration
-          if (trimmed.includes(' class ') && trimmed.includes('public')) break;
-        }
       }
-
       result.push(lines[i]);
     }
     return result.join('\n');
@@ -605,14 +429,10 @@ function transformEnvVarToBoolean(value) {
 }
 
 function truncate(s, size = 255) {
-  if (s === undefined || s === null) {
-    return '';
-  }
-  const str = s.toString();
-  if (str.trim().length < size) {
-    return str;
+  if (s.toString().trim().length < size) {
+    return s.toString();
   }
-  return `${str.substring(0, size)}...`;
+  return `${s.toString().substring(0, size)}...`;
 }
 
 export {