search

trustlines-protocol / relay

MIT License
8 stars 7 forks source link

Lower indexer required confirmation in tests to 50 #590

Closed cducrest closed 3 years ago

cducrest commented 3 years ago

This speeds up tests using the indexer a lot. This should also make them non-flaky as the tests won't have to wait for as long for the indexer to sync.

compojoom commented 3 years ago

does the medium class actually bring us anything?

cducrest commented 3 years ago

It's the default one, I just made it explicit. The bigger class made the test like 2-3% faster which could just be random, however they did help me to debug how to make the test faster and non-flaky because they failed in a more stable way.