Can I get some help with tests? I wrote a quick failing test with Assert.Equal(true, false);, but running ./build.ps1 -Test all I'm getting is:

Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
No test matches the given testcase filter `FullyQualifiedName~Test.en_US_Linux`

The ToArray() cannot be removed without changing how the filtering is done, because you cannot delete items from a collection while enumerating it.

I updated the code to get rid of ToArray(), as well as merging this fix with #3454.

The ToArray() cannot be removed without changing how the filtering is done, because you cannot delete items from a collection while enumerating it.

Good catch, thanks. There is a better pattern to do it on the fly though:

for (int i = 0; i < collection.Length; i++) {
    if (predicate(collection[i])) {
        // postdec cancels out increment, so next loop still uses same index
        collection.RemoveAt(i--);
    }
}

I personally like it as it's compact and efficient, other people find it hacky. Up to you.

🎉 v2.3.0-beta0 has been released which incorporates this pull request. 🎉