[Webkit-unassigned] [Bug 159224] Use a regex to check if a test step is for JavaScriptCore
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Jun 28 14:06:52 PDT 2016
https://bugs.webkit.org/show_bug.cgi?id=159224
--- Comment #3 from Alexey Proskuryakov <ap at webkit.org> ---
Comment on attachment 282279
--> https://bugs.webkit.org/attachment.cgi?id=282279
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=282279&action=review
> Tools/BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/BuildbotCombinedQueueView.js:112
> - } else if (mostRecentFinishedIteration.failedTestSteps.length === 1 && ["jscore-test", "webkit-32bit-jsc-test", "webkit-jsc-cloop-test"].indexOf(mostRecentFinishedIteration.failedTestSteps[0].name) >= 0) {
> + } else if (mostRecentFinishedIteration.failedTestSteps.length === 1 && /jsc/.test(mostRecentFinishedIteration.failedTestSteps[0].name) === true) {
Would it make sense to make sure that "test" is in the name too? That would decrease the chances of something like "build-jsc" or "configure-jsc-dependencies" to confuse the logic.
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160628/2cec5551/attachment.html>
More information about the webkit-unassigned
mailing list