rebase_dump_accessibility_tree_test.py incompatible with polygerrit (404) |
||||
Issue descriptionRunning tools/accessibility/rebase_dump_accessibility_tree_test.py when the code review tool is configured to be polygerrit leads to a 404: For example, it attempts to access: https://codereview.chromium.org/525855 It should access: https://chromium-review.googlesource.com/c/525855/
,
Jun 8 2017
For example, https://chromium-review.googlesource.com/changes/525855/ provides JSON (unfortunately does not appear to be legal JSON as first characters are )]}'\n Might be complicated, but we can use the JSON results something like: let messageIndex = data.messages.length, currentResults = {}; while (messageIndex -- && !currentResults.complete) { let message = data.messages[messageIndex].message; if (message.includes('\n\nDry run: Try jobs \n')) { if (message.includes('\nDry run: Try jobs failed on the following builders:')) { failed(message, currentResults); } else { succeeded(message, currentResults) } } }
,
Jun 11 2018
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jun 13 2018
,
Oct 18
The script has been updated multiple times since this bug. |
||||
►
Sign in to add a comment |
||||
Comment 1 by aleventhal@chromium.org
, Jun 8 2017