Create a test for the 'database is locked' and 'database table is locked' error #823

Closed
opened 2025-11-02 03:37:52 -06:00 by GiteaMirror · 5 comments
Owner

Originally created by @typeless on GitHub (Jun 19, 2017).

I tried to re-use the 'NewIssue' integration test for the purpose, creating a test with two goroutines each looping the test 100 times. But it takes quite long time to run and not that reliable given the number of iterations are not that high.

Any thoughts?

Originally created by @typeless on GitHub (Jun 19, 2017). I tried to re-use the 'NewIssue' integration test for the purpose, creating a test with two goroutines each looping the test 100 times. But it takes quite long time to run and not that reliable given the number of iterations are not that high. Any thoughts?
GiteaMirror added the type/testing label 2025-11-02 03:37:52 -06:00
Author
Owner

@bkcsoft commented on GitHub (Jun 19, 2017):

This is easy actually, just make all tests run in parallel ;)

@bkcsoft commented on GitHub (Jun 19, 2017): This is easy actually, just make all tests run in parallel ;)
Author
Owner

@typeless commented on GitHub (Jun 19, 2017):

@bkcsoft Would it not affect the CI running time? I suppose the Drone CI server doesn't provide too many cores for free instances.

Anyway I will still submit the PR for troubleshooting the database locking issue. And we can discuss whether to add it to the 'official' test suite later.

@typeless commented on GitHub (Jun 19, 2017): @bkcsoft Would it not affect the CI running time? I suppose the Drone CI server doesn't provide too many cores for free instances. Anyway I will still submit the PR for troubleshooting the database locking issue. And we can discuss whether to add it to the 'official' test suite later.
Author
Owner

@bkcsoft commented on GitHub (Jun 19, 2017):

Running the test in parallel would make it run faster, if we allow to run multiple processes 😉

@bkcsoft commented on GitHub (Jun 19, 2017): Running the test in parallel would make it run faster, if we allow to run multiple processes 😉
Author
Owner

@lunny commented on GitHub (Jun 23, 2017):

testing for #2040

@lunny commented on GitHub (Jun 23, 2017): testing for #2040
Author
Owner

@lunny commented on GitHub (Oct 24, 2018):

Closed by #5144 since the test-sqlite has been reenabled on that PR.

@lunny commented on GitHub (Oct 24, 2018): Closed by #5144 since the test-sqlite has been reenabled on that PR.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#823