db: Fix a arg mismatch between ...any and any on db.Client.NamedExec (#…

…1126)
justtrackio · Jul 19, 2024 · f490c9f · f490c9f
1 parent 2b9f43d
commit f490c9f
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 29 deletions.
diff --git a/pkg/db/client.go b/pkg/db/client.go
@@ -58,7 +58,7 @@ type Client interface {
 	GetSingleScalarValue(ctx context.Context, query string, args ...interface{}) (int, error)
 	GetResult(ctx context.Context, query string, args ...interface{}) (*Result, error)
 	Exec(ctx context.Context, query string, args ...interface{}) (sql.Result, error)
-	NamedExec(ctx context.Context, query string, args ...interface{}) (sql.Result, error)
+	NamedExec(ctx context.Context, query string, arg interface{}) (sql.Result, error)
 	ExecMultiInTx(ctx context.Context, sqlers ...Sqler) (results []sql.Result, err error)
 	BindNamed(query string, arg interface{}) (string, []interface{}, error)
 	Prepare(ctx context.Context, query string) (*sql.Stmt, error)
@@ -209,10 +209,10 @@ func (c *ClientSqlx) Exec(ctx context.Context, query string, args ...interface{}
 	return res.(sql.Result), err
 }
 
-func (c *ClientSqlx) NamedExec(ctx context.Context, query string, args ...interface{}) (sql.Result, error) {
-	c.logger.Debug("> %s %q", query, args)
+func (c *ClientSqlx) NamedExec(ctx context.Context, query string, arg interface{}) (sql.Result, error) {
+	c.logger.Debug("> %s %q", query, arg)
 
-	return c.db.NamedExecContext(ctx, query, args)
+	return c.db.NamedExecContext(ctx, query, arg)
 }
 
 func (c *ClientSqlx) ExecMultiInTx(ctx context.Context, sqlers ...Sqler) (results []sql.Result, err error) {

diff --git a/pkg/db/mocks/Client.go b/pkg/db/mocks/Client.go