Skip to content

Commit 958945a

Browse files
Ambient Code Botclaude
andcommitted
Remove duplicate legacy key test case from table-driven tests
The table-driven case "legacy authentication key only should NOT produce OIDC configs" duplicated the dedicated TestExtractOIDCConfigsIgnoresLegacyKey function. Keep the dedicated function for clearer regression intent. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent be23504 commit 958945a

1 file changed

Lines changed: 0 additions & 23 deletions

File tree

controller/cmd/main_test.go

Lines changed: 0 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -100,29 +100,6 @@ authentication:
100100
},
101101
wantCount: 1, // localhost issuer should be skipped
102102
},
103-
{
104-
name: "legacy authentication key only should NOT produce OIDC configs",
105-
configmap: &corev1.ConfigMap{
106-
ObjectMeta: metav1.ObjectMeta{
107-
Name: "jumpstarter-controller",
108-
Namespace: "default",
109-
},
110-
Data: map[string]string{
111-
"authentication": `
112-
jwt:
113-
- issuer:
114-
url: "https://dex.example.com"
115-
audiences:
116-
- "jumpstarter"
117-
claimMappings:
118-
username:
119-
claim: "email"
120-
prefix: ""
121-
`,
122-
},
123-
},
124-
wantNil: true,
125-
},
126103
{
127104
name: "missing config key returns nil",
128105
configmap: &corev1.ConfigMap{

0 commit comments

Comments
 (0)