Been investigating some weirdness in the Quoter:

These functions are not marked view because they rely on calling non-view functions and reverting to compute the result. They are also not gas efficient and should not be called on-chain.

https://docs.uniswap.org/reference/periphery/interfaces/IQuoter

Found@shanefontaine's posthttps://medium.com/authereum/getting-ethereum-transaction-revert-reasons-the-easy-way-24203a4d1844, but would need the equivalent for Python lol

All v3 tests are now passing for me locally. Not sure why CI is failing...

However, two tests for v2 are failing, as well as one for v1.

Probably going to merge this for now, we can continue the work in a follow-up PR.

I just remembered that CI is likely failing due not having access to secrets in PRs (maybe it's just because the branch is on my own fork?).

After some debugging, I got the tests to print this:

<Result MissingSchema("Invalid URL '': No schema supplied. Perhaps you meant http://?")>

Tried setting the secret in my own fork, but no success.

Gonna merge this and hope for the best...