Sorting tokens in fixture does not work

[sz-piotr]

https://github.com/Uniswap/uniswap-v2-core/blob/4dd59067c76dea4a0e8e4bfdda41877a6b16dedc/test/shared/fixtures.ts#L40

Because ethers returns a string from this method call, .address always results in undefined, thus the order of tokens is never changed.

[moodysalem]

Good catch! I guess token0 and token1 must always be the same for the fixture, so we never had failing tests out of it.

[Devilla]

Good catch! I guess token0 and token1 must always be the same for the fixture, so we never had failing tests out of it.

Sir, how can token0 and token1 be same for pairFixture ?

[moodysalem]

I mean same addresses across test runs (but token0 != token1), since we use the same mnemonic for every test run

[Devilla]

Okay so it'll fail for different pair address and pass for same pair address right?

[ghost]

I think the best fix would be to lowercase the addresses of the token and just compare like how it is done from the factory contract as well.