FuelLabs/fuel-core

Add roundtrip compression tests to show all coin types are covered

Open

#3.073 aberto em 1 de ago. de 2025

Ver no GitHub
 (0 comments) (0 reactions) (1 assignee)Rust (2.861 forks)batch import
good first issue

Métricas do repositório

Stars
 (57.084 stars)
Métricas de merge de PR
 (Mesclagem média 8h 53m) (1 fundiu PR em 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.

Guia do colaborador

Direção de pesquisa
Examine a discussão do PR vinculada (https://github.com/FuelLabs/fuel core/pull/3071#discussion r2247556627) para entender por que as saídas Variable e Change estavam faltando no CompressionContext. Localize os testes de ida e volta existentes, como postcard roundtrip v0, e adicione testes semelhantes que cubram todos os tipos de moeda, incluindo Variable e Change. Certifique se de que os testes verifiquem a ida e volta de compressão e descompressão para cada tipo.
Pilha de tecnologia
rust
Domain
testing
Tipo Issue
Teste
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
Rust programmingunderstanding of fuel core compressionbasic testing in Rust
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70