LazySuite will, by default, keep a cache of tests that are being added to it. This cache never gets removed, so when running a very large test suite, older test cases do not get freed, causing memory pressure over the long term.
This patch instead removes elements from the cache as they get read. Since the test are supposed to be iterated over just once, I believe this does not change the outward spec of this class, even if internal details have changed.
LazySuitewill, by default, keep a cache of tests that are being added to it. This cache never gets removed, so when running a very large test suite, older test cases do not get freed, causing memory pressure over the long term.This patch instead removes elements from the cache as they get read. Since the test are supposed to be iterated over just once, I believe this does not change the outward spec of this class, even if internal details have changed.