Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add methods for migrate #174

Merged
merged 23 commits into from
Jun 10, 2023
Merged

feat: add methods for migrate #174

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
72c32e5
feat: add methods for migrate
devhaozi Jun 10, 2023
d0d0000
fix: lint
devhaozi Jun 10, 2023
ebfc2b3
fix: lint
devhaozi Jun 10, 2023
e8f872f
fix: tests
devhaozi Jun 10, 2023
23c8857
fix: tests
devhaozi Jun 10, 2023
7bd3d1a
fix: tests
devhaozi Jun 10, 2023
6486eb8
fix: tests
devhaozi Jun 10, 2023
3c1a0b7
fix: tests
devhaozi Jun 10, 2023
739c2d7
fix: tests
devhaozi Jun 10, 2023
2e6adae
fix: tests
devhaozi Jun 10, 2023
0c5aad8
Merge branch 'master' into haozi/add-migrate
devhaozi Jun 10, 2023
afc3402
Update database/console/migrate_fresh_command.go
devhaozi Jun 10, 2023
75ae1fc
Update database/console/migrate_refresh_command.go
devhaozi Jun 10, 2023
1ac1dc7
Update database/console/migrate_fresh_command.go
devhaozi Jun 10, 2023
30a7aa2
Update database/console/migrate_refresh_command.go
devhaozi Jun 10, 2023
5d16a8b
Update database/console/migrate_refresh_command_test.go
devhaozi Jun 10, 2023
71a3729
Update database/console/migrate_reset_command.go
devhaozi Jun 10, 2023
a2ef5d5
Update database/console/migrate_reset_command_test.go
devhaozi Jun 10, 2023
bf62b69
Update database/console/migrate_rollback_command_test.go
devhaozi Jun 10, 2023
8a90455
Update database/console/migrate_status_command.go
devhaozi Jun 10, 2023
238e5a8
Update database/console/migrate_status_command.go
devhaozi Jun 10, 2023
17c6efe
feat: update tests
devhaozi Jun 10, 2023
466d810
feat: update tests
devhaozi Jun 10, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: tests
  • Loading branch information
@devhaozi
devhaozi committed Jun 10, 2023
commit 2e6adaeaaedbeb9d518fa006aa96b6c4311f28e2
15 changes: 5 additions & 10 deletions database/console/migrate_rollback_command_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ func TestMigrateRollbackCommand(t *testing.T) {
mockConfig *configmock.Config
pool *dockertest.Pool
resource *dockertest.Resource
query ormcontract.Query
_ ormcontract.Query
devhaozi marked this conversation as resolved.
Show resolved Hide resolved
)

beforeEach := func() {
Expand All @@ -34,7 +34,7 @@ func TestMigrateRollbackCommand(t *testing.T) {
setup: func() {
var err error
docker := gorm.NewMysqlDocker()
pool, resource, query, err = docker.New()
pool, resource, _, err = docker.New()
assert.Nil(t, err)
mockConfig = docker.MockConfig
createMysqlMigrations()
Expand All @@ -46,7 +46,7 @@ func TestMigrateRollbackCommand(t *testing.T) {
setup: func() {
var err error
docker := gorm.NewPostgresqlDocker()
pool, resource, query, err = docker.New()
pool, resource, _, err = docker.New()
assert.Nil(t, err)
mockConfig = docker.MockConfig
createPostgresqlMigrations()
Expand All @@ -57,7 +57,7 @@ func TestMigrateRollbackCommand(t *testing.T) {
setup: func() {
var err error
docker := gorm.NewSqlserverDocker()
pool, resource, query, err = docker.New()
pool, resource, _, err = docker.New()
assert.Nil(t, err)
mockConfig = docker.MockConfig
createSqlserverMigrations()
Expand All @@ -68,7 +68,7 @@ func TestMigrateRollbackCommand(t *testing.T) {
setup: func() {
var err error
docker := gorm.NewSqliteDocker("goravel")
pool, resource, query, err = docker.New()
pool, resource, _, err = docker.New()
assert.Nil(t, err)
mockConfig = docker.MockConfig
createSqliteMigrations()
Expand All @@ -90,11 +90,6 @@ func TestMigrateRollbackCommand(t *testing.T) {
migrateRollbackCommand := NewMigrateRollbackCommand(mockConfig)
assert.Nil(t, migrateRollbackCommand.Handle(mockContext))

devhaozi marked this conversation as resolved.
Show resolved Hide resolved
var agent Agent
err := query.Where("name", "goravel").First(&agent)
assert.Nil(t, err)
assert.True(t, agent.ID == 0)

if pool != nil && test.name != "sqlite" {
assert.Nil(t, pool.Purge(resource))
}
Expand Down
2 changes: 1 addition & 1 deletion database/console/migrate_status_command_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ func TestMigrateStatusCommand(t *testing.T) {
migrateStatusCommand := NewMigrateStatusCommand(mockConfig)
assert.Nil(t, migrateStatusCommand.Handle(mockContext))

res, err := query.Table("migrations").Where("dirty", 0).Update("dirty", 1)
res, err := query.Table("migrations").Where("dirty", false).Update("dirty", true)
assert.Nil(t, err)
assert.Equal(t, int64(1), res.RowsAffected)

Expand Down