-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(sdk): add a test that has assertion statement values as objects #245
Conversation
… into add-test-with-object-assertions
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you do a run with the java and go branches to verify the changes?
It's in the comment |
Filewatcher sets assertion values as JSON objects like
this adds a test that makes sure that all SDKs can properly deserialize such manifests when they
have assertion verification disabled.
In order to make this work we needed to pass through an option that disables assertion verification.
Requires opentdf/platform#1833 and opentdf/java-sdk#219 to be
merged before it will pass
successful run against SDK branches: https://github.com/opentdf/tests/actions/runs/12992325319/job/36231902231