Skip to content
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

Add gluecose conformance tests #19

Merged
merged 3 commits into from
Mar 29, 2022

Conversation

qmuntal
Copy link
Contributor

@qmuntal qmuntal commented Mar 29, 2022

This PR checks in several conformance samples from https://github.com/gluecose/test-vectors and test that go-cose can sign and verify them.

Additionally, if the sample allows it, our CBOR output is compared against the sample output.

This new test suite deprecates all tests contained in sign_verify_cose_wg_examples_test.go, which were difficult to run and only did partial validations.

conformance_test.go Outdated Show resolved Hide resolved
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
@SteveLasker
Copy link
Contributor

Awesome seeing these. Did you mean to make two PRs? #19 and #19 ? They seem to impact the same files

@qmuntal
Copy link
Contributor Author

qmuntal commented Mar 29, 2022

Awesome seeing these. Did you mean to make two PRs? #19 and #19 ? They seem to impact the same files

This PR supersedes #18. I'll close it when we metge #19 .

@qmuntal
Copy link
Contributor Author

qmuntal commented Mar 29, 2022

@SteveLasker @thomas-fossati this is ready to merge.

@qmuntal qmuntal mentioned this pull request Mar 29, 2022
Copy link
Contributor

@thomas-fossati thomas-fossati left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome work, 🚢 it!

@SteveLasker SteveLasker merged commit 9a28930 into veraison:main Mar 29, 2022
@SteveLasker
Copy link
Contributor

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants