Skip to content
This repository has been archived by the owner on Dec 2, 2024. It is now read-only.

Merge main into next-node (2022-08-12) #659

Merged
merged 11 commits into from
Aug 12, 2022

Conversation

koslambrou
Copy link
Contributor

Pre-submit checklist:

  • Branch
    • Tests are provided (if possible)
    • Commit sequence broadly makes sense
    • Key commits have useful messages
    • Formatting, PNG optimization, etc. are updated
  • PR
    • Self-reviewed the diff
    • Useful pull request description
    • Reference the ADR in the PR and reference the PR in the ADR (if revelant)
    • Reviewer requested

zeme-wana and others added 11 commits July 29, 2022 15:06
Co-authored-by: Konstantinos Lambrou-Latreille <[email protected]>
…ation rules when validating transaction in the emulator (#639)
…e balancing (#600)

* Calculate minAda on the change output when balancing

* Better coin selection algorithm

* Add more explanation

* Re-enable prop_Uniswap

* Disabled prop_Uniswap again

This reverts commit eceb8c3.

* PR feedback
…sV1ScriptFilter` and `Ledger.Constraints.TxConstraints.collectFromTheScript` as unrelated functions. (#644)

The rationale is that if someone uses `collectFromPlutusV1ScriptFilter`,
the constraints will fail with a `TypedValidatorMissing` error for
lookups, even though we're not using typed validators.
* Add mustSpendAtLeast Constraint tests

* Change MustSpendAtLeast to use Datum and included phase-2 failure test

* Spilt 'higherThanScriptBalance' test into two and introduced ownPaymentPubKeyHash script lookup (2 tests failing)

* Comment out failing tests due to bug PLT-665
...moved some test specs into "TxConstraints directory.
@koslambrou koslambrou force-pushed the kll/merge-main-into-next-node-2022-08-11 branch from 8616bfe to 593ffaf Compare August 12, 2022 12:19
@koslambrou koslambrou merged commit 19e1e6c into next-node Aug 12, 2022
@koslambrou koslambrou deleted the kll/merge-main-into-next-node-2022-08-11 branch August 12, 2022 13:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants