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.

Sign up for GitHub

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

Jump to bottom

Add regression tests #47

Open
lidel opened this issue Aug 1, 2022 · 0 comments
Open

Add regression tests #47

lidel opened this issue Aug 1, 2022 · 0 comments

Comments

@lidel
Copy link
Member

lidel commented Aug 1, 2022 •
edited
Loading

We should have a test for each:

  • CIDv0 (QmbWqxBEKC3P8tqsKc98xmWNzrzDtRLMiMPL8wBuTGsMnR)
  • CIDv1 base32
    • dag-pb
    • raw
    • dag-json
    • dag-cbor
  • CIDv1 base36
    • libp2p-key codec
  • PeerID (not a CID, but pasting 12D3KooWJHxkQKX8C5KAyqEPhn2ssT2in4TExyG9SXxi519tycL9 should show meaningful error as noted in Support legacy (raw) PeerIDs in Base58btc #48)
@lidel lidel mentioned this issue Aug 1, 2022
Merged
6 tasks
@hacdias hacdias mentioned this issue Aug 2, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@lidel