- Replace newPool/reregPool by a singular regPool. - Remove addPoolDeposits. - Update test examples to use this new way of handling pool deposits. - Update ledger state golden test.
Also, * Use `mempty` deposits for genesis stake pools per specification. * Remove `epochStatePoolParamsL` lens. * Add prUTxOStateL, prChainAccountStateL, prCertStateL lenses to PoolReap exports.
* Add `spsDeposit` field to `StakePoolState` * Remove `psDeposits` field from `PState` data constructor * Update `mkStakePoolState` to take deposit parameter as first argument * Replace `psDepositsL` and `psDepositsCompactL` lenses with `psDepositsG` and `psDepositsCompactG` getters * Remove `payPoolDeposit` and `refundPoolDeposit` functions * Update `EncCBOR`/`DecCBOR` instances for `PState` to handle new structure * Add lenses for `StakePoolState` fields
Closes #4606
Co-authored-by: teodanciu <[email protected]>
Also update the pulser when modifying the protocol version in Conway
Reverts #5188
The problem lies in the fact that starting with Conway era number of cost model can vary, while number of parameters in any genesis file must stay the same, since that was the number of parameters with which era was initiated with. This PR: * Fixes the parsing where addition of new cost model parameters in a newer version of plutus results in a failure, unless new parameters are added to the genesis file, which would be a wrong thing to do. * Fixes the total number of parameters with which Conway era has started with. This was a not really a problem, since parsing for cost model parameters in Conway did not enforce the initial number * Start enforcing the initial number of parameters in the Conway Genesis
* Generalise compatible tests by using certificates with deposits * Move tests that are reliant on certs w/out deposits and only run them pre-Dijkstra