docs: add test case details and sample data for verifier tests #19
docs: add test case details and sample data for verifier tests #19
Conversation
\cc @jimmarino |
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.
The BpnCredential Type doesent have CX in it
docs/testkit/sample_input/v-t_0003.a_vc_embedded_forged_claim.json
Outdated
Show resolved
Hide resolved
docs/testkit/sample_input/v-t_0003.b_vc_embedded_altered_credentialsubject.json
Outdated
Show resolved
Hide resolved
docs/testkit/sample_input/v-t_0003.c_vc_embedded_altered_claim.json
Outdated
Show resolved
Hide resolved
docs/testkit/sample_input/v-t_0004.a_vc_embedded_forged_proof.json
Outdated
Show resolved
Hide resolved
docs/testkit/sample_input/v-t_0004.b_vc_embedded_invalid_proof.json
Outdated
Show resolved
Hide resolved
docs/testkit/sample_input/v-t_0002_valid_vc_embedded_proof.json
Outdated
Show resolved
Hide resolved
Co-authored-by: Maximilian Schmidt (ZF Friedrichshafen AG) <104348964+maximilianschmidt-zf@users.noreply.github.com>
"credentialSubject": { | ||
"http://schema.org/identifier": "BPNL000000000000", | ||
"id": "did:web:localhost:BPNL000000000000", | ||
"type": "https://org.eclipse.tractusx/businessPartnerData#BpnCredential" |
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.
Have these types changed to C-X? @wolf4ood
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.
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.
this is not specific to a Catena-X or any other VC, so if anything, we should remove those references.
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.
Hey I think there is a typo happend at proof and the bpn credential
docs/testkit/sample_input/v-t_0002_valid_vc_embedded_proof.json
Outdated
Show resolved
Hide resolved
docs/testkit/sample_input/v-t_0003.a_vc_embedded_forged_claim.json
Outdated
Show resolved
Hide resolved
docs/testkit/sample_input/v-t_0003.b_vc_embedded_altered_credentialsubject.json
Outdated
Show resolved
Hide resolved
docs/testkit/sample_input/v-t_0003.c_vc_embedded_altered_claim.json
Outdated
Show resolved
Hide resolved
docs/testkit/sample_input/v-t_0004.a_vc_embedded_forged_proof.json
Outdated
Show resolved
Hide resolved
docs/testkit/sample_input/v-t_0004.b_vc_embedded_invalid_proof.json
Outdated
Show resolved
Hide resolved
Co-authored-by: Maximilian Schmidt (ZF Friedrichshafen AG) <104348964+maximilianschmidt-zf@users.noreply.github.com>
yep correct. i just copy-pasted it... |
merging, as all comments have been addressed, and re-reviews are open for multiple days |
WHAT
adds test specification and some sample test data for verifier tests. Issuer tests will follow in another PR, as well as more test data/.
WHY
to have a specified test set for verifier tests