Commit5b08f8b

authored

fix: change createWorkspace to use dbtime.Time (#19414)

The `createWorkspace` function was updated to use an injected Clock,which makes it possible to mock time in tests:https://github.com/coder/coder/pull/19264/files#diff-46f90baab52ea3ad914acbde30d656dbc8e46f5918d19bc056c445a1dc502482R1130For database operations, however, it is recommended to use `dbtime.Time`since it rounds to the microsecond, the smallest unit of precisionsupported by Postgres.

1 parent1a601c3 commit5b08f8bCopy full SHA for 5b08f8b

File tree

-1

lines changed

-1

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -636,7 +636,7 @@ func createWorkspace(`
`636`	`636`	`)`
`637`	`637`
`638`	`638`	`// Use injected Clock to allow time mocking in tests`
`639`		`-now:=api.Clock.Now()`
	`639`	`+now:=dbtime.Time(api.Clock.Now())`
`640`	`640`
`641`	`641`	`templateVersionPresetID:=req.TemplateVersionPresetID`
`642`	`642`

Comments

(0)