cedar-policy / cedar-go

Apache License 2.0
44 stars 7 forks source link

Fix corpus tests #16

Closed khieta closed 1 day ago

khieta commented 1 month ago

Issue #, if available:

Description of changes:

Hi 👋 I've been working on standardizing Cedar's JSON interfaces (including the integration tests), and made a couple edits recently that will affect cedar-go. I believe the changes I've made here will fix any breaks.

Tested with: go test -count=1 -v -tags corpus ./integration_tests/...

Related PRs:

philhassey commented 1 month ago

I'm holding off merging this until our error alerts for broken tests are working! Thanks!

patjakdev commented 1 day ago

I've added your changes to https://github.com/cedar-policy/cedar-go/pull/22.

The integration tests are now running successfully! Thank you :)