FWIW, I also do not think the orphan removal was any bearing in this test case. At least if I understand the use case correctly. It's not that the collection is generally immutable, he just happens to be using an immutable version in this use case. And here that should be fine.

Generally speaking you are correct. Just talking about this specific example