mattn / go-sqlite3 / 988

52%

master: 54%

Pull #744

travis-ci

Fix context cancellation racy handling

[why]
Context cancellation goroutine is not in sync with Next() method lifetime.
It leads to sql.ErrNoRows instead of context.Canceled often (easy to reproduce).
It leads to interruption of next query executed on same connection (harder to reproduce).

[how]
Do query in goroutine, wait when interruption done.

[testing]
Add unit test that reproduces error cases.

42 of 42 new or added lines in 1 file covered. (100.0%)

951 of 1854 relevant lines covered (51.29%)

922252.48 hits per line