Merge pull request 'adding in file path option' (#2) from new/file-path into main

Reviewed-on: #2

option.go

@@ -1,15 +1,26 @@
 package lite
 
-type Option func(d *database)
+import "errors"
+
+var (
+	ErrBadFilename error = errors.New("database filename cannot be empty")
+)
+
+type Option func(d *database) error
 
 func WithFile(name string) Option {
-	return func(d *database) {
+	return func(d *database) error {
+		if name == "" {
+			return ErrBadFilename
+		}
 		d.file = name
+		return nil
 	}
 }
 
 func WithPath(path string) Option {
-	return func(d *database) {
+	return func(d *database) error {
 		d.path = path
+		return nil
 	}
 }

sqlite.go

@@ -4,7 +4,7 @@ import (
 	"context"
 	"database/sql"
 	"fmt"
-	"os"
+	"path"
 
 	"github.com/jmoiron/sqlx"
 	_ "github.com/ncruces/go-sqlite3/driver"
@@ -18,7 +18,6 @@ type database struct {
 }
 
 type Database interface {
-	Connect() error
 	Close() error
 	Exec(ctx context.Context, query string, args ...any) (sql.Result, error)
 	Query(ctx context.Context, query string, args ...any) (*sqlx.Rows, error)
@@ -27,35 +26,28 @@ type Database interface {
 }
 
 func New(opts ...Option) (Database, error) {
-	home, err := os.UserHomeDir()
-	if err != nil {
-		return nil, err
-	}
-
 	defaultFile := "lite-db"
 
 	db := &database{
 		file: defaultFile,
-		path: home,
 	}
 
 	for _, opt := range opts {
 		opt(db)
 	}
 
-	return db, nil
+	filePath := db.file
+	if db.path != "" {
+		filePath = path.Join(db.path, db.file)
 	}
 
-// connects to database file
+	slite, err := sqlx.Open("sqlite3", fmt.Sprintf("file:%s", filePath))
-func (d *database) Connect() error {
-	db, err := sqlx.Open("sqlite3", fmt.Sprintf("file:%s/%s", d.path, d.file))
 	if err != nil {
-		return err
+		return nil, err
 	}
+	db.sqlite = slite
 
-	d.sqlite = db
+	return db, nil
-
-	return nil
 }
 
 // closes database file connection