update according to comments

database/pgsql/complex_test.go

@@ -134,46 +134,31 @@ func TestConcurrency(t *testing.T) {
 
 func TestCaching(t *testing.T) {
 	store, err := openDatabaseForTest("Caching", false)
-	if !assert.Nil(t, err) {
+	require.Nil(t, err)
-		t.FailNow()
-	}
 	defer store.Close()
+
 	nsFeatures, vulnerabilities := testGenRandomVulnerabilityAndNamespacedFeature(t, store)
 	tx, err := store.Begin()
-	if !assert.Nil(t, err) {
+	require.Nil(t, err)
-		t.FailNow()
-	}
 
 	require.Nil(t, tx.PersistNamespacedFeatures(nsFeatures))
-	if err := tx.Commit(); err != nil {
+	require.Nil(t, tx.Commit())
-		panic(err)
-	}
 
 	tx, err = store.Begin()
-	if !assert.Nil(t, err) {
+	require.Nil(t, tx.Commit())
-		t.FailNow()
-	}
 
 	require.Nil(t, tx.InsertVulnerabilities(vulnerabilities))
-	if err := tx.Commit(); err != nil {
+	require.Nil(t, tx.Commit())
-		panic(err)
-	}
 
 	tx, err = store.Begin()
-	if !assert.Nil(t, err) {
+	require.Nil(t, err)
-		t.FailNow()
-	}
 	defer tx.Rollback()
 
 	affected, err := tx.FindAffectedNamespacedFeatures(nsFeatures)
-	if !assert.Nil(t, err) {
+	require.Nil(t, err)
-		t.FailNow()
-	}
 
 	for _, ansf := range affected {
-		if !assert.True(t, ansf.Valid) {
+		require.True(t, ansf.Valid)
-			t.FailNow()
-		}
 
 		expectedAffectedNames := []string{}
 		for _, vuln := range vulnerabilities {

database/pgsql/feature.go

@@ -270,7 +270,6 @@ func (tx *pgSession) FindAffectedNamespacedFeatures(features []database.Namespac
 	if err != nil {
 		return nil, handleError("searchNamespacedFeaturesVulnerabilities", err)
 	}
-
 	defer rows.Close()
 
 	for rows.Next() {