Skip to content

Added custom generator to address, enabled some Huddle tests#5505

Open
Soupstraw wants to merge 12 commits intomasterfrom
jj/custom-generators
Open

Added custom generator to address, enabled some Huddle tests#5505
Soupstraw wants to merge 12 commits intomasterfrom
jj/custom-generators

Conversation

@Soupstraw
Copy link
Contributor

@Soupstraw Soupstraw commented Jan 7, 2026

Description

This PR adds custom generators to address and reward_account, which allows us to enable some Huddle roundtrip tests.

Checklist

  • Commits in meaningful sequence and with useful messages.
  • Tests added or updated when needed.
  • CHANGELOG.md files updated for packages with externally visible changes.
    NOTE: New section is never added with the code changes. (See RELEASING.md).
  • Versions updated in .cabal and CHANGELOG.md files when necessary, according to the
    versioning process.
  • Version bounds in .cabal files updated when necessary.
    NOTE: If bounds change in a cabal file, that package itself must have a version increase. (See RELEASING.md).
  • Code formatted (use scripts/fourmolize.sh).
  • Cabal files formatted (use scripts/cabal-format.sh).
  • CDDL files are up to date (use scripts/gen-cddl.sh)
  • hie.yaml updated (use scripts/gen-hie.sh).
  • Self-reviewed the diff.

@Soupstraw Soupstraw force-pushed the jj/custom-generators branch 4 times, most recently from b07eb66 to b0ca63b Compare January 8, 2026 12:16
@Soupstraw Soupstraw changed the title Added custom generator to address Added custom generator to address, enabled some Huddle tests Jan 8, 2026
@Soupstraw Soupstraw force-pushed the jj/custom-generators branch 3 times, most recently from 216f332 to 19cec52 Compare January 8, 2026 12:26
@Soupstraw Soupstraw marked this pull request as ready for review January 8, 2026 12:27
@Soupstraw Soupstraw requested a review from a team as a code owner January 8, 2026 12:27
@Soupstraw Soupstraw force-pushed the jj/custom-generators branch 2 times, most recently from f2589a9 to 512a628 Compare January 8, 2026 12:31
Copy link
Collaborator

@lehins lehins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!
I did not realize that we do not have proper decoders implemented for bytes. We need to fix this for Dijkstra: #5513

@Soupstraw Soupstraw force-pushed the jj/custom-generators branch from 512a628 to 9614b6a Compare January 28, 2026 12:15
@Soupstraw Soupstraw requested a review from lehins January 28, 2026 13:37
@Soupstraw Soupstraw force-pushed the jj/custom-generators branch from 501a2c3 to 8f52c71 Compare January 29, 2026 11:36
Copy link
Contributor

@aniketd aniketd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! 🙌

Copy link
Collaborator

@lehins lehins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice! Thank you.
This will probably need to be adjusted when we start using cuddle version that uses QuickCheck's Gen, instead of StatefulGen, but compiler will remind us about it.

@lehins lehins enabled auto-merge February 3, 2026 12:33
@lehins lehins disabled auto-merge February 3, 2026 12:34
@lehins
Copy link
Collaborator

lehins commented Feb 3, 2026

Needs a manual rebase due to some conflicts not visible in Github UI

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