Skip to content

Add compression coverage for dynamic coin outputs - #3309

Open
0xjc65eth wants to merge 1 commit into
FuelLabs:masterfrom
0xjc65eth:test-compression-dynamic-outputs
Open

0xjc65eth wants to merge 1 commit into
FuelLabs:masterfrom
0xjc65eth:test-compression-dynamic-outputs

Conversation

@0xjc65eth

Copy link
Copy Markdown

Summary

  • add a regression test that builds a block with Coin, Change, and Variable outputs
  • assert CompressionContext maps each output UTXO back to the expected tx pointer and output index

Why
Issue #3073 notes that Change and Variable outputs were previously missed when constructing the CompressionContext, and existing tests did not catch it. The implementation currently handles them; this test locks that behavior in.

Testing

  • cargo test -p fuel-core-compression-service compression_context_indexes_coin_change_and_variable_outputs
  • git diff --check -- crates/services/compression/src/temporal_registry.rs

Closes #3073.

@0xjc65eth
0xjc65eth requested a review from xgreenx as a code owner May 26, 2026 13:26
@cursor

cursor Bot commented May 26, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Test-only change with no runtime or storage behavior modifications.

Overview
Adds a regression test in temporal_registry.rs that builds a block with Coin, Change, and Variable outputs and checks CompressionContext::create_from_block plus lookup return the expected tx_pointer and output_index for each UTXO.

No production logic changes in this diff—the test documents expected behavior for dynamic coin outputs (issue #3073) so future regressions in output indexing are caught.

Reviewed by Cursor Bugbot for commit 6a5f8a9. Bugbot is set up for automated code reviews on this repo. Configure here.

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.

Add roundtrip compression tests to show all coin types are covered

1 participant