FuelLabs/fuel-core

Add roundtrip compression tests to show all coin types are covered

Open

#3 073 ouverte le 1 août 2025

Voir sur GitHub
 (0 commentaires) (0 réactions) (1 assigné)Rust (2 861 forks)batch import
good first issue

Métriques du dépôt

Stars
 (57 084 stars)
Métriques de merge PR
 (Merge moyen 8h 53m) (1 PR mergée en 30 j)

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.

Guide contributeur

Direction de recherche
Examinez la discussion PR liée (https://github.com/FuelLabs/fuel core/pull/3071#discussion r2247556627) pour comprendre pourquoi les sorties Variable et Change étaient absentes de CompressionContext. Localisez les tests aller retour existants, tels que postcard roundtrip v0, et ajoutez des tests similaires couvrant tous les types de coin, y compris Variable et Change. Assurez vous que les tests vérifient l'aller retour de compression et de décompression pour chaque type.
Stack technique
rust
Domaine
testing
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Rust programmingunderstanding of fuel core compressionbasic testing in Rust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70