osmosis icon indicating copy to clipboard operation
osmosis copied to clipboard

Published 20 hours ago • verified publisher icon osmosis-labs

TWAP: Make integration tests for relevant methods working on 3 asset pools

Open ValarDragon opened this issue 3 years ago • 1 comments

Background

Most of the TWAP tests, are centered around 2 asset pools. All of the store management and code is written to explicitly work with more pairs, and should work, with a few unit tests for this.

We should make multi-asset pool tests at the api_test and hook_test layers.

Acceptance Criteria

  • [ ] Multi asset pool test for Endblock flow
  • [ ] Multi asset pool test for GetArithmeticTwap

ValarDragon avatar Jul 23 '22 13:07 ValarDragon

  • [ ] https://github.com/osmosis-labs/osmosis/pull/2364
  • [ ] https://github.com/osmosis-labs/osmosis/pull/2420

mattverse avatar Aug 16 '22 11:08 mattverse

Closing in favor of #2538

czarcas7ic avatar Sep 06 '22 02:09 czarcas7ic