FuelLabs/fuel-core

Add roundtrip compression tests to show all coin types are covered

Open

#3,073 opened on Aug 1, 2025

View on GitHub
 (0 comments) (0 reactions) (1 assignee)Rust (2,861 forks)batch import
good first issue

Repository metrics

Stars
 (57,084 stars)
PR merge metrics
 (Avg merge 8h 53m) (1 merged PR in 30d)

Description

Followup to https://github.com/FuelLabs/fuel-core/pull/3071#discussion_r2247556627

We were missing Variable and Change output when constructing the CompressionContext, but no tests were breaking. We need to clarify the expected behavior with tests.

Currently there are some roundtrip compression tests (e.g. postcard_roundtrip_v0) but they don't cover the aforementioned cases.

Contributor guide

Research direction
Examine the linked PR discussion (https://github.com/FuelLabs/fuel core/pull/3071#discussion r2247556627) to understand why Variable and Change output were missing from CompressionContext. Locate the existing roundtrip tests, such as postcard roundtrip v0, and add similar tests covering all coin types, including Variable and Change. Ensure the tests verify compression and decompression roundtrip for each type.
Tech stack
rust
Domain
testing
Issue type
Test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Mostly clear
Prerequisites
Rust programmingunderstanding of fuel core compressionbasic testing in Rust
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70