fix: flaky TestGetPendingGates - add status assertion

internal/approval/gate_test.go

@@ -47,7 +47,9 @@ func TestGetPendingGates(t *testing.T) {
 	pending := manager.GetPendingGates()
 	assert.Equal(t, 1, len(pending))
 	if len(pending) > 0 {
-		assert.Equal(t, gate2.ID, pending[0].ID)
+		// Don't rely on map iteration order - just check it's the unapproved gate
+		assert.Equal(t, StatusPending, pending[0].Decision.Status)
+		assert.Equal(t, gate2.ID, pending[0].ID) // gate2 is the only pending one
 	}
 }