Add bash script demonstrating browser shell-wrapper pipe-hang bug (Brave + Chromium)#3748
Draft
Add bash script demonstrating browser shell-wrapper pipe-hang bug (Brave + Chromium)#3748
Conversation
Copilot
AI
changed the title
[WIP] Fix frontend tests hanging issue on Ubuntu 24.04
Fix frontend test:headless hanging at end
Mar 18, 2026
Copilot
AI
changed the title
Fix frontend test:headless hanging at end
Fix frontend test:headless hanging at end; add devcontainer
Mar 18, 2026
Copilot
AI
changed the title
Fix frontend test:headless hanging at end; add devcontainer
Add devcontainer with Brave browser for verifying frontend test fix
Mar 19, 2026
Copilot
AI
changed the title
Add devcontainer with Brave browser for verifying frontend test fix
Install Brave browser in devcontainer and use it for frontend tests via CHROMIUM_BIN
Mar 19, 2026
Copilot
AI
changed the title
Install Brave browser in devcontainer and use it for frontend tests via CHROMIUM_BIN
Fix karma test run hanging at end when using Brave browser
Mar 19, 2026
Copilot
AI
changed the title
Fix karma test run hanging at end when using Brave browser
Add bash script demonstrating browser shell-wrapper pipe-hang bug (Brave + Chromium)
Mar 19, 2026
Collaborator
|
I preserved the results locally. |
Fix frontend tests hanging by adding --watch=false to test:headless script Add devcontainer and verify the test:headless fix inside it Update devcontainer to install and use Brave browser for frontend tests Co-authored-by: marksvc <7265309+marksvc@users.noreply.github.com> Install Brave browser in devcontainer and use it for frontend tests via CHROMIUM_BIN Co-authored-by: marksvc <7265309+marksvc@users.noreply.github.com> Fix karma hang: point CHROMIUM_BIN at brave binary directly, not the shell wrapper Co-authored-by: marksvc <7265309+marksvc@users.noreply.github.com> Add bash script demonstrating the Brave pipe-hang bug Co-authored-by: marksvc <7265309+marksvc@users.noreply.github.com> Extend pipe-hang test to also test Chromium browser (Debian wrapper and direct binary) Co-authored-by: marksvc <7265309+marksvc@users.noreply.github.com> chore(clientapp): update angular packages to patched versions Agent-Logs-Url: https://github.com/sillsdev/web-xforge/sessions/92599b64-061a-43ba-9a38-46f8ba43c085 Co-authored-by: marksvc <7265309+marksvc@users.noreply.github.com> rename devcontainer dir
4a34905 to
2878158
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #3748 +/- ##
=======================================
Coverage 81.02% 81.02%
=======================================
Files 630 630
Lines 40632 40632
Branches 6603 6598 -5
=======================================
Hits 32922 32922
Misses 6676 6676
Partials 1034 1034 ☔ View full report in Codecov by Sentry. |
Agent-Logs-Url: https://github.com/sillsdev/web-xforge/sessions/e83ebc75-61ab-46b9-ae7e-5b975eed9049 Co-authored-by: marksvc <7265309+marksvc@users.noreply.github.com>
Agent-Logs-Url: https://github.com/sillsdev/web-xforge/sessions/e83ebc75-61ab-46b9-ae7e-5b975eed9049 Co-authored-by: marksvc <7265309+marksvc@users.noreply.github.com>
Agent-Logs-Url: https://github.com/sillsdev/web-xforge/sessions/e83ebc75-61ab-46b9-ae7e-5b975eed9049 Co-authored-by: marksvc <7265309+marksvc@users.noreply.github.com>
Agent-Logs-Url: https://github.com/sillsdev/web-xforge/sessions/e83ebc75-61ab-46b9-ae7e-5b975eed9049 Co-authored-by: marksvc <7265309+marksvc@users.noreply.github.com>
Agent-Logs-Url: https://github.com/sillsdev/web-xforge/sessions/e83ebc75-61ab-46b9-ae7e-5b975eed9049 Co-authored-by: marksvc <7265309+marksvc@users.noreply.github.com>
Agent-Logs-Url: https://github.com/sillsdev/web-xforge/sessions/e83ebc75-61ab-46b9-ae7e-5b975eed9049 Co-authored-by: marksvc <7265309+marksvc@users.noreply.github.com>
Agent-Logs-Url: https://github.com/sillsdev/web-xforge/sessions/e83ebc75-61ab-46b9-ae7e-5b975eed9049 Co-authored-by: marksvc <7265309+marksvc@users.noreply.github.com>
Agent-Logs-Url: https://github.com/sillsdev/web-xforge/sessions/e83ebc75-61ab-46b9-ae7e-5b975eed9049 Co-authored-by: marksvc <7265309+marksvc@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
/home/runner/work/web-xforge/web-xforge/.devcontainer/Dockerfileto omit NodeSource setup while keeping Node.js install and Brave install in-containerapp,db) and verify container startuptimeout 180 npm run test:headless ...only inside the devcontainerOriginal prompt
Created from VS Code.
💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.
This change is