From c71fb694bb928b5d8cd253b1fd5b7f8975d6b5ab Mon Sep 17 00:00:00 2001
From: rcourtman <courtmanr@gmail.com>
Date: Mon, 1 Dec 2025 23:08:36 +0000
Subject: [PATCH] test: Add edge case test for addQueryParam URL parse error

Add test case for URL containing control character (null byte)
which triggers url.Parse error and returns path unchanged.

Coverage improved from 88.9% to 100%.
---
 internal/api/oidc_handlers_test.go | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/internal/api/oidc_handlers_test.go b/internal/api/oidc_handlers_test.go
index 9e7f31d..161f823 100644
--- a/internal/api/oidc_handlers_test.go
+++ b/internal/api/oidc_handlers_test.go
@@ -187,6 +187,15 @@ func TestAddQueryParam(t *testing.T) {
 			value: "bar",
 			want:  "/page?foo=bar#section",
 		},
+
+		// Invalid URL (control character causes parse error)
+		{
+			name:  "path with control character returns unchanged",
+			path:  "/page\x00invalid",
+			key:   "foo",
+			value: "bar",
+			want:  "/page\x00invalid",
+		},
 	}
 
 	for _, tc := range tests {