Skip to content

Commit

Permalink
fix: avoid testing.T log after test finished
Browse files Browse the repository at this point in the history
  • Loading branch information
jkaflik committed May 17, 2023
1 parent 7629aa0 commit aa43e90
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions tests/issues/957_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -34,10 +34,6 @@ func Test957(t *testing.T) {

// when the client is configured to use the test environment
opts := clickhouse_tests.ClientOptionsFromEnv(testEnv, clickhouse.Settings{})
opts.Debug = true
opts.Debugf = func(format string, v ...interface{}) {
t.Logf(format, v...)
}
// and the client is configured to have only 1 connection
opts.MaxIdleConns = 2
opts.MaxOpenConns = 1
Expand All @@ -49,7 +45,8 @@ func Test957(t *testing.T) {
// then the client should be able to execute queries for 1 second
deadline := time.Now().Add(time.Second)
for time.Now().Before(deadline) {
_, err := conn.Query(ctx, "SELECT 1")
rows, err := conn.Query(ctx, "SELECT 1")
require.NoError(t, err)
rows.Close()
}
}

0 comments on commit aa43e90

Please sign in to comment.