Fix: Update tests to align with lenient schema behavior

Recent schema changes (commits 77e8ab0-6588b38) made validation more
lenient to support ATAK formats by:
- Making color attributes optional (argb/value)
- Changing height/height_unit base type to xs:string
- Making link attributes optional

Old tests expected strict validation that rejected valid ATAK formats.

Changes:
1. Updated test "bad" cases to test actual type violations:
   - color: non-integer argb instead of missing attributes
   - height: non-decimal value attribute instead of string content
   - height_unit: non-integer value attribute instead of string content
   - link: invalid datetime instead of missing attributes

2. Updated regression test timestamps from 2025-10-23 to 2025-11-01
   to pass 24-hour validation window

All tests now pass ✅

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: pdfinn

shape_regression_test.go

@@ -9,7 +9,7 @@ import (
 // Regression test: ATAK accepts circles without link, cotlib should too.
 func TestEllipseWithoutLink(t *testing.T) {
 	xml := `<?xml version='1.0' encoding='UTF-8'?>
-<event version='2.0' uid='test-no-link' type='u-r-b-c-c' time='2025-10-23T02:00:00Z' start='2025-10-23T02:00:00Z' stale='2025-10-24T02:00:00Z' how='h-e'>
+<event version='2.0' uid='test-no-link' type='u-r-b-c-c' time='2025-11-01T02:00:00Z' start='2025-11-01T02:00:00Z' stale='2025-11-02T02:00:00Z' how='h-e'>
 	<point lat='1.0' lon='1.0' hae='0' ce='10' le='10'/>
 	<detail>
 		<shape>
@@ -33,7 +33,7 @@ func TestEllipseWithoutLink(t *testing.T) {
 // TestEllipseWithLink ensures ellipse shapes still work WITH <link> element.
 func TestEllipseWithLink(t *testing.T) {
 	xml := `<?xml version='1.0' encoding='UTF-8'?>
-<event version='2.0' uid='test-with-link' type='u-r-b-c-c' time='2025-10-23T02:00:00Z' start='2025-10-23T02:00:00Z' stale='2025-10-24T02:00:00Z' how='h-e'>
+<event version='2.0' uid='test-with-link' type='u-r-b-c-c' time='2025-11-01T02:00:00Z' start='2025-11-01T02:00:00Z' stale='2025-11-02T02:00:00Z' how='h-e'>
 	<point lat='1.0' lon='1.0' hae='0' ce='10' le='10'/>
 	<detail>
 		<shape>
@@ -71,9 +71,9 @@ func TestColorFlexibleAttributes(t *testing.T) {
 		name string
 		xml  string
 	}{
-		{"only_argb", `<event version='2.0' uid='t1' type='a-f-G' time='2025-10-23T02:00:00Z' start='2025-10-23T02:00:00Z' stale='2025-10-24T02:00:00Z' how='h-e'><point lat='1.0' lon='1.0' hae='0' ce='10' le='10'/><detail><color argb='-1'/></detail></event>`},
-		{"only_value", `<event version='2.0' uid='t2' type='a-f-G' time='2025-10-23T02:00:00Z' start='2025-10-23T02:00:00Z' stale='2025-10-24T02:00:00Z' how='h-e'><point lat='1.0' lon='1.0' hae='0' ce='10' le='10'/><detail><color value='-1'/></detail></event>`},
-		{"both", `<event version='2.0' uid='t3' type='a-f-G' time='2025-10-23T02:00:00Z' start='2025-10-23T02:00:00Z' stale='2025-10-24T02:00:00Z' how='h-e'><point lat='1.0' lon='1.0' hae='0' ce='10' le='10'/><detail><color value='-1' argb='-1'/></detail></event>`},
+		{"only_argb", `<event version='2.0' uid='t1' type='a-f-G' time='2025-11-01T02:00:00Z' start='2025-11-01T02:00:00Z' stale='2025-11-02T02:00:00Z' how='h-e'><point lat='1.0' lon='1.0' hae='0' ce='10' le='10'/><detail><color argb='-1'/></detail></event>`},
+		{"only_value", `<event version='2.0' uid='t2' type='a-f-G' time='2025-11-01T02:00:00Z' start='2025-11-01T02:00:00Z' stale='2025-11-02T02:00:00Z' how='h-e'><point lat='1.0' lon='1.0' hae='0' ce='10' le='10'/><detail><color value='-1'/></detail></event>`},
+		{"both", `<event version='2.0' uid='t3' type='a-f-G' time='2025-11-01T02:00:00Z' start='2025-11-01T02:00:00Z' stale='2025-11-02T02:00:00Z' how='h-e'><point lat='1.0' lon='1.0' hae='0' ce='10' le='10'/><detail><color value='-1' argb='-1'/></detail></event>`},
 	}
 
 	ctx := context.Background()

validation_test.go

@@ -718,12 +718,12 @@ func TestTAKDetailSchemaValidation(t *testing.T) {
 		if err := evt.Validate(); err != nil {
 			t.Fatalf("valid height rejected: %v", err)
 		}
-		evt.Detail.Height.Raw = []byte(`<height>bad</height>`)
+		evt.Detail.Height.Raw = []byte(`<height value="not-a-decimal"/>`)
 		if err := evt.Validate(); err == nil {
 			t.Fatal("expected error for invalid height")
 		}
 		evt.Detail.Height.Raw = []byte(`<height>1</height>`)
-		evt.Detail.HeightUnit.Raw = []byte(`<height_unit>x</height_unit>`)
+		evt.Detail.HeightUnit.Raw = []byte(`<height_unit value="not-an-integer"/>`)
 		if err := evt.Validate(); err == nil {
 			t.Fatal("expected error for invalid height_unit")
 		}
@@ -759,7 +759,7 @@ func TestTAKDetailSchemaValidation(t *testing.T) {
 		if err := evt.Validate(); err != nil {
 			t.Fatalf("valid link detail rejected: %v", err)
 		}
-		evt.Detail.LinkDetail.Raw = []byte(`<link uid="u"/>`)
+		evt.Detail.LinkDetail.Raw = []byte(`<link production_time="not-a-datetime"/>`)
 		if err := evt.Validate(); err == nil {
 			t.Fatal("expected error for invalid link detail")
 		}

validator/schema_test.go

@@ -279,19 +279,19 @@ func TestValidateRemainingDetailSchemas(t *testing.T) {
 			name:   "color",
 			schema: "tak-details-color",
 			good:   []byte(`<color argb="1"/>`),
-			bad:    []byte(`<color/>`),
+			bad:    []byte(`<color argb="not-an-integer"/>`),
 		},
 		{
 			name:   "height",
 			schema: "tak-details-height",
 			good:   []byte(`<height>1</height>`),
-			bad:    []byte(`<height>bad</height>`),
+			bad:    []byte(`<height value="not-a-decimal"/>`),
 		},
 		{
 			name:   "height_unit",
 			schema: "tak-details-height_unit",
 			good:   []byte(`<height_unit>1</height_unit>`),
-			bad:    []byte(`<height_unit>x</height_unit>`),
+			bad:    []byte(`<height_unit value="not-an-integer"/>`),
 		},
 		{
 			name:   "labels_on",
@@ -303,7 +303,7 @@ func TestValidateRemainingDetailSchemas(t *testing.T) {
 			name:   "link",
 			schema: "tak-details-link",
 			good:   []byte(`<link uid="u" type="a-f-G" relation="p-p"/>`),
-			bad:    []byte(`<link uid="u"/>`),
+			bad:    []byte(`<link production_time="not-a-datetime"/>`),
 		},
 		{
 			name:   "emergency",