cedar-policy / cedar

Implementation of the Cedar Policy Language
https://www.cedarpolicy.com
Apache License 2.0
892 stars 80 forks source link

fix accepting `tags` as an attr name #1209

Closed cdisselkoen closed 2 months ago

cdisselkoen commented 2 months ago

Description of changes

Bugfix for unreleased code, specifically #1204. 1204 accidentally disallowed tags as an attribute name, which would be a breaking change. This PR fixes that, and adds appropriate tests. Also moves around a couple test utilities so that they can be used by the new test.

Issue #, if available

Checklist for requesting a review

The change in this PR is (choose one, and delete the other options):

I confirm that this PR (choose one, and delete the other options):

I confirm that cedar-spec (choose one, and delete the other options):