Investigate TODOs in tests

[shaobo-he-aws]

Describe the improvement you'd like to request

For instance,

cedar/cedar-policy-validator/src/json_schema.rs

Line 2986 in 33bd811

assert_matches!(&**element, Type::Type{ ty: TypeVariant::String, ..}); // TODO: why is this `TypeVariant::String` in this case but `EntityOrCommon { "String" }` in all the other cases in this test? Do we accept common types as the element type for sets?

Describe alternatives you've considered

No response

Additional context

No response

Is this something that you'd be interested in working on?

• 👋 I may be able to implement this internal improvement
• ⚠️ This feature might incur a breaking change