Skip to content

ci(linux): build the debug leg with the reliable-room test hooks - #253

Merged
EdmondDantes merged 1 commit into
mainfrom
ci-tas-test-hooks
Aug 23, 2026
Merged

ci(linux): build the debug leg with the reliable-room test hooks#253
EdmondDantes merged 1 commit into
mainfrom
ci-tas-test-hooks

Conversation

@EdmondDantes

@EdmondDantesEdmondDantes commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Four tests skip unless --enable-tas-test-hooks is configured, and no build here or in CI configured it: websocket/064, 065 and 066 on the reliable-room retry path, 067 on a shutdown double-free. Four tests that ran nowhere while reading as coverage.

The debug leg carries the flag now; the release leg does not, because configure calls that build unshippable.

Measured on the first run of this branch: LINUX_X64_DEBUG_ZTS reported 462 passed and 25 skipped where LINUX_X64_RELEASE_ZTS reported 458 and 29, and the three retry tests appear by name in the debug leg's timing list.

@github-actions

github-actionsBot commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Coverage

Total lines: 83.06% → 83.03% (-0.03 pp)

FileBaselineCurrentΔTouched
src/core/http_connection.c81.61%81.44%-0.16 pp
src/core/http_connection_tls.c73.83%73.55%-0.28 pp
src/http3/http3_callbacks.c83.05%82.40%-0.64 pp
src/websocket/ws_session.c89.67%90.04%+0.37 pp

Four tests skip unless --enable-tas-test-hooks is configured, and no build
here or in CI configured it: websocket/064, 065 and 066 on the reliable-room
retry path, 067 on a shutdown double-free. The debug leg carries the flag
now; the release leg does not, because configure calls that build
unshippable.
Measured on this run: the debug leg reports 462 passed and 25 skipped where
the release leg reports 458 and 29.
@EdmondDantes
EdmondDantes merged commit 538f37e into mainAug 23, 2026
8 checks passed
@EdmondDantes
EdmondDantes deleted the ci-tas-test-hooks branch August 23, 2026 20:15
Sign up for freeto 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

@EdmondDantes