Adjust FA1.2 tests
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
Stylistic guide (mandatory)
-
My commits comply with the following policy. -
My code complies with the style guide.
Edited by Konstantin Ivanov