Skip to content

Fix release test failures (scan-token race, layout test on small screens) - #727

Merged
Hirogen merged 1 commit into
Developmentfrom
fix/release-1.50-tests
Oct 3, 2026
Merged

Hirogen merged 1 commit into
Developmentfrom
fix/release-1.50-tests

Conversation

@Hirogen

@Hirogen Hirogen commented Oct 3, 2026 •

Copy link
Copy Markdown
Collaborator

The v1.50.0 release build (run 37122052131) failed in the Test target:

  • LineVisibilityTracker race: the scan task read its CancellationTokenSource.Token only once it started running. If the tracker was disposed or the scan restarted before then, the task failed with ObjectDisposedException. The token is now read while the lock is held. The new regression test Dispose_BeforeTheScanTaskStarts_LeavesNoFaultedScan failed before the fix.
  • HighlightDialogLayoutTests: Windows caps a form at the screen size, so on the 1024×768 runner the 1.5× and 2× minimum sizes can't be reached. Those cases are now marked inconclusive instead of failing. groupBoxSelection now has a Name so overlap messages identify it.

LineVisibilityTracker read the scan's cancellation token inside the
queued task; disposing or restarting before the task started faulted it
with ObjectDisposedException. Read the token under the lock instead.

HighlightDialogLayoutTests assumed the scaled minimum size fits the
screen; Windows caps forms at the screen size, so 1.5x/2x failed on the
1024x768 CI runner. Treat that case as inconclusive. Name
groupBoxSelection so overlap messages identify it.
@Hirogen Hirogen changed the title Fix scan-token race and screen-size-dependent layout test Fix release test failures (scan-token race, layout test on small screens) Oct 3, 2026
@Hirogen
Hirogen merged commit 15fe6bc into Development Oct 3, 2026
3 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