Skip to content

Conversation

@muditchaudhary
Copy link
Contributor

@muditchaudhary muditchaudhary commented Nov 9, 2025

Cedar 4.4 introduced Enumerated Entity Types, which allow entities to be restricted to a predefined set of values. While this feature requires no implementation changes in CedarJava, this PR adds test cases to verify the functionality works as expected with the existing codebase.

Signed-off-by: Mudit Chaudhary <chmudit@amazon.com>
Signed-off-by: Mudit Chaudhary <chmudit@amazon.com>
@muditchaudhary muditchaudhary force-pushed the 4.4_release/features/enumTypes branch from 258abeb to fabbdd2 Compare November 9, 2025 16:52
@muditchaudhary muditchaudhary marked this pull request as ready for review November 9, 2025 17:09
@muditchaudhary muditchaudhary changed the title Add test coverage for Enumerated Entity Types support Adds test coverage for Enumerated Entity Types support Nov 9, 2025
"permit("
+ " principal,"
+ " action == Action::\"CreateList\","
+ " resource == Application::\"TinyTODO\"" // Typo in enum value
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: Is this case substantially different than givenEnumSchemaAndWrongCaseEnumValueReturnsInvalid?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually not. We can remove one of these.

@muditchaudhary muditchaudhary merged commit f4a8c40 into cedar-policy:main Dec 4, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants