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

Simple fuzz test for v0.14 <-> v0.15 interoperability #270

Draft
wants to merge 7 commits into
base: marco/capture-interface
Choose a base branch
from

Conversation

MarcoPolo
Copy link
Contributor

I wrote this small test as a sanity check to make sure v0.15 did not accidentally break anything in a subtle ways. It pulls in v0.14 into the compattest/internal/prev folder and fuzzes against it.

We probably don't need to merge this. Running this locally was enough to increase my confidence that we aren't accidentally breaking something.

Note that we also have transport interop tests in go-libp2p as well that would fail if v0.15 were not compatible. The goal with the fuzz tests is to hopefully check edge cases as well.

web3-bot and others added 7 commits February 16, 2025 12:35
BREAKING CHANGE: Updating the Go version to 1.23
* refactor: keep same api for SplitFirst/SplitLast

Not worth breaking this function, and it saves us some pain with
updating downstream deps

* add defensive copy
…ent (#272)

* Backwards compatible Encapsulate/Decapsulate/Join

* remove EncapsulateC

* feat: Add Multiaddr.AppendComponent

* remove .Empty (#274)

prefer the simpler `len` check for Multiaddr and `== nil` for
`*Component`.

* export AsMultiaddrer interface

* rename AsMultiaddr to Multiaddr

Export Multiaddrer interface
git-subtree-dir: compattest/internal/prev
git-subtree-split: 37363a0
@MarcoPolo MarcoPolo force-pushed the marco/backwards-compat branch from 24b8ec0 to bf94599 Compare February 24, 2025 20:08
@p-shahi p-shahi mentioned this pull request Feb 26, 2025
9 tasks
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