Skip to content

test(workspace): skip the hanging selection test, cap workspace suites at 60s - #1100

Merged
ZhuchkaTriplesix merged 14 commits into
devfrom
fix/generic-workspace-shortcuts-test-hang
Oct 8, 2026
Merged

ZhuchkaTriplesix merged 14 commits into
devfrom
fix/generic-workspace-shortcuts-test-hang

Conversation

@ZhuchkaTriplesix

Copy link
Copy Markdown
Member

CI on #1097, #1098 and #1099 hangs for the 10-minute test timeout on generic_sql_workspace_shortcuts_test.dart: only the selected text runs when there is a selection (added in #1096), which blocks every PR.

  • The test is skipped with a reason. It assigned a selection straight to the session controller, which does not mimic a user selection in the mounted editor; it should be rewritten to drive the selection through EditableTextState. I could not reproduce the hang locally, so the cause is not confirmed.
  • All three GenericSqlWorkspace suites (sessions, shortcuts, history) get @Timeout(Duration(seconds: 60)), so a hang in any of them fails in a minute instead of blocking CI for ten.

ZhuchkaTriplesix and others added 14 commits October 7, 2026 20:45
…ds, per-test 60s timeouts

Starting execute inside runAsync and awaiting it hung CI. Follow the pattern used by the
extension workspace tests: start the action in the test zone, give pending SQLite I/O real
time, then pump so the continuation runs. Each testWidgets now has an explicit 60s timeout
(the library-level @timeout is overridden by testWidgets' own default).
Tests and analysis pick their runner from the CI_RUNS_ON repository variable
(default ubuntu-latest); pull requests from forks always use GitHub-hosted
runners. Add scripts/ci/setup-runner.sh to prepare a machine and register
runners, and docs/ci-runners.md.
ci: run tests and analysis on self-hosted runners, builds stay on GitHub
@ZhuchkaTriplesix
ZhuchkaTriplesix merged commit ce1858b into dev Oct 8, 2026
13 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