Commonize OIDC handler test cases with table-driven generation

[kbukum1]

Commonize OIDC handler test cases with table-driven generation

Follow-up from #108 — reduces repetitive test code while preserving identical coverage.

Problem

oidc_handling_test.go had 65 hand-written test cases (13 ecosystems × 5 OIDC providers) with near-identical structure. Adding a new provider required copying ~13 blocks; adding a new ecosystem required copying ~5 blocks. Each block repeated the same provider credential fields, handler factory, and assertion logic.

Approach

Introduce buildEcosystemCases helper and oidcProviderFields shared function:

• oidcProviderFields(provider) — returns provider-specific credential fields (aws-region, tenant-id, etc.) shared across all ecosystems
• buildEcosystemCases(name, factory, logLabel, variants) — generates test cases by merging ecosystem URL config with provider fields, deriving log lines and auth URLs automatically
• Each ecosystem defines only what varies: credential URL fields per provider, with explicit overrides for special cases

Special cases preserved with explicit overrides:

• Docker/Helm JFrog: host-derived log targets
• Go proxy/Terraform Azure: host-based credentials
• Python GCP: index-url with /simple suffix stripping
• RubyGems: dual url+host credential fields
• NuGet: service index discovery mocks and resource URL registration
• Composer JFrog: dual registry+url fields

Result

• 1846 → 685 lines (~63% reduction)
• Same 65 test cases with identical names and assertions
• Adding a new provider = add 1 entry to oidcProviderFields + 1 line per ecosystem
• Adding a new ecosystem = add 1 buildEcosystemCases call with 5 variants
• Test runner and assertion logic unchanged

[Copilot]

Pull request overview

Refactors oidc_handling_test.go to generate OIDC handler URL/auth/log assertion cases via shared, table-driven helpers, reducing repetitive per-ecosystem/per-provider boilerplate while aiming to keep the same coverage.

Changes:

• Adds oidcProviderFields + buildEcosystemCases helpers and supporting structs to generate OIDC test cases across providers.
• Rewrites TestOIDCURLsAreAuthenticated to use provider variants per ecosystem (including special-case overrides like NuGet mocks and Python /simple behavior).
• Normalizes provider token-exchange endpoint mocks within the shared test runner.

Show a summary per file

File	Description
internal/handlers/oidc_handling_test.go	Replaces many hand-written OIDC URL/auth test cases with helper-generated, table-driven cases and provider-specific token-exchange mocks.

Copilot's findings

• Files reviewed: 1/1 changed files
• Comments generated: 2

[Copilot]

Copilot's findings

• Files reviewed: 1/1 changed files
• Comments generated: 1

[Copilot]

Copilot's findings

• Files reviewed: 1/1 changed files
• Comments generated: 2