Skip to content

Commit

Permalink
fix: adds context to all methods
Browse files Browse the repository at this point in the history
Signed-off-by: Neeraj Gartia <neerajgartia211002@gmail.com>
  • Loading branch information
NeerajGartia21 committed Jun 26, 2023
1 parent 8922832 commit eb6ce52
Showing 1 changed file with 10 additions and 10 deletions.
20 changes: 10 additions & 10 deletions pkg/server/infrastructure/datastore/mysql/mysql.go
Original file line number Diff line number Diff line change
Expand Up @@ -50,13 +50,13 @@ func New(ctx context.Context, cfg datastore.Config) (datastore.DataStore, error)
}

for _, v := range model.GetRegisterModels() {
if err := db.AutoMigrate(v); err != nil {
if err := db.WithContext(ctx).AutoMigrate(v); err != nil {
return nil, err
}

Check warning on line 55 in pkg/server/infrastructure/datastore/mysql/mysql.go

View check run for this annotation

Codecov / codecov/patch

pkg/server/infrastructure/datastore/mysql/mysql.go#L52-L55

Added lines #L52 - L55 were not covered by tests
}

m := &mysql{
client: *db,
client: *db.WithContext(ctx),

Check warning on line 59 in pkg/server/infrastructure/datastore/mysql/mysql.go

View check run for this annotation

Codecov / codecov/patch

pkg/server/infrastructure/datastore/mysql/mysql.go#L59

Added line #L59 was not covered by tests
database: cfg.Database,
}
return m, nil
Expand All @@ -73,7 +73,7 @@ func (m *mysql) Add(ctx context.Context, entity datastore.Entity) error {
entity.SetCreateTime(time.Now())
entity.SetUpdateTime(time.Now())

if dbAdd := m.client.Create(entity); dbAdd.Error != nil {
if dbAdd := m.client.WithContext(ctx).Create(entity); dbAdd.Error != nil {
if match := errors.Is(dbAdd.Error, gorm.ErrDuplicatedKey); match {
return datastore.ErrRecordExist
}
Expand Down Expand Up @@ -114,7 +114,7 @@ func (m *mysql) Get(ctx context.Context, entity datastore.Entity) error {
return datastore.ErrTableNameEmpty
}

Check warning on line 115 in pkg/server/infrastructure/datastore/mysql/mysql.go

View check run for this annotation

Codecov / codecov/patch

pkg/server/infrastructure/datastore/mysql/mysql.go#L110-L115

Added lines #L110 - L115 were not covered by tests

if dbGet := m.client.First(entity); dbGet.Error != nil {
if dbGet := m.client.WithContext(ctx).First(entity); dbGet.Error != nil {
if errors.Is(dbGet.Error, gorm.ErrRecordNotFound) {
return datastore.ErrRecordNotExist
}
Expand All @@ -132,7 +132,7 @@ func (m *mysql) Put(ctx context.Context, entity datastore.Entity) error {
return datastore.ErrTableNameEmpty
}
entity.SetUpdateTime(time.Now())
if dbPut := m.client.Model(entity).Updates(entity); dbPut.Error != nil {
if dbPut := m.client.WithContext(ctx).Model(entity).Updates(entity); dbPut.Error != nil {
if errors.Is(dbPut.Error, gorm.ErrRecordNotFound) {
return datastore.ErrRecordNotExist
}
Expand All @@ -150,7 +150,7 @@ func (m *mysql) IsExist(ctx context.Context, entity datastore.Entity) (bool, err
return false, datastore.ErrTableNameEmpty
}

Check warning on line 151 in pkg/server/infrastructure/datastore/mysql/mysql.go

View check run for this annotation

Codecov / codecov/patch

pkg/server/infrastructure/datastore/mysql/mysql.go#L146-L151

Added lines #L146 - L151 were not covered by tests

if dbExist := m.client.First(entity); dbExist.Error != nil {
if dbExist := m.client.WithContext(ctx).First(entity); dbExist.Error != nil {
if errors.Is(dbExist.Error, gorm.ErrRecordNotFound) {
return false, nil
}
Expand All @@ -173,7 +173,7 @@ func (m *mysql) Delete(ctx context.Context, entity datastore.Entity) error {
return err
}

Check warning on line 174 in pkg/server/infrastructure/datastore/mysql/mysql.go

View check run for this annotation

Codecov / codecov/patch

pkg/server/infrastructure/datastore/mysql/mysql.go#L172-L174

Added lines #L172 - L174 were not covered by tests

if dbDelete := m.client.Model(entity).Delete(entity); dbDelete.Error != nil {
if dbDelete := m.client.WithContext(ctx).Model(entity).Delete(entity); dbDelete.Error != nil {
klog.Errorf("delete document failure %w", dbDelete.Error)
return datastore.NewDBError(dbDelete.Error)
}

Check warning on line 179 in pkg/server/infrastructure/datastore/mysql/mysql.go

View check run for this annotation

Codecov / codecov/patch

pkg/server/infrastructure/datastore/mysql/mysql.go#L176-L179

Added lines #L176 - L179 were not covered by tests
Expand Down Expand Up @@ -260,7 +260,7 @@ func (m *mysql) List(ctx context.Context, entity datastore.Entity, op *datastore
})

Check warning on line 260 in pkg/server/infrastructure/datastore/mysql/mysql.go

View check run for this annotation

Codecov / codecov/patch

pkg/server/infrastructure/datastore/mysql/mysql.go#L217-L260

Added lines #L217 - L260 were not covered by tests
}
var list []datastore.Entity
rows, err := m.client.Model(entity).Clauses(clauses...).Rows()
rows, err := m.client.WithContext(ctx).Model(entity).Clauses(clauses...).Rows()
if err != nil {
return nil, datastore.NewDBError(err)
}
Expand All @@ -274,7 +274,7 @@ func (m *mysql) List(ctx context.Context, entity datastore.Entity, op *datastore
if err != nil {
return nil, datastore.NewDBError(err)
}
err = m.client.ScanRows(rows, &item)
err = m.client.WithContext(ctx).ScanRows(rows, &item)
if err != nil {
return nil, datastore.NewDBError(fmt.Errorf("row scan failure %w", err))
}
Expand Down Expand Up @@ -310,7 +310,7 @@ func (m *mysql) Count(ctx context.Context, entity datastore.Entity, filterOption
Exprs: exprs,
})
}
if dbCount := m.client.Model(entity).Clauses(clauses...).Count(&count); dbCount.Error != nil {
if dbCount := m.client.WithContext(ctx).Model(entity).Clauses(clauses...).Count(&count); dbCount.Error != nil {
return 0, datastore.NewDBError(dbCount.Error)
}
return count, nil

Check warning on line 316 in pkg/server/infrastructure/datastore/mysql/mysql.go

View check run for this annotation

Codecov / codecov/patch

pkg/server/infrastructure/datastore/mysql/mysql.go#L291-L316

Added lines #L291 - L316 were not covered by tests
Expand Down

0 comments on commit eb6ce52

Please sign in to comment.