Fixtures cleanup - Githubissues

curveresearch / curvesim

Simulates Curve Finance pools

MIT License

155 stars 32 forks source link

Fixtures cleanup #241

Closed allt0ld closed 1 year ago

allt0ld commented 1 year ago

Description

Added tricrypto_views.vy and tricrypto_factory.vy to curvesim.test.fixtures.curve + corresponding fixtures as tricrypto_ng.vy and its vyper_tricrypto fixture rely on these external contracts for functionality/parameters.
Replaced test_tricrypto.get_math with tricrypto_math fixture (of tricrypto_math.vy) to avoid slow boa.load_partial and for cleanliness.

Hygiene checklist

[x] Changelog entry
[x] Everything public has a Numpy-style docstring (modules, public functions, classes, and public methods)
[x] Commit history is cleaned-up with minor changes squashed together and descriptive commit messages following Tim Pope's style

Cute Animal Picture

github-actions[bot] commented 1 year ago

Coverage report

The coverage rate went from 74.96% to 75.17% :arrow_up: The branch rate is 61%.

None of the new lines are part of the tested code. Therefore, there is no coverage data about them.