Renamed builds to remove adapter word duplication

Author: vgarvardt

README.md

@@ -28,7 +28,7 @@ import (
 
 	"github.com/jackc/pgx"
 	pg "github.com/vgarvardt/go-oauth2-pg"
-	pgxAdapter "github.com/vgarvardt/go-oauth2-pg/adapter/pgx"
+	pgxAdapter "github.com/vgarvardt/go-oauth2-pg/pgx_adapter"
 	"gopkg.in/oauth2.v3/manage"
 )
 
@@ -39,7 +39,7 @@ func main() {
 	manager := manage.NewDefaultManager()
 
 	// use PostgreSQL token store with pgx.Connection adapter
-	store, _ := pg.NewStore(pgxAdapter.NewConnAdapter(pgxConn), pg.WithGCInterval(time.Minute))
+	store, _ := pg.NewStore(pgxAdapter.NewConn(pgxConn), pg.WithGCInterval(time.Minute))
 	defer store.Close()
 
 	manager.MapTokenStorage(store)

pgx_adapter/adapter.go

@@ -6,34 +6,34 @@ import (
 	pgxHelpers "github.com/vgarvardt/pgx-helpers"
 )
 
-// ConnPoolAdapter is the adapter type for PGx connection pool connection type
-type ConnPoolAdapter struct {
+// ConnPool is the adapter type for PGx connection pool connection type
+type ConnPool struct {
 	conn *pgx.ConnPool
 }
 
-// NewConnPoolAdapter instantiates PGx connection pool adapter
-func NewConnPoolAdapter(conn *pgx.ConnPool) *ConnPoolAdapter {
-	return &ConnPoolAdapter{conn}
+// NewConnPool instantiates PGx connection pool adapter
+func NewConnPool(conn *pgx.ConnPool) *ConnPool {
+	return &ConnPool{conn}
 }
 
-// ConnAdapter is the adapter type for PGx connection connection type
-type ConnAdapter struct {
+// Conn is the adapter type for PGx connection connection type
+type Conn struct {
 	conn *pgx.Conn
 }
 
-// NewConnAdapter instantiates PGx connection adapter
-func NewConnAdapter(conn *pgx.Conn) *ConnAdapter {
-	return &ConnAdapter{conn}
+// NewConn instantiates PGx connection adapter
+func NewConn(conn *pgx.Conn) *Conn {
+	return &Conn{conn}
 }
 
 // Exec runs a query and returns an error if any
-func (a *ConnPoolAdapter) Exec(query string, args ...interface{}) error {
+func (a *ConnPool) Exec(query string, args ...interface{}) error {
 	_, err := a.conn.Exec(query, args...)
 	return err
 }
 
 // SelectOne runs a select query and scans the object into a struct or returns an error
-func (a *ConnPoolAdapter) SelectOne(dst interface{}, query string, args ...interface{}) error {
+func (a *ConnPool) SelectOne(dst interface{}, query string, args ...interface{}) error {
 	row := a.conn.QueryRow(query, args...)
 	if err := pgxHelpers.ScanStruct(row, dst); err != nil {
 		if err == pgx.ErrNoRows {
@@ -46,13 +46,13 @@ func (a *ConnPoolAdapter) SelectOne(dst interface{}, query string, args ...inter
 }
 
 // Exec runs a query and returns an error if any
-func (a *ConnAdapter) Exec(query string, args ...interface{}) error {
+func (a *Conn) Exec(query string, args ...interface{}) error {
 	_, err := a.conn.Exec(query, args...)
 	return err
 }
 
 // SelectOne runs a select query and scans the object into a struct or returns an error
-func (a *ConnAdapter) SelectOne(dst interface{}, query string, args ...interface{}) error {
+func (a *Conn) SelectOne(dst interface{}, query string, args ...interface{}) error {
 	row := a.conn.QueryRow(query, args...)
 	if err := pgxHelpers.ScanStruct(row, dst); err != nil {
 		if err == pgx.ErrNoRows {

pgx_adapter/adapter_test.go

@@ -53,7 +53,7 @@ func generateTableName() string {
 	return fmt.Sprintf("token_%d", time.Now().UnixNano())
 }
 
-func TestNewConnAdapter(t *testing.T) {
+func TestNewConn(t *testing.T) {
 	l := new(memoryLogger)
 
 	pgxConnConfig, err := pgx.ParseURI(uri)
@@ -68,7 +68,7 @@ func TestNewConnAdapter(t *testing.T) {
 		assert.NoError(t, pgxConn.Close())
 	}()
 
-	adapter := NewConnAdapter(pgxConn)
+	adapter := NewConn(pgxConn)
 	tableName := generateTableName()
 
 	store, err := pg.NewStore(adapter, pg.WithLogger(l), pg.WithTableName(tableName), pg.WithGCInterval(time.Second))
@@ -80,7 +80,7 @@ func TestNewConnAdapter(t *testing.T) {
 	runStoreTest(t, store, l)
 }
 
-func TestNewConnPoolAdapter(t *testing.T) {
+func TestNewConnPool(t *testing.T) {
 	l := new(memoryLogger)
 
 	pgxConnConfig, err := pgx.ParseURI(uri)
@@ -95,7 +95,7 @@ func TestNewConnPoolAdapter(t *testing.T) {
 
 	defer pgXConnPool.Close()
 
-	adapter := NewConnPoolAdapter(pgXConnPool)
+	adapter := NewConnPool(pgXConnPool)
 	tableName := generateTableName()
 
 	store, err := pg.NewStore(adapter, pg.WithLogger(l), pg.WithTableName(tableName), pg.WithGCInterval(time.Second))

sql_adapter/adapter.go

@@ -11,14 +11,14 @@ type Adapter struct {
 	conn *sqlx.DB
 }
 
-// NewSQL instantiates sqlx.DB connection adapter from sql.DB connection
-func NewSQL(conn *sql.DB) *Adapter {
+// New instantiates sqlx.DB connection adapter from sql.DB connection
+func New(conn *sql.DB) *Adapter {
 	// The driverName of the original database is required for named query support - we do not use it here
 	return &Adapter{sqlx.NewDb(conn, "")}
 }
 
-// NewSQLx instantiates sqlx.DB connection adapter
-func NewSQLx(conn *sqlx.DB) *Adapter {
+// NewX instantiates sqlx.DB connection adapter
+func NewX(conn *sqlx.DB) *Adapter {
 	return &Adapter{conn}
 }
 

sql_adapter/adapter_test.go

@@ -41,7 +41,7 @@ func generateTableName() string {
 	return fmt.Sprintf("token_%d", time.Now().UnixNano())
 }
 
-func TestNewSQL(t *testing.T) {
+func TestNew(t *testing.T) {
 	l := new(memoryLogger)
 
 	conn, err := sql.Open("pgx", uri)
@@ -51,7 +51,7 @@ func TestNewSQL(t *testing.T) {
 		assert.NoError(t, conn.Close())
 	}()
 
-	adapter := NewSQL(conn)
+	adapter := New(conn)
 	tableName := generateTableName()
 
 	store, err := pg.NewStore(adapter, pg.WithLogger(l), pg.WithTableName(tableName), pg.WithGCInterval(time.Second))
@@ -63,7 +63,7 @@ func TestNewSQL(t *testing.T) {
 	runStoreTest(t, store, l)
 }
 
-func TestNewSQLx(t *testing.T) {
+func TestNewX(t *testing.T) {
 	l := new(memoryLogger)
 
 	conn, err := sql.Open("pgx", uri)
@@ -73,7 +73,7 @@ func TestNewSQLx(t *testing.T) {
 		assert.NoError(t, conn.Close())
 	}()
 
-	adapter := NewSQLx(sqlx.NewDb(conn, ""))
+	adapter := NewX(sqlx.NewDb(conn, ""))
 	tableName := generateTableName()
 
 	store, err := pg.NewStore(adapter, pg.WithLogger(l), pg.WithTableName(tableName), pg.WithGCInterval(time.Second))