feat: add tests using invocation test vectors#116
feat: add tests using invocation test vectors#116alanshaw wants to merge 4 commits intoucan-wg:mainfrom
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 |
|
Closed in favor of #119 |
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