Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 1, 2025

Adds property-based testing for ProgressiveList in the proptest operations suite.

Changes

  • ProgressiveSpec<T> - Specification struct without length limit (no N type parameter), matching ProgressiveList's unbounded nature
  • arb_op_progressive - Strategy generating only implemented operations: Len, Push, Iter, TreeHash, Debase
  • apply_ops_progressive_list - Applies operations to both ProgressiveList and spec, verifying equivalence
  • Test module - Tests for u8, u64, Hash256, and Large types

Example

progressive_list_test!(u8, u8);
progressive_list_test!(hash256, Hash256, arb_hash256());

Unimplemented operations (Get, Set, IterFrom, PopFront, Rebase, etc.) are handled as no-ops for exhaustive matching but excluded from generation.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Add iterator for ProgTree structure Add ProgressiveList to proptest suite Dec 1, 2025
Copilot AI requested a review from michaelsproul December 1, 2025 07:23
@michaelsproul michaelsproul marked this pull request as ready for review December 2, 2025 06:16
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.02%. Comparing base (07a808e) to head (e32d558).

Additional details and impacted files
@@                 Coverage Diff                  @@
##           progressive-list      #89      +/-   ##
====================================================
+ Coverage             69.35%   73.02%   +3.66%     
====================================================
  Files                    24       24              
  Lines                  1426     1427       +1     
====================================================
+ Hits                    989     1042      +53     
+ Misses                  437      385      -52     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@michaelsproul michaelsproul merged commit edd833b into progressive-list Dec 2, 2025
12 checks passed
@michaelsproul michaelsproul deleted the copilot/sub-pr-82 branch December 2, 2025 06:33
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.

3 participants