Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 0ff419b

Browse files
authoredJun 15, 2024··
Fix GitHub Issue #214 (#274)
1 parent e0e796f commit 0ff419b

File tree

3 files changed

+11
-7
lines changed

3 files changed

+11
-7
lines changed
 

‎dist/index.js

Lines changed: 5 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/index.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎src/model/results-check.ts

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -76,10 +76,12 @@ const ResultsCheck = {
7676
const pullRequest = github.context.payload.pull_request;
7777
const headSha = (pullRequest && pullRequest.head.sha) || github.context.sha;
7878

79+
// Check max length for https://github.com/game-ci/unity-test-runner/issues/214
7980
const maxLength = 65_534;
80-
if (output.length > maxLength) {
81-
core.warning(`Output too long (${output.length}) truncating to ${maxLength}`);
82-
output = output.slice(0, maxLength);
81+
if (output.text.length > maxLength) {
82+
core.warning(`Test details of ${output.text.length} surpass limit of ${maxLength}`);
83+
output.text =
84+
'Test details omitted from GitHub UI due to length. See console logs for details.';
8385
}
8486

8587
core.info(`Posting results for ${headSha}`);

0 commit comments

Comments
 (0)
Please sign in to comment.