Skip to content

Commit a2cebf5

Browse files
authored
test(artifact): serialize deadline-bound drain test (#1334)
The unsettled-queue full-export test has a 30-second context to catch real stalls, but it also performs repeated real SQLite and Docbank writes. Since the artifact suite was parallelized, shared-runner contention can consume that deadline even while export progresses, failing both plain and race jobs at different write sites. Keep this single deadline-bound integration test in the serial phase. Its real-store coverage, behavioral assertions, and hang guard remain intact; other artifact tests remain parallel and production behavior is unchanged. Co-authored-by: Wes McKinney <wesm@users.noreply.github.com>
1 parent 66bc76f commit a2cebf5

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

internal/artifact/sync_test.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -499,7 +499,9 @@ func TestDrainArtifactSyncExportsReturnsMoreAtRoundBudget(t *testing.T) {
499499
func TestDrainArtifactSyncFullExportReturnsMoreWhenQueueDoesNotSettle(
500500
t *testing.T,
501501
) {
502-
t.Parallel()
502+
// Serial: this exercises the full drain cap through real SQLite and
503+
// Docbank writes under a fixed deadline. Package-wide I/O contention can
504+
// otherwise consume the deadline without the drain being stuck.
503505

504506
database := testExportDB(t)
505507
seedSession(t, database, "sess-1", "alpha")

0 commit comments

Comments
 (0)