From 1ae6ca764d50a6a0f3d30009a287623337b534c7 Mon Sep 17 00:00:00 2001 From: Lars Buitinck Date: Fri, 5 Jun 2015 16:33:55 +0200 Subject: [PATCH] Cosmetics: use WaitGroup instead of channel in test --- sqlite3_test/sqltest.go | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/sqlite3_test/sqltest.go b/sqlite3_test/sqltest.go index fc82782..06cfd0e 100644 --- a/sqlite3_test/sqltest.go +++ b/sqlite3_test/sqltest.go @@ -275,12 +275,11 @@ func TestPreparedStmt(t *testing.T) { } const nRuns = 10 - ch := make(chan bool) + var wg sync.WaitGroup for i := 0; i < nRuns; i++ { + wg.Add(1) go func() { - defer func() { - ch <- true - }() + defer wg.Done() for j := 0; j < 10; j++ { count := 0 if err := sel.QueryRow().Scan(&count); err != nil && err != sql.ErrNoRows { @@ -294,9 +293,7 @@ func TestPreparedStmt(t *testing.T) { } }() } - for i := 0; i < nRuns; i++ { - <-ch - } + wg.Wait() } // Benchmarks need to use panic() since b.Error errors are lost when