refactored

mtojek · mtojek · commit d96a34a10fec · 2024-09-17T13:17:42.000+02:00
diff --git a/coderd/database/dbauthz/dbauthz.go b/coderd/database/dbauthz/dbauthz.go
@@ -898,13 +898,6 @@ func (q *querier) customRoleCheck(ctx context.Context, role database.CustomRole)
 	return nil
 }
 
-func (q *querier) DeleteOldNotificationReportGeneratorLogs(ctx context.Context, arg database.DeleteOldNotificationReportGeneratorLogsParams) error {
-	if err := q.authorizeContext(ctx, policy.ActionDelete, rbac.ResourceSystem); err != nil {
-		return err
-	}
-	return q.db.DeleteOldNotificationReportGeneratorLogs(ctx, arg)
-}
-
 func (q *querier) AcquireLock(ctx context.Context, id int64) error {
 	return q.db.AcquireLock(ctx, id)
 }
diff --git a/coderd/database/dbauthz/dbauthz_test.go b/coderd/database/dbauthz/dbauthz_test.go
@@ -2735,12 +2735,6 @@ func (s *MethodTestSuite) TestSystemFunctions() {
 			Value: "value",
 		}).Asserts(rbac.ResourceSystem, policy.ActionCreate)
 	}))
-	s.Run("DeleteOldNotificationReportGeneratorLogs", s.Subtest(func(db database.Store, check *expects) {
-		check.Args(database.DeleteOldNotificationReportGeneratorLogsParams{
-			Before:                 dbtime.Now(),
-			NotificationTemplateID: uuid.New(),
-		}).Asserts(rbac.ResourceSystem, policy.ActionDelete)
-	}))
 	s.Run("GetFailedWorkspaceBuildsByTemplateID", s.Subtest(func(db database.Store, check *expects) {
 		check.Args(database.GetFailedWorkspaceBuildsByTemplateIDParams{
 			TemplateID: uuid.New(),
diff --git a/coderd/database/dbmetrics/dbmetrics.go b/coderd/database/dbmetrics/dbmetrics.go

Original file line number	Diff line number	Diff line change
`@@ -898,13 +898,6 @@ func (q *querier) customRoleCheck(ctx context.Context, role database.CustomRole)`
`898`	`898`	`return nil`
`899`	`899`	`}`
`900`	`900`
`901`		`-func (q *querier) DeleteOldNotificationReportGeneratorLogs(ctx context.Context, arg database.DeleteOldNotificationReportGeneratorLogsParams) error {`
`902`		`- if err := q.authorizeContext(ctx, policy.ActionDelete, rbac.ResourceSystem); err != nil {`
`903`		`- return err`
`904`		`- }`
`905`		`- return q.db.DeleteOldNotificationReportGeneratorLogs(ctx, arg)`
`906`		`-}`
`907`		`-`
`908`	`901`	`func (q *querier) AcquireLock(ctx context.Context, id int64) error {`
`909`	`902`	`return q.db.AcquireLock(ctx, id)`
`910`	`903`	`}`