Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TINY-11177: Vastly improve remote testing #145

Draft
wants to merge 53 commits into
base: master
Choose a base branch
from

v15.0.0-alpha.9

44518a7
Select commit
Loading
Failed to load commit list.
Draft

TINY-11177: Vastly improve remote testing #145

v15.0.0-alpha.9
44518a7
Select commit
Loading
Failed to load commit list.
Task list completed / task-list-completed succeeded Jan 17, 2025 in 0s

3 / 3 tasks completed

All tasks have been completed

Details

Required Tasks

Task Status
Remote testing was slow because bedrock sent session POST requests to /start before each it block, and then again to /results after each it block. It would wait for these requests to finish before continuing. Incomplete
/start is now sent at startup and after every 50 tests. This can be made larger if we find it's still too slow. Incomplete
/results is now only sent for failure and skip, not pass. Incomplete
The bedrock UI doesn't block on these status requests anymore, it just throws them into an array and uses Promise.all() at the very end to wait for them. Incomplete
Unfortunately the bedrock CLI console HUD depended on receiving these status updates, so I had to adjust it to account for the missing reports. Incomplete
The bedrock HTML page now has a fake icon to remove the favicon.ico request which probably also slowed things down. Incomplete
A proper keep-alive header is now set, instead of using the default 5 seconds which was causing occasional 502 Bad Gateway errors. Incomplete
Reverted #142 which was designed to solve this problem but server-side was the wrong answer. Incomplete
Changelog entry added Completed
package.json versions have not been changed (done by Lerna on release) Completed
Tests have been added (if applicable) Completed
For stable releases, all dependencies must be stable Incomplete
For release candidates, all dependencies must be release candidates or stable Incomplete