Skip to content
Snippets Groups Projects

convert a bunch of dbutil.DB to database.DB

Created by: camdencheek

Now that all the stores have mockable interface versions, we can start using them. I've already begun the process in order to nail down the pattern, but in order to allow mocking across the entire codebase, we need to replace usage of dbutil.DB with database.DB so mocks can be propagated.

Progresses #26113

This commit just updates one of our graphql resolvers to use database.DB instead of dbutil.DB, then fixes the type errors. Pretty amazing how interconnected it all is.

This is a very mechanical change. The process was just:

  1. Update a resolver to use database.DB instead of dbutil.DB
  2. Fix any type errors by changing the input type to database.DB, or if there is no input type, wrap the object with database.NewDB()
  3. Repeat 2

There will be more like this, but given that these changes hit a huge portion of the code base, keeping them smaller will hopefully help with merge conflicts.

Merge request reports

Approval is optional

Merged by avatar (Aug 23, 2025 12:04pm UTC)

Merge details

  • Changes merged into main with a93de68b.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading