Remove non-functional optional test in field selection (#1161)

Author: TristonianJones

cel/cel_test.go

@@ -2669,6 +2669,22 @@ func TestOptionalValuesEval(t *testing.T) {
 		in   map[string]any
 		out  any
 	}{
+		{
+			expr: `has({'foo': optional.none()}.foo)`,
+			out:  types.True,
+		},
+		{
+			expr: `has({'foo': optional.none()}.foo.value)`,
+			out:  types.False,
+		},
+		{
+			expr: `has({?'foo': optional.none()}.foo)`,
+			out:  types.False,
+		},
+		{
+			expr: `has({?'foo': optional.none()}.foo.value)`,
+			out:  "no such key: foo",
+		},
 		{
 			expr: `{}.?invalid`,
 			out:  types.OptionalNone,

interpreter/interpretable.go

@@ -194,9 +194,6 @@ func (q *testOnlyQualifier) Qualify(vars Activation, obj any) (any, error) {
 	if unk, isUnk := out.(types.Unknown); isUnk {
 		return unk, nil
 	}
-	if opt, isOpt := out.(types.Optional); isOpt {
-		return opt.HasValue(), nil
-	}
 	return present, nil
 }