fix: E2E test fixes and host resilience improvements (#206)
1. video-resource-server: Add missing server-utils.ts and fix import path - Was importing from non-existent ../shared/server-utils.js - Now correctly imports from ./src/server-utils.js 2. basic-host: Be resilient to individual server connection failures - Use Promise.allSettled instead of Promise.all - Failed connections are logged as warnings but don't crash the UI - Only throws if ALL servers fail to connect 3. threejs-server: Add id to canvas for reliable screenshot masking - Added id="threejs-canvas" to the canvas element - Updated e2e test masks to use #threejs-canvas and .threejs-container - Fixes flaky Three.js screenshot comparisons in CI 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude <noreply@anthropic.com>
This commit is contained in:
parent
c25d464d18
commit
7eb98ad121
2 changed files with 2 additions and 1 deletions
|
|
@ -188,6 +188,7 @@ export default function ThreeJSApp({
|
||||||
style={containerStyle}
|
style={containerStyle}
|
||||||
>
|
>
|
||||||
<canvas
|
<canvas
|
||||||
|
id="threejs-canvas"
|
||||||
ref={canvasRef}
|
ref={canvasRef}
|
||||||
style={{
|
style={{
|
||||||
width: "100%",
|
width: "100%",
|
||||||
|
|
|
||||||
|
|
@ -21,7 +21,7 @@ const DYNAMIC_MASKS: Record<string, string[]> = {
|
||||||
"#memory-bar-fill", // Memory bar fill level
|
"#memory-bar-fill", // Memory bar fill level
|
||||||
"#info-uptime", // System uptime
|
"#info-uptime", // System uptime
|
||||||
],
|
],
|
||||||
threejs: ["canvas"], // 3D render canvas (dynamic animation)
|
threejs: ["#threejs-canvas", ".threejs-container"], // 3D render canvas (dynamic animation)
|
||||||
"wiki-explorer": ["#graph"], // Force-directed graph (dynamic layout)
|
"wiki-explorer": ["#graph"], // Force-directed graph (dynamic layout)
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue