fix: Move output to stderr #2123 (#2125)

This change moves informational command outputs from console.log (prints on stdout) to console.warn/console.error (prints on stderr)
to enable stdout processing in pipelines.

Co-authored-by: Anoop M D <anoop.md1421@gmail.com>
This commit is contained in:
Johannes Zorn 2024-06-05 17:05:28 +02:00 committed by GitHub
parent f9b33dde38
commit 1b4d9b8f6e
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -406,8 +406,7 @@ const handler = async function (argv) {
return aSequence - bSequence; return aSequence - bSequence;
}); });
} else { } else {
console.log(chalk.yellow('Running Folder Recursively \n')); console.warn(chalk.yellow('Running Folder Recursively \n'));
bruJsons = getBruFilesRecursively(filename, testsOnly); bruJsons = getBruFilesRecursively(filename, testsOnly);
} }
} }
@ -461,7 +460,7 @@ const handler = async function (argv) {
if (nextRequestIdx >= 0) { if (nextRequestIdx >= 0) {
currentRequestIndex = nextRequestIdx; currentRequestIndex = nextRequestIdx;
} else { } else {
console.error("Could not find request with name '" + nextRequestName + "'"); console.warn("Could not find request with name '" + nextRequestName + "'");
currentRequestIndex++; currentRequestIndex++;
} }
} else { } else {
@ -471,7 +470,7 @@ const handler = async function (argv) {
const summary = printRunSummary(results); const summary = printRunSummary(results);
const totalTime = results.reduce((acc, res) => acc + res.response.responseTime, 0); const totalTime = results.reduce((acc, res) => acc + res.response.responseTime, 0);
console.log(chalk.dim(chalk.grey(`Ran all requests - ${totalTime} ms`))); console.error(chalk.dim(chalk.grey(`Ran all requests - ${totalTime} ms`)));
if (outputPath && outputPath.length) { if (outputPath && outputPath.length) {
const outputDir = path.dirname(outputPath); const outputDir = path.dirname(outputPath);
@ -494,15 +493,14 @@ const handler = async function (argv) {
makeHtmlOutput(outputJson, outputPath); makeHtmlOutput(outputJson, outputPath);
} }
console.log(chalk.dim(chalk.grey(`Wrote results to ${outputPath}`))); console.error(chalk.dim(chalk.grey(`Wrote results to ${outputPath}`)));
} }
if (summary.failedAssertions + summary.failedTests + summary.failedRequests > 0) { if (summary.failedAssertions + summary.failedTests + summary.failedRequests > 0) {
process.exit(constants.EXIT_STATUS.ERROR_FAILED_COLLECTION); process.exit(constants.EXIT_STATUS.ERROR_FAILED_COLLECTION);
} }
} catch (err) { } catch (err) {
console.log('Something went wrong'); console.error(chalk.red('Something went wrong: ' + err.message));
console.error(chalk.red(err.message));
process.exit(constants.EXIT_STATUS.ERROR_GENERIC); process.exit(constants.EXIT_STATUS.ERROR_GENERIC);
} }
}; };