This exploits the fact that we're using a string and not, say, `None` to indicate the "all" case. So I find it a bit weird here. It's also an uncommon use-case, and so should be tested after the "all" case.