cedar-policy / cedar

Implementation of the Cedar Policy Language

Home Page:https://www.cedarpolicy.com

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Make integration test JSON consistent with FFI

khieta opened this issue · comments

Describe the improvement you'd like to request

The cedar_testing module defines a JSON format for integration tests. We should make it consistent with the JSON format in the ffi module, reusing definitions where possible.

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

Started looking into this as part of #757. There's actually not too much reuse currently possible since cedar_policy::ffi relies on the public APIs in cedar_policy, while the integration testing code relies on internal APIs in cedar_policy_core and cedar_policy_validator. It's not simple to change this right now because the DRT/Lean code in cedar-spec also needs these internal functions & types. So I think this issue is blocked on cedar-spec#75