Skip to content

Commit

Permalink
Adjust bugFiler to support new crash format and STB for crashes (#7748)
Browse files Browse the repository at this point in the history
  • Loading branch information
jmaher committed Jul 18, 2023
1 parent ea91730 commit 1fa90c0
Showing 1 changed file with 11 additions and 2 deletions.
13 changes: 11 additions & 2 deletions ui/shared/BugFiler.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -191,6 +191,8 @@ export class BugFilerClass extends React.Component {
/.*test_.*\.xhtml/, // mochitest-chrome
/.*browser_.*\.html/, // b-c
/.*browser_.*\.js/, // b-c
/.*\.ini/, // when we have a failure on shutdown (crash/leak/timeout)
/.*\.toml/, // when we have a failure on shutdown (crash/leak/timeout)
/.*org.mozilla.geckoview.test.*/, // junit
].some((regexp) => regexp.test(summaryString));

Expand All @@ -204,6 +206,13 @@ export class BugFilerClass extends React.Component {
/.*\.xht(\?.*| )\|/,
/.*\.mp4 \|/, // reftest specific
/.*\.webm \|/, // reftest specific
/ \| .*\.js(\?.*)?/, // crash format
/ \| .*\.html(\?.*)?/,
/ \| .*\.htm(\?.*)?/,
/ \| .*\.xhtml(\?.*)?/,
/ \| .*\.xht(\?.*)?/,
/ \| .*.mp4/, // reftest specific
/ \| .*\.webm/, // reftest specific
].some((regexp) => regexp.test(summaryString));
}

Expand All @@ -221,8 +230,8 @@ export class BugFilerClass extends React.Component {
trimParams = true;
}

// If not crash|leak
if (!crash && !isAssertion && isTestPath) {
// If not leak
if (!isAssertion && isTestPath) {
const parts = summaryString.split(' | ');
// split('?') is for removing `?params...` from the test name
if (parts.length === 2 || parts.length === 1) {
Expand Down

0 comments on commit 1fa90c0

Please sign in to comment.