Split Failures returns empty when by_approach=False

This worked previously and probably broke with the most recent update where an extra join was added for efficiency. Need to introduce unit tests to catch things like this in future releases. I'm opening this as an issue to remember to come back to it.