Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

chore(coderd/database/dbauthz): migrate more tests to mocked db#19300

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
Emyrk merged 8 commits intomainfromhugodutka/dbauthz-mock-2
Aug 20, 2025

Conversation

hugodutka
Copy link
Contributor

@hugodutkahugodutka requested a review fromEmyrkAugust 11, 2025 18:00
@hugodutkahugodutka changed the titlechore(coderd/database/dbauthz): migrate License, Organization OIDC claims, WorkspaceProxy, and ProvisionerKeys tests to use mocked dbchore(coderd/database/dbauthz): migrate License, Organization OIDC claims, WorkspaceProxy, and ProvisionerKeys testsAug 11, 2025
@hugodutkahugodutka changed the titlechore(coderd/database/dbauthz): migrate License, Organization OIDC claims, WorkspaceProxy, and ProvisionerKeys testschore(coderd/database/dbauthz): migrate more tests to mocked dbAug 11, 2025
Base automatically changed fromhugodutka/dbauthz-mock-1 tomainAugust 13, 2025 14:30
blink-sobotand others added5 commitsAugust 14, 2025 08:16
…n OIDC claims, WorkspaceProxy, ProvisionerKeys)\n\n- Convert tests to s.Mocked\n- Add gomock expectations for fetch and authorized paths\n- Remove unused imports\n\nCo-authored-by: hugodutka <28019628+hugodutka@users.noreply.github.com>
- Convert DBCrypt subtests to s.Mocked with gomock expectations- Remove reliance on real DB for setup- Keeps consistent style with prior groups
- Convert CryptoKeys subtests to s.Mocked with gomock expectations- Return proper values for methods that return rows- Aligns with prior mocked migration groups
- Convert UserSecrets subtests to s.Mocked using gomock- Add expectations for prefetch (GetUserSecret) and CRUD paths- Aligns with recent mocked migrations for CryptoKeys/DBCrypt
@hugodutkahugodutkaforce-pushed thehugodutka/dbauthz-mock-2 branch fromb615cb1 tod5f03c2CompareAugust 14, 2025 08:24
@EmyrkEmyrkenabled auto-merge (squash)August 20, 2025 13:57
@EmyrkEmyrk merged commitd536b91 intomainAug 20, 2025
26 checks passed
@EmyrkEmyrk deleted the hugodutka/dbauthz-mock-2 branchAugust 20, 2025 14:10
@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsAug 20, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.
Reviewers

@EmyrkEmyrkEmyrk approved these changes

Assignees

@hugodutkahugodutka

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@hugodutka@Emyrk
[8]ページ先頭
©2009-2025 Movatter.jp