-
Notifications
You must be signed in to change notification settings - Fork 9
feat: add tests using invocation test vectors #116
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
base: main
Are you sure you want to change the base?
Conversation
|
Summarised the failures here: ^^^ Probably needs discussion - do we need a delegation in this case? I had assumed no if the invocation is signed by the subject. ^^^ I possibly have the delegations the wrong way around in the vectors. I think the language in these two sections has caused this confusion: ^^^ I think this one is actually the same problem as above. ^^^ I believe this one might be failing because of #75 |
We had discussion with Brooke about this very question, pretty sure go-ucan is doing the right thing. Also, I believe https://github.com/hugomrdias/iso-repo/tree/main/packages/iso-ucan is doing the same. I'll need to find some time to look at the rest. |
ACK. Yes it looks that way. I'll fix the vectors and send a PR to clarify the spec language. |
|
Ok fixed the proof ordering in the vectors and now we're down to just the 2 failures: |
Is that a thing? What would be the purpose? |
It's a smaller payload for this case and it is redundant to include a proof for it. For example, perhaps you crate a bucket identified by You might also want to use this when a service needs to perform some other task on itself as part of a larger invocation. Like a |
I made 20 test vectors for invocations and added tests here that use them against the implementation.
Some are failing. I believe 1 because of #75 but please lets discuss the others :)
refs ucan-wg/spec#195