Skip to content

Adjust FA1.2 tests

Konstantin Ivanov requested to merge better-approvable-ledger-tests into master

Description

Problem: we have a test suite checking that at zero transfer no changes are applied to the contract, but this restriction is too strict and the original FA1.2 interface does not require such thing. Recently this became a problem in one of our projects.

Solution: move this test suite to ManagedLedger tests, and in general ApprovableLedger tests leave a test on that zero transfer just does not modify participants' balances.

Checklist for your Merge Request

Related changes (conditional)

  • Tests (see short guidelines)

    • If I added new functionality, I added tests covering it.
    • If I fixed a bug, I added a regression test to prevent the bug from silently reappearing again.
  • Documentation

    • I checked whether I should update the docs and did so if necessary:
    • I updated changelog files of all affected packages released to Hackage if my changes are externally visible.

Stylistic guide (mandatory)

Edited by Konstantin Ivanov

Merge request reports