Skip to content

Commit

Permalink
test: mark 'test/parallel/test-sqlite.js' as flaky
Browse files Browse the repository at this point in the history
The current test is large and can time out. It should be split
into multiple smaller tests as done in nodejs#54014. However, that
approach appears to change GC behavior such that the database
files are not cleaned up quickly enough on Windows. Forcing
any unfinalized SQL statements to be GC'ed appears to fix the
problem. Mark the original test as flaky until the necessary
code changes are made.
  • Loading branch information
cjihrig committed Jul 25, 2024
1 parent 36e44f1 commit 72ed735
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions test/parallel/parallel.status
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,9 @@ test-fs-read-stream-concurrent-reads: PASS, FLAKY
# https://github.com/nodejs/node/issues/52630
test-error-serdes: PASS, FLAKY

# https://github.com/nodejs/node/issues/54006
test-sqlite: PASS, FLAKY

[$system==win32]

# Windows on ARM
Expand Down

0 comments on commit 72ed735

Please sign in to comment.