Commitf34431b

committed

Correct dbauthz test

Signed-off-by: Danny Kopping <danny@coder.com>

1 parentf075341 commitf34431bCopy full SHA for f34431b

File tree

-2

lines changed

-2

lines changed

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -2493,7 +2493,8 @@ func (s *MethodTestSuite) TestSystemFunctions() {`
`2493`	`2493`	`}))`
`2494`	`2494`	`s.Run("FetchNewMessageMetadata",s.Subtest(func(db database.Store,check*expects) {`
`2495`	`2495`	`// TODO: update this test once we have a specific role for notifications`
`2496`		`-check.Args(database.FetchNewMessageMetadataParams{}).Asserts(rbac.ResourceSystem,policy.ActionRead)`
	`2496`	`+u:=dbgen.User(s.T(),db, database.User{})`
	`2497`	`+check.Args(database.FetchNewMessageMetadataParams{UserID:u.ID}).Asserts(rbac.ResourceSystem,policy.ActionRead)`
`2497`	`2498`	`}))`
`2498`	`2499`	`s.Run("GetNotificationMessagesByStatus",s.Subtest(func(db database.Store,check*expects) {`
`2499`	`2500`	`// TODO: update this test once we have a specific role for notifications`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -1870,7 +1870,7 @@ func (q *FakeQuerier) FetchNewMessageMetadata(ctx context.Context, arg database.`
`1870`	`1870`	`return database.FetchNewMessageMetadataRow{},err`
`1871`	`1871`	`}`
`1872`	`1872`
`1873`		`-user,err:=q.GetUserByID(ctx,arg.UserID)`
	`1873`	`+user,err:=q.getUserByIDNoLock(arg.UserID)`
`1874`	`1874`	`iferr!=nil {`
`1875`	`1875`	`return database.FetchNewMessageMetadataRow{},xerrors.Errorf("fetch user: %w",err)`
`1876`	`1876`	`}`

Comments

(0)