erabo/spiegel-keyman
1
0
Fork 0
mirror of https://github.com/keymanapp/keyman.git synced 2026-08-05 00:15:32 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

maint(web): escape single quotes in TC service messages

Browse source 
This fixes a problem where TC shows an error because it can't find the
end of the service message for a failed test. Single quotes (and some
other characters) have to be escaped with `|'` inside of a message.

Build-bot: skip:all build:web
Test-bot: skip
This commit is contained in:
Eberhard Beilharz 2025-12-19 10:07:29 +01:00
parent 52812dd18f
commit 0dbbcc452e
No known key found for this signature in database
GPG key ID: E9140597606020D3
1 changed files with 11 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
common/test/resources/playwright-TC-reporter.ts
View file

@ -62,6 +62,15 @@ class TestNode {
TestNode.OpenNodes.push(this.id);
}
private escape(message: string): string {
// TeamCity escaping rules for: ' | [ ] \n \r \uNNNN
// See: https://www.jetbrains.com/help/teamcity/service-messages.html#Escaped+Values
return message?.replace(/['|\[\]]/g, (matched) => `|${matched}`)
.replace(/\n/g, '|n')
.replace(/\r/g, '|r')
.replace(/[\u0080-\uFFFF]/g, c => `|0x${c.charCodeAt(0).toString(16).padStart(4, '0')}`) ?? '';
}
private getTestResult(result: TestResult): { msgTitle: string, details: string } {
if (!result) {
return null;
@ -72,9 +81,9 @@ class TestNode {
case 'failed':
case 'interrupted':
case 'timedOut':
return { msgTitle: 'testFailed', details: `message='${result.error?.message}' details='${result.error?.value ?? result.error?.cause}'` };
return { msgTitle: 'testFailed', details: `message='${this.escape(result.error?.message)}' details='${this.escape(result.error?.value ?? result.error?.cause)}'` };
case 'skipped':
return { msgTitle: 'testIgnored', details: `message='${result.annotations?.toString() ?? ''}'` };
return { msgTitle: 'testIgnored', details: `message='${this.escape(result.annotations?.toString()) ?? ''}'` };
}
}