From 9ca6997a577d128eabdd2b545c000be0808c595c Mon Sep 17 00:00:00 2001
From: Molecule AI Core-BE <core-be@agents.moleculesai.app>
Date: Sat, 16 May 2026 22:06:51 +0000
Subject: [PATCH] test(handlers): add validateWorkspaceID pure function
 coverage
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

6 test cases for validateWorkspaceID (untested on main):
- valid v4 UUID
- valid v1 UUID
- empty string → error
- non-UUID string → error
- short UUID → error
- invalid hex char → error

Contributes to workspace_crud.go test coverage gap.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
---
 .../workspace_crud_validators_test.go         | 40 +++++++++++++++++++
 1 file changed, 40 insertions(+)

diff --git a/workspace-server/internal/handlers/workspace_crud_validators_test.go b/workspace-server/internal/handlers/workspace_crud_validators_test.go
index 74f0b346f..3c3772845 100644
--- a/workspace-server/internal/handlers/workspace_crud_validators_test.go
+++ b/workspace-server/internal/handlers/workspace_crud_validators_test.go
@@ -165,3 +165,43 @@ func TestValidateWorkspaceFields_YAMLCharsAllowedInEmptyName(t *testing.T) {
 		t.Errorf("empty name with valid role: expected nil, got %v", err)
 	}
 }
+
+// ─── validateWorkspaceID ───────────────────────────────────────────────────────
+
+func TestValidateWorkspaceID_ValidUUIDv4(t *testing.T) {
+	if err := validateWorkspaceID("550e8400-e29b-41d4-a716-446655440000"); err != nil {
+		t.Errorf("valid v4 UUID: expected nil, got %v", err)
+	}
+}
+
+func TestValidateWorkspaceID_ValidUUIDv1(t *testing.T) {
+	// UUIDv1 format is also accepted by uuid.Parse.
+	if err := validateWorkspaceID("6ba7b810-9dad-11d1-80b4-00c04fd430c8"); err != nil {
+		t.Errorf("valid v1 UUID: expected nil, got %v", err)
+	}
+}
+
+func TestValidateWorkspaceID_EmptyString(t *testing.T) {
+	if err := validateWorkspaceID(""); err == nil {
+		t.Error("empty string: expected error, got nil")
+	}
+}
+
+func TestValidateWorkspaceID_NotAUuid(t *testing.T) {
+	if err := validateWorkspaceID("not-a-uuid"); err == nil {
+		t.Error("not-a-uuid: expected error, got nil")
+	}
+}
+
+func TestValidateWorkspaceID_WrongLength(t *testing.T) {
+	if err := validateWorkspaceID("550e8400-e29b-41d4-a716"); err == nil {
+		t.Error("short UUID: expected error, got nil")
+	}
+}
+
+func TestValidateWorkspaceID_InvalidCharacters(t *testing.T) {
+	// 'g' is not a valid hex character.
+	if err := validateWorkspaceID("550e8400-e29b-41d4-a716-44665544000g"); err == nil {
+		t.Error("invalid hex char: expected error, got nil")
+	}
+}
-- 
2.52.0