tests(devtools): fix webserver #12236
Conversation
| # - CORS (Access-Control-Allow-Origin header) | ||
|
|
||
| ln -s "$DEVTOOLS_PATH/out/Default/resources/inspector" "$DEVTOOLS_PATH/test/webtests/http/tests/inspector-sources" | ||
| ln -s "$latest_content_shell/out/Release/resources/inspector" "$DEVTOOLS_PATH/test/webtests/http/tests/inspector-sources" |
There was a problem hiding this comment.
How I debugged...
I removed the output redirection of the webserver to dev/null and noticed 404s. integration_runner.js was missing. I realized line wasn't doing what the comment suggested... the inspector-sources file is meant to come from content_shell, the reason is that the compiled devtools framework code should match the content shell binary.
I guess integration_runner.js was being generated by devtools before, but no longer is. the incorrect symbolic link was overlooked because of that.
There was a problem hiding this comment.
Scratch that. DT team refactored the BUILD rules, and so we weren't building the right thing any more. And the output folder changed slightly.
| ++++++++ testExportHtml | ||
|
|
||
| # of .lh-audit divs (original): 137 | ||
| # of .lh-audit divs (original): 136 |
There was a problem hiding this comment.
file last changed with csp-xss https://github.com/GoogleChrome/lighthouse/commits/master/third-party/chromium-webtests/webtests/http/tests/devtools/lighthouse/lighthouse-export-run-expected.txt
I don't see why this would have changed, since the audit is in experimental, so doesn't show in DT.
3 participants
Fixes #12209