Skip to content

chore(coderd/database/dbauthz): migrate TestSystemFunctions to mocked DB #19301

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

hugodutka
Copy link
Contributor

Related to coder/internal#869

@hugodutka hugodutka requested a review from Emyrk August 11, 2025 18:35
@hugodutka hugodutka force-pushed the hugodutka/dbauthz-mock-2 branch from b615cb1 to d5f03c2 Compare August 14, 2025 08:24
@hugodutka hugodutka force-pushed the hugodutka/dbauthz-mock-3 branch from bf65b40 to 31de126 Compare August 14, 2025 08:24
Base automatically changed from hugodutka/dbauthz-mock-2 to main August 20, 2025 14:10
hugodutka and others added 4 commits August 20, 2025 14:54
- Convert many System functions to s.Mocked with gomock
- Add expectations for read/update/create/delete system paths
- Fix UpdateWorkspaceBuildCostByID expectation (returns error)
- Migrate AcquireProvisionerJob and ProvisionerJob update helpers
- Convert InsertWorkspaceAgent/UpsertWorkspaceApp and related to s.Mocked
- Keep RBAC assertions consistent
@hugodutka hugodutka force-pushed the hugodutka/dbauthz-mock-3 branch from 31de126 to 3dd08e2 Compare August 20, 2025 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant