fix nilaway

database/console/driver/sqlite.go

@@ -19,11 +19,9 @@ func init() {
 	database.Register("sqlite", &Sqlite{})
 }
 
-var DefaultMigrationsTable = "schema_migrations"
 var (
-	ErrDatabaseDirty  = fmt.Errorf("database is dirty")
-	ErrNilConfig      = fmt.Errorf("no config")
-	ErrNoDatabaseName = fmt.Errorf("no database name")
+	DefaultMigrationsTable = "schema_migrations"
+	ErrNilConfig           = fmt.Errorf("no config")
 )
 
 type Config struct {

database/console/migration/migrate.go

@@ -2,7 +2,6 @@ package migration
 
 import (
 	"database/sql"
-	"errors"
 	"fmt"
 
 	"github.com/golang-migrate/migrate/v4"
@@ -14,6 +13,7 @@ import (
 	"github.com/goravel/framework/contracts/database"
 	"github.com/goravel/framework/database/console/driver"
 	databasedb "github.com/goravel/framework/database/db"
+	"github.com/goravel/framework/errors"
 	"github.com/goravel/framework/support"
 )
 
@@ -28,7 +28,7 @@ func getMigrate(config config.Config) (*migrate.Migrate, error) {
 	configBuilder := databasedb.NewConfigBuilder(config, connection)
 	writeConfigs := configBuilder.Writes()
 	if len(writeConfigs) == 0 {
-		return nil, errors.New("not found database configuration")
+		return nil, errors.OrmDatabaseConfigNotFound
 	}
 
 	switch dbDriver {
@@ -110,6 +110,6 @@ func getMigrate(config config.Config) (*migrate.Migrate, error) {
 
 		return migrate.NewWithDatabaseInstance(dir, "sqlserver", instance)
 	default:
-		return nil, errors.New("database driver only support mysql, postgres, sqlite and sqlserver")
+		return nil, errors.OrmDriverNotSupported
 	}
 }

database/console/migration/migrate_fresh_command.go

@@ -1,14 +1,14 @@
 package migration
 
 import (
-	"errors"
 	"strings"
 
 	"github.com/golang-migrate/migrate/v4"
 
 	"github.com/goravel/framework/contracts/config"
 	"github.com/goravel/framework/contracts/console"
 	"github.com/goravel/framework/contracts/console/command"
+	"github.com/goravel/framework/errors"
 	"github.com/goravel/framework/support/color"
 )
 

database/console/migration/migrate_make_command.go

@@ -1,12 +1,11 @@
 package migration
 
 import (
-	"errors"
-
 	"github.com/goravel/framework/contracts/config"
 	"github.com/goravel/framework/contracts/console"
 	"github.com/goravel/framework/contracts/console/command"
 	"github.com/goravel/framework/contracts/database/migration"
+	"github.com/goravel/framework/errors"
 	"github.com/goravel/framework/support/color"
 )
 
@@ -47,7 +46,7 @@ func (r *MigrateMakeCommand) Handle(ctx console.Context) error {
 		name, err = ctx.Ask("Enter the migration name", console.AskOption{
 			Validate: func(s string) error {
 				if s == "" {
-					return errors.New("the migration name cannot be empty")
+					return errors.MigrationNameIsRequired
 				}
 
 				return nil

database/console/migration/migrate_refresh_command.go

@@ -1,7 +1,6 @@
 package migration
 
 import (
-	"errors"
 	"strconv"
 	"strings"
 
@@ -10,6 +9,7 @@ import (
 	"github.com/goravel/framework/contracts/config"
 	"github.com/goravel/framework/contracts/console"
 	"github.com/goravel/framework/contracts/console/command"
+	"github.com/goravel/framework/errors"
 	"github.com/goravel/framework/support/color"
 )
 

database/console/migration/migrate_reset_command.go

@@ -1,13 +1,12 @@
 package migration
 
 import (
-	"errors"
-
 	"github.com/golang-migrate/migrate/v4"
 
 	"github.com/goravel/framework/contracts/config"
 	"github.com/goravel/framework/contracts/console"
 	"github.com/goravel/framework/contracts/console/command"
+	"github.com/goravel/framework/errors"
 	"github.com/goravel/framework/support/color"
 )
 

database/console/migration/migrate_rollback_command.go

@@ -1,7 +1,6 @@
 package migration
 
 import (
-	"errors"
 	"strconv"
 
 	_ "github.com/go-sql-driver/mysql"
@@ -11,6 +10,7 @@ import (
 	"github.com/goravel/framework/contracts/config"
 	"github.com/goravel/framework/contracts/console"
 	"github.com/goravel/framework/contracts/console/command"
+	"github.com/goravel/framework/errors"
 	"github.com/goravel/framework/support/color"
 )
 

database/console/seed_command.go

@@ -1,22 +1,20 @@
 package console
 
 import (
-	"errors"
-	"fmt"
-
 	"github.com/goravel/framework/contracts/config"
 	"github.com/goravel/framework/contracts/console"
 	"github.com/goravel/framework/contracts/console/command"
-	"github.com/goravel/framework/contracts/database/seeder"
+	contractsseeder "github.com/goravel/framework/contracts/database/seeder"
+	"github.com/goravel/framework/errors"
 	"github.com/goravel/framework/support/color"
 )
 
 type SeedCommand struct {
 	config config.Config
-	seeder seeder.Facade
+	seeder contractsseeder.Facade
 }
 
-func NewSeedCommand(config config.Config, seeder seeder.Facade) *SeedCommand {
+func NewSeedCommand(config config.Config, seeder contractsseeder.Facade) *SeedCommand {
 	return &SeedCommand{
 		config: config,
 		seeder: seeder,
@@ -85,19 +83,19 @@ func (receiver *SeedCommand) ConfirmToProceed(force bool) error {
 		return nil
 	}
 
-	return errors.New("application in production use --force to run this command")
+	return errors.DBForceIsRequiredInProduction
 }
 
 // GetSeeders returns a seeder instances
-func (receiver *SeedCommand) GetSeeders(names []string) ([]seeder.Seeder, error) {
+func (receiver *SeedCommand) GetSeeders(names []string) ([]contractsseeder.Seeder, error) {
 	if len(names) == 0 {
 		return receiver.seeder.GetSeeders(), nil
 	}
-	var seeders []seeder.Seeder
+	var seeders []contractsseeder.Seeder
 	for _, name := range names {
 		seeder := receiver.seeder.GetSeeder(name)
 		if seeder == nil {
-			return nil, fmt.Errorf("no seeder of %s found", name)
+			return nil, errors.DBSeederNotFound.Args(name)
 		}
 		seeders = append(seeders, seeder)
 	}

database/console/seed_command_test.go

@@ -6,6 +6,7 @@ import (
 	"github.com/stretchr/testify/suite"
 
 	"github.com/goravel/framework/contracts/database/seeder"
+	"github.com/goravel/framework/errors"
 	configmocks "github.com/goravel/framework/mocks/config"
 	consolemocks "github.com/goravel/framework/mocks/console"
 	seedermocks "github.com/goravel/framework/mocks/database/seeder"
@@ -51,7 +52,7 @@ func (s *SeedCommandTestSuite) TestConfirmToProceed() {
 
 	s.mockContext.On("OptionBool", "force").Return(false).Once()
 	err = s.seedCommand.ConfirmToProceed(false)
-	s.EqualError(err, "application in production use --force to run this command")
+	s.ErrorIs(err, errors.DBForceIsRequiredInProduction)
 }
 
 func (s *SeedCommandTestSuite) TestGetSeeders() {

database/migration/sql_driver_test.go

@@ -20,10 +20,13 @@ type SqlDriverSuite struct {
 	suite.Suite
 	mockConfig        *mocksconfig.Config
 	driverToTestQuery map[contractsdatabase.Driver]*gorm.TestQuery
-	driver            *SqlDriver
 }
 
 func TestSqlDriverSuite(t *testing.T) {
+	if env.IsWindows() {
+		t.Skip("Skipping tests of using docker")
+	}
+
 	suite.Run(t, &SqlDriverSuite{})
 }
 

database/service_provider.go

@@ -67,18 +67,19 @@ func (r *ServiceProvider) Boot(app foundation.Application) {
 }
 
 func (r *ServiceProvider) registerCommands(app foundation.Application) {
-	if artisanFacade := app.MakeArtisan(); artisanFacade != nil {
-		config := app.MakeConfig()
-		schema := app.MakeSchema()
-		seeder := app.MakeSeeder()
+	artisan := app.MakeArtisan()
+	config := app.MakeConfig()
+	schema := app.MakeSchema()
+	seeder := app.MakeSeeder()
 
-		artisanFacade.Register([]contractsconsole.Command{
+	if artisan != nil && config != nil && schema != nil && seeder != nil {
+		artisan.Register([]contractsconsole.Command{
 			consolemigration.NewMigrateMakeCommand(config, schema),
 			consolemigration.NewMigrateCommand(config, schema),
 			consolemigration.NewMigrateRollbackCommand(config),
 			consolemigration.NewMigrateResetCommand(config),
-			consolemigration.NewMigrateRefreshCommand(config, artisanFacade),
-			consolemigration.NewMigrateFreshCommand(config, artisanFacade),
+			consolemigration.NewMigrateRefreshCommand(config, artisan),
+			consolemigration.NewMigrateFreshCommand(config, artisan),
 			consolemigration.NewMigrateStatusCommand(config),
 			console.NewModelMakeCommand(),
 			console.NewObserverMakeCommand(),

errors/list.go

@@ -42,6 +42,9 @@ var (
 	CryptMissingIVKey        = New("decrypt payload error: missing iv key")
 	CryptMissingValueKey     = New("decrypt payload error: missing value key")
 
+	DBForceIsRequiredInProduction = New("application in production use --force to run this command")
+	DBSeederNotFound              = New("not found %s seeder")
+
 	EventListenerNotBind = New("event %v doesn't bind listeners")
 
 	FilesystemDefaultDiskNotSet   = New("please set default disk")
@@ -62,6 +65,7 @@ var (
 	LogDriverNotSupported      = New("invalid driver: %s, only support stack, single, daily, custom").SetModule(ModuleLog)
 	LogEmptyLogFilePath        = New("empty log file path").SetModule(ModuleLog)
 
+	MigrationNameIsRequired    = New("migration name cannot be empty")
 	MigrationUnsupportedDriver = New("unsupported migration driver: %s")
 
 	OrmDatabaseConfigNotFound      = New("not found database configuration")

support/docker/docker.go

@@ -15,7 +15,7 @@ const (
 	TestModelNormal
 
 	// Switch this value to control the test model.
-	TestModel = TestModelMinimum
+	TestModel = TestModelNormal
 )
 
 type ContainerType string