testpoolswap - add auto and true arguments to display optimal path and prices for composite swap

[thedoublejay]

What would you like to be added:

We would like to add auto and true arguments for testpoolswap so we can use get the optimal path and prices for a swap.

Why is this needed:

Currently, the path finding algorithm is done on the client side (both path and prices) and it's causing some noticeable performance issues. Note, address can be from anyone (not validated if you are the owner).

testpoolswap '{"from":"address","tokenFrom":"DFI","amountFrom":1,"to":"address","tokenTo":"GOOGL"}' auto true

Sample response:
{
  “path”: “auto”,
  “pools”: [
    “5",
    “17”
  ],
  “amount”: “35102.57338662@15”
}

[defichain-bot]

@thedoublejay: Thanks for opening an issue, it is currently awaiting triage.

The triage/accepted label can be added by foundation members by writing /triage accepted in a comment.

Details

I am a bot created to help the DeFiCh developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the DeFiCh/oss-governance-bot repository.

[defichain-bot]

@thedoublejay: There are no 'area' labels on this issue. Adding an appropriate label will greatly expedite the process for us. You can add as many area as you see fit. If you are unsure what to do you can ignore this!

You can add area labels by leaving a /area comment.

Details

I am a bot created to help the DeFiCh developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the DeFiCh/oss-governance-bot repository.

[fuxingloh]

/triage accepted
/area packages
/priority important-soon
/assign @eli-lim

Similar to #48 but expanding on top of the existing testpoolswap interface as requested by @thedoublejay.

[eli-lim]

Linked - DeFiCh/ain#1114

Do we want to present the "optimal path" - whether it's a direct or composite swap?
If so, DeFiCh/ain#1115 will need to be adjusted very slightly

[eli-lim]

@thedoublejay To reduce computation load on defichain, shall we expose this optimal path finding as an endpoint on Whale instead?

[thedoublejay]

@eli-lim Yup! Let's go with optimal path.

[thedoublejay]

@eli-lim Ok understand your point now. We can move this calculation on Whale side.