Skip to content

Faster integration tests - #287

Merged
azat merged 2 commits into
mainfrom
faster-integration-tests
Aug 10, 2026
Merged

Faster integration tests#287
azat merged 2 commits into
mainfrom
faster-integration-tests

Conversation

@azat

@azat azat commented Aug 10, 2026

Copy link
Copy Markdown
Owner

Before

running 7 tests
test tests::integration ... ok
test tests::test_pane_click_focus ... ok
test tests::test_query_logs_view ... ok
test tests::test_settings_align_keeps_logs ... ok
test tests::test_table_logs_pane ... ok
test tests::test_panes ... ok
test tests::test_queries_view ... ok

test result: ok. 7 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 43.43s

And sometimes we even had tests that took > 60 seconds

test tests::test_settings_align_keeps_logs has been running for over 60 seconds
test tests::test_table_logs_pane has been running for over 60 seconds

Now:

running 7 tests
test tests::integration ... ok
test tests::test_panes ... ok
test tests::test_query_logs_view ... ok
test tests::test_queries_view ... ok
test tests::test_pane_click_focus ... ok
test tests::test_settings_align_keeps_logs ... ok
test tests::test_table_logs_pane ... ok

test result: ok. 7 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 13.01s

azat and others added 2 commits August 10, 2026 11:29
Relative "now" in SQL time bounds was rendered as now(), which has
second precision: rows failing event_time_microseconds <= end_time_
within the current second only appeared on the next background refresh
(--delay_interval, 30s by default) - the cause of the bimodal 30s
stalls in the TUI tests.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The Refresh sent per received frame kept the TUI loop out of its 30ms
idle wait, spinning it (and wait_for) in lockstep at full speed - a
core burned for the whole suite. The loop emits a frame every
iteration anyway, so just consume them.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@azat
azat merged commit 4c30155 into main Aug 10, 2026
9 checks passed
@azat
azat deleted the faster-integration-tests branch August 10, 2026 09:34
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