Commit7c46f76

authored

chore: fetch templates by id without lock in dbfake (#6351)

1 parentacbe968 commit7c46f76Copy full SHA for 7c46f76

File tree

-2

lines changed

-2

lines changed

Lines changed: 6 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -936,7 +936,7 @@ func (q *fakeQuerier) GetAuthorizedWorkspaces(ctx context.Context, arg database.`
`936`	`936`	`}`
`937`	`937`
`938`	`938`	`ifarg.TemplateName!="" {`
`939`		`-template,err:=q.GetTemplateByID(ctx,workspace.TemplateID)`
	`939`	`+template,err:=q.getTemplateByIDNoLock(ctx,workspace.TemplateID)`
`940`	`940`	`iferr==nil&&!strings.EqualFold(arg.TemplateName,template.Name) {`
`941`	`941`	`continue`
`942`	`942`	`}`
`@@ -1617,10 +1617,14 @@ func (q *fakeQuerier) ParameterValues(_ context.Context, arg database.ParameterV`
`1617`	`1617`	`returnparameterValues,nil`
`1618`	`1618`	`}`
`1619`	`1619`
`1620`		`-func (q*fakeQuerier)GetTemplateByID(_ context.Context,id uuid.UUID) (database.Template,error) {`
	`1620`	`+func (q*fakeQuerier)GetTemplateByID(ctx context.Context,id uuid.UUID) (database.Template,error) {`
`1621`	`1621`	`q.mutex.RLock()`
`1622`	`1622`	`deferq.mutex.RUnlock()`
`1623`	`1623`
	`1624`	`+returnq.getTemplateByIDNoLock(ctx,id)`
	`1625`	`+}`
	`1626`	`+`
	`1627`	`+func (q*fakeQuerier)getTemplateByIDNoLock(_ context.Context,id uuid.UUID) (database.Template,error) {`
`1624`	`1628`	`for_,template:=rangeq.templates {`
`1625`	`1629`	`iftemplate.ID==id {`
`1626`	`1630`	`returntemplate,nil`

Comments

(0)