onflow / atree

Atree provides scalable arrays and scalable ordered maps.
https://onflow.org
Apache License 2.0
39 stars 13 forks source link

Smoke tests need to check recently added data deduplication feature #349

Closed fxamacker closed 8 months ago

fxamacker commented 9 months ago

Issue To Be Solved

Currently, the data deduplication feature added by PR #342 is not being checked by smoke tests.

We need to smoke test encoding/decoding to the new compact map format (which deduplicates nested Cadence composite types).

Suggested Solution

Add nested composite types to smoke tests. Make sure smoke tests include encoding/decoding with compact map format to test data deduplication.

fxamacker commented 8 months ago

This is closed by #350