8000 refactor: organize OAuth2 provider tests into dedicated packages by ThomasK33 · Pull Request #18747 · coder/coder · GitHub
[go: up one dir, main page]
Skip to content

refactor: organize OAuth2 provider tests into dedicated packages #18747

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.

Sign up for GitHub

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

Merged
merged 1 commit into from
Jul 3, 2025
Merged

refactor: organize OAuth2 provider tests into dedicated packages #18747

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
refactor(oauth2): migrate test files to oauth2provider package 
- Create oauth2provider/metadata_test.go with OAuth2 metadata endpoint tests
- Create oauth2provider/validation_test.go with comprehensive client validation tests
- Create oauth2provider/provider_test.go with provider app and registration tests
- Remove duplicated tests from coderd/oauth2_test.go, keep integration tests
- Fix lint issues in oauth2providertest package naming
- Improve test organization with unit tests separated from integration tests
- Maintain 100% test coverage while enabling faster test execution

Change-Id: I99aac9f53df95eed3135d895781831a4a2749f6a
Signed-off-by: Thomas Kosiewski <tk@coder.com>
  • Loading branch information
@ThomasK33
ThomasK33 committed Jul 3, 2025
commit 860ba25707860302b2c5f71f01f6b2fe6222fc3c
Loading
Loading
0