Skip to content

Gate WebSocket and FileReader behavior in native CI - #913

Merged
wieslawsoltes merged 1 commit into
fix/native-websocket-fixture-link-910from
test/native-websocket-fixture-contract-910
Sep 22, 2026
Merged

wieslawsoltes merged 1 commit into
fix/native-websocket-fixture-link-910from
test/native-websocket-fixture-contract-910

Conversation

@wieslawsoltes

@wieslawsoltes wieslawsoltes commented Sep 22, 2026 •

Copy link
Copy Markdown
Collaborator

Result

The macOS 12 NuGet runtime build now links webscene_native_engine_tests after PR #911. This follow-up exposes its existing websocket-file-reader filter as a dedicated CTest, so the top of stack #912 directly gates cross-socket fairness, browser WebSocket behavior, and FileReader scheduling without relying on the broad aggregate test reaching those cases.

Gate

The top PR's macOS 12 NuGet Build osx-arm64 job must compile, link, and pass webscene_native_websocket_file_reader. Linux/macOS native builds and portable contracts remain stack gates. The earlier package run reached 91 CTests and found seven other runtime failures; those are being tracked separately, and this PR does not suppress them.

GitHub stack #912: #909 (floating conversion) → #911 (white-box link) → this focused test gate. Validate this top PR, then merge ready layers atomically.

@wieslawsoltes wieslawsoltes changed the title Gate WebSocket fixture link and behavior directly Gate WebSocket and FileReader behavior in native CI Sep 22, 2026
@wieslawsoltes

Copy link
Copy Markdown
Collaborator Author

The first macOS 12 package run after the #911 link fix compiled and linked all 91 native CTests; the focused form target passed, while seven broader runtime tests failed. WebScene #914 records exact test names, logs, existing owners, and the full-green follow-up. This top PR adds an explicit webscene_native_websocket_file_reader gate so stack #912 can be assessed for its own WebSocket behavior before merging.

@wieslawsoltes
wieslawsoltes merged commit 8c90c5c into main Sep 22, 2026
9 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant