tests: Move common function assert_approx() into util.py #16917

pull fridokus wants to merge 1 commits into bitcoin:master from fridokus:master changing 3 files +9 −13

fridokus commented at 12:59 PM on September 19, 2019: contributor

To reduce code duplication, move assert_approx into common framework util.py.

assert_approx() is used in two functional tests.
Test: Move common function assert_approx() into util.py 96299a9d6c
fanquake added the label Tests on Sep 19, 2019
theStack commented at 8:11 PM on September 19, 2019: member

ACK 96299a9 LGTM, and all functional tests pass (the Travis fail seems to be unrelated).
practicalswift commented at 9:28 PM on September 19, 2019: contributor

ACK 96299a9d6c0a6b9125a58a63ee3147e55d1b086b -- DRY is good and diff looks correct
fanquake renamed this:
~~Test: Move common function assert_approx() into util.py~~
tests: Move common function assert_approx() into util.py
on Sep 20, 2019
fanquake approved
fanquake commented at 8:53 AM on September 20, 2019: member

ACK 96299a9d6c0a6b9125a58a63ee3147e55d1b086b - thanks for contributing 🍻
fanquake referenced this in commit a73775e4d5 on Sep 20, 2019
fanquake merged this on Sep 20, 2019
fanquake closed this on Sep 20, 2019
fridokus commented at 11:48 AM on September 20, 2019: contributor

🍻
deadalnix referenced this in commit 2dca88b1ae on Jun 15, 2020
vijaydasmp referenced this in commit ec39bf685c on Dec 7, 2021
vijaydasmp referenced this in commit 21e452573e on Dec 10, 2021
vijaydasmp referenced this in commit 10719173a9 on Dec 10, 2021
vijaydasmp referenced this in commit 193ad8f69a on Dec 10, 2021
vijaydasmp referenced this in commit e117d00f7a on Dec 10, 2021
vijaydasmp referenced this in commit 0651dfb969 on Dec 13, 2021
vijaydasmp referenced this in commit f8ee2436bf on Dec 13, 2021
vijaydasmp referenced this in commit a7d94ed55f on Dec 15, 2021
DrahtBot locked this on Dec 16, 2021

Contributors

Labels