Skip to content

[Test] Fix CrashReportTest to support non-product builds - #164

Open
sendaoYan wants to merge 1 commit into
bytedance:jdk17u-target8from
sendaoYan:issue-163
Open

[Test] Fix CrashReportTest to support non-product builds#164
sendaoYan wants to merge 1 commit into
bytedance:jdk17u-target8from
sendaoYan:issue-163

Conversation

@sendaoYan

Copy link
Copy Markdown
Collaborator

The hs_err JRE version line format differs between product and non-product builds. For product builds the line contains "(build )", while for fastdebug/slowdebug builds it contains "(fastdebug build )" or "(slowdebug build )". The test regex previously only matched the product build format, causing the test to fail on fastdebug/slowdebug builds.

Fix the regex to optionally match the debug level prefix before "build":
(build -> ((?:fastdebug |slowdebug )?build

Issue: #163

…g) builds

The hs_err JRE version line format differs between product and non-product
builds. For product builds the line contains "(build <version>)", while for
fastdebug/slowdebug builds it contains "(fastdebug build <version>)" or
"(slowdebug build <version>)". The test regex previously only matched the
product build format, causing the test to fail on fastdebug/slowdebug builds.

Fix the regex to optionally match the debug level prefix before "build":
  \(build  ->  \((?:fastdebug |slowdebug )?build

Issue: bytedance#163
@sendaoYan

Copy link
Copy Markdown
Collaborator Author

github action job报错,看起来像是环境问题

Retrying in 19.513 seconds
Error: Bad Gateway
Error: Failed to resolve action download info.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant