Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Proof approach in cardano-ledger-tests has to be scrapped. #4874

Open
lehins opened this issue Feb 4, 2025 · 0 comments
Open

Proof approach in cardano-ledger-tests has to be scrapped. #4874

lehins opened this issue Feb 4, 2025 · 0 comments
Assignees
Labels
💳 technical-debt Issues related to technical debt we introduced

Comments

@lehins
Copy link
Collaborator

lehins commented Feb 4, 2025

Throughout development of Alonzo and Babbage there was an attempt at creating a new kind of generator and testing approach (i.e. for the aware it used to be called the Omega era). It is mostly contained in Test.Cardano.Ledger.Generic

This approach is overly complex that results in a lot of unnecessary boilerplate code. Also, it defies the type safety our usual type class approach for us.

It needs to go. That being said, there is a lot of useful functionality in there for generating valid transactions. We need to scrap whatever useful stuff we can find from this approach and move it into individual era ImpSpec modules

@lehins lehins added the 💳 technical-debt Issues related to technical debt we introduced label Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💳 technical-debt Issues related to technical debt we introduced
Projects
None yet
Development

No branches or pull requests

2 participants