Skip to content
Snippets Groups Projects

database: update dbutil-based constructors

Closed Warren Gifford requested to merge jc/update-dbutil-constructors into main

Created by: unknwon

Updates a few dbutil-based constructors to accept datbase.DB instead, as they are not mockable in our tests.

Subscribed team please let me know if some dbutil.DBs are on purpose and can break if not using it.

This is the result of a time-boxed effort.

Test plan

Unit tests


Part of https://github.com/sourcegraph/sourcegraph/issues/26113

Merge request reports

Closed by avatar (Apr 13, 2025 2:47am UTC)

Loading

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