Skip to content

Commit 6494fdd

Browse files
committed
Use one line if for error handling
1 parent d63981d commit 6494fdd

File tree

2 files changed

+5
-11
lines changed

2 files changed

+5
-11
lines changed

database/testutils.go

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -51,8 +51,7 @@ func getTestLogging() *logging.Logging {
5151
func getTestDb(logs *logging.Logging) *DB {
5252
var defaultOptions Options
5353

54-
err := defaults.Set(&defaultOptions)
55-
if err != nil {
54+
if err := defaults.Set(&defaultOptions); err != nil {
5655
utils.PrintErrorThenExit(err, 1)
5756
}
5857

@@ -71,13 +70,11 @@ func getTestDb(logs *logging.Logging) *DB {
7170
}
7271

7372
func initTestDb(db *DB) {
74-
_, err := db.Query("DROP TABLE IF EXISTS user")
75-
if err != nil {
73+
if _, err := db.Query("DROP TABLE IF EXISTS user"); err != nil {
7674
utils.PrintErrorThenExit(err, 1)
7775
}
7876

79-
_, err = db.Query(`CREATE TABLE user ("id" INTEGER PRIMARY KEY, "name" VARCHAR(255) DEFAULT '', "age" INTEGER DEFAULT 0, "email" VARCHAR(255) DEFAULT '')`)
80-
if err != nil {
77+
if _, err := db.Query(`CREATE TABLE user ("id" INTEGER PRIMARY KEY, "name" VARCHAR(255) DEFAULT '', "age" INTEGER DEFAULT 0, "email" VARCHAR(255) DEFAULT '')`); err != nil {
8178
utils.PrintErrorThenExit(err, 1)
8279
}
8380
}
@@ -91,8 +88,7 @@ func prefillTestDb(db *DB) {
9188
}
9289

9390
for _, entity := range entities {
94-
_, err := db.NamedExec(`INSERT INTO user ("id", "name", "age", "email") VALUES (:id, :name, :age, :email)`, entity)
95-
if err != nil {
91+
if _, err := db.NamedExec(`INSERT INTO user ("id", "name", "age", "email") VALUES (:id, :name, :age, :email)`, entity); err != nil {
9692
utils.PrintErrorThenExit(err, 1)
9793
}
9894
}

database/upsert_test.go

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -129,8 +129,7 @@ func TestUpsertStreamed(t *testing.T) {
129129

130130
time.Sleep(time.Second)
131131

132-
err := db.Select(&actual, "SELECT * FROM user")
133-
if err != nil {
132+
if err := db.Select(&actual, "SELECT * FROM user"); err != nil {
134133
t.Fatalf("cannot select from database: %v", err)
135134
}
136135

@@ -139,7 +138,6 @@ func TestUpsertStreamed(t *testing.T) {
139138

140139
return actual, upsertError
141140
}))
142-
143141
}
144142
}
145143

0 commit comments

Comments
 (0)