Makes aclitem test types more specific

pull/210/head
Manni Wood 2016-11-12 12:36:55 -05:00
parent 104c01df21
commit 96b652cc95
1 changed files with 8 additions and 8 deletions

View File

@ -643,9 +643,9 @@ func TestNullX(t *testing.T) {
} }
} }
func assertAclItemSlicesEqual(t *testing.T, query, scan interface{}) { func assertAclItemSlicesEqual(t *testing.T, query, scan []pgx.AclItem) {
if !reflect.DeepEqual(query, *(scan.(*[]pgx.AclItem))) { if !reflect.DeepEqual(query, scan) {
t.Errorf("failed to encode aclitem[]\n EXPECTED: %v\n ACTUAL: %v", query, *(scan.(*[]pgx.AclItem))) t.Errorf("failed to encode aclitem[]\n EXPECTED: %v\n ACTUAL: %v", query, scan)
} }
} }
@ -658,20 +658,20 @@ func TestAclArrayDecoding(t *testing.T) {
sql := "select $1::aclitem[]" sql := "select $1::aclitem[]"
tests := []struct { tests := []struct {
query interface{} query []pgx.AclItem
scan interface{} scan []pgx.AclItem
}{ }{
{ {
[]pgx.AclItem{"=r/postgres"}, []pgx.AclItem{"=r/postgres"},
&[]pgx.AclItem{}, []pgx.AclItem{},
}, },
{ {
[]pgx.AclItem{"=r/postgres", "postgres=arwdDxt/postgres"}, []pgx.AclItem{"=r/postgres", "postgres=arwdDxt/postgres"},
&[]pgx.AclItem{}, []pgx.AclItem{},
}, },
} }
for i, tt := range tests { for i, tt := range tests {
err := conn.QueryRow(sql, tt.query).Scan(tt.scan) err := conn.QueryRow(sql, tt.query).Scan(&tt.scan)
if err != nil { if err != nil {
t.Errorf(`%d. error reading array: %v`, i, err) t.Errorf(`%d. error reading array: %v`, i, err)
if pgerr, ok := err.(pgx.PgError); ok { if pgerr, ok := err.(pgx.PgError); ok {