diff --git a/mongo/database_test.go b/mongo/database_test.go index 34b0634666ab5a879551e4b08db296de8176faba..473eda73646bf04ae36cd5d8b7b88002df7d6966 100644 --- a/mongo/database_test.go +++ b/mongo/database_test.go @@ -113,7 +113,7 @@ func TestTruncate(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -123,7 +123,7 @@ func TestTruncate(t *testing.T) { collections, err := sess.Collections() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, name := range collections { @@ -131,7 +131,7 @@ func TestTruncate(t *testing.T) { // Pointing the collection. col, err := sess.Collection(name) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // The collection may ot may not exists. @@ -142,7 +142,7 @@ func TestTruncate(t *testing.T) { err = col.Truncate() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -159,7 +159,7 @@ func TestAppend(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -171,7 +171,7 @@ func TestAppend(t *testing.T) { if err != nil { // We can use the collection even if it does not exists. if err != db.ErrCollectionDoesNotExists { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -247,7 +247,7 @@ func TestResultCount(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } defer sess.Close() @@ -261,7 +261,7 @@ func TestResultCount(t *testing.T) { total, err := res.Count() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if total == 0 { @@ -280,7 +280,7 @@ func TestResultFetch(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -289,7 +289,7 @@ func TestResultFetch(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Testing map @@ -320,7 +320,7 @@ func TestResultFetch(t *testing.T) { t.Fatalf("Expecting a name.") } } else { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -350,7 +350,7 @@ func TestResultFetch(t *testing.T) { t.Fatalf("Expecting a name.") } } else { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -380,7 +380,7 @@ func TestResultFetch(t *testing.T) { t.Fatalf("Expecting a name.") } } else { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -393,7 +393,7 @@ func TestResultFetch(t *testing.T) { err = res.All(&all_rows_m) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, single_row_m := range all_rows_m { @@ -412,7 +412,7 @@ func TestResultFetch(t *testing.T) { err = res.All(&all_rows_s) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, single_row_s := range all_rows_s { @@ -431,7 +431,7 @@ func TestResultFetch(t *testing.T) { err = res.All(&all_rows_t) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, single_row_t := range all_rows_t { @@ -449,7 +449,7 @@ func TestUpdate(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -459,7 +459,7 @@ func TestUpdate(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Value @@ -474,7 +474,7 @@ func TestUpdate(t *testing.T) { err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Updating with a map @@ -485,13 +485,13 @@ func TestUpdate(t *testing.T) { err = res.Update(row_m) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if value.Name != row_m["name"] { @@ -506,13 +506,13 @@ func TestUpdate(t *testing.T) { err = res.Update(row_s) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if value.Name != row_s.Name { @@ -527,13 +527,13 @@ func TestUpdate(t *testing.T) { err = res.Update(row_t) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if value.Name != row_t.Value1 { @@ -551,7 +551,7 @@ func TestFunction(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -561,7 +561,7 @@ func TestFunction(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } row_s := struct { @@ -593,7 +593,7 @@ func TestRemove(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -603,7 +603,7 @@ func TestRemove(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Getting the first artist. @@ -616,7 +616,7 @@ func TestRemove(t *testing.T) { err = res.One(&first) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } res = artist.Find(db.Cond{"_id": first.Id}) @@ -625,7 +625,7 @@ func TestRemove(t *testing.T) { err = res.Remove() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -639,7 +639,7 @@ func TestDataTypes(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -653,7 +653,7 @@ func TestDataTypes(t *testing.T) { id, err := dataTypes.Append(testValues) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Trying to get the same subject we added. @@ -662,7 +662,7 @@ func TestDataTypes(t *testing.T) { exists, err := res.Count() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if exists == 0 { @@ -689,7 +689,7 @@ func BenchmarkAppendRaw(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -706,7 +706,7 @@ func BenchmarkAppendRaw(b *testing.B) { for i := 0; i < b.N; i++ { err := col.Insert(map[string]string{"name": "Hayao Miyazaki"}) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } } @@ -715,7 +715,7 @@ func BenchmarkAppendDbItem(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -727,7 +727,7 @@ func BenchmarkAppendDbItem(b *testing.B) { for i := 0; i < b.N; i++ { _, err = artist.Append(map[string]string{"name": "Leonardo DaVinci"}) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } } @@ -736,7 +736,7 @@ func BenchmarkAppendStruct(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -748,7 +748,7 @@ func BenchmarkAppendStruct(b *testing.B) { for i := 0; i < b.N; i++ { _, err = artist.Append(struct{ Name string }{"John Lennon"}) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } } diff --git a/mysql/database_test.go b/mysql/database_test.go index 6469cbda865e2ee460a5e6aeb3086f9c6c975e8d..33c352b5759165bd9ca3c6fe31660b44ef6d7117 100644 --- a/mysql/database_test.go +++ b/mysql/database_test.go @@ -123,7 +123,7 @@ func TestTruncate(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -133,7 +133,7 @@ func TestTruncate(t *testing.T) { collections, err := sess.Collections() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, name := range collections { @@ -141,7 +141,7 @@ func TestTruncate(t *testing.T) { // Pointing the collection. col, err := sess.Collection(name) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Since this is a SQL collection (table), the structure must exists before @@ -153,7 +153,7 @@ func TestTruncate(t *testing.T) { err = col.Truncate() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -170,7 +170,7 @@ func TestAppend(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -180,7 +180,7 @@ func TestAppend(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Appending a map. @@ -227,7 +227,7 @@ func TestResultCount(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } defer sess.Close() @@ -241,7 +241,7 @@ func TestResultCount(t *testing.T) { total, err := res.Count() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if total == 0 { @@ -260,7 +260,7 @@ func TestResultFetch(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -269,7 +269,7 @@ func TestResultFetch(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Testing map @@ -293,7 +293,7 @@ func TestResultFetch(t *testing.T) { t.Fatalf("Expecting a name.") } } else { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -323,7 +323,7 @@ func TestResultFetch(t *testing.T) { t.Fatalf("Expecting a name.") } } else { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -353,7 +353,7 @@ func TestResultFetch(t *testing.T) { t.Fatalf("Expecting a name.") } } else { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -366,7 +366,7 @@ func TestResultFetch(t *testing.T) { err = res.All(&all_rows_m) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, single_row_m := range all_rows_m { @@ -385,7 +385,7 @@ func TestResultFetch(t *testing.T) { err = res.All(&all_rows_s) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, single_row_s := range all_rows_s { @@ -404,7 +404,7 @@ func TestResultFetch(t *testing.T) { err = res.All(&all_rows_t) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, single_row_t := range all_rows_t { @@ -422,7 +422,7 @@ func TestUpdate(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -432,7 +432,7 @@ func TestUpdate(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Value @@ -447,7 +447,7 @@ func TestUpdate(t *testing.T) { err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Updating with a map @@ -458,13 +458,13 @@ func TestUpdate(t *testing.T) { err = res.Update(row_m) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if value.Name != row_m["name"] { @@ -479,13 +479,13 @@ func TestUpdate(t *testing.T) { err = res.Update(row_s) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if value.Name != row_s.Name { @@ -500,13 +500,13 @@ func TestUpdate(t *testing.T) { err = res.Update(row_t) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if value.Name != row_t.Value1 { @@ -524,7 +524,7 @@ func TestFunction(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -534,7 +534,7 @@ func TestFunction(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } row_s := struct { @@ -566,7 +566,7 @@ func TestRemove(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -576,7 +576,7 @@ func TestRemove(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Getting the artist with id = 1 @@ -586,7 +586,7 @@ func TestRemove(t *testing.T) { err = res.Remove() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -600,7 +600,7 @@ func TestDataTypes(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -614,7 +614,7 @@ func TestDataTypes(t *testing.T) { id, err := dataTypes.Append(testValues) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Trying to get the same subject we added. @@ -623,7 +623,7 @@ func TestDataTypes(t *testing.T) { exists, err := res.Count() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if exists == 0 { @@ -650,7 +650,7 @@ func BenchmarkAppendRaw(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -664,7 +664,7 @@ func BenchmarkAppendRaw(b *testing.B) { for i := 0; i < b.N; i++ { _, err := driver.Exec(`INSERT INTO artist (name) VALUES('Hayao Miyazaki')`) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } } @@ -677,7 +677,7 @@ func BenchmarkAppendDbItem(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -689,7 +689,7 @@ func BenchmarkAppendDbItem(b *testing.B) { for i := 0; i < b.N; i++ { _, err = artist.Append(map[string]string{"name": "Leonardo DaVinci"}) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } } @@ -703,7 +703,7 @@ func BenchmarkAppendDbItem_Transaction(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -713,19 +713,19 @@ func BenchmarkAppendDbItem_Transaction(b *testing.B) { err = sess.Begin() if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } for i := 0; i < b.N; i++ { _, err = artist.Append(map[string]string{"name": "Isaac Asimov"}) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } err = sess.End() if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } @@ -734,7 +734,7 @@ func BenchmarkAppendStruct(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -746,7 +746,7 @@ func BenchmarkAppendStruct(b *testing.B) { for i := 0; i < b.N; i++ { _, err = artist.Append(struct{ Name string }{"John Lennon"}) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } } diff --git a/postgresql/database_test.go b/postgresql/database_test.go index dc0f928276c302144253ec7d01bb46c89a74e0b2..fe67d1a94e2d79df7067650b140f8be0cefd226b 100644 --- a/postgresql/database_test.go +++ b/postgresql/database_test.go @@ -123,7 +123,7 @@ func TestTruncate(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -133,7 +133,7 @@ func TestTruncate(t *testing.T) { collections, err := sess.Collections() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, name := range collections { @@ -141,7 +141,7 @@ func TestTruncate(t *testing.T) { // Pointing the collection. col, err := sess.Collection(name) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Since this is a SQL collection (table), the structure must exists before @@ -153,7 +153,7 @@ func TestTruncate(t *testing.T) { err = col.Truncate() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -170,7 +170,7 @@ func TestAppend(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -180,7 +180,7 @@ func TestAppend(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Appending a map. @@ -227,7 +227,7 @@ func TestResultCount(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } defer sess.Close() @@ -241,7 +241,7 @@ func TestResultCount(t *testing.T) { total, err := res.Count() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if total == 0 { @@ -260,7 +260,7 @@ func TestResultFetch(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -269,7 +269,7 @@ func TestResultFetch(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Testing map @@ -293,7 +293,7 @@ func TestResultFetch(t *testing.T) { t.Fatalf("Expecting a name.") } } else { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -323,7 +323,7 @@ func TestResultFetch(t *testing.T) { t.Fatalf("Expecting a name.") } } else { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -353,7 +353,7 @@ func TestResultFetch(t *testing.T) { t.Fatalf("Expecting a name.") } } else { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -366,7 +366,7 @@ func TestResultFetch(t *testing.T) { err = res.All(&all_rows_m) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, single_row_m := range all_rows_m { @@ -385,7 +385,7 @@ func TestResultFetch(t *testing.T) { err = res.All(&all_rows_s) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, single_row_s := range all_rows_s { @@ -404,7 +404,7 @@ func TestResultFetch(t *testing.T) { err = res.All(&all_rows_t) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, single_row_t := range all_rows_t { @@ -422,7 +422,7 @@ func TestUpdate(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -432,7 +432,7 @@ func TestUpdate(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Value @@ -452,13 +452,13 @@ func TestUpdate(t *testing.T) { err = res.Update(row_m) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if value.Name != row_m["name"] { @@ -473,13 +473,13 @@ func TestUpdate(t *testing.T) { err = res.Update(row_s) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if value.Name != row_s.Name { @@ -494,13 +494,13 @@ func TestUpdate(t *testing.T) { err = res.Update(row_t) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if value.Name != row_t.Value1 { @@ -518,7 +518,7 @@ func TestFunction(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -528,7 +528,7 @@ func TestFunction(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } row_s := struct { @@ -560,7 +560,7 @@ func TestRemove(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -570,7 +570,7 @@ func TestRemove(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Getting the artist with id = 1 @@ -580,7 +580,7 @@ func TestRemove(t *testing.T) { err = res.Remove() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -594,7 +594,7 @@ func TestDataTypes(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -608,7 +608,7 @@ func TestDataTypes(t *testing.T) { id, err := dataTypes.Append(testValues) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Trying to get the same subject we added. @@ -617,7 +617,7 @@ func TestDataTypes(t *testing.T) { exists, err := res.Count() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if exists == 0 { @@ -644,7 +644,7 @@ func BenchmarkAppendRaw(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -658,7 +658,7 @@ func BenchmarkAppendRaw(b *testing.B) { for i := 0; i < b.N; i++ { _, err := driver.Exec(`INSERT INTO artist (name) VALUES('Hayao Miyazaki')`) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } } @@ -671,7 +671,7 @@ func BenchmarkAppendDbItem(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -683,7 +683,7 @@ func BenchmarkAppendDbItem(b *testing.B) { for i := 0; i < b.N; i++ { _, err = artist.Append(map[string]string{"name": "Leonardo DaVinci"}) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } } @@ -697,7 +697,7 @@ func BenchmarkAppendDbItem_Transaction(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -707,19 +707,19 @@ func BenchmarkAppendDbItem_Transaction(b *testing.B) { err = sess.Begin() if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } for i := 0; i < b.N; i++ { _, err = artist.Append(map[string]string{"name": "Isaac Asimov"}) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } err = sess.End() if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } @@ -728,7 +728,7 @@ func BenchmarkAppendStruct(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -740,7 +740,7 @@ func BenchmarkAppendStruct(b *testing.B) { for i := 0; i < b.N; i++ { _, err = artist.Append(struct{ Name string }{"John Lennon"}) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } } diff --git a/ql/database_test.go b/ql/database_test.go index 11afddceb73b21151fd35e170deba74321ef8a4b..9771fb8339157666a0911d8af44b31174dd35b2c 100644 --- a/ql/database_test.go +++ b/ql/database_test.go @@ -114,7 +114,7 @@ func TestTruncate(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -124,7 +124,7 @@ func TestTruncate(t *testing.T) { collections, err := sess.Collections() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, name := range collections { @@ -132,7 +132,7 @@ func TestTruncate(t *testing.T) { // Pointing the collection. col, err := sess.Collection(name) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Since this is a SQL collection (table), the structure must exists before @@ -144,7 +144,7 @@ func TestTruncate(t *testing.T) { err = col.Truncate() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -161,7 +161,7 @@ func TestAppend(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -171,7 +171,7 @@ func TestAppend(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Appending a map. @@ -218,7 +218,7 @@ func TestResultCount(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } defer sess.Close() @@ -232,7 +232,7 @@ func TestResultCount(t *testing.T) { total, err := res.Count() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if total == 0 { @@ -251,7 +251,7 @@ func TestResultFetch(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -260,7 +260,7 @@ func TestResultFetch(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Testing map @@ -284,7 +284,7 @@ func TestResultFetch(t *testing.T) { t.Fatalf("Expecting a name.") } } else { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -314,7 +314,7 @@ func TestResultFetch(t *testing.T) { t.Fatalf("Expecting a name.") } } else { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -344,7 +344,7 @@ func TestResultFetch(t *testing.T) { t.Fatalf("Expecting a name.") } } else { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -357,7 +357,7 @@ func TestResultFetch(t *testing.T) { err = res.All(&all_rows_m) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, single_row_m := range all_rows_m { @@ -376,7 +376,7 @@ func TestResultFetch(t *testing.T) { err = res.All(&all_rows_s) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, single_row_s := range all_rows_s { @@ -395,7 +395,7 @@ func TestResultFetch(t *testing.T) { err = res.All(&all_rows_t) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, single_row_t := range all_rows_t { @@ -413,7 +413,7 @@ func TestUpdate(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -423,7 +423,7 @@ func TestUpdate(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Value @@ -438,7 +438,7 @@ func TestUpdate(t *testing.T) { err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Updating with a map @@ -449,13 +449,13 @@ func TestUpdate(t *testing.T) { err = res.Update(row_m) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if value.Name != row_m["name"] { @@ -470,13 +470,13 @@ func TestUpdate(t *testing.T) { err = res.Update(row_s) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if value.Name != row_s.Name { @@ -491,13 +491,13 @@ func TestUpdate(t *testing.T) { err = res.Update(row_t) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if value.Name != row_t.Value1 { @@ -515,7 +515,7 @@ func TestFunction(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -525,7 +525,7 @@ func TestFunction(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } row_s := struct { @@ -557,7 +557,7 @@ func TestRemove(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -567,7 +567,7 @@ func TestRemove(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Getting the artist with id = 1 @@ -577,7 +577,7 @@ func TestRemove(t *testing.T) { err = res.Remove() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -643,7 +643,7 @@ func BenchmarkAppendRaw(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -657,7 +657,7 @@ func BenchmarkAppendRaw(b *testing.B) { for i := 0; i < b.N; i++ { _, err := driver.Exec(`INSERT INTO artist (name) VALUES('Hayao Miyazaki')`) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } } @@ -670,7 +670,7 @@ func BenchmarkAppendDbItem(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -682,7 +682,7 @@ func BenchmarkAppendDbItem(b *testing.B) { for i := 0; i < b.N; i++ { _, err = artist.Append(map[string]string{"name": "Leonardo DaVinci"}) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } } @@ -696,7 +696,7 @@ func BenchmarkAppendDbItem_Transaction(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -706,19 +706,19 @@ func BenchmarkAppendDbItem_Transaction(b *testing.B) { err = sess.Begin() if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } for i := 0; i < b.N; i++ { _, err = artist.Append(map[string]string{"name": "Isaac Asimov"}) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } err = sess.End() if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } @@ -727,7 +727,7 @@ func BenchmarkAppendStruct(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -739,7 +739,7 @@ func BenchmarkAppendStruct(b *testing.B) { for i := 0; i < b.N; i++ { _, err = artist.Append(struct{ Name string }{"John Lennon"}) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } } diff --git a/sqlite/database.go b/sqlite/database.go index d27334fa6820b6d2c4f9cce2671c5eb55b8e4388..24f42c2b8f277e1e9ade83fa3c4119e2e4e61911 100644 --- a/sqlite/database.go +++ b/sqlite/database.go @@ -29,12 +29,12 @@ import ( // This hack is not required anymore. // See: https://github.com/mattn/go-sqlite3/issues/40 //_ "github.com/xiam/gosqlite3" + _ "github.com/mattn/go-sqlite3" "log" "os" "reflect" "regexp" "strings" - _ "github.com/mattn/go-sqlite3" "upper.io/db" ) diff --git a/sqlite/database_test.go b/sqlite/database_test.go index 22a70d0e71b6489d8638e5ff9bc004d30008b072..364817b2ffa38c4e8a355fb4abe19f0a6c4bc61c 100644 --- a/sqlite/database_test.go +++ b/sqlite/database_test.go @@ -115,7 +115,7 @@ func TestTruncate(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -125,7 +125,7 @@ func TestTruncate(t *testing.T) { collections, err := sess.Collections() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, name := range collections { @@ -133,7 +133,7 @@ func TestTruncate(t *testing.T) { // Pointing the collection. col, err := sess.Collection(name) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Since this is a SQL collection (table), the structure must exists before @@ -145,7 +145,7 @@ func TestTruncate(t *testing.T) { err = col.Truncate() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -162,7 +162,7 @@ func TestAppend(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -172,7 +172,7 @@ func TestAppend(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Appending a map. @@ -219,7 +219,7 @@ func TestResultCount(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } defer sess.Close() @@ -233,7 +233,7 @@ func TestResultCount(t *testing.T) { total, err := res.Count() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if total == 0 { @@ -252,7 +252,7 @@ func TestResultFetch(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -261,7 +261,7 @@ func TestResultFetch(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Testing map @@ -285,7 +285,7 @@ func TestResultFetch(t *testing.T) { t.Fatalf("Expecting a name.") } } else { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -315,7 +315,7 @@ func TestResultFetch(t *testing.T) { t.Fatalf("Expecting a name.") } } else { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -345,7 +345,7 @@ func TestResultFetch(t *testing.T) { t.Fatalf("Expecting a name.") } } else { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -358,7 +358,7 @@ func TestResultFetch(t *testing.T) { err = res.All(&all_rows_m) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, single_row_m := range all_rows_m { @@ -377,7 +377,7 @@ func TestResultFetch(t *testing.T) { err = res.All(&all_rows_s) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, single_row_s := range all_rows_s { @@ -396,7 +396,7 @@ func TestResultFetch(t *testing.T) { err = res.All(&all_rows_t) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } for _, single_row_t := range all_rows_t { @@ -414,7 +414,7 @@ func TestUpdate(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -424,7 +424,7 @@ func TestUpdate(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Value @@ -439,7 +439,7 @@ func TestUpdate(t *testing.T) { err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Updating with a map @@ -450,13 +450,13 @@ func TestUpdate(t *testing.T) { err = res.Update(row_m) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if value.Name != row_m["name"] { @@ -471,13 +471,13 @@ func TestUpdate(t *testing.T) { err = res.Update(row_s) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if value.Name != row_s.Name { @@ -492,13 +492,13 @@ func TestUpdate(t *testing.T) { err = res.Update(row_t) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } err = res.One(&value) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if value.Name != row_t.Value1 { @@ -516,7 +516,7 @@ func TestFunction(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -526,7 +526,7 @@ func TestFunction(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } row_s := struct { @@ -558,7 +558,7 @@ func TestRemove(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -568,7 +568,7 @@ func TestRemove(t *testing.T) { artist, err := sess.Collection("artist") if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Getting the artist with id = 1 @@ -578,7 +578,7 @@ func TestRemove(t *testing.T) { err = res.Remove() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } } @@ -592,7 +592,7 @@ func TestDataTypes(t *testing.T) { sess, err := db.Open(wrapperName, settings) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // We should close the database when it's no longer in use. @@ -606,7 +606,7 @@ func TestDataTypes(t *testing.T) { id, err := dataTypes.Append(testValues) if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } // Trying to get the same subject we added. @@ -615,7 +615,7 @@ func TestDataTypes(t *testing.T) { exists, err := res.Count() if err != nil { - t.Fatalf(err.Error()) + t.Fatal(err) } if exists == 0 { @@ -642,7 +642,7 @@ func BenchmarkAppendRaw(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -656,7 +656,7 @@ func BenchmarkAppendRaw(b *testing.B) { for i := 0; i < b.N; i++ { _, err := driver.Exec(`INSERT INTO artist (name) VALUES("Hayao Miyazaki")`) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } } @@ -669,7 +669,7 @@ func BenchmarkAppendDbItem(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -681,7 +681,7 @@ func BenchmarkAppendDbItem(b *testing.B) { for i := 0; i < b.N; i++ { _, err = artist.Append(map[string]string{"name": "Leonardo DaVinci"}) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } } @@ -695,7 +695,7 @@ func BenchmarkAppendDbItem_Transaction(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -705,19 +705,19 @@ func BenchmarkAppendDbItem_Transaction(b *testing.B) { err = sess.Begin() if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } for i := 0; i < b.N; i++ { _, err = artist.Append(map[string]string{"name": "Isaac Asimov"}) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } err = sess.End() if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } @@ -726,7 +726,7 @@ func BenchmarkAppendStruct(b *testing.B) { sess, err := db.Open(wrapperName, settings) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } defer sess.Close() @@ -738,7 +738,7 @@ func BenchmarkAppendStruct(b *testing.B) { for i := 0; i < b.N; i++ { _, err = artist.Append(struct{ Name string }{"John Lennon"}) if err != nil { - b.Fatalf(err.Error()) + b.Fatal(err) } } }