Commit 1816d46
committed
test: adapt test-linux-perf to V8 changes
Refs: v8/v8@9a31804
PR-URL: #38273
Backport-PR-URL: #38991
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Mary Marchini <oss@mmarchini.me>1 parent 32961c4 commit 1816d46
1 file changed
Lines changed: 11 additions & 8 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -96,17 +96,20 @@ for (const perfArgs of perfArgsList) { | |||
| 96 | 96 | output += perfScript.stdout; | |
| 97 | 97 | } | |
| 98 | 98 | ||
| 99 | - const interpretedFunctionOneRe = /InterpretedFunction:functionOne/; | ||
| 100 | - const compiledFunctionOneRe = /LazyCompile:\*functionOne/; | ||
| 101 | - const interpretedFunctionTwoRe = /InterpretedFunction:functionTwo/; | ||
| 102 | - const compiledFunctionTwoRe = /LazyCompile:\*functionTwo/; | ||
| 99 | + const interpretedFunctionOneRe = /~functionOne/; | ||
| 100 | + const compiledFunctionOneRe = /\*functionOne/; | ||
| 101 | + const interpretedFunctionTwoRe = /~functionTwo/; | ||
| 102 | + const compiledFunctionTwoRe = /\*functionTwo/; | ||
| 103 | 103 | ||
| 104 | + function makeAssertMessage(message) { | ||
| 105 | + return message + '\nPerf output:\n\n' + output; | ||
| 106 | + } | ||
| 104 | 107 | ||
| 105 | 108 | assert.ok(output.match(interpretedFunctionOneRe), | |
| 106 | - "Couldn't find interpreted functionOne()"); | ||
| 109 | + makeAssertMessage("Couldn't find interpreted functionOne()")); | ||
| 107 | 110 | assert.ok(output.match(compiledFunctionOneRe), | |
| 108 | - "Couldn't find compiled functionOne()"); | ||
| 111 | + makeAssertMessage("Couldn't find compiled functionOne()")); | ||
| 109 | 112 | assert.ok(output.match(interpretedFunctionTwoRe), | |
| 110 | - "Couldn't find interpreted functionTwo()"); | ||
| 113 | + makeAssertMessage("Couldn't find interpreted functionTwo()")); | ||
| 111 | 114 | assert.ok(output.match(compiledFunctionTwoRe), | |
| 112 | - "Couldn't find compiled functionTwo"); | ||
| 115 | + makeAssertMessage("Couldn't find compiled functionTwo")); | ||
0 commit comments