From 8387c1d196b6232ae84d937737010984f0ffb359 Mon Sep 17 00:00:00 2001 From: Prashansa Kulshrestha Date: Mon, 1 Jun 2026 13:27:22 +0530 Subject: [PATCH] tests: removed unrelated tests from konnect test suite Konnect CI takes >20 minutes. To avoid random timeouts, and reduce time-to-merge, some tests that make no sense to be tested against Konnect are skipped here. This would give a little buffer to avoid timeouts. A proper fix for this increased CI time would be to split testing across multiple orgs. While we do not implement that, we wish to keep the CI time under 25 minutes. Hence, the buffer. --- tests/integration/diagnostic_policy_test.go | 4 ++++ tests/integration/file_convert_test.go | 3 +++ tests/integration/lint_test.go | 2 ++ tests/integration/render_test.go | 1 + tests/integration/validate_test.go | 1 + 5 files changed, 11 insertions(+) diff --git a/tests/integration/diagnostic_policy_test.go b/tests/integration/diagnostic_policy_test.go index d8e24915f..8d85ee01e 100644 --- a/tests/integration/diagnostic_policy_test.go +++ b/tests/integration/diagnostic_policy_test.go @@ -8,6 +8,7 @@ import ( ) func Test_RenderDiagnosticSeverityOverrides_RouteRegexWarningAsError(t *testing.T) { + skipWhenKonnect(t) _, err := render( "testdata/render/005-diagnostics/route-regex.yaml", "-E", "route-regex-path-format", @@ -17,6 +18,7 @@ func Test_RenderDiagnosticSeverityOverrides_RouteRegexWarningAsError(t *testing. } func Test_RenderDiagnosticSeverityOverrides_OIDCSeverityOverrides(t *testing.T) { + skipWhenKonnect(t) t.Run("defaults to hard error", func(t *testing.T) { _, err := render("testdata/render/005-diagnostics/oidc-missing-config.yaml") require.Error(t, err) @@ -44,12 +46,14 @@ func Test_RenderDiagnosticSeverityOverrides_OIDCSeverityOverrides(t *testing.T) } func Test_RenderDiagnosticSeverityOverrides_DefaultWarningsRemainWarnings(t *testing.T) { + skipWhenKonnect(t) output, err := render("testdata/render/005-diagnostics/route-regex.yaml") require.NoError(t, err) assert.Contains(t, output, "svc-route-regex") } func Test_RenderDiagnosticSeverityOverrides_InvalidCode(t *testing.T) { + skipWhenKonnect(t) _, err := render( "testdata/render/005-diagnostics/route-regex.yaml", "-E", "not-a-real-code", diff --git a/tests/integration/file_convert_test.go b/tests/integration/file_convert_test.go index 70373ce67..fd0ec02ba 100644 --- a/tests/integration/file_convert_test.go +++ b/tests/integration/file_convert_test.go @@ -11,6 +11,7 @@ import ( ) func Test_FileConvert(t *testing.T) { + skipWhenKonnect(t) tests := []struct { name string convertCmdSourceFormat string @@ -75,6 +76,7 @@ func Test_FileConvert(t *testing.T) { } func Test_FileConvert_NoExpandEnvVars(t *testing.T) { + skipWhenKonnect(t) tests := []struct { name string additionalArgs []string @@ -124,6 +126,7 @@ func Test_FileConvert_NoExpandEnvVars(t *testing.T) { } func Test_FileConvert_28xTo34x(t *testing.T) { + skipWhenKonnect(t) originalCwd, err := os.Getwd() require.NoError(t, err) defer func() { diff --git a/tests/integration/lint_test.go b/tests/integration/lint_test.go index 0951b15bc..2c8e99208 100644 --- a/tests/integration/lint_test.go +++ b/tests/integration/lint_test.go @@ -16,6 +16,7 @@ import ( ) func Test_LintPlain(t *testing.T) { + skipWhenKonnect(t) tests := []struct { name string stateFile string @@ -50,6 +51,7 @@ type lintErrors struct { } func Test_LintStructured(t *testing.T) { + skipWhenKonnect(t) tests := []struct { name string stateFile string diff --git a/tests/integration/render_test.go b/tests/integration/render_test.go index 8f91ee9fb..36682e1a3 100644 --- a/tests/integration/render_test.go +++ b/tests/integration/render_test.go @@ -8,6 +8,7 @@ import ( ) func Test_RenderPlain(t *testing.T) { + skipWhenKonnect(t) tests := []struct { name string stateFile string diff --git a/tests/integration/validate_test.go b/tests/integration/validate_test.go index d3a03f7ca..7bde0198a 100644 --- a/tests/integration/validate_test.go +++ b/tests/integration/validate_test.go @@ -123,6 +123,7 @@ func Test_Validate_Konnect(t *testing.T) { } func Test_Validate_File(t *testing.T) { + skipWhenKonnect(t) setup(t) tests := []struct {