Skip to content

Commit 6246aa3

Browse files
Rohit Patilclaude
andcommitted
Complete OIDC test migration to OpenShift Tests Extension framework
- Migrate test/e2e-oidc to OTE framework with Ginkgo v2 integration - Implement dual-mode pattern supporting both Go tests and Ginkgo - Add complete OIDC authentication test workflow (not just preconditions) - Fix suite configuration to prevent duplicate test execution - Resolve all unused variables and code issues - Use proper OTE tags: [OIDC][Serial][Disruptive] TestExternalOIDCWithKeycloak [Timeout:3h] Changes: * test/e2e-oidc/external_oidc.go: New Ginkgo test registration with complete OIDC test implementation * test/e2e-oidc/external_oidc_test.go: Simplified Go test wrapper for backward compatibility * cmd/cluster-authentication-operator-tests-ext/main.go: Add e2e-oidc import and fix serial suite qualifier Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
1 parent e35c554 commit 6246aa3

4 files changed

Lines changed: 968 additions & 1139 deletions

File tree

cmd/cluster-authentication-operator-tests-ext/main.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ import (
1515
_ "github.com/openshift/cluster-authentication-operator/test/e2e"
1616
_ "github.com/openshift/cluster-authentication-operator/test/e2e-encryption-kms"
1717
_ "github.com/openshift/cluster-authentication-operator/test/e2e-encryption-perf"
18+
_ "github.com/openshift/cluster-authentication-operator/test/e2e-oidc"
1819

1920
"k8s.io/klog/v2"
2021
)
@@ -75,11 +76,12 @@ func prepareOperatorTestsRegistry() (*oteextension.Registry, error) {
7576
// The following suite runs tests that must execute serially (one at a time)
7677
// because they modify cluster-wide resources like OAuth configuration.
7778
// Tests tagged with [Serial] and any of [Operator], [OIDC], [Templates], [Tokens] are included in this suite.
79+
// Excludes [Disruptive] tests which run in the disruptive suite instead.
7880
extension.AddSuite(oteextension.Suite{
7981
Name: "openshift/cluster-authentication-operator/operator/serial",
8082
Parallelism: 1,
8183
Qualifiers: []string{
82-
`name.contains("[Serial]") && (name.contains("[Operator]") || name.contains("[OIDC]") || name.contains("[Templates]") || name.contains("[Tokens]"))`,
84+
`name.contains("[Serial]") && !name.contains("[Disruptive]") && (name.contains("[Operator]") || name.contains("[OIDC]") || name.contains("[Templates]") || name.contains("[Tokens]"))`,
8385
},
8486
})
8587

0 commit comments

Comments
 (0)